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
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?
Add test cases for delete_build() function
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
Adding more test cases.
The test for key 'last_used' had to be broken down separately since it contained a time constraint check.