RepoAuditor 0.2.69__tar.gz → 0.2.70__tar.gz

{repoauditor-0.2.69 → repoauditor-0.2.70}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: RepoAuditor
-Version: 0.2.69
+Version: 0.2.70
 Summary: Add your description here
 Project-URL: Homepage, https://github.com/gt-sse-center/RepoAuditor
 Project-URL: Documentation, https://github.com/gt-sse-center/RepoAuditor

{repoauditor-0.2.69 → repoauditor-0.2.70}/src/RepoAuditor/Plugins/GitHubBase/Module.py

@@ -58,6 +58,17 @@ class GitHubBaseModule(Module):
     @override
     def GenerateInitialData(self, dynamic_args: dict[str, Any]) -> Optional[dict[str, Any]]:
         """Generate the initial data to be used in the `dynamic_args`, such as session info, etc."""
+        # Read the PAT (if provided) as a filename or a value.
+        # This also decouples the PAT reading logic from the GitHubSession class (which may be mocked for testing).
+        github_pat = dynamic_args.get("pat")
+        if github_pat:
+            potential_file = Path(github_pat)
+            if potential_file.is_file():
+                with potential_file.open("r") as f:
+                    github_pat = f.read().strip()
+        # Re-assign github_pat so it can be used within the subclassed modules.
+        dynamic_args["pat"] = github_pat
+
         # Create a GitHub API session
         dynamic_args["session"] = _GitHubSession(dynamic_args["url"], dynamic_args.get("pat"))
 
@@ -90,12 +101,6 @@ class _GitHubSession(requests.Session):
         )
 
         if github_pat:
-            potential_file = Path(github_pat)
-
-            if potential_file.is_file():
-                with potential_file.open("r") as f:
-                    github_pat = f.read().strip()
-
             self.headers["Authorization"] = f"Bearer {github_pat}"
 
         github_url = github_url.removesuffix("/")

{repoauditor-0.2.69 → repoauditor-0.2.70}/src/RepoAuditor/__init__.py

@@ -2,4 +2,4 @@ APP_NAME = "RepoAuditor"
 
 # Note that the default value below will be used until the value
 # here is explicitly updated by the Continuous Integration system.
-__version__ = "0.2.69"
+__version__ = "0.2.70"

{repoauditor-0.2.69 → repoauditor-0.2.70}/tests/Plugins/CommunityStandards_EndToEndTest.py

@@ -334,10 +334,7 @@ def args() -> list[str]:
 # ----------------------------------------------------------------------
 @pytest.fixture
 def pat_args(args) -> list[str]:
-    _github_pat_filename = (Path(__file__).parent / "github_pat.txt").resolve()
-    CheckPATFileExists(_github_pat_filename)
+    github_pat_filename = (Path(__file__).parent / "github_pat.txt").resolve()
+    CheckPATFileExists(github_pat_filename)
 
-    with _github_pat_filename.open() as f:
-        pat_value = f.read().strip()
-
-    return args + ["--CommunityStandards-pat", pat_value]
+    return args + ["--CommunityStandards-pat", str(github_pat_filename)]

{repoauditor-0.2.69 → repoauditor-0.2.70}/tests/Plugins/GitHubEnterprise_EndToEndTest.py

@@ -39,15 +39,12 @@ def args() -> list[str]:
 # ----------------------------------------------------------------------
 @pytest.fixture
 def pat_args(args) -> list[str]:
-    _github_pat_filename = (Path(__file__).parent / "github_gatech_pat.txt").resolve()
+    github_pat_filename = (Path(__file__).parent / "github_gatech_pat.txt").resolve()
     parsed = urllib.parse.urlparse(GetGithubUrl("enterprise"))
     parsed = parsed._replace(path="")
-    CheckPATFileExists(_github_pat_filename, github_url=parsed.geturl())
+    CheckPATFileExists(github_pat_filename, github_url=parsed.geturl())
 
-    with _github_pat_filename.open() as f:
-        pat_value = f.read().strip()
-
-    return args + ["--GitHub-pat", pat_value]
+    return args + ["--GitHub-pat", str(github_pat_filename)]
 
 
 # ----------------------------------------------------------------------

{repoauditor-0.2.69 → repoauditor-0.2.70}/tests/Plugins/GitHubModule/fixtures.py

@@ -18,6 +18,6 @@ def session():
 
     s = _GithubSession_()
     s.github_url = "https://github.com/gt-sse-center/RepoAuditor"
-    s.pat = "dummy_github_pat.txt"
+    s.pat = "github_pat_dummy"
     s.is_enterprise = False
     return s

{repoauditor-0.2.69 → repoauditor-0.2.70}/tests/Plugins/GitHub_EndToEndTest.py

@@ -470,10 +470,7 @@ def args() -> list[str]:
 # ----------------------------------------------------------------------
 @pytest.fixture
 def pat_args(args) -> list[str]:
-    _github_pat_filename = (Path(__file__).parent / "github_pat.txt").resolve()
-    CheckPATFileExists(_github_pat_filename)
+    github_pat_filename = (Path(__file__).parent / "github_pat.txt").resolve()
+    CheckPATFileExists(github_pat_filename)
 
-    with _github_pat_filename.open() as f:
-        pat_value = f.read().strip()
-
-    return args + ["--GitHub-pat", pat_value]
+    return args + ["--GitHub-pat", str(github_pat_filename)]