pagure

#3212 Add tests for getting commit author stats
Merged by pingou. Opened by lsedlar.
lsedlar/pagure task-tests  into  master

Download 3212.patch

Handling non-existing repo is done by catching exception as that is easier to mock.

I think I liked the previous approach more, there are much more things that can raise a GitError exception that a non-existing repo :(

What about including in this PR the last two commits of https://pagure.io/pagure/pull-request/3203 ?

Ok, I'll update it.

rebased onto 7933adae27fd756188938a914192ab956e573d98

Commits added, task code restored to original version, rebased on current master.

Pretty please pagure-ci rebuild

Looks good and tests are passing, let's rebase and merge :)

rebased onto d6299fc345205db9dea5d9568f26788f9d5c0d59

Pretty please pagure-ci rebuild

Ci is currently failing on jenkins, I'm running the tests locally and will merge if they pass :)

All good :)

Pull-Request has been merged by pingou
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.