aboutsummaryrefslogtreecommitdiff
path: root/contributing.md
diff options
context:
space:
mode:
authorheavydemon21 <nielsstunnebrink1@gmail.com>2024-11-20 14:52:52 +0100
committerheavydemon21 <nielsstunnebrink1@gmail.com>2024-11-20 14:52:52 +0100
commita01c2dc46daeffa0a5c92230d1513047a072c04d (patch)
treeb0df9754cf9c8bd900594816f9bdac20679308c3 /contributing.md
parent81664b46f5cb7c28d629ddafe4cbea7e717d62d2 (diff)
parentf78e8011ee77ba5303a5e608cc81c3b303d35943 (diff)
Merge branch 'master' into niels/RenderingParticle
Diffstat (limited to 'contributing.md')
-rw-r--r--contributing.md23
1 files changed, 22 insertions, 1 deletions
diff --git a/contributing.md b/contributing.md
index 5b0c79d..9c95851 100644
--- a/contributing.md
+++ b/contributing.md
@@ -20,7 +20,7 @@ that you can click on to open them.
# Code style
- Formatting nitty-gritty is handled by clang-format/clang-tidy (run `make
- format` in the root folder of this repository to format all sources files)
+ format` or `make lint`)
- <details><summary>
ASCII only
</summary><table><tr><th>Good</th><th>Bad</th></tr><tr><td>
@@ -798,6 +798,27 @@ that you can click on to open them.
resolving merge conflicts when multiple sources were added by different
people to the same CMakeLists.txt easier.
+## GoogleTest-specific
+
+- Unit tests are not *required* to follow all code standards
+- <details><summary>
+ Private/protected members may be accessed using preprocessor tricks
+ </summary>
+
+ ```cpp
+ // include unrelated headers before
+
+ #define private public
+ #define protected public
+
+ // headers included after *will* be affected
+ ```
+ </details>
+- Each test source file defines tests within a single test suite (first
+ parameter of `TEST()` / `TEST_F()` macro)
+- Test source files match their suite name (or test fixture name in the case of
+ tests that use a fixture)
+
# Structure
- Files are placed in the appropriate directory: