Closed nickbianco closed 5 months ago
(the diff's big because things were moved into anonymous namespaces etc?)
Yes, essentially. The helper functions were also at the bottom of the cpp file, so it wasn't the standard "wrap existing test functions with Catch2 sections" type of change.
Fixes issue #3555
Brief summary of changes
Main test conversion is self-explanatory. Also updated the headers in
testModelProcessor.cpp
.Testing I've completed
Looking for feedback on...
CHANGELOG.md (choose one)
This change is![Reviewable](https://reviewable.io/review_button.svg)