pki icon indicating copy to clipboard operation
pki copied to clipboard

Add pytests to CMake Test matrix

Open SilleBille opened this issue 4 years ago • 2 comments

This patch:

  • includes pytests under tests/python/ dir in the CMake test matrix. Though the unit tests exited, they were never executed.

  • Fixes the deprecated pytest methods.

Note

The pytests were added to the CMake test matrix instead of adding them to the spec file. This ensures that the pytests are executed in other platforms (Debian).

Signed-off-by: Dinesh Prasanth M K <[email protected]>

SilleBille avatar May 18 '20 23:05 SilleBille

Hmmm. pytest fails because it fails to access the modules.. The packages are not installed yet and so, the test fails. Any thoughts? @edewata @cipherboy

SilleBille avatar May 18 '20 23:05 SilleBille

digging deeper, this can be achieved using tox. We have several setup.py files lying around and I will resubmit this PR after cleaning them up.

The only caveat is that, it's difficult for me to add this to CMake. Do you oppose executing pytest in spec files?

SilleBille avatar May 19 '20 01:05 SilleBille