pagure

#5187 An attempt to fix pip/tox tests
Closed by ngompa. Opened by abitrolly.
abitrolly/pagure fix-tests  into  master

Download 5187.patch

Currently tests that are run with installed native Fedora 32 dependencies are passing, but tests when Pagure is installed with pip are not. Pinning dependencies will most likely fix the problem.

pretty please pagure-ci rebuild

We probably want to have the tests passing with pygit2 >= 1.0 since Fedora 32 ships with it too.

Need to compare all dependencies. Right now the failure rate for #4120 is this.

============= 24 failed, 1677 passed, 9 skipped in 1703.72 seconds =============

1 new commit added

  • Successful F32 tests are done with pygit2 1.2.1

Tests that are failing are actually Centos Python 2 tests.

We can have logs and better test stats if @pingou can take a look at https://pagure.io/centos-infra/issue/360 -

rebased onto c1198d32e3d127aee5e211dc4b11ad892dc8d10e

pretty please pagure-ci rebuild

I don't see much changes in this PR, so I guess this one should be closed in favor of https://pagure.io/pagure/pull-request/5365
I agree with the version pinning part, my plan is to do it as soon all tests are passing again and then taking this state / package versions as baseline.

Pull-Request has been closed by ngompa
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.