Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move all test to test folder + pytest parametrization #67

Open
EnricoTrizio opened this issue May 22, 2023 · 2 comments
Open

Move all test to test folder + pytest parametrization #67

EnricoTrizio opened this issue May 22, 2023 · 2 comments
Labels
documentation Improvements or additions to documentation

Comments

@EnricoTrizio
Copy link
Collaborator

Up to now, some tests are at the end of the specific files and some are defined in the test folder.

We should move all of them to the test folder and use whenever needed the pytest parametrize functions

@EnricoTrizio EnricoTrizio added the documentation Improvements or additions to documentation label May 22, 2023
@EnricoTrizio
Copy link
Collaborator Author

Maybe we should also remove all the unused prints once we move everything to the test folder or replace them with asserts + intelligible error messages

@EnricoTrizio
Copy link
Collaborator Author

We should also cover in the tests the several checks that we may have that could raise errors
i.e. using pytest.raises

with pytest.raises(ValueError, match='What an horrible error!'):
      some_command_meant_to_raise_an_error()

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

1 participant