-
Notifications
You must be signed in to change notification settings - Fork 247
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
[GeoMechanicsApplication] Re-enable settlement test using the C++ route #12877
Merged
Merged
Changes from all commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
1b5e75f
Run `settlement_workflow.py` in subdirectory `python`
avdg81 4e57780
Renamed a Python test script and a test class
avdg81 0335747
Added an incomplete Python test script for the C++ route
avdg81 cf10af3
Introduced an intermediate test fixture
avdg81 790e9a0
Added two new attributes to the base test fixture
avdg81 a4fa4ac
Renamed a Python test script
avdg81 344e420
Added a test fixture for the C++ route
avdg81 cd2c352
Fixed an incorrect Python module name
avdg81 c844a52
Also corrected an import statement
avdg81 022daac
Added more checks for the C++ route
avdg81 32386ad
Add a lambda for the "should cancel" callback
avdg81 c4e4f36
Attempt to fix test failures
avdg81 064d8fe
Use proper callback functions in the C++ route test
avdg81 52f6e09
Defined a map that contains the time steps to check
avdg81 ba5bc8f
Added time, output filename and expected displacements
avdg81 08b3fe6
Extracted a method for checking the displacements
avdg81 044f63d
Extracted a method for defining the expected displacements
avdg81 de7ea3f
Defined methods for checking the nodal stresses
avdg81 99090e9
Also the test case for the C++ route now checks the nodal stresses
avdg81 8b0cf23
Updated the descriptions of the test fixtures
avdg81 a76b3e0
Removed the C++ unit test for the settlement workflow
avdg81 4f18573
Removed the now unused original output files
avdg81 e0bb35d
Merge remote-tracking branch 'origin/master' into geo/12865-re-enable…
avdg81 249c737
Use `def_static` rather than `def` to bind static member functions
avdg81 d0b280e
Removed an unused `#include`
avdg81 029e63b
Removed an unused test script
avdg81 cedcf3f
Added a missing `tearDown` call of the base class
avdg81 b8383d2
Moved a function definition to another line in the file
avdg81 3edbf35
Don't use a loop counter when we don't need one
avdg81 b981d8f
Processed the review suggestions
avdg81 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
63 changes: 0 additions & 63 deletions
63
applications/GeoMechanicsApplication/tests/cpp_tests/test_settlement_workflow.cpp
This file was deleted.
Oops, something went wrong.
210 changes: 171 additions & 39 deletions
210
applications/GeoMechanicsApplication/tests/settlement_workflow.py
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
Oops, something went wrong.
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.
For those interested, this include is needed such that python (lambda) functions can be converted to C++ ones (since we pass functions to the exported RunStage function)