You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The current code coverage does not include the C++ interface and its tests. We should fix this.
We might also want to streamline the three interface tests into one test. Pro: It is cleaner. Con: The current tests serve as examples on how to use the interface for different situations. Combining the tests will make them less useful as examples.
Solution
According to @mrp089, the including the interface tests in the code coverage might just require adding the code from the main CMakeLists to the CMakeLists inside tests/test_interface/.
Additional context
No response
Code of Conduct
I agree to follow this project's Code of Conduct and Contributing Guidelines
The text was updated successfully, but these errors were encountered:
Problem
The current code coverage does not include the C++ interface and its tests. We should fix this.
We might also want to streamline the three interface tests into one test. Pro: It is cleaner. Con: The current tests serve as examples on how to use the interface for different situations. Combining the tests will make them less useful as examples.
Solution
According to @mrp089, the including the interface tests in the code coverage might just require adding the code from the main CMakeLists to the CMakeLists inside
tests/test_interface/
.Additional context
No response
Code of Conduct
The text was updated successfully, but these errors were encountered: