[libcxx-commits] [libcxx] Update testing documentation with testing guidelines. (PR #87928)
Louis Dionne via libcxx-commits
libcxx-commits at lists.llvm.org
Thu Apr 11 11:44:11 PDT 2024
================
@@ -180,6 +180,42 @@ The tests of libc++ are stored in libc++'s testing related subdirectories:
``libcxx/test/libcxx``. The structure of this directories follows the
structure of ``libcxx/test/std``.
+Principles of testing
+---------------------
+
+Tests are a practical way to validate the correctness of the code. As such, they contain pragmatic trade offs between
+the cost of writing and maintaining the tests and the value they provide. Please consider the following principles when
+writing tests:
+
+- **Consider the next reader**
+
+ Tests should be obvious to the future reader. Avoid too much boiler plate or other
+ distractions. Ensure each test has enough context to understand what it is testing. Avoid gratuitous use of advanced
+ test features or abstractions.
----------------
ldionne wrote:
Add: We also strongly encourage the intent of a test and the reason for its existence to be documented in plain text comments.
This would address Mark's comment below.
https://github.com/llvm/llvm-project/pull/87928
More information about the libcxx-commits
mailing list