RepoAuditor 0.2.72__tar.gz → 0.2.75__tar.gz

{repoauditor-0.2.72 → repoauditor-0.2.75}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: RepoAuditor
-Version: 0.2.72
+Version: 0.2.75
 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.72 → repoauditor-0.2.75}/src/RepoAuditor/Plugins/CommunityStandards/Impl/ExistsRequirementImpl.py

@@ -80,8 +80,8 @@ class ExistsRequirementImpl(Requirement):
         query_data: dict[str, Any],
         requirement_args: dict[str, Any],
     ) -> Requirement.EvaluateImplResult:
-        # Check if `dynamic_arg_name` is `not unrequired`.
-        if not requirement_args[self.dynamic_arg_name]:
+        # Check if dynamic argument is `not unrequired`.
+        if not requirement_args.get(self.dynamic_arg_name, False):
             for location in self.possible_locations:
                 # Get full file path in temporary repo directory and check if it exists
                 # This checks both upper case and lower case variants

{repoauditor-0.2.72 → repoauditor-0.2.75}/src/RepoAuditor/Plugins/GitHub/ClassicBranchProtectionQuery.py

@@ -115,20 +115,20 @@ class ClassicBranchProtectionQuery(Query):
             "ClassicBranchProtectionQuery",
             ExecutionStyle.Parallel,
             [
-                RequirePullRequests(),
-                RequireApprovals(),
+                AllowDeletions(),
+                AllowMainlineForcePushes(),
                 DismissStalePullRequestApprovals(),
-                RequireCodeOwnerReview(),
-                RequireApprovalMostRecentPush(),
-                RequireStatusChecksToPass(),
-                RequireUpToDateBranches(),
+                DoNotAllowBypassSettings(),
                 EnsureStatusChecks(),
+                RequireApprovalMostRecentPush(),
+                RequireApprovals(),
+                RequireCodeOwnerReview(),
                 RequireConversationResolution(),
-                RequireSignedCommits(),
                 RequireLinearHistory(),
-                DoNotAllowBypassSettings(),
-                AllowDeletions(),
-                AllowMainlineForcePushes(),
+                RequirePullRequests(),
+                RequireSignedCommits(),
+                RequireStatusChecksToPass(),
+                RequireUpToDateBranches(),
             ],
         )
 

{repoauditor-0.2.72 → repoauditor-0.2.75}/src/RepoAuditor/Plugins/GitHub/ClassicBranchProtectionRequirements/Impl/ClassicValueRequirementImpl.py

@@ -40,11 +40,11 @@ class ClassicValueRequirementImpl(ValueRequirementImpl):
 
             resolution = textwrap.dedent(
                 f"""\
-                1) Visit '{{session.github_url}}/settings/branches'
-                2) Locate the 'Branch protection rules' section
-                3) Click the 'Edit' button next to the branch '{{branch}}'
-                4) Locate the '{github_settings_section}' section
-                5) Ensure that {github_settings_value_display} is {{__expected_value}}
+                1) Visit '{{session.github_url}}/settings/branches'.
+                2) Locate the 'Branch protection rules' section.
+                3) Click the 'Edit' button next to the branch '{{branch}}'.
+                4) Locate the '{github_settings_section}' section.
+                5) Ensure that {github_settings_value_display} is '{{__expected_value}}'.
                 """,
             )
 

{repoauditor-0.2.72 → repoauditor-0.2.75}/src/RepoAuditor/Plugins/GitHub/Impl/EnableRequirementImpl.py

@@ -81,7 +81,8 @@ class EnableRequirementImpl(Requirement):
     ) -> Requirement.EvaluateImplResult:
         expected_value = self.enabled_by_default
 
-        if requirement_args[self.dynamic_arg_name]:
+        # The value of the dynamic arg doesn't matter as much as whether it was provided
+        if requirement_args.get(self.dynamic_arg_name, False):
             expected_value = not expected_value
             provide_rationale = False
         else:

{repoauditor-0.2.72 → repoauditor-0.2.75}/src/RepoAuditor/Plugins/GitHub/Impl/EnableRulesetRequirementImpl.py

@@ -61,7 +61,8 @@ class EnableRulesetRequirementImpl(EnableRequirementImpl):
     ) -> Requirement.EvaluateImplResult:
         expected_rule_enabled = self.enabled_by_default
 
-        if requirement_args[self.dynamic_arg_name]:
+        # The value of the dynamic arg doesn't matter as much as whether it was provided
+        if requirement_args.get(self.dynamic_arg_name, False):
             expected_rule_enabled = not expected_rule_enabled
             provide_rationale = False
         else:

