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

Rename utility functions to fix running tests with 'nose' #75

Merged
merged 1 commit into from
Feb 5, 2024

Conversation

pjbriggs
Copy link
Member

@pjbriggs pjbriggs commented Feb 5, 2024

Rename the create_test_file and delete_test_file functions in test/common.py to create_file and delete_file (i.e. remove the _test component), to fix errors when running unit tests with nose. Without the changes, nosetests treats these helper function as test cases (which then fail).

@pjbriggs pjbriggs added the bug label Feb 5, 2024
@pjbriggs pjbriggs merged commit 4854072 into devel Feb 5, 2024
8 checks passed
@pjbriggs pjbriggs deleted the fix-nosetests branch February 5, 2024 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant