RepoAuditor 0.2.67__tar.gz → 0.2.68__tar.gz

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: RepoAuditor
-Version: 0.2.67
+Version: 0.2.68
 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.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/ClassicBranchProtectionRequirements/AllowDeletions.py

@@ -22,7 +22,7 @@ class AllowDeletions(ClassicEnableRequirementImpl):
         super().__init__(
             "AllowDeletions",
             False,
-            "true",
+            "enabled",
             "Rules applied to everyone including administrators",
             "Allow deletions",
             lambda data: data["branch_protection_data"]["allow_deletions"]["enabled"],

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/ClassicBranchProtectionRequirements/AllowMainlineForcePushes.py

@@ -22,7 +22,7 @@ class AllowMainlineForcePushes(ClassicEnableRequirementImpl):
         super().__init__(
             "AllowMainlineForcePushes",
             False,
-            "true",
+            "enabled",
             "Rules applied to everyone including administrators",
             "Allow force pushes",
             lambda data: data["branch_protection_data"]["allow_force_pushes"]["enabled"],

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/ClassicBranchProtectionRequirements/DismissStalePullRequestApprovals.py

@@ -23,7 +23,7 @@ class DismissStalePullRequestApprovals(ClassicEnableRequirementImpl):
         super().__init__(
             "DismissStalePullRequestApprovals",
             True,
-            "false",
+            "disabled",
             "Protect matching branches",
             "Require a pull request before merging -> Dismiss stale pull request approvals when new commits are pushed",
             _GetValue,

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/ClassicBranchProtectionRequirements/DoNotAllowBypassSettings.py

@@ -22,7 +22,7 @@ class DoNotAllowBypassSettings(ClassicEnableRequirementImpl):
         super().__init__(
             "DoNotAllowBypassSettings",
             True,
-            "false",
+            "disabled",
             "Protect matching branches",
             "Do not allow bypassing the above settings",
             lambda data: data["branch_protection_data"]["enforce_admins"]["enabled"],

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/ClassicBranchProtectionRequirements/EnsureStatusChecks.py

@@ -57,7 +57,7 @@ class EnsureStatusChecks(Requirement):
     def GetDynamicArgDefinitions(self) -> dict[str, TypeDefinitionItemType]:
         """Get the definitions for the arguments to this requirement."""
         return {
-            "disable": (
+            "disabled": (
                 bool,
                 typer.Option(
                     default=False,
@@ -73,7 +73,7 @@ class EnsureStatusChecks(Requirement):
         query_data: dict[str, Any],
         requirement_args: dict[str, Any],
     ) -> Requirement.EvaluateImplResult:
-        if requirement_args["disable"]:
+        if requirement_args["disabled"]:
             return Requirement.EvaluateImplResult(
                 EvaluateResult.DoesNotApply,
                 "The status check requirement has been explicitly disabled.",

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/ClassicBranchProtectionRequirements/RequireApprovalMostRecentPush.py

@@ -23,7 +23,7 @@ class RequireApprovalMostRecentPush(ClassicEnableRequirementImpl):
         super().__init__(
             "RequireApprovalMostRecentPush",
             True,
-            "false",
+            "disabled",
             "Protect matching branches",
             "Require a pull request before merging -> Require approval of the most recent reviewable push",
             _GetValue,

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/ClassicBranchProtectionRequirements/RequireCodeOwnerReview.py

@@ -23,7 +23,7 @@ class RequireCodeOwnerReview(ClassicEnableRequirementImpl):
         super().__init__(
             "RequireCodeOwnerReview",
             False,
-            "true",
+            "enabled",
             "Protect matching branches",
             "Require a pull request before merging -> Require review from Code Owners",
             _GetValue,

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/ClassicBranchProtectionRequirements/RequireConversationResolution.py

@@ -22,7 +22,7 @@ class RequireConversationResolution(ClassicEnableRequirementImpl):
         super().__init__(
             "RequireConversationResolution",
             True,
-            "false",
+            "disabled",
             "Protect matching branches",
             "Require conversation resolution before merging",
             lambda data: data["branch_protection_data"]["required_conversation_resolution"]["enabled"],

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/ClassicBranchProtectionRequirements/RequireLinearHistory.py

@@ -22,7 +22,7 @@ class RequireLinearHistory(ClassicEnableRequirementImpl):
         super().__init__(
             "RequireLinearHistory",
             False,
-            "true",
+            "enabled",
             "Protect matching branches",
             "Require linear history",
             lambda data: data["branch_protection_data"]["required_linear_history"]["enabled"],

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/ClassicBranchProtectionRequirements/RequirePullRequests.py

@@ -22,7 +22,7 @@ class RequirePullRequests(ClassicEnableRequirementImpl):
         super().__init__(
             "RequirePullRequests",
             True,
-            "false",
+            "disabled",
             "Protect matching branches",
             "Require a pull request before merging",
             lambda data: "required_pull_request_reviews" in data["branch_protection_data"],

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/ClassicBranchProtectionRequirements/RequireSignedCommits.py

@@ -22,7 +22,7 @@ class RequireSignedCommits(ClassicEnableRequirementImpl):
         super().__init__(
             "RequireSignedCommits",
             True,
-            "false",
+            "disabled",
             "Protect matching branches",
             "Require signed commits",
             lambda data: data["branch_protection_data"]["required_signatures"]["enabled"],

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/ClassicBranchProtectionRequirements/RequireStatusChecksToPass.py

@@ -22,7 +22,7 @@ class RequireStatusChecksToPass(ClassicEnableRequirementImpl):
         super().__init__(
             "RequireStatusChecksToPass",
             True,
-            "false",
+            "disabled",
             "Protect matching branches",
             "Require status checks to pass before merging",
             lambda data: "required_status_checks" in data["branch_protection_data"],

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/ClassicBranchProtectionRequirements/RequireUpToDateBranches.py

@@ -23,7 +23,7 @@ class RequireUpToDateBranches(ClassicEnableRequirementImpl):
         super().__init__(
             "RequireUpToDateBranches",
             True,
-            "false",
+            "disabled",
             "Protect matching branches",
             "Require status checks to pass before merging -> Require branches to be up to date before merging",
             _GetValue,

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/DefaultBranchRequirements/Protected.py

@@ -56,7 +56,7 @@ class Protected(Requirement):
     def GetDynamicArgDefinitions(self) -> dict[str, TypeDefinitionItemType]:
         """Get the definitions for the arguments to this requirement."""
         return {
-            "false": (
+            "disabled": (
                 bool,
                 typer.Option(
                     default=False,
@@ -78,7 +78,7 @@ class Protected(Requirement):
         if is_protected is None:
             return CreateIncompleteDataResult()
 
-        if requirement_args["false"]:
+        if requirement_args["disabled"]:
             expected_value = False
             provide_rationale = False
         else:

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/StandardRequirements/AutoMerge.py

@@ -22,7 +22,7 @@ class AutoMerge(StandardEnableRequirementImpl):
         super().__init__(
             "AutoMerge",
             True,
-            "false",
+            "disabled",
             "settings",
             "Pull Requests",
             "Allow auto-merge",

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/StandardRequirements/DeleteHeadBranches.py

@@ -22,7 +22,7 @@ class DeleteHeadBranches(StandardEnableRequirementImpl):
         super().__init__(
             "DeleteHeadBranches",
             True,
-            "false",
+            "disabled",
             "settings",
             "Pull Requests",
             "Automatically delete head branches",

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/StandardRequirements/MergeCommit.py

@@ -22,7 +22,7 @@ class MergeCommit(StandardEnableRequirementImpl):
         super().__init__(
             "MergeCommit",
             True,
-            "false",
+            "disabled",
             "settings",
             "Pull Requests",
             "Allow merge commits",

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/StandardRequirements/Private.py

@@ -55,7 +55,7 @@ class Private(Requirement):
     def GetDynamicArgDefinitions(self) -> dict[str, TypeDefinitionItemType]:
         """Get the definitions for the arguments to this requirement."""
         return {
-            "true": (
+            "enabled": (
                 bool,
                 typer.Option(
                     default=False,
@@ -76,7 +76,7 @@ class Private(Requirement):
         if result is None:
             return CreateIncompleteDataResult()
 
-        expect_private = requirement_args["true"]
+        expect_private = requirement_args["enabled"]
 
         if result != expect_private:
             query_data["__expected_visibility"] = "private" if expect_private else "public"

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/StandardRequirements/RebaseMergeCommit.py

@@ -22,7 +22,7 @@ class RebaseMergeCommit(StandardEnableRequirementImpl):
         super().__init__(
             "RebaseMergeCommit",
             False,
-            "true",
+            "enabled",
             "settings",
             "Pull Requests",
             "Allow rebase merging",

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/StandardRequirements/SquashCommitMerge.py

@@ -22,7 +22,7 @@ class SquashCommitMerge(StandardEnableRequirementImpl):
         super().__init__(
             "SquashCommitMerge",
             False,
-            "true",
+            "enabled",
             "settings",
             "Pull Requests",
             "Allow squash merging",

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/StandardRequirements/SuggestUpdatingPullRequestBranches.py

@@ -22,7 +22,7 @@ class SuggestUpdatingPullRequestBranches(StandardEnableRequirementImpl):
         super().__init__(
             "SuggestUpdatingPullRequestBranches",
             False,
-            "true",
+            "enabled",
             "settings",
             "Pull Requests",
             "Always suggest updating pull request branches",

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/StandardRequirements/SupportDiscussions.py

@@ -22,7 +22,7 @@ class SupportDiscussions(StandardEnableRequirementImpl):
         super().__init__(
             "SupportDiscussions",
             False,
-            "true",
+            "enabled",
             "settings",
             "Features",
             "Discussions",

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/StandardRequirements/SupportIssues.py

@@ -22,7 +22,7 @@ class SupportIssues(StandardEnableRequirementImpl):
         super().__init__(
             "SupportIssues",
             True,
-            "false",
+            "disabled",
             "settings",
             "Features",
             "Issues",

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/StandardRequirements/SupportProjects.py

@@ -22,7 +22,7 @@ class SupportProjects(StandardEnableRequirementImpl):
         super().__init__(
             "SupportProjects",
             True,
-            "false",
+            "disabled",
             "settings",
             "Features",
             "Projects",

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/StandardRequirements/SupportWikis.py

@@ -22,7 +22,7 @@ class SupportWikis(StandardEnableRequirementImpl):
         super().__init__(
             "SupportWikis",
             True,
-            "false",
+            "disabled",
             "settings",
             "Features",
             "Wikis",

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/StandardRequirements/TemplateRepository.py

@@ -22,7 +22,7 @@ class TemplateRepository(StandardEnableRequirementImpl):
         super().__init__(
             "TemplateRepository",
             False,
-            "true",
+            "enabled",
             "settings",
             "General",
             "Template repository",

{repoauditor-0.2.67 → repoauditor-0.2.68}/src/RepoAuditor/Plugins/GitHub/StandardRequirements/WebCommitSignoff.py

@@ -22,7 +22,7 @@ class WebCommitSignoff(StandardEnableRequirementImpl):
         super().__init__(
             "WebCommitSignoff",
             True,
-            "false",
+            "disabled",
             "settings",
             "General",
             "Require contributors to sign off on web-based commits",

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

{repoauditor-0.2.67 → repoauditor-0.2.68}/tests/Plugins/GitHubEnterprise_EndToEndTest.py

@@ -32,7 +32,7 @@ def args() -> list[str]:
         GetGithubUrl("enterprise"),
         # Currently GitHub Enterprise does not support GitHub hosted runners.
         # As a result, no status checks show up to select and this will always fail.
-        "--GitHub-EnsureStatusChecks-disable",
+        "--GitHub-EnsureStatusChecks-disabled",
     ]
 
 
@@ -63,14 +63,14 @@ def test_Successful(pat_args, snapshot):
 class TestStandard:
     # ----------------------------------------------------------------------
     def test_NoAutoMerge(self, pat_args, snapshot):
-        result = CliRunner().invoke(app, pat_args + ["--GitHub-AutoMerge-false"])
+        result = CliRunner().invoke(app, pat_args + ["--GitHub-AutoMerge-disabled"])
 
         assert result.exit_code == -1, result.output
         assert ScrubDurationGithuburlAndSpaces(result.stdout) == snapshot
 
     # ----------------------------------------------------------------------
     def test_NoDeleteHeadBranches(self, pat_args, snapshot):
-        result = CliRunner().invoke(app, pat_args + ["--GitHub-DeleteHeadBranches-false"])
+        result = CliRunner().invoke(app, pat_args + ["--GitHub-DeleteHeadBranches-disabled"])
 
         assert result.exit_code == -1, result.output
         assert ScrubDurationGithuburlAndSpaces(result.stdout) == snapshot
@@ -78,21 +78,21 @@ class TestStandard:
     # ----------------------------------------------------------------------
     @pytest.mark.skip(reason="Not enabled by Enterprise Administrator")
     def test_NoDependabotSecurityUpdates(self, pat_args, snapshot):
-        result = CliRunner().invoke(app, pat_args + ["--GitHub-DependabotSecurityUpdates-false"])
+        result = CliRunner().invoke(app, pat_args + ["--GitHub-DependabotSecurityUpdates-disabled"])
 
         assert result.exit_code == -1, result.output
         assert ScrubDurationGithuburlAndSpaces(result.stdout) == snapshot
 
     # ----------------------------------------------------------------------
     def test_NoMergeCommit(self, pat_args, snapshot):
-        result = CliRunner().invoke(app, pat_args + ["--GitHub-MergeCommit-false"])
+        result = CliRunner().invoke(app, pat_args + ["--GitHub-MergeCommit-disabled"])
 
         assert result.exit_code == -1, result.output
         assert ScrubDurationGithuburlAndSpaces(result.stdout) == snapshot
 
     # ----------------------------------------------------------------------
     def test_YesRebaseMergeCommit(self, pat_args, snapshot):
-        result = CliRunner().invoke(app, pat_args + ["--GitHub-RebaseMergeCommit-true"])
+        result = CliRunner().invoke(app, pat_args + ["--GitHub-RebaseMergeCommit-enabled"])
 
         assert result.exit_code == -1, result.output
         assert ScrubDurationGithuburlAndSpaces(result.stdout) == snapshot
@@ -101,7 +101,7 @@ class TestStandard:
     @pytest.mark.skip(reason="Not enabled by Enterprise Administrator")
     def test_NoSecretScanning(self, pat_args, snapshot):
         """Test is secret scanning is not disabled."""
-        result = CliRunner().invoke(app, pat_args + ["--GitHub-SecretScanning-false"])
+        result = CliRunner().invoke(app, pat_args + ["--GitHub-SecretScanning-disabled"])
 
         assert result.exit_code == -1, result.output
         assert ScrubDurationGithuburlAndSpaces(result.stdout) == snapshot
@@ -110,63 +110,63 @@ class TestStandard:
     @pytest.mark.skip(reason="Not enabled by Enterprise Administrator")
     def test_NoSecretScanningPushProtection(self, pat_args, snapshot):
         """Test is secret scanning push protection is not disabled."""
-        result = CliRunner().invoke(app, pat_args + ["--GitHub-SecretScanningPushProtection-false"])
+        result = CliRunner().invoke(app, pat_args + ["--GitHub-SecretScanningPushProtection-disabled"])
 
         assert result.exit_code == -1, result.output
         assert ScrubDurationGithuburlAndSpaces(result.stdout) == snapshot
 
     # ----------------------------------------------------------------------
     def test_YesSquashCommitMerge(self, pat_args, snapshot):
-        result = CliRunner().invoke(app, pat_args + ["--GitHub-SquashCommitMerge-true"])
+        result = CliRunner().invoke(app, pat_args + ["--GitHub-SquashCommitMerge-enabled"])
 
         assert result.exit_code == -1, result.output
         assert ScrubDurationGithuburlAndSpaces(result.stdout) == snapshot
 
     # ----------------------------------------------------------------------
     def test_YesSuggestUpdatingPullRequestBranches(self, pat_args, snapshot):
-        result = CliRunner().invoke(app, pat_args + ["--GitHub-SuggestUpdatingPullRequestBranches-true"])
+        result = CliRunner().invoke(app, pat_args + ["--GitHub-SuggestUpdatingPullRequestBranches-enabled"])
 
         assert result.exit_code == -1, result.output
         assert ScrubDurationGithuburlAndSpaces(result.stdout) == snapshot
 
     # ----------------------------------------------------------------------
     def test_YesSupportDiscussions(self, pat_args, snapshot):
-        result = CliRunner().invoke(app, pat_args + ["--GitHub-SupportDiscussions-true"])
+        result = CliRunner().invoke(app, pat_args + ["--GitHub-SupportDiscussions-enabled"])
 
         assert result.exit_code == -1, result.output
         assert ScrubDurationGithuburlAndSpaces(result.stdout) == snapshot
 
     # ----------------------------------------------------------------------
     def test_NoSupportIssues(self, pat_args, snapshot):
-        result = CliRunner().invoke(app, pat_args + ["--GitHub-SupportIssues-false"])
+        result = CliRunner().invoke(app, pat_args + ["--GitHub-SupportIssues-disabled"])
 
         assert result.exit_code == -1, result.output
         assert ScrubDurationGithuburlAndSpaces(result.stdout) == snapshot
 
     # ----------------------------------------------------------------------
     def test_NoSupportProjects(self, pat_args, snapshot):
-        result = CliRunner().invoke(app, pat_args + ["--GitHub-SupportProjects-false"])
+        result = CliRunner().invoke(app, pat_args + ["--GitHub-SupportProjects-disabled"])
 
         assert result.exit_code == -1, result.output
         assert ScrubDurationGithuburlAndSpaces(result.stdout) == snapshot
 
     # ----------------------------------------------------------------------
     def test_NoSupportWikis(self, pat_args, snapshot):
-        result = CliRunner().invoke(app, pat_args + ["--GitHub-SupportWikis-false"])
+        result = CliRunner().invoke(app, pat_args + ["--GitHub-SupportWikis-disabled"])
 
         assert result.exit_code == -1, result.output
         assert ScrubDurationGithuburlAndSpaces(result.stdout) == snapshot
 
     # ----------------------------------------------------------------------
     def test_YesTemplateRepository(self, pat_args, snapshot):
-        result = CliRunner().invoke(app, pat_args + ["--GitHub-TemplateRepository-true"])
+        result = CliRunner().invoke(app, pat_args + ["--GitHub-TemplateRepository-enabled"])
 
         assert result.exit_code == -1, result.output
         assert ScrubDurationGithuburlAndSpaces(result.stdout) == snapshot
 
     # ----------------------------------------------------------------------
     def test_NoWebCommitSignoff(self, pat_args, snapshot):
-        result = CliRunner().invoke(app, pat_args + ["--GitHub-WebCommitSignoff-false"])
+        result = CliRunner().invoke(app, pat_args + ["--GitHub-WebCommitSignoff-disabled"])
 
         assert result.exit_code == -1, result.output
         assert ScrubDurationGithuburlAndSpaces(result.stdout) == snapshot
@@ -203,7 +203,7 @@ class TestStandard:
     def test_PrivateValue(self, pat_args, snapshot):
         result = CliRunner().invoke(
             app,
-            pat_args + ["--GitHub-Private-true"],
+            pat_args + ["--GitHub-Private-enabled"],
         )
 
         assert result.exit_code == -1, result.output
@@ -227,7 +227,7 @@ class TestStandard:
     def test_Protected(self, pat_args, snapshot):
         result = CliRunner().invoke(
             app,
-            pat_args + ["--GitHub-Protected-false"],
+            pat_args + ["--GitHub-Protected-disabled"],
         )
 
         assert result.exit_code == -1, result.output
@@ -240,7 +240,7 @@ class TestClassic:
     def test_AllowDeletions(self, pat_args, snapshot):
         result = CliRunner().invoke(
             app,
-            pat_args + ["--GitHub-AllowDeletions-true"],
+            pat_args + ["--GitHub-AllowDeletions-enabled"],
         )
 
         assert result.exit_code == -1, result.output
@@ -250,7 +250,7 @@ class TestClassic:
     def test_AllowForcePush(self, pat_args, snapshot):
         result = CliRunner().invoke(
             app,
-            pat_args + ["--GitHub-AllowMainlineForcePushes-true"],
+            pat_args + ["--GitHub-AllowMainlineForcePushes-enabled"],
         )
 
         assert result.exit_code == -1, result.output
@@ -260,7 +260,7 @@ class TestClassic:
     def test_DismissStalePullRequestApprovals(self, pat_args, snapshot):
         result = CliRunner().invoke(
             app,
-            pat_args + ["--GitHub-DismissStalePullRequestApprovals-false"],
+            pat_args + ["--GitHub-DismissStalePullRequestApprovals-disabled"],
         )
 
         assert result.exit_code == -1, result.output
@@ -270,7 +270,7 @@ class TestClassic:
     def test_DoNotAllowBypassSettings(self, pat_args, snapshot):
         result = CliRunner().invoke(
             app,
-            pat_args + ["--GitHub-DoNotAllowBypassSettings-false"],
+            pat_args + ["--GitHub-DoNotAllowBypassSettings-disabled"],
         )
 
         assert result.exit_code == -1, result.output
@@ -291,7 +291,7 @@ class TestClassic:
     def test_RequireApprovalMostRecentPush(self, pat_args, snapshot):
         result = CliRunner().invoke(
             app,
-            pat_args + ["--GitHub-RequireApprovalMostRecentPush-false"],
+            pat_args + ["--GitHub-RequireApprovalMostRecentPush-disabled"],
         )
 
         assert result.exit_code == -1, result.output
@@ -311,7 +311,7 @@ class TestClassic:
     def test_RequireCodeOwnerReview(self, pat_args, snapshot):
         result = CliRunner().invoke(
             app,
-            pat_args + ["--GitHub-RequireCodeOwnerReview-true"],
+            pat_args + ["--GitHub-RequireCodeOwnerReview-enabled"],
         )
 
         assert result.exit_code == -1, result.output
@@ -321,7 +321,7 @@ class TestClassic:
     def test_RequireConversationResolution(self, pat_args, snapshot):
         result = CliRunner().invoke(
             app,
-            pat_args + ["--GitHub-RequireConversationResolution-false"],
+            pat_args + ["--GitHub-RequireConversationResolution-disabled"],
         )
 
         assert result.exit_code == -1, result.output
@@ -331,7 +331,7 @@ class TestClassic:
     def test_RequireLinearHistory(self, pat_args, snapshot):
         result = CliRunner().invoke(
             app,
-            pat_args + ["--GitHub-RequireLinearHistory-true"],
+            pat_args + ["--GitHub-RequireLinearHistory-enabled"],
         )
 
         assert result.exit_code == -1, result.output
@@ -341,7 +341,7 @@ class TestClassic:
     def test_RequirePullRequests(self, pat_args, snapshot):
         result = CliRunner().invoke(
             app,
-            pat_args + ["--GitHub-RequirePullRequests-false"],
+            pat_args + ["--GitHub-RequirePullRequests-disabled"],
         )
 
         assert result.exit_code == -1, result.output
@@ -351,7 +351,7 @@ class TestClassic:
     def test_RequireSignedCommits(self, pat_args, snapshot):
         result = CliRunner().invoke(
             app,
-            pat_args + ["--GitHub-RequireSignedCommits-false"],
+            pat_args + ["--GitHub-RequireSignedCommits-disabled"],
         )
 
         assert result.exit_code == -1, result.output
@@ -361,7 +361,7 @@ class TestClassic:
     def test_RequireStatusChecksToPass(self, pat_args, snapshot):
         result = CliRunner().invoke(
             app,
-            pat_args + ["--GitHub-RequireStatusChecksToPass-false"],
+            pat_args + ["--GitHub-RequireStatusChecksToPass-disabled"],
         )
 
         assert result.exit_code == -1, result.output
@@ -371,7 +371,7 @@ class TestClassic:
     def test_RequireUpToDateBranches(self, pat_args, snapshot):
         result = CliRunner().invoke(
             app,
-            pat_args + ["--GitHub-RequireUpToDateBranches-false"],
+            pat_args + ["--GitHub-RequireUpToDateBranches-disabled"],
         )
 
         assert result.exit_code == -1, result.output

{repoauditor-0.2.67 → repoauditor-0.2.68}/tests/Plugins/GitHubModule/ClassicBranchProtectionRequirements/DismissStalePullRequestApprovals_UnitTest.py

@@ -70,6 +70,6 @@ class TestDismissStalePullRequestApprovals:
     def test_successful(self, requirement, query_data):
         """Test successful"""
         query_data["branch_protection_data"]["required_pull_request_reviews"]["dismiss_stale_reviews"] = True
-        requirement_args = {"false": False}
+        requirement_args = {"disabled": False}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.Success

{repoauditor-0.2.67 → repoauditor-0.2.68}/tests/Plugins/GitHubModule/ClassicBranchProtectionRequirements/EnsureStatusChecks_UnitTest.py

@@ -35,7 +35,7 @@ def requirement():
 class TestEnsureStatusChecks:
     def test_disabled(self, requirement, query_data):
         """Test disabled requirement"""
-        requirement_args = {"disable": True}
+        requirement_args = {"disabled": True}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert "The status check requirement has been explicitly disabled." in result.context
@@ -43,7 +43,7 @@ class TestEnsureStatusChecks:
     def test_required_status_checks_missing(self, requirement, query_data):
         """Test when `required_status_checks` is missing"""
         query_data["branch_protection_data"] = {}
-        requirement_args = {"disable": False}
+        requirement_args = {"disabled": False}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
@@ -51,7 +51,7 @@ class TestEnsureStatusChecks:
     def test_required_status_checks_none(self, requirement, query_data):
         """Test when `required_status_checks` is None"""
         query_data["branch_protection_data"] = {"required_status_checks": None}
-        requirement_args = {"disable": False}
+        requirement_args = {"disabled": False}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
@@ -59,7 +59,7 @@ class TestEnsureStatusChecks:
     def test_checks_missing(self, requirement, query_data):
         """Test when `checks` is missing"""
         query_data["branch_protection_data"]["required_status_checks"] = {}
-        requirement_args = {"disable": False}
+        requirement_args = {"disabled": False}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.Error
         assert "No status checks are configured." in result.context
@@ -67,7 +67,7 @@ class TestEnsureStatusChecks:
     def test_checks_none(self, requirement, query_data):
         """Test when `checks` is None"""
         query_data["branch_protection_data"]["required_status_checks"] = {"checks": None}
-        requirement_args = {"disable": False}
+        requirement_args = {"disabled": False}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.Error
         assert "No status checks are configured." in result.context
@@ -75,7 +75,7 @@ class TestEnsureStatusChecks:
     def test_Successful(self, requirement, query_data):
         """Test successful"""
         query_data["branch_protection_data"]["required_status_checks"]["checks"] = ["check1", "check2"]
-        requirement_args = {"disable": False}
+        requirement_args = {"disabled": False}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.Success