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
A number of tests in the Core test directory identify a specific token via a /* testMarker */ like comment in the inc file(s).
When the test case file inc gets larger, it is quite easy to accidentally have a duplicate "test marker" comment in the file (non unique name) and the test would end up not testing what it should be testing and this would go unnoticed.
It would be good to have a minimal test - probably with a helper function running the actual test code in the AbstractMethodUnitTest class - to verify that an inc file doesn't contain duplicate test markers.
This would be a QA check on the tests, but can be run as a test itself.
The text was updated successfully, but these errors were encountered:
A number of tests in the
Core
test directory identify a specific token via a/* testMarker */
like comment in theinc
file(s).When the test case file
inc
gets larger, it is quite easy to accidentally have a duplicate "test marker" comment in the file (non unique name) and the test would end up not testing what it should be testing and this would go unnoticed.It would be good to have a minimal test - probably with a helper function running the actual test code in the
AbstractMethodUnitTest
class - to verify that aninc
file doesn't contain duplicate test markers.This would be a QA check on the tests, but can be run as a test itself.
The text was updated successfully, but these errors were encountered: