RepoAuditor 0.2.66__tar.gz → 0.2.67__tar.gz

{repoauditor-0.2.66 → repoauditor-0.2.67}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: RepoAuditor
-Version: 0.2.66
+Version: 0.2.67
 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.66 → repoauditor-0.2.67}/src/RepoAuditor/Plugins/GitHub/Impl/EnableRulesetRequirementImpl.py

@@ -103,8 +103,7 @@ class EnableRulesetRequirementImpl(EnableRequirementImpl):
 
             return self.EvaluateImplResult(
                 EvaluateResult.Error,
-                f"{self.github_settings_value} rule active in ruleset '{ruleset['name']}'."
-                f"It should be set to {expected_rule_enabled}",
+                f"{self.github_settings_value} rule active in ruleset '{ruleset['name']}'. It should be set to {expected_rule_enabled}",
                 provide_resolution=True,
                 provide_rationale=provide_rationale,
             )
@@ -115,7 +114,7 @@ class EnableRulesetRequirementImpl(EnableRequirementImpl):
 
         return self.EvaluateImplResult(
             EvaluateResult.Error,
-            f"No active branch ruleset requiring {self.github_settings_value} found",
+            f"No active branch ruleset with {self.github_settings_value} found",
             provide_resolution=True,
             provide_rationale=provide_rationale,
         )

{repoauditor-0.2.66 → repoauditor-0.2.67}/src/RepoAuditor/Plugins/GitHub/RulesetRequirements/RequirePullRequests.py

