search

QuTech-Delft / OpenSquirrel

A flexible (Python-based) quantum program compiler
Apache License 2.0
4 stars 1 forks source link

215 unittest to pytest #217

Closed S-Linde closed 2 months ago

S-Linde commented 2 months ago

This pull request converts the unittest style tests to the pytest format. All tests have been updated with the exception of the tests for the quantify scheduler. These tests should probably be redesigned to use quantify_scheduler instead of mocking its behavior.

Closes #215

S-Linde commented 2 months ago

Great work, Stan!

Just a few quesstion:

  • Are there any changes in the way we have to run tests now? Or we still run them with python -m pytest -vv or pytest -vv? I suppose yes.
  • Are we introducing any dependencies? I suppose not.
  • Any doc file you think needs updating? E.g., README.md, or CONTRIBUTING.md.

Thanks!

No, everything stays the same. We already had pytest as a dependency and we used it to run our tests, so nothing is changed in those regards.