fix: excluding TYPE_CHECKING should have been the line not the branch

Author: nedbat

CHANGES.rst

@@ -23,6 +23,10 @@ upgrading your version of coverage.py.
 Unreleased
 ----------
 
+- Fix: the exclusion for ``if TYPE_CHECKING:`` was wrong: it marked the branch
+  as partial, but it should have been a line exclusion so the entire clause
+  would be excluded.
+
 - Fix: changed where .pth files are written for ``patch = subprocess``, closing
   `issue 2006`_.
 

coverage/config.py

@@ -147,6 +147,7 @@ def getregexlist(self, section: str, option: str) -> list[str]:
 DEFAULT_EXCLUDE = [
     r"#\s*(pragma|PRAGMA)[:\s]?\s*(no|NO)\s*(cover|COVER)",
     r"^\s*(((async )?def .*?)?\)(\s*->.*?)?:\s*)?\.\.\.\s*(#|$)",
+    r"if (typing\.)?TYPE_CHECKING:",
 ]
 
 # The default partial branch regexes, to be modified by the user.
@@ -160,7 +161,6 @@ def getregexlist(self, section: str, option: str) -> list[str]:
 DEFAULT_PARTIAL_ALWAYS = [
     "while (True|1|False|0):",
     "if (True|1|False|0):",
-    r"if (typing\.)?TYPE_CHECKING:",
 ]
 
 

tests/test_arcs.py

@@ -2164,8 +2164,8 @@ def test_default(self) -> None:
             if typing.TYPE_CHECKING:    # only for mypy
                 g = 12
             """,
-            lines=[1,2,3,4,5,6,7,8,9,10,11,12],
-            missing="12",
+            lines=[1,2,3,4,5,6,7,8,9,10],
+            missing="",
             branchz="23 24 56 57 89 8A BC B.",
             branchz_missing="",
         )