@@ -89,7 +89,7 @@ of thing a unit test never catches.
8989The renderer is tested against a real OpenGL context (` tests/renderer/ ` ). Because
9090CI runners are headless, these run on top of a software GL implementation:
9191` xvfb ` on Linux and Mesa on Windows. Keep this in mind when adding renderer
92- tests — anything that assumes a specific GPU, driver extension or exact pixel
92+ tests or anything that assumes a specific GPU, driver extension or exact pixel
9393output will be flaky.
9494
9595### Editor / manual UI tests
@@ -129,13 +129,13 @@ grouped into four GoogleTest executables, each with a matching `ctest` label:
129129
130130Tests are only built when ` NEXO_BUILD_TESTS ` is ` ON ` (the default for the
131131` default ` and ` coverage ` presets; off for ` minimal ` ). When adding a new test
132- file, register it in the relevant ` tests/<area>/CMakeLists.txt ` — ` add_executable `
132+ file, register it in the relevant ` tests/<area>/CMakeLists.txt ` , ` add_executable `
133133does not pick files up automatically.
134134
135135## Naming and file conventions
136136
137137- ** File name:** ` <Subject>.test.cpp ` , where ` <Subject> ` matches the class or unit
138- under test — e.g. ` Coordinator.test.cpp ` , ` Camera.test.cpp ` , ` Matrix.test.cpp ` .
138+ under test : e.g. ` Coordinator.test.cpp ` , ` Camera.test.cpp ` , ` Matrix.test.cpp ` .
139139- ** Header banner:** every test file starts with the standard NEXO banner block
140140 filled in with author, date and a one-line description, like every other source
141141 file in the repository. Copy it from an existing test.
@@ -209,7 +209,7 @@ expected to come with tests that keep coverage at or above 80%.
209209
210210> [ !CAUTION]
211211> 80% is a floor, not a goal. Coverage on its own proves only that a line ran,
212- > not that its behaviour was checked — a green gate with shallow assertions is
212+ > not that its behaviour was checked, a green gate with shallow assertions is
213213> worse than the same number with tests that actually exercise behaviour.
214214
215215## Continuous integration
@@ -236,14 +236,14 @@ sonar-scanner); the other legs still build and run the tests.
236236
237237- Add tests for the logic you introduce or change, in the same PR.
238238- Make sure the whole suite passes locally before opening the PR.
239- - If you cannot reasonably test something, say so in the PR description — the
239+ - If you cannot reasonably test something, say so in the PR description. The
240240 pull request template has a dedicated section for this, including a
241241 "I need help with writing tests" option. Use it; that is a valid answer.
242242
243243** Core team / maintainers**
244244
245245- Keep the SonarCloud quality gate green and unblock it when it flags real gaps.
246- - Review the * tests* in a PR, not only the production code — coverage of the right
246+ - Review the * tests* in a PR, not only the production code, coverage of the right
247247 behaviour is part of the review.
248248- Add or extend integration coverage when landing major features, and keep the
249249 editor checklists in sync with UI changes.
@@ -254,7 +254,7 @@ sonar-scanner); the other legs still build and run the tests.
254254The following are not expected to carry unit-test coverage and are excluded from
255255coverage metrics:
256256
257- - The editor (` editor/ ` ) — validated through the manual checklists in ` tests/editor/ ` .
257+ - The editor (` editor/ ` ) : validated through the manual checklists in ` tests/editor/ ` .
258258- The tests and examples themselves (` tests/ ` , ` examples/ ` ).
259259- Generated or third-party code vendored under ` vcpkg/ ` and ` external/ ` .
260260- Thin, logic-free glue (entry points such as ` main ` , trivial getters/setters).
@@ -265,7 +265,7 @@ sign-off in the PR.
265265## Writing a test: a worked example
266266
267267A minimal GoogleTest fixture following the conventions above. The banner header
268- is abbreviated here for readability — copy the full one from an existing file.
268+ is abbreviated here for readability, copy the full one from an existing file.
269269
270270``` c++
271271// // Coordinator.test.hpp //////////////////////////////////////////////////////
0 commit comments