#98 Add test cases for delete_build() function
Merged by mikem. Opened by sijis.
sijis/koji feature/add_test_cases_2  into  master

Download 98.patch

Adding more test cases.

The test for key 'last_used' had to be broken down separately since it contained a time constraint check.

I'm not that familar with using the mock module. Is it really correct to make this self-referential?

You can. Ralph would know the true 'magic' of that. He was using that
snippet for another test case he was using and told me about it.

Sijis
On Jun 7, 2016 4:20 PM, pagure@pagure.io wrote:

mikem commented on the pull-request: Add test cases for delete_build() function that you are following:
I'm not that familar with using the mock module. Is it really correct to make this self-referential?

To reply, visit the link below or just reply to this email
https://pagure.io/koji/pull-request/98

Commit dfaa593f fixes this pull-request

Pull-Request has been merged by mikem@redhat.com

Metadata