You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Per our discussion on the dnf charm library pull request here #75 (comment), we realized that there is potentially a better way to launch tests without having to use pytest's built-in flag --ignore=... to ignore specific tests. In the discussion thread, it was suggested to use @pytest.mark.<test_suite> with pytest -m <test_suite>.
I also wanted to suggest potentially using cleantest for helping with the overhaul of the tests for the charm libraries. I noticed in the tox.ini file that there is a standalone integration environment for just setting up an LXD test instance:
cleantest can handle both bringing up and tearing down the Ubuntu test instance. This way the LXD logic does not have to live inside of tox.ini and the integration environment can be dedicated for just launching the integration tests.
The text was updated successfully, but these errors were encountered:
Per our discussion on the dnf charm library pull request here #75 (comment), we realized that there is potentially a better way to launch tests without having to use pytest's built-in flag
--ignore=...
to ignore specific tests. In the discussion thread, it was suggested to use@pytest.mark.<test_suite>
withpytest -m <test_suite>
.I also wanted to suggest potentially using cleantest for helping with the overhaul of the tests for the charm libraries. I noticed in the tox.ini file that there is a standalone integration environment for just setting up an LXD test instance:
operator-libs-linux/tox.ini
Lines 70 to 88 in d0ebc9f
cleantest can handle both bringing up and tearing down the Ubuntu test instance. This way the LXD logic does not have to live inside of tox.ini and the integration environment can be dedicated for just launching the integration tests.
The text was updated successfully, but these errors were encountered: