The test method used in test_utilities.hpp is unnecessary as the same is accomplished with the implementation methods found in utilities.cpp. Removing the test specific method reduces code branching in the unit tests and tests code representative of what will be run on a system.
Test Plan
Manual testing
Modify existing tests to use the already existing method
Release Plan
check in PR
Acceptance Criteria
[ ] Code has been implemented, reviewed and merged.
[ ] Test plan has been completed
[ ] Release plan has been completed
Once all above items are checked, this story can be moved to done.
dabonnie
commented
4 years ago
Description
The test method used in test_utilities.hpp is unnecessary as the same is accomplished with the implementation methods found in utilities.cpp. Removing the test specific method reduces code branching in the unit tests and tests code representative of what will be run on a system.
Test Plan
Release Plan
Acceptance Criteria
Once all above items are checked, this story can be moved to done.