diff options
| author | WBoerenkamps <wrj.boerenkamps@student.avans.nl> | 2024-11-20 20:31:11 +0100 |
|---|---|---|
| committer | WBoerenkamps <wrj.boerenkamps@student.avans.nl> | 2024-11-20 20:31:11 +0100 |
| commit | 3a5a61399cea453c5fc25e3b999ef9a751fd4448 (patch) | |
| tree | 5fd9d88d7e71696ffd7b52e956b44c3ade92dd81 /contributing.md | |
| parent | 5ec30d8915debef409f6db6f8ee9b822fb24d46b (diff) | |
| parent | b6aea9de9c9c8d86856bcd6c3b521e385bc00a69 (diff) | |
Merge branch 'master' of https://github.com/lonkaars/crepe into wouter/events
Diffstat (limited to 'contributing.md')
| -rw-r--r-- | contributing.md | 23 |
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: |