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

just a question on tcgapi_tests.py #20

Open
gpepus opened this issue Aug 27, 2020 · 1 comment
Open

just a question on tcgapi_tests.py #20

gpepus opened this issue Aug 27, 2020 · 1 comment

Comments

@gpepus
Copy link

gpepus commented Aug 27, 2020

my experience with pytest is such that if you don't name a pytest test file starting with test_ then you break the pytest automation or you have to adjust the pytest configuration file -- which is extra work. WHY did Seagate change test_tcgapi.py to tcgapi_test.py?

@adithya-22
Copy link
Contributor

Hi Greg, the file name can either start with "test_" or end with "_test" for the pytest framework to be able to discover the tests. https://docs.pytest.org/en/reorganize-docs/new-docs/user/naming_conventions.html
The reason we changed the name is to increase readability by grouping the source code and the corresponding test file below it.

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

No branches or pull requests

2 participants