ci: Don't include unit tests in calculation of test coverage (#1436)

* chore: release 1.13.1

Release-As: 1.13.1

* fix: Don't include tests in calculating test coverage

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* Owl-bot approved way of removing tests inclusion in calculation

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* Moved to right location

* Blackened

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* Changed coverage level

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Sam Goodman <[email protected]>
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

Author: 3 people

noxfile.py

@@ -172,7 +172,6 @@ def default(session):
         "--quiet",
         f"--junitxml=unit_{session.python}_sponge_log.xml",
         "--cov=google",
-        "--cov=tests/unit",
         "--cov-append",
         "--cov-config=.coveragerc",
         "--cov-report=",
@@ -268,7 +267,7 @@ def cover(session):
     test runs (not system test runs), and then erases coverage data.
     """
     session.install("coverage", "pytest-cov")
-    session.run("coverage", "report", "--show-missing", "--fail-under=99")
+    session.run("coverage", "report", "--show-missing", "--fail-under=98")
 
     session.run("coverage", "erase")
 

owlbot.py

@@ -85,7 +85,7 @@
 # ----------------------------------------------------------------------------
 
 templated_files = common.py_library(
-    cov_level=99,
+    cov_level=98,
     system_test_python_versions=["3.8"],
     unit_test_python_versions=["3.6", "3.7", "3.8", "3.9"],
     unit_test_extras=["testing"],
@@ -123,4 +123,7 @@
 # Don't treat docs warnings as errors
 s.replace("noxfile.py", """["']-W["'],  # warnings as errors""", "")
 
+# Don't include tests in calculation of test coverage
+s.replace("noxfile.py", """        \"--cov=tests/unit\",""", "")
+
 s.shell.run(["nox", "-s", "blacken"], hide_output=False)

samples/model-builder/noxfile.py

@@ -171,11 +171,7 @@ def lint(session: nox.sessions.Session) -> None:
 def blacken(session: nox.sessions.Session) -> None:
     """Run black. Format code to uniform standard."""
     session.install(BLACK_VERSION)
-    python_files = [
-        path
-        for path in os.listdir(".")
-        if path.endswith(".py") or path.endswith("experiment_tracking")
-    ]
+    python_files = [path for path in os.listdir(".") if path.endswith(".py")]
 
     session.run("black", *python_files)
 
@@ -184,7 +180,6 @@ def blacken(session: nox.sessions.Session) -> None:
 # format = isort + black
 #
 
-
 @nox.session
 def format(session: nox.sessions.Session) -> None:
     """
@@ -234,7 +229,9 @@ def _session_tests(
 
     if os.path.exists("requirements-test.txt"):
         if os.path.exists("constraints-test.txt"):
-            session.install("-r", "requirements-test.txt", "-c", "constraints-test.txt")
+            session.install(
+                "-r", "requirements-test.txt", "-c", "constraints-test.txt"
+            )
         else:
             session.install("-r", "requirements-test.txt")
         with open("requirements-test.txt") as rtfile:
@@ -247,9 +244,9 @@ def _session_tests(
         post_install(session)
 
     if "pytest-parallel" in packages:
-        concurrent_args.extend(["--workers", "auto", "--tests-per-worker", "auto"])
+        concurrent_args.extend(['--workers', 'auto', '--tests-per-worker', 'auto'])
     elif "pytest-xdist" in packages:
-        concurrent_args.extend(["-n", "auto"])
+        concurrent_args.extend(['-n', 'auto'])
 
     session.run(
         "pytest",
@@ -279,7 +276,7 @@ def py(session: nox.sessions.Session) -> None:
 
 
 def _get_repo_root() -> Optional[str]:
-    """Returns the root folder of the project."""
+    """ Returns the root folder of the project. """
     # Get root of this repository. Assume we don't have directories nested deeper than 10 items.
     p = Path(os.getcwd())
     for i in range(10):