-
Notifications
You must be signed in to change notification settings - Fork 248
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] gtests #11896
Closed
Closed
Changes from 2 commits
Commits
Show all changes
53 commits
Select commit
Hold shift + click to select a range
cfdbe49
Prepare to build gtests
roigcarlo 0c6e1ee
Removing references to runkratos
roigcarlo 0eec903
Enable gtesting
roigcarlo 622723b
Add target for the core
roigcarlo a48cce7
Moving mpi related test utilities out of the core
roigcarlo 6caf59a
Fixing mpi dependant tests
roigcarlo 88d416b
Refactor core and mpicore tests
roigcarlo 9042bd9
Updating tests
roigcarlo da0b2a5
Adding gtest
roigcarlo b47ee25
Patch for cmake 3.27
roigcarlo 95ab5f4
Mpi tests
roigcarlo b45c82a
Adding kernel destructor and fixing mpi test with individual kernel i…
roigcarlo 14186bf
Restoring internal macros checks
roigcarlo 8c34e88
Fixing problem comparing array_1d with ZeroVector(x) from boost
roigcarlo d5550bc
Fixing cstr cmp in text excpetion
roigcarlo 35020c4
Fixing cstr cmp in code location
roigcarlo c0c3240
Test_checks now checks the checks class. Checkmate
roigcarlo 956125b
Fixing logger test (bad copy paste)
roigcarlo b064b3c
Merge branch 'master' into ci/gtests
roigcarlo d48fa49
Min cmake version
roigcarlo 74f10ef
Missing include
roigcarlo dcc5bfb
Using distributed expects
roigcarlo 28de457
Merge branch 'master' into ci/gtests
roigcarlo e455d15
Removing testing_ folder
roigcarlo d12daff
Merge branch 'master' into ci/gtests
roigcarlo ea4aba4
Merge conflicts
roigcarlo 40002cc
declared size_t
6a4aefa
compilable version without MPI
d51f66b
added testing folder
40283f5
Merge branch 'master' into ci/gtests
72aefb5
added file(GLOB_RECURSE KRATOS_TEST_SOURCES
600be3c
changed MPISynchronousPointSynchronization
ceaa848
made comparison of pointers before calling the macros
1d36556
moved cpp_tests_utilities.* from utilities to tests/cpp_tests/utilities
44a9a1c
added USE_MPI OFF to test Deltares TC
d00247f
corrected include for test_suite.h
e8c1a31
added tests/cpp_tests/*.cpp to KRATOS_TEST_UTILITIES_SOURCES to compi…
b687b5e
Merge branch 'master' into geo/gtests
9c63731
added include of test_suite.h
ae8570c
- moved cpp_tests_utilities.* to tests/test_utilities
6632c22
commented Factory, it seems it is not used in Kratos yet.
32cc295
modifications in CMakeLists.txt files
361f77b
CMake files
9a3911a
reverted libs
397c1af
modifications
5cab920
added KRATOS_SOURCE to working directories
5dd3345
removed second definition of KRATOS_TEST_CASE
ef3d3e7
changed comparison of strings
7650717
removed USE_MPI set
39fccb8
changed size_t to unsigned long to compile on Windows
76b1b5a
added/removed KRATOS_API(KRATOS_CORE)
2857cf8
commented KRATOS_TEST_CASE_IN_SUITE to avoid the macro redefinition w…
d06607d
used std::filesystem::path instead of std::string
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
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.
this change does not seem right, should work either way. Can you elaborate please?
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.
In this branch KRATOS_EXPECT_EQ is replaced with EXPECT_EQ which tests memory allocations for C strings. I guess these two lines were used to check KRATOS_EXPECT_* functionality for C strings. Perhaps, it is better to remove them now.