Re: [osinfo-db PATCH v2] tests: Add Resources tests

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 <fidencio@xxxxxxxxxx>
---
  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 <crobinso@xxxxxxxxxx>

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
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Fedora Users]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux