Change exclusion syntax from ./tests to tests/*

Copilot · nathanjmcdougall · Copilot · commit ab7bfb5ca0b0 · 2025-11-05T19:36:28.000Z
Co-authored-by: nathanjmcdougall &lt;18602289+nathanjmcdougall@users.noreply.github.com&gt;
diff --git a/src/usethis/_integrations/sonarqube/config.py b/src/usethis/_integrations/sonarqube/config.py
@@ -79,8 +79,8 @@ def get_sonar_project_properties() -> str:  # noqa: PLR0912
     if source_dir_str == ".":
         sources = "./"
         # When using flat layout, exclude tests directory to avoid double indexing
-        if "./tests" not in exclusions:
-            exclusions.insert(0, "./tests")
+        if "tests/*" not in exclusions:
+            exclusions.insert(0, "tests/*")
     else:
         sources = f"./{source_dir_str}"
 
diff --git a/tests/usethis/_integrations/sonarqube/test_sonarqube_config.py b/tests/usethis/_integrations/sonarqube/test_sonarqube_config.py
@@ -100,7 +100,7 @@ def test_different_python_version(self, tmp_path: Path):
 sonar.tests=./tests
 sonar.python.coverage.reportPaths=coverage.xml
 sonar.verbose=false
-sonar.exclusions=./tests
+sonar.exclusions=tests/*
 """
         )
 
@@ -129,7 +129,7 @@ def test_no_pin_python(self, tmp_path: Path):
 sonar.tests=./tests
 sonar.python.coverage.reportPaths=coverage.xml
 sonar.verbose=false
-sonar.exclusions=./tests
+sonar.exclusions=tests/*
 """
         )
 
@@ -159,7 +159,7 @@ def test_different_project_key(self, tmp_path: Path):
 sonar.tests=./tests
 sonar.python.coverage.reportPaths=coverage.xml
 sonar.verbose=false
-sonar.exclusions=./tests
+sonar.exclusions=tests/*
 """
         )
 
@@ -192,7 +192,7 @@ def test_set_verbose_true(self, tmp_path: Path):
 sonar.tests=./tests
 sonar.python.coverage.reportPaths=coverage.xml
 sonar.verbose=true
-sonar.exclusions=./tests
+sonar.exclusions=tests/*
 """
         )
 
@@ -239,7 +239,7 @@ def test_patch_version_ignored(self, tmp_path: Path):
 sonar.tests=./tests
 sonar.python.coverage.reportPaths=coverage.xml
 sonar.verbose=false
-sonar.exclusions=./tests
+sonar.exclusions=tests/*
 """
         )
 
@@ -276,7 +276,7 @@ def test_exclusions(self, tmp_path: Path):
 sonar.tests=./tests
 sonar.python.coverage.reportPaths=coverage.xml
 sonar.verbose=false
-sonar.exclusions=./tests, **/Dockerfile, src/notebooks/**/*
+sonar.exclusions=tests/*, **/Dockerfile, src/notebooks/**/*
 """
         )
 
@@ -307,7 +307,7 @@ def test_different_coverage_file_location(self, tmp_path: Path):
 sonar.tests=./tests
 sonar.python.coverage.reportPaths=./test-reports/cov.xml
 sonar.verbose=false
-sonar.exclusions=./tests
+sonar.exclusions=tests/*
 """
         )
 
@@ -325,7 +325,7 @@ def test_missing_coverage_file_location_error(self, tmp_path: Path):
                 get_sonar_project_properties()
 
     def test_flat_layout_exclusions_already_has_tests(self, tmp_path: Path):
-        # When using flat layout and ./tests is already in exclusions,
+        # When using flat layout and tests/* is already in exclusions,
         # it should not be added again.
 
         with change_cwd(tmp_path), PyprojectTOMLManager():
@@ -337,7 +337,7 @@ def test_flat_layout_exclusions_already_has_tests(self, tmp_path: Path):
             )
             PyprojectTOMLManager().set_value(
                 keys=["tool", "usethis", "sonarqube", "exclusions"],
-                value=["./tests", "**/Dockerfile"],
+                value=["tests/*", "**/Dockerfile"],
             )
             PyprojectTOMLManager().set_value(
                 keys=["tool", "coverage", "xml", "output"], value="coverage.xml"
@@ -357,7 +357,7 @@ def test_flat_layout_exclusions_already_has_tests(self, tmp_path: Path):
 sonar.tests=./tests
 sonar.python.coverage.reportPaths=coverage.xml
 sonar.verbose=false
-sonar.exclusions=./tests, **/Dockerfile
+sonar.exclusions=tests/*, **/Dockerfile
 """
         )
 
