|
FhSim
3.1.0
Marine systems simulation
|
| ID | 0075 |
| Class | DOCUMENTATION |
| Severity | 1 |
| Status | ready |
| Models | — |
| Found | 2026-09-26, OpenWiki pass |
| Decision needed |
doc/user/testing_report.md:4 dates the report to d29101c, 2026-09-24, and lines 12-13 say "No test in this library is `DISABLED_` or `_NEEDS_GUI` (`grep -rn "DISABLED_|_NEEDS_GUI"
tests/*.cpp` is empty), so every test that exists runs in CI." At HEAD grep -rn "TEST(.*_NEEDS_GUI" tests finds seven:
tests/ValuesDisplay_Test.cpp:102,110,116 (ValuesDisplay_AcceptsEveryCorner_NEEDS_GUI, _RejectsAnUnknownCorner_NEEDS_GUI, _SetsUpOneInportPerSignal_NEEDS_GUI);tests/Vessel_Test.cpp:122 (Vessel_RendersWithoutMeshName_NEEDS_GUI);tests/SeineOperation_Test.cpp:218,226 (SeineOperation_ShowStatusDefaultsToFalse_NEEDS_GUI, SeineOperation_AcceptsShowStatus_NEEDS_GUI);tests/TrawlDoor_Test.cpp:58 (TrawlDoor_RendersWithoutMesh_NEEDS_GUI).CI runs with --gtest_filter=-*_NEEDS_GUI (quoted at testing_report.md:11), so these seven do not run there. The report's own row for ValuesDisplay (testing_report.md:72) already mentions a _NEEDS_GUI case, contradicting line 12.
Readers only: the report overstates what CI covers.
Regenerate or amend the report at current HEAD: list the seven _NEEDS_GUI tests, say that CI skips them, and update the Vessel, SeineOperation and TrawlDoor rows.
The grep above, compared with the report's text.
None; documentation only.