{repoauditor-0.2.72 → repoauditor-0.2.75}/src/RepoAuditor/Plugins/GitHub/Impl/ValueRequirementImpl.py

@@ -84,7 +84,7 @@ class ValueRequirementImpl(Requirement):
         query_data: dict[str, Any],
         requirement_args: dict[str, Any],
     ) -> Requirement.EvaluateImplResult:
-        expected_value = requirement_args["value"]
+        expected_value = requirement_args.get("value", self.default_value)
         query_data["__expected_value"] = f"{expected_value}"
 
         result = self.get_configuration_value_func(query_data)

{repoauditor-0.2.72 → repoauditor-0.2.75}/src/RepoAuditor/Plugins/GitHub/StandardQuery.py

@@ -212,33 +212,27 @@ class StandardQuery(Query):
             "StandardQuery",
             ExecutionStyle.Parallel,
             [
-                # metadata
+                AutoMerge(),
+                DefaultBranch(),
+                DeleteHeadBranches(),
                 Description(),
+                DependabotSecurityUpdates(),
                 License(),
-                # settings page
-                TemplateRepository(),
-                WebCommitSignoff(),
-                DefaultBranch(),
-                # settings page (Features)
-                SupportWikis(),
-                SupportIssues(),
-                SupportDiscussions(),
-                SupportProjects(),
-                # settings page (Pull Requests)
                 MergeCommit(),
                 MergeCommitMessage(),
-                SquashCommitMerge(),
-                SquashMergeCommitMessage(),
-                RebaseMergeCommit(),
-                SuggestUpdatingPullRequestBranches(),
-                AutoMerge(),
-                DeleteHeadBranches(),
-                # settings page (Danger Zone)
                 Private(),
-                # settings/security_analysis
-                DependabotSecurityUpdates(),
+                RebaseMergeCommit(),
                 SecretScanning(),
                 SecretScanningPushProtection(),
+                SquashCommitMerge(),
+                SquashMergeCommitMessage(),
+                SuggestUpdatingPullRequestBranches(),
+                SupportDiscussions(),
+                SupportIssues(),
+                SupportProjects(),
+                SupportWikis(),
+                TemplateRepository(),
+                WebCommitSignoff(),
             ],
         )
 

{repoauditor-0.2.72 → repoauditor-0.2.75}/src/RepoAuditor/Plugins/GitHub/StandardRequirements/Impl/StandardValueRequirementImpl.py

@@ -44,9 +44,9 @@ class StandardValueRequirementImpl(
 
             resolution = textwrap.dedent(
                 f"""\
-                1) Visit '{{session.github_url}}/{github_settings_url_suffix}'
-                2) Locate the '{github_settings_section}' section
-                3) Ensure that {github_settings_value_display} is set to {{__expected_value}}
+                1) Visit '{{session.github_url}}/{github_settings_url_suffix}'.
+                2) Locate the '{github_settings_section}' section.
+                3) Ensure that {github_settings_value_display} is set to '{{__expected_value}}'.
                 """,
             )
 

{repoauditor-0.2.72 → repoauditor-0.2.75}/src/RepoAuditor/Plugins/GitHub/StandardRequirements/Private.py

@@ -71,7 +71,7 @@ class Private(Requirement):
         query_data: dict[str, Any],
         requirement_args: dict[str, Any],
     ) -> Requirement.EvaluateImplResult:
-        expect_private = requirement_args["enabled"]
+        expect_private = requirement_args.get("enabled", False)
         query_data["__expected_visibility"] = "private" if expect_private else "public"
 
         result = query_data["standard"].get("private", None)

{repoauditor-0.2.72 → repoauditor-0.2.75}/src/RepoAuditor/Requirement.py

