koji

#4432 avoid loading system hub config in unit test
Merged by mikem. Opened by mikem.
mikem/koji fix-session-mock  into  master

Download 4432.patch

Fixes https://pagure.io/koji/issue/4431

The unit test was never meant to read the system config. This adjustment to the mock setup solves the issue.

@jcupova

Looks good to me. All of the Python3 tests for me now.

Thank you @mikem !

Metadata Update from @mikem:
- Pull-request tagged with: no_qe

mock now is only available in py2 testenv, so it should be from unitest import mock here

mock now is only available in py2 testenv, so it should be from unitest import mock here

ah, yet another test whose origin straddled that change. #4150 was only recently merged, but was mostly written in 2024, before #4239 was merged

1 new commit added

  • fix mock import

:thumbsup:

rebased onto 7c2d768d225ecccd17fdf1b5ca71790ae1118369

Commit 5619342e fixes this pull-request

Pull-Request has been merged by mikem
Metadata
None

no_qe

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.