-
Notifications
You must be signed in to change notification settings - Fork 16
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
Cleanup test suite #161
Merged
Merged
Cleanup test suite #161
Changes from 62 commits
Commits
Show all changes
74 commits
Select commit
Hold shift + click to select a range
315849e
split up the read_main_input function
yardasol f13237e
rename the helper function for read_main_input
yardasol 9407d2f
split up app.run()
yardasol 02cc71b
split up app.run()
yardasol ea39880
Merge branch 'master' into cleaning-exercise
yardasol a96216c
add back in missing content from master merge
yardasol fc0b0ae
move tests directory outside of the saltproc module
yardasol 22da911
move unit tests to their own directory; move input and data files to …
yardasol cb5688e
split test_app into unit test and integration test components
yardasol eb1adce
move run to top of file
yardasol d8594a4
remove global variables; rename variables
yardasol 7ff3ee4
add arguments to some of the app functions
yardasol 7c1536e
Merge branch 'cleaning-exercise' into cleanup-tests
yardasol d3d56ec
reorder functions
yardasol 019eed6
replace os.path with pathlib.Path; fix import statements
yardasol 2679fbf
get test_app unit test working
yardasol c804f8c
move pytest.ini to repo base
yardasol ff31b67
remove tests packages from saltproc package
yardasol 3188df9
add missing h5 files
yardasol 3749f09
add openmc option to _create_depcode_object
yardasol 327118b
fix _create_simulation_object function signature
yardasol 7eb9e1d
cleanup run_no_reproc
yardasol 56aabaa
cleanup run_no_reproc
yardasol 9225563
Merge remote-tracking branch 'origin/cleanup-tests' into cleanup-tests
yardasol c42be3c
cleanup run_no_reproc
yardasol 8f47876
Merge remote-tracking branch 'origin/cleanup-tests' into cleanup-tests
yardasol ac7d01e
Merge remote-tracking branch 'origin/cleanup-tests' into cleanup-tests
yardasol 0e7e9bf
Merge remote-tracking branch 'origin/cleanup-tests' into cleanup-tests
yardasol a66deaa
combine 4
yardasol f71ecd6
patch last commit
yardasol 1db6aac
Merge remote-tracking branch 'origin/cleanup-tests' into cleanup-tests
yardasol d107d88
expand test_suite scope
yardasol b13a08f
remove duplicate input files
yardasol 827787f
cleanup run_constant_reprocessing
yardasol d667372
Merge remote-tracking branch 'origin/cleanup-tests' into cleanup-tests
yardasol 0abb5b2
cleanup run_constant_reprocessing
yardasol 34bb854
rename some files in run_no_reprocessing for consistency
yardasol e98b3ba
make input file and reference data file names for test suite more clear
yardasol fd65385
pass filename changes to test_app.py; pytest and import statement adj…
yardasol af5bc03
cleanup basic_reprocessing integration test
yardasol b55fc39
remove cruft from run_constant_reprocessing
yardasol 10790a6
update openmc test files
yardasol ad3fd35
update serpent test files
yardasol 3f0f190
update handling of geometry file for DepcodeOpenMC
yardasol b77afc9
grammar fix in error message
yardasol 22628ea
add conftest.py file; rework test_depcode.py
yardasol d116d56
overhaul sparger, simulator, separator unit tests
yardasol fd02250
overhaul test_materialflow.py
yardasol a901325
cleanup fixtures
yardasol 05a3793
add database storage integration test
yardasol 8f3aea3
fix typo in test_input.json for run_no_reprocessing
yardasol 0af3f9f
fix major structural conflicts
yardasol 8ec7fa1
Merge remote-tracking branch 'upstream/master' into cleanup-tests
yardasol 267ccc1
update test-saltproc workflow
yardasol f9a8ddb
fix docstrings
yardasol 32f79bd
pep8 fixes
yardasol c4d1abe
fix test-saltproc workflow
yardasol 1867e4e
update syntax in docpages
yardasol 97af5a3
update readme
yardasol 616348a
update release notes
yardasol 2bbd049
fix triggering paths for test-saltproc workflow
yardasol c94b445
fix new syntax inconsistencies
yardasol ba49bca
fix new syntax inconsistencies
yardasol 49be51f
remove cruft terminal output
yardasol c0bfe49
process_input_file -> process_file; path_input_file -> dot_file
yardasol 4d849ca
Add descriptions for object fixtures
yardasol 94e749d
remove old comments
yardasol bba890f
Remov cruft pytest marker
yardasol 9b06262
Merge remote-tracking branch 'origin/cleanup-tests' into cleanup-tests
yardasol a1315fc
reorder function definitions to be consitent w style of software
yardasol 3a6135a
expand testing instructions
yardasol c0bcedd
Merge remote-tracking branch 'origin/cleanup-tests' into cleanup-tests
yardasol 8e2192b
fix test input file names
yardasol 51e7a8d
remove paths to share dir is tools/ci scripts
yardasol File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,5 @@ | ||
[pytest] | ||
python_files = test*.py | ||
munkm marked this conversation as resolved.
Show resolved
Hide resolved
|
||
filterwarnings = | ||
error | ||
ignore::Warning |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
If one wants to run just the unit tests and not the integration tests, how would they do that here?
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.
You can specify the path to the test(s) you want when calling
pytest
. Alternatively, you can use the--ignore
flag followed by the path(s) to the test(s) you don't want to test.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.
Ok, I think that's worth explicitly stating here for people who may want to run unit tests locally when doing development. Can you please add it back in with the updated paths?