PR#255: Add a few tests - - Pagure.io

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

fedpkg

#255 Add a few tests

Merged Jul 25, 2018 by cqi. Opened Jul 24, 2018 by cqi.

cqi/fedpkg more-tests into master

Download 255.patch

no initial comment

lsedlar commented Jul 24, 2018

Flake8 is complaining about unused variable here.

onosek commented Jul 25, 2018

too many blank lines warning

onosek commented Jul 25, 2018

typo?

rebased onto f3cc8a99e2e5d2f2286110bfb6d3c30961efe9bf

Jul 25, 2018

5 new commits added

Fix flake8 errors and typo in tests
Add tests for some commands
Convert test case for utils.py as normal test case
Add some tests for BugzillaClient
Add tests for utils.py

Jul 25, 2018

cqi commented Jul 25, 2018

Thank you guys. Issues are fixed.

onosek commented Jul 25, 2018

OK for me.

rebased onto 9dd651f769aa0eaee4cad7ca68ca32cdb58ffa13

Jul 25, 2018

rebased onto 9dd651f769aa0eaee4cad7ca68ca32cdb58ffa13

Jul 25, 2018

cqi commented Jul 25, 2018

Rebase and fix conflicts.

cqi commented Jul 25, 2018

Commit 32037609 fixes this pull-request

Pull-Request has been merged by cqi

Jul 25, 2018

Pull-Request has been merged by cqi

Jul 25, 2018

Metadata

Assignee

None

Tags

No Tags

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

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

© Red Hat, Inc. and others.