Fix CI test collection: remove conflicting tests/__init__.py and add testpaths

GladwinJohnson · Copilot · GladwinJohnson · commit ededbe2c30a3 · 2026-04-07T06:55:16.000-07:00
- Remove msal-key-attestation/tests/__init__.py to prevent namespace
  conflict with the main tests/ package (caused all 24 existing tests
  to fail with 'ModuleNotFoundError: No module named tests.test_*')
- Add [tool:pytest] testpaths=tests to setup.cfg so bare 'pytest' only
  collects the main test suite
- Fix remaining CodeQL alert: avoid printing result dict data on failure

Co-authored-by: Copilot &lt;223556219+Copilot@users.noreply.github.com&gt;
diff --git a/msal-key-attestation/tests/__init__.py b/msal-key-attestation/tests/__init__.py
diff --git a/sample/msi_v2_sample.py b/sample/msi_v2_sample.py
@@ -79,9 +79,7 @@ def main():
     )
 
     if "access_token" not in result:
-        print("ERROR: Token acquisition failed:",
-              result.get("error", "unknown"), "-",
-              result.get("error_description", "no description"))
+        print("ERROR: Token acquisition failed. Check logs for details.")
         sys.exit(1)
 
     token_type = result.get("token_type", "unknown")
diff --git a/setup.cfg b/setup.cfg
@@ -72,3 +72,6 @@ broker =
 [options.packages.find]
 exclude =
     tests
+
+[tool:pytest]
+testpaths = tests

Original file line number	Diff line number	Diff line change
`@@ -79,9 +79,7 @@ def main():`
`79`	`79`	`)`
`80`	`80`
`81`	`81`	`if "access_token" not in result:`
`82`		`- print("ERROR: Token acquisition failed:",`
`83`		`- result.get("error", "unknown"), "-",`
`84`		`- result.get("error_description", "no description"))`
	`82`	`+ print("ERROR: Token acquisition failed. Check logs for details.")`
`85`	`83`	`sys.exit(1)`
`86`	`84`
`87`	`85`	`token_type = result.get("token_type", "unknown")`