search

Exawind / openturbine

A flexible multibody structural dynamics code for wind turbines
https://exawind.github.io/openturbine/
MIT License
18 stars 7 forks source link

Parameterize the unit test suite #115

Closed faisal-bhuiyan closed 2 months ago

faisal-bhuiyan commented 7 months ago

Many of the tests in OpenTurbine are identical in function (example in the following comment). Parameterization would help to comprehend the the extents of the testing better.

faisal-bhuiyan commented 7 months ago

Minor nitpick: some of these tests would be good candidates for a parameterization refactor. Example: https://github.com/Exawind/openturbine/blob/45aab4dbf501976640405be9250204455447dcd6/tests/unit_tests/gen_alpha_poc/test_math_utilities.cpp#L8-L30

Originally posted by @psakievich in https://github.com/Exawind/openturbine/discussions/108#discussioncomment-8400136

faisal-bhuiyan commented 2 months ago

This is a general suggestion which I believe we are complying with in our test suite at this point in time. Closing.