@@ -14,11 +14,11 @@ class RequirePullRequests(EnableRulesetRequirementImpl):
 
     def __init__(self) -> None:
         super().__init__(
-            name="RequirePullRequests",
+            name="RequirePullRequestsRule",
             enabled_by_default=True,
             dynamic_arg_name="disabled",
             github_ruleset_type="pull_request",
-            github_ruleset_value="Pull Requests",
+            github_ruleset_value="Require a pull request before merging",
             get_configuration_value_func=lambda rule: rule.get("type", "") == "pull_request",
             resolution="{__expected_value} pull request rule in repository rulesets",
             rationale="Pull request reviews help maintain code quality and collaboration",

{repoauditor-0.2.66 → repoauditor-0.2.67}/src/RepoAuditor/Plugins/GitHub/RulesetRequirements/RequireSignedCommits.py

@@ -14,11 +14,11 @@ class RequireSignedCommits(EnableRulesetRequirementImpl):
 
     def __init__(self) -> None:
         super().__init__(
-            name="RequireSignedCommits",
+            name="RequireSignedCommitsRule",
             enabled_by_default=True,
             dynamic_arg_name="disabled",
             github_ruleset_type="required_signatures",
-            github_ruleset_value="Signed Commits",
+            github_ruleset_value="Require signed commits",
             get_configuration_value_func=lambda rule: rule.get("type", "") == "required_signatures",
             resolution="Enable commit signing requirement in repository rulesets",
             rationale="Signed commits ensure commit authenticity",

{repoauditor-0.2.66 → repoauditor-0.2.67}/src/RepoAuditor/Plugins/GitHub/RulesetRequirements/RequireStatusChecks.py

@@ -14,11 +14,11 @@ class RequireStatusChecks(EnableRulesetRequirementImpl):
 
     def __init__(self) -> None:
         super().__init__(
-            name="RequireStatusChecks",
+            name="RequireStatusChecksRule",
             enabled_by_default=True,
             dynamic_arg_name="disabled",
             github_ruleset_type="required_status_checks",
-            github_ruleset_value="Status Checks",
+            github_ruleset_value="Require status checks to pass",
             get_configuration_value_func=lambda rule: rule.get("type", "") == "required_status_checks",
             resolution="Configure required status checks in repository rulesets",
             rationale="Status checks ensure code quality and compatibility",

{repoauditor-0.2.66 → repoauditor-0.2.67}/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.66"
+__version__ = "0.2.67"

{repoauditor-0.2.66 → repoauditor-0.2.67}/tests/Plugins/GitHubModule/RulesetRequirements_UnitTest.py

@@ -84,7 +84,7 @@ class TestRequirePullRequests:
         context = {"rules": []}
         result = requirement.Evaluate(context, {"disabled": False})
         assert result.result == EvaluateResult.Error
-        assert "No active branch ruleset requiring 'Pull Requests' found" in result.context
+        assert f"No active branch ruleset with {requirement.github_settings_value} found" in result.context
 
 
 # ----------------------------------------------------------------------
@@ -144,7 +144,7 @@ class TestRequireSignedCommits:
         context = {"rules": []}
         result = requirement.Evaluate(context, {"disabled": False})
         assert result.result == EvaluateResult.Error
-        assert "No active branch ruleset requiring 'Signed Commits' found" in result.context
+        assert f"No active branch ruleset with {requirement.github_settings_value} found" in result.context
 
 
 # ----------------------------------------------------------------------
@@ -187,4 +187,4 @@ class TestRequireStatusChecks:
         context = {"rules": []}
         result = requirement.Evaluate(context, {"disabled": False})
         assert result.result == EvaluateResult.Error
-        assert "No active branch ruleset requiring 'Status Checks' found" in result.context
+        assert f"No active branch ruleset with {requirement.github_settings_value} found" in result.context

{repoauditor-0.2.66 → repoauditor-0.2.67}/tests/Plugins/GitHub_EndToEndTest.py

@@ -427,7 +427,7 @@ class TestRulesets:
 
     def test_RequireStatusChecks(self, pat_args, snapshot):
         """Test for requirement `Require status checks to pass` is disabled in the ruleset for the `main` branch."""
-        result = CliRunner().invoke(app, pat_args + ["--GitHub-RequireStatusChecks-disabled"])
+        result = CliRunner().invoke(app, pat_args + ["--GitHub-RequireStatusChecksRule-disabled"])
 
         # Since the repo ruleset has the requirement enabled,
         # we should get an error.
@@ -436,7 +436,7 @@ class TestRulesets:
 
     def test_RequirePullRequests(self, pat_args, snapshot):
         """Test for requirement `Require a pull request before merging` is disabled in the ruleset for the `main` branch."""
-        result = CliRunner().invoke(app, pat_args + ["--GitHub-RequirePullRequests-disabled"])
+        result = CliRunner().invoke(app, pat_args + ["--GitHub-RequirePullRequestsRule-disabled"])
 
         # Since the repo ruleset has the requirement disabled,
         # we should get an error.
@@ -445,7 +445,7 @@ class TestRulesets:
 
     def test_RequireSignedCommits(self, pat_args, snapshot):
         """Test for requirement `Require signed commits` is disabled in the ruleset for the `main` branch."""
-        result = CliRunner().invoke(app, pat_args + ["--GitHub-RequireSignedCommits-disabled"])
+        result = CliRunner().invoke(app, pat_args + ["--GitHub-RequireSignedCommitsRule-disabled"])
 
         # Since the repo ruleset has the requirement disabled,
         # we should get an error.

{repoauditor-0.2.66 → repoauditor-0.2.67}/tests/Plugins/__snapshots__/GitHub_EndToEndTest.ambr

@@ -823,7 +823,7 @@
   │ │                                                                                                                  │ │
   │ │ Validates that 'Pull Requests' is set to the expected value.                                                     │ │
   │ │                                                                                                                  │ │
-  │ │ ERROR: 'Pull Requests' rule active in ruleset 'Protect Main'.It should be set to False                           │ │
+  │ │ ERROR: 'Pull Requests' rule active in ruleset 'Protect Main'. It should be set to False                           │ │
   │ │                                                                                                                  │ │
   │ │ Resolution                                                                                                       │ │
   │ │ ----------                                                                                                       │ │
@@ -873,7 +873,7 @@
   │ │                                                                                                                  │ │
   │ │ Validates that 'Signed Commits' is set to the expected value.                                                    │ │
   │ │                                                                                                                  │ │
-  │ │ ERROR: 'Signed Commits' rule active in ruleset 'Protect Main'.It should be set to False                          │ │
+  │ │ ERROR: 'Signed Commits' rule active in ruleset 'Protect Main'. It should be set to False                          │ │
   │ │                                                                                                                  │ │
   │ │ Resolution                                                                                                       │ │
   │ │ ----------                                                                                                       │ │
@@ -923,7 +923,7 @@
   │ │                                                                                                                  │ │
   │ │ Validates that 'Status Checks' is set to the expected value.                                                     │ │
   │ │                                                                                                                  │ │
-  │ │ ERROR: 'Status Checks' rule active in ruleset 'Protect Main'.It should be set to False                           │ │
+  │ │ ERROR: 'Status Checks' rule active in ruleset 'Protect Main'. It should be set to False                           │ │
   │ │                                                                                                                  │ │
   │ │ Resolution                                                                                                       │ │
   │ │ ----------                                                                                                       │ │
@@ -945,6 +945,126 @@
   Results: DONE! (-1, <scrubbed duration>)
   
   
+  '''
+# ---
+# name: TestRulesets.test_RequirePullRequests
+  '''
+  Processing 1 module...
+    Processing 'GitHub' (1 of 1)...
+  
+  
+  
+  
+    DONE! (-1, <scrubbed duration>)
+  DONE! (-1, <scrubbed duration>)
+  
+  
+  ╭─────────────────────────────────────────────────────── GitHub ───────────────────────────────────────────────────────╮
+  │                                                                                                                      │
+  │ Validates GitHub configuration settings.                                                                             │
+  │                                                                                                                      │
+  │ Successful:     37 (94.87%)                                                                                          │
+  │ Warnings:       0 (0.00%)                                                                                            │
+  │ Errors:         1 (2.56%)                                                                                            │
+  │ Skipped:        1 (2.56%)                                                                                            │
+  │                                                                                                                      │
+  │ ╭─ [Error] RequirePullRequestsRule ────────────────────────────────────────────────────────────────────────────────╮ │
+  │ │                                                                                                                  │ │
+  │ │ Validates that 'Require a pull request before merging' is set to the expected value.                             │ │
+  │ │                                                                                                                  │ │
+  │ │ ERROR: No active branch ruleset with 'Require a pull request before merging' found                               │ │
+  │ │                                                                                                                  │ │
+  │ │ Resolution                                                                                                       │ │
+  │ │ ----------                                                                                                       │ │
+  │ │   Enable pull request requirements in repository rulesets                                                        │ │
+  │ │                                                                                                                  │ │
+  │ ╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯ │
+  │                                                                                                                      │
+  ╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
+  
+  Results: DONE! (-1, <scrubbed duration>)
+  
+  
+  '''
+# ---
+# name: TestRulesets.test_RequireSignedCommits
+  '''
+  Processing 1 module...
+    Processing 'GitHub' (1 of 1)...
+  
+  
+  
+  
+    DONE! (-1, <scrubbed duration>)
+  DONE! (-1, <scrubbed duration>)
+  
+  
+  ╭─────────────────────────────────────────────────────── GitHub ───────────────────────────────────────────────────────╮
+  │                                                                                                                      │
+  │ Validates GitHub configuration settings.                                                                             │
+  │                                                                                                                      │
+  │ Successful:     37 (94.87%)                                                                                          │
+  │ Warnings:       0 (0.00%)                                                                                            │
+  │ Errors:         1 (2.56%)                                                                                            │
+  │ Skipped:        1 (2.56%)                                                                                            │
+  │                                                                                                                      │
+  │ ╭─ [Error] RequireSignedCommitsRule ───────────────────────────────────────────────────────────────────────────────╮ │
+  │ │                                                                                                                  │ │
+  │ │ Validates that 'Require signed commits' is set to the expected value.                                            │ │
+  │ │                                                                                                                  │ │
+  │ │ ERROR: No active branch ruleset with 'Require signed commits' found                                              │ │
+  │ │                                                                                                                  │ │
+  │ │ Resolution                                                                                                       │ │
+  │ │ ----------                                                                                                       │ │
+  │ │   Enable commit signing requirement in repository rulesets                                                       │ │
+  │ │                                                                                                                  │ │
+  │ ╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯ │
+  │                                                                                                                      │
+  ╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
+  
+  Results: DONE! (-1, <scrubbed duration>)
+  
+  
+  '''
+# ---
+# name: TestRulesets.test_RequireStatusChecks
+  '''
+  Processing 1 module...
+    Processing 'GitHub' (1 of 1)...
+  
+  
+  
+  
+    DONE! (-1, <scrubbed duration>)
+  DONE! (-1, <scrubbed duration>)
+  
+  
+  ╭─────────────────────────────────────────────────────── GitHub ───────────────────────────────────────────────────────╮
+  │                                                                                                                      │
+  │ Validates GitHub configuration settings.                                                                             │
+  │                                                                                                                      │
+  │ Successful:     37 (94.87%)                                                                                          │
+  │ Warnings:       0 (0.00%)                                                                                            │
+  │ Errors:         1 (2.56%)                                                                                            │
+  │ Skipped:        1 (2.56%)                                                                                            │
+  │                                                                                                                      │
+  │ ╭─ [Error] RequireStatusChecksRule ────────────────────────────────────────────────────────────────────────────────╮ │
+  │ │                                                                                                                  │ │
+  │ │ Validates that 'Require status checks to pass' is set to the expected value.                                     │ │
+  │ │                                                                                                                  │ │
+  │ │ ERROR: No active branch ruleset with 'Require status checks to pass' found                                       │ │
+  │ │                                                                                                                  │ │
+  │ │ Resolution                                                                                                       │ │
+  │ │ ----------                                                                                                       │ │
+  │ │   Configure required status checks in repository rulesets                                                        │ │
+  │ │                                                                                                                  │ │
+  │ ╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯ │
+  │                                                                                                                      │
+  ╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
+  
+  Results: DONE! (-1, <scrubbed duration>)
+  
+  
   '''
 # ---
 # name: TestStandard.test_DefaultBranchValue