@@ -87,6 +87,9 @@ class Requirement(ABC):
         If `requires_explicit_include` is True, the requirement must be explicitly included on the command line.
         """
         self.name = name
+        # Use description template so we can update this description for each requirement
+        self.description_template = description
+        # Use the description as is as backup
         self.description = description
         self.style = style
 
@@ -113,7 +116,7 @@ class Requirement(ABC):
         result_info = self._EvaluateImpl(query_data, requirement_args)
 
         # Fill in templates in description string
-        self.description = self.description.format(**query_data)
+        self.description = self.description_template.format(**query_data)
 
         if result_info.result == EvaluateResult.Error:
             return Requirement.EvaluateInfo(

{repoauditor-0.2.72 → repoauditor-0.2.75}/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.72"
+__version__ = "0.2.75"

{repoauditor-0.2.72 → repoauditor-0.2.75}/tests/Plugins/GitHubModule/ClassicBranchProtectionRequirements/DismissStalePullRequestApprovals_UnitTest.py

@@ -32,44 +32,44 @@ def requirement():
     return DismissStalePullRequestApprovals()
 
 
-@pytest.fixture(name="requirement_args")
-def requirement_args_fixture():
-    return {"disabled": False}
-
-
 class TestDismissStalePullRequestApprovals:
-    def test_required_pull_request_reviews_missing(self, requirement, query_data, requirement_args):
+    def test_required_pull_request_reviews_missing(self, requirement, query_data):
         """Test when `required_pull_request_reviews` is missing."""
         query_data["branch_protection_data"] = {}
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
 
-    def test_required_pull_request_reviews_none(self, requirement, query_data, requirement_args):
+    def test_required_pull_request_reviews_none(self, requirement, query_data):
         """Test when `required_pull_request_reviews` is None"""
         query_data["branch_protection_data"]["required_pull_request_reviews"] = None
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
 
-    def test_dismiss_stale_reviews_missing(self, requirement, query_data, requirement_args):
+    def test_dismiss_stale_reviews_missing(self, requirement, query_data):
         """Test when `dismiss_stale_reviews` is missing"""
         query_data["branch_protection_data"]["required_pull_request_reviews"] = {}
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
 
-    def test_dismiss_stale_reviews_none(self, requirement, query_data, requirement_args):
+    def test_dismiss_stale_reviews_none(self, requirement, query_data):
         """Test when `dismiss_stale_reviews` is None"""
         query_data["branch_protection_data"]["required_pull_request_reviews"] = {
             "dismiss_stale_reviews": None
         }
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
 
-    def test_successful(self, requirement, query_data, requirement_args):
+    def test_successful(self, requirement, query_data):
         """Test successful"""
         query_data["branch_protection_data"]["required_pull_request_reviews"]["dismiss_stale_reviews"] = True
+        requirement_args = {"disabled": False}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.Success

{repoauditor-0.2.72 → repoauditor-0.2.75}/tests/Plugins/GitHubModule/ClassicBranchProtectionRequirements/RequireApprovalMostRecentPush_UnitTest.py

@@ -32,38 +32,37 @@ def requirement():
     return RequireApprovalMostRecentPush()
 
 
-@pytest.fixture(name="requirement_args")
-def requirement_args_fixture():
-    return {"disabled": False}
-
-
 class TestRequireApprovalMostRecentPush:
-    def test_required_pull_request_reviews_missing(self, requirement, query_data, requirement_args):
+    def test_required_pull_request_reviews_missing(self, requirement, query_data):
         """Test when `required_pull_request_reviews` is missing"""
         query_data["branch_protection_data"] = {}
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
 
-    def test_required_pull_request_reviews_none(self, requirement, query_data, requirement_args):
+    def test_required_pull_request_reviews_none(self, requirement, query_data):
         """Test when `required_pull_request_reviews` is None"""
         query_data["branch_protection_data"]["required_pull_request_reviews"] = None
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
 
-    def test_require_last_push_approval_missing(self, requirement, query_data, requirement_args):
+    def test_require_last_push_approval_missing(self, requirement, query_data):
         """Test when `require_last_push_approval` is missing"""
         query_data["branch_protection_data"]["required_pull_request_reviews"] = {}
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
 
-    def test_require_last_push_approval_none(self, requirement, query_data, requirement_args):
+    def test_require_last_push_approval_none(self, requirement, query_data):
         """Test when `require_last_push_approval` is None"""
         query_data["branch_protection_data"]["required_pull_request_reviews"][
             "require_last_push_approval"
         ] = None
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None

{repoauditor-0.2.72 → repoauditor-0.2.75}/tests/Plugins/GitHubModule/ClassicBranchProtectionRequirements/RequireApprovals_UnitTest.py

@@ -30,47 +30,47 @@ def requirement():
     return RequireApprovals()
 
 
-@pytest.fixture(name="requirement_args")
-def requirement_args_fixture():
-    return {"value": "1"}
-
-
 class TestRequireApprovals:
-    def test_required_pull_request_reviews_missing(self, requirement, query_data, requirement_args):
+    def test_required_pull_request_reviews_missing(self, requirement, query_data):
         """Test when `required_pull_request_reviews` is missing"""
         query_data["branch_protection_data"] = {}
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
 
-    def test_required_pull_request_reviews_none(self, requirement, query_data, requirement_args):
+    def test_required_pull_request_reviews_none(self, requirement, query_data):
         """Test when `required_pull_request_reviews` is None"""
         query_data["branch_protection_data"]["required_pull_request_reviews"] = None
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
 
-    def test_required_approving_review_count_missing(self, requirement, query_data, requirement_args):
+    def test_required_approving_review_count_missing(self, requirement, query_data):
         """Test when `required_approving_review_count` is missing"""
         query_data["branch_protection_data"]["required_pull_request_reviews"] = {}
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
 
-    def test_required_approving_review_count_none(self, requirement, query_data, requirement_args):
+    def test_required_approving_review_count_none(self, requirement, query_data):
         """Test when `required_approving_review_count` is None"""
         query_data["branch_protection_data"]["required_pull_request_reviews"][
             "required_approving_review_count"
         ] = None
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
 
-    def test_IncorrectValue(self, requirement, query_data, requirement_args):
+    def test_IncorrectValue(self, requirement, query_data):
         """Test incorrect"""
         query_data["branch_protection_data"]["required_pull_request_reviews"][
             "required_approving_review_count"
         ] = "2"
+        requirement_args = {"value": "1"}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.Error
         assert (
@@ -78,10 +78,11 @@ class TestRequireApprovals:
             in result.context
         )
 
-    def test_Successful(self, requirement, query_data, requirement_args):
+    def test_Successful(self, requirement, query_data):
         """Test successful"""
         query_data["branch_protection_data"]["required_pull_request_reviews"][
             "required_approving_review_count"
         ] = "1"
+        requirement_args = {"value": "1"}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.Success

{repoauditor-0.2.72 → repoauditor-0.2.75}/tests/Plugins/GitHubModule/ClassicBranchProtectionRequirements/RequireCodeOwnerReview_UnitTest.py

@@ -32,47 +32,47 @@ def requirement():
     return RequireCodeOwnerReview()
 
 
-@pytest.fixture(name="requirement_args")
-def requirement_args_fixture():
-    return {"enabled": True}
-
-
 class TestRequireCodeOwnerReview:
-    def test_required_pull_request_reviews_missing(self, requirement, query_data, requirement_args):
+    def test_required_pull_request_reviews_missing(self, requirement, query_data):
         """Test when `required_pull_request_reviews` is missing"""
         query_data["branch_protection_data"] = {}
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
 
-    def test_required_pull_request_reviews_none(self, requirement, query_data, requirement_args):
+    def test_required_pull_request_reviews_none(self, requirement, query_data):
         """Test when `required_pull_request_reviews` is None"""
         query_data["branch_protection_data"]["required_pull_request_reviews"] = None
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
 
-    def test_require_code_owner_reviews_missing(self, requirement, query_data, requirement_args):
+    def test_require_code_owner_reviews_missing(self, requirement, query_data):
         """Test when `require_code_owner_reviews` is missing"""
         query_data["branch_protection_data"]["required_pull_request_reviews"] = {}
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
 
-    def test_require_code_owner_reviews_none(self, requirement, query_data, requirement_args):
+    def test_require_code_owner_reviews_none(self, requirement, query_data):
         """Test when `require_code_owner_reviews` is None"""
         query_data["branch_protection_data"]["required_pull_request_reviews"][
             "require_code_owner_reviews"
         ] = None
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
 
-    def test_IncorrectValue(self, requirement, query_data, requirement_args):
+    def test_IncorrectValue(self, requirement, query_data):
         """Test incorrect"""
         query_data["branch_protection_data"]["required_pull_request_reviews"][
             "require_code_owner_reviews"
         ] = False
+        requirement_args = {"enabled": True}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.Error
         assert (
@@ -80,10 +80,11 @@ class TestRequireCodeOwnerReview:
             in result.context
         )
 
-    def test_Successful(self, requirement, query_data, requirement_args):
+    def test_Successful(self, requirement, query_data):
         """Test successful"""
         query_data["branch_protection_data"]["required_pull_request_reviews"][
             "require_code_owner_reviews"
         ] = True
+        requirement_args = {"enabled": True}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.Success

{repoauditor-0.2.72 → repoauditor-0.2.75}/tests/Plugins/GitHubModule/ClassicBranchProtectionRequirements/RequireUpToDateBranches_UnitTest.py

@@ -24,36 +24,35 @@ def requirement():
     return RequireUpToDateBranches()
 
 
-@pytest.fixture(name="requirement_args")
-def requirement_args_fixture():
-    return {"disabled": True}
-
-
 class TestRequireUpToDateBranches:
-    def test_required_status_checks_missing(self, requirement, query_data, requirement_args):
+    def test_required_status_checks_missing(self, requirement, query_data):
         """Test when `required_status_checks` is missing"""
         query_data["branch_protection_data"] = {}
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
 
-    def test_required_status_checks_none(self, requirement, query_data, requirement_args):
+    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 = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
 
-    def test_strict_missing(self, requirement, query_data, requirement_args):
+    def test_strict_missing(self, requirement, query_data):
         """Test when `strict` is missing"""
         query_data["branch_protection_data"]["required_status_checks"] = {}
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
 
-    def test_strict_none(self, requirement, query_data, requirement_args):
+    def test_strict_none(self, requirement, query_data):
         """Test when `strict` is None"""
         query_data["branch_protection_data"]["required_status_checks"] = {"strict": None}
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.DoesNotApply
         assert result.context is None
@@ -69,8 +68,9 @@ class TestRequireUpToDateBranches:
             in result.context
         )
 
-    def test_Successful(self, requirement, query_data, requirement_args):
+    def test_Successful(self, requirement, query_data):
         """Test successful"""
         query_data["branch_protection_data"]["required_status_checks"]["strict"] = False
+        requirement_args = {"disabled": True}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.Success

{repoauditor-0.2.72 → repoauditor-0.2.75}/tests/Plugins/GitHubModule/StandardQueryRequirements/DependabotSecurityUpdates_UnitTest.py

@@ -33,43 +33,43 @@ def requirement():
     return DependabotSecurityUpdates()
 
 
-@pytest.fixture(name="requirement_args")
-def requirement_args_fixture():
-    return {"disabled": False}
-
-
 class TestDependabotSecurityUpdates:
-    def test_security_and_analysis_missing(self, requirement, query_data, requirement_args):
+    def test_security_and_analysis_missing(self, requirement, query_data):
         """Test if `security_and_analysis` is missing"""
         query_data["standard"] = {}
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.Warning
         assert "Incomplete data was encountered" in result.context
 
-    def test_dependabot_security_updates_missing(self, requirement, query_data, requirement_args):
+    def test_dependabot_security_updates_missing(self, requirement, query_data):
         """Test if `dependabot_security_updates` is missing"""
         query_data["standard"]["security_and_analysis"] = {}
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.Warning
         assert "Incomplete data was encountered" in result.context
 
-    def test_status_missing(self, requirement, query_data, requirement_args):
+    def test_status_missing(self, requirement, query_data):
         """Test if `status` is missing"""
         query_data["standard"]["security_and_analysis"]["dependabot_security_updates"] = {}
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.Warning
         assert "Incomplete data was encountered" in result.context
 
-    def test_status_none(self, requirement, query_data, requirement_args):
+    def test_status_none(self, requirement, query_data):
         """Test if `status` is None"""
         query_data["standard"]["security_and_analysis"]["dependabot_security_updates"]["status"] = None
+        requirement_args = {}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.Warning
         assert "Incomplete data was encountered" in result.context
 
-    def test_Incorrect(self, requirement, query_data, requirement_args):
+    def test_Incorrect(self, requirement, query_data):
         """Result doesn't match expected value"""
         query_data["standard"]["security_and_analysis"]["dependabot_security_updates"]["status"] = True
+        requirement_args = {"disabled": False}
         result = requirement.Evaluate(query_data, requirement_args)
         assert result.result == EvaluateResult.Error
         assert (

{repoauditor-0.2.72 → repoauditor-0.2.75}/tests/Plugins/GitHubModule/StandardQueryRequirements/License_UnitTest.py

@@ -37,7 +37,7 @@ class TestLicense:
         query_data["standard"] = {}
         result = requirement.Evaluate(
             query_data,
-            requirement_args={"value": requirement.default_value},
+            requirement_args={},
         )
         assert result.result == EvaluateResult.Warning
         assert "Incomplete data was encountered" in result.context
@@ -49,7 +49,7 @@ class TestLicense:
         query_data["standard"]["license"] = None
         result = requirement.Evaluate(
             query_data,
-            requirement_args={"value": requirement.default_value},
+            requirement_args={},
         )
         assert result.result == EvaluateResult.Error
         assert "'License' must be set to 'MIT License' (it is currently set to '')." in result.context
@@ -58,6 +58,6 @@ class TestLicense:
         """Test successful"""
         result = requirement.Evaluate(
             query_data,
-            requirement_args={"value": requirement.default_value},
+            requirement_args={},
         )
         assert result.result == EvaluateResult.Success