dougshidong / PHiLiP

Parallel High-Order Library for PDEs through hp-adaptive Discontinuous Galerkin methods
Other
45 stars 37 forks source link

Unify Navier-Stokes and RANS source term unit test #184

Open cyanmcg opened 1 year ago

cyanmcg commented 1 year ago

tests/unit_tests/navier_stokes_unit_test/reynolds_averaged_navier_stokes_sa_neg_manufactured_solution_source.cpp is largely a copy paste from tests/unit_tests/navier_stokes_unit_test/navier_stokes_manufactured_solution_source.cpp with the difference of calling a different physics object and explicitly creating a manufactured solution. Would be better to have some sort of input variable that switches the original file between RANS and NS functionality; then in the CMakeList.txt, you pass the appropriate argument when you define the ctest's "COMMAND"; see tests/unit_tests/grid/gmsh_reader/gmsh_reader.cpp and tests/unit_tests/grid/gmsh_reader/CMakeLists.txt for reference

_Originally posted by @jbrillon in https://github.com/dougshidong/PHiLiP/pull/172#discussion_r988171234_