On 3/20/19 6:27 AM, Fabiano FidĂȘncio wrote:
Those tests are going to ensure our database doesn't have:
- more than one resources element for the very same arch;
- minimum resources bigger than recommended resources;
- recommended resources bigger than maximum resources;
- recommended resources bigger than network install resources;
- network resources bigger than maximum resources;

All those tests were previously part of test-os, from libosinfo

Signed-off-by: Fabiano FidĂȘncio <[email protected]>
---
  tests/osinfo.py         |  58 +++++++++++++++++++++++
  tests/test_resources.py | 101 ++++++++++++++++++++++++++++++++++++++++
  2 files changed, 159 insertions(+)
  create mode 100644 tests/test_resources.py

Reviewed-by: Cole Robinson <[email protected]>

But add that list of bits from the commit message to a block comment somewhere in the test file, since at a glance it's not immediately clear what is being tested.

For string formats I recommend using double quotes, and embedding single quotes. Using single quotes for both means you have to escape them all which just adds visual noise IMO.

- Cole

_______________________________________________
Libosinfo mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libosinfo

Reply via email to