greenwave

#177 fix urlparse invocation
Merged by ralph. Opened by dcallagh.
dcallagh/greenwave urlparse-urlparse-urlparse  into  master

Download 177.patch

urlparse is the module, urlparse.urlparse() is the function. Error found
by pylint.

I guess this means we have no test coverage for this @gnaponie ? The tests pass with or without this fix...

Yup. We should get that function covered. Thanks for the jenkins build fix, @dcallagh!

Pull-Request has been merged by ralph

@dcallagh It is just mocked in greenwave/tests/test_policies.py::test_remote_original_spec_nvr_rule_policy
Because I didn't know how to "fake" koji. I've tried to create a subprocess like we do for resultsdb, waiverdb, dist-git in the functional-tests, but it would be always a "fake" instance. I'm not sure about how to test it in a "real" way.
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.