-
-
Notifications
You must be signed in to change notification settings - Fork 346
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
Require pytest for test runners #1167
Conversation
@speth and @bryanwweber … this should be it (last thing I added was a meaningful error with instructions if |
64d78c5
to
f07ea4c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. I'll leave this open for the moment to see if @bryanwweber has any thoughts.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM thanks @ischoegl!
cantera_root = os.path.relpath(__file__).split(os.sep)[:-1] + ['..', '..'] | ||
os.chdir(os.sep.join(cantera_root + ['test', 'work'])) | ||
CANTERA_ROOT = Path(__file__).parents[2] | ||
os.chdir(str(CANTERA_ROOT / "test" / "work")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not a big fan of chdir
, I find that it leads to confusion about where files will be placed. Hopefully we can just get rid of this file entirely someday, since we don't need all the stuff that we had for compatibility with unittest
that are in this file.
Changes proposed in this pull request
pytest
for Python test suitepytest
features can be used in unit testsIf applicable, fill in the issue number this pull request is fixing
Addresses Cantera/enhancements#128
Checklist
scons build
&scons test
) and unit tests address code coverage