Documentation: kunit: Modular tests should not depend on KUNIT=y
authorGeert Uytterhoeven <geert+renesas@glider.be>
Tue, 2 May 2023 10:06:38 +0000 (12:06 +0200)
committerShuah Khan <skhan@linuxfoundation.org>
Mon, 8 May 2023 14:39:49 +0000 (08:39 -0600)
When the documentation was updated for modular tests, the dependency on
"KUNIT=y" was forgotten to be updated, now encouraging people to create
tests that cannot be enabled when the KUNIT framework itself is modular.
Fix this by changing the dependency to "KUNIT".

Document when it is appropriate (and required) to depend on "KUNIT=y".

Fixes: c9ef2d3e3f3b ("KUnit: Docs: make start.rst example Kconfig follow style.rst")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: David Gow <davidgow@google.com>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
Documentation/dev-tools/kunit/start.rst

index c736613..9619a04 100644 (file)
@@ -256,9 +256,12 @@ Now we are ready to write the test cases.
 
        config MISC_EXAMPLE_TEST
                tristate "Test for my example" if !KUNIT_ALL_TESTS
-               depends on MISC_EXAMPLE && KUNIT=y
+               depends on MISC_EXAMPLE && KUNIT
                default KUNIT_ALL_TESTS
 
+Note: If your test does not support being built as a loadable module (which is
+discouraged), replace tristate by bool, and depend on KUNIT=y instead of KUNIT.
+
 3. Add the following lines to ``drivers/misc/Makefile``:
 
 .. code-block:: make