-
Notifications
You must be signed in to change notification settings - Fork 226
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
Try to add the framework for multi-test again #349
Conversation
…CorrectionComplex
note that the cmake file is modified for using the unit test
start adding the testing for K and Epsilon equations
# Conflicts: # tests/CMakeLists.txt
…ions, modify kp with friction velocity, change the position of WALLFUNC
… that their wall viscous is not zero
@FengWang3119 I think that the reason for failed CI is because your code is inconsistent with master branch, |
@FengWang3119 It seems that you have mixed up several things in one branch which is not typical git based developing practice. |
I have updated my code to the latest one. I have no idea about the git deloping process, I will try to learn it |
… previously it can not be paralleled, now it seems can
@FengWang3119 Many compiling errors in linux CI is due to the strict compiling rule in linux (such as initialization order and unused variables.) CI fails in Windows due to some code |
Add the multi-tests framework by using Google Test Fixture, the basic setting is constructed once and several tests can be conducted together.
The code can run on my computer, hope it can pass CI, the testing fixture part is in folder "test_turbulent_module"