t/README: Add a note about the dangers of coverage chasing

[thirdparty/git.git] / t / README

diff --git a/t/README b/t/README
index 468876de0d3db10d7c6b39fcade93ad998f2d0a2..e49534a1bf48017b48eaf8a93f6b7c226dad8c98 100644 (file)
--- a/t/README
+++ b/t/README
@@ -271,6 +271,15 @@ Do:
  - Check the test coverage for your tests. See the "Test coverage"
    below.
 
+   Don't blindly follow test coverage metrics, they're a good way to
+   spot if you've missed something. If a new function you added
+   doesn't have any coverage you're probably doing something wrong,
+   but having 100% coverage doesn't necessarily mean that you tested
+   everything.
+
+   Tests that are likely to smoke out future regressions are better
+   than tests that just inflate the coverage metrics.
+
 Don't:
 
  - exit() within a <script> part.