]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
doc/dev: update unit tests 2 of 7 38134/head
authorZac Dover <zac.dover@gmail.com>
Tue, 17 Nov 2020 14:02:32 +0000 (00:02 +1000)
committerZac Dover <zac.dover@gmail.com>
Wed, 18 Nov 2020 08:13:35 +0000 (18:13 +1000)
This commit updates the "What does 'make
check' mean" section of the "Unit Tests" chapter
of the Developer Guide. It makes the wording a
bit less editorial and attempts to reduce the
reader's cognitive load.

Signed-off-by: Zac Dover <zac.dover@gmail.com>
doc/dev/developer_guide/tests-unit-tests.rst

index 06a9d063dac3e8a764308387a2aafc5e58f5f4c9..e39f4edaa63f074d6a01d988b110c52789eec888 100644 (file)
@@ -12,33 +12,36 @@ and multi-machine clusters to run.
 What does "make check" mean?
 ----------------------------
 
-After compiling Ceph, the code can be run through a battery of tests covering
-various aspects of Ceph. For historical reasons, this battery of tests is
-often referred to as ``make check`` even though the actual command used to run
-the tests is now ``ctest``. For inclusion in this battery of tests, a test
-must:
+After Ceph has been compiled, its code can be run through a battery of
+tests that cover various aspects of Ceph. For historical reasons, this
+battery of tests is often referred to as ``make check`` even though the
+actual command used to run the tests is now ``ctest``. In order to be
+included in this battery of tests, a test must:
 
 * bind ports that do not conflict with other tests
 * not require root access
 * not require more than one machine to run
 * complete within a few minutes
 
-For simplicity, we will refer to this class of tests as "make check tests" or
-"unit tests", to distinguish them from the more complex "integration tests"
-that are run via the `teuthology framework`_.
+For the sake of simplicity, this class of tests is referred to as "make
+check tests" or "unit tests". This is meant to distinguish these tests from
+the more complex "integration tests" that are run via the `teuthology
+framework`_.
 
-While it is possible to run ``ctest`` directly, it can be tricky to correctly
-set up your environment. Fortunately, a script is provided to make it easier
-run the unit tests on your code. It can be run from the top-level directory of
-the Ceph source tree by doing::
+While it is possible to run ``ctest`` directly, it can be tricky to
+correctly set up your environment for it. Fortunately, a script is provided
+to make it easier to run the unit tests on your code. This script can be
+run from the top-level directory of the Ceph source tree by running the
+following command:
 
-    $ ./run-make-check.sh
+.. prompt:: bash $
 
+   ./run-make-check.sh
 
 You will need a minimum of 8GB of RAM and 32GB of free disk space for this
 command to complete successfully on x86_64 (other architectures may have
 different constraints). Depending on your hardware, it can take from 20
-minutes to three hours to complete, but it's worth the wait.
+minutes to three hours to complete.
 
 How unit tests are declared
 ---------------------------