greenwave

#190 Fix testing correct NVR and omit testing ID
Closed by lholecek. Opened by lholecek.
lholecek/greenwave omit-id-test  into  master

Download 190.patch

Fixes following test run:

TEST=true python -m pytest -k test_make_a_decision_with_verbose_flag

The problem is that database is not dropped after test case is executed
so even the ID counters are not reset.

Normal test execution worked since it expected some data added by
previous test case.

It could be good idea to randomize test case execution order.

Oh yeah I ran into this one in PR#184 as well. My solution there was to just assert that the set of results matches the response we got from ResultsDB when we originally inserted them.

https://pagure.io/greenwave/pull-request/184#6_281

But :+1: if you want to merge this now anyway.

Oh yeah I ran into this one in PR#184 as well. My solution there was to just assert that the set of results matches the response we got from ResultsDB when we originally inserted them.

Nice. Then I'm closing this one.

Pull-Request has been closed by lholecek
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.