RepoAuditor 0.2.6__tar.gz → 0.2.8__tar.gz

{repoauditor-0.2.6 → repoauditor-0.2.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: RepoAuditor
-Version: 0.2.6
+Version: 0.2.8
 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.6 → repoauditor-0.2.8}/pyproject.toml

@@ -104,12 +104,12 @@ select = [
     "PIE",
     "T20",
     "PYI",
-    # "PT", # 5
+    "PT",
     "Q",
     "RSE",
     "RET",
     "SLF",
-    # "SIM", # 4
+    "SIM",
     "SLOT",
     # "TID", # 9
     "TD",

{repoauditor-0.2.6 → repoauditor-0.2.8}/src/RepoAuditor/EntryPoint.py

@@ -293,11 +293,7 @@ def EntryPoint(
                 display_rationale=not no_rationale,
             )
         except Exception as ex:
-            if dm.is_debug:
-                error = traceback.format_exc()
-            else:
-                error = str(ex)
-
+            error = traceback.format_exc() if dm.is_debug else str(ex)
             dm.WriteError(error)
 
 

{repoauditor-0.2.6 → repoauditor-0.2.8}/src/RepoAuditor/ExecuteModules.py

@@ -71,7 +71,7 @@ def Execute(
             elif module_info.module.style == ExecutionStyle.Sequential:
                 sequential.append((index, module_info))
             else:
-                assert False, module_info.module.style  # pragma: no cover
+                raise RuntimeError(module_info.module.style)  # pragma: no cover
 
         if len(parallel) == 1:
             sequential.append(parallel[0])
@@ -179,69 +179,71 @@ def Execute(
                 all_results[all_results_index] = transformed_results
 
         for index, (all_results_index, module_info) in enumerate(sequential):
-            with modules_dm.Nested(
-                "Processing '{}' ({} of {})...".format(
-                    module_info.module.name,
-                    index + 1 + len(parallel),
-                    len(module_infos),
-                ),
-            ) as this_module_dm:
+            with (
+                modules_dm.Nested(
+                    "Processing '{}' ({} of {})...".format(
+                        module_info.module.name,
+                        index + 1 + len(parallel),
+                        len(module_infos),
+                    ),
+                ) as this_module_dm,
                 # rich.progress needs to output to sys.stdout
-                with this_module_dm.YieldStdout() as stdout_context:
-                    stdout_context.persist_content = False
-
-                    # Technically speaking, it would be more correct to use `stdout_context.stream` here
-                    # rather than referencing `sys.stdout` directly, but it is really hard to work with mocked
-                    # stream as mocks will create mocks for everything called on the mock. Use sys.stdout
-                    # directly to avoid that particular problem.
-                    from unittest.mock import Mock, MagicMock
-
-                    assert stdout_context.stream is sys.stdout or isinstance(
-                        stdout_context.stream, (Mock, MagicMock)
-                    ), stdout_context.stream
-
-                    with Progress(
-                        *Progress.get_default_columns(),
-                        TimeElapsedColumn(),
-                        "{task.fields[status]}",
-                        console=Capabilities.Get(sys.stdout).CreateRichConsole(sys.stdout),
-                        transient=True,
-                        refresh_per_second=10,
-                    ) as progress_bar:
-                        progress_bar_task_id = progress_bar.add_task(
-                            stdout_context.line_prefix,
-                            status="",
-                            total=module_info.module.GetNumRequirements(),
-                            visible=True,
+                this_module_dm.YieldStdout() as stdout_context,
+            ):
+                stdout_context.persist_content = False
+
+                # Technically speaking, it would be more correct to use `stdout_context.stream` here
+                # rather than referencing `sys.stdout` directly, but it is really hard to work with mocked
+                # stream as mocks will create mocks for everything called on the mock. Use sys.stdout
+                # directly to avoid that particular problem.
+                from unittest.mock import Mock, MagicMock
+
+                assert stdout_context.stream is sys.stdout or isinstance(
+                    stdout_context.stream, (Mock, MagicMock)
+                ), stdout_context.stream
+
+                with Progress(
+                    *Progress.get_default_columns(),
+                    TimeElapsedColumn(),
+                    "{task.fields[status]}",
+                    console=Capabilities.Get(sys.stdout).CreateRichConsole(sys.stdout),
+                    transient=True,
+                    refresh_per_second=10,
+                ) as progress_bar:
+                    progress_bar_task_id = progress_bar.add_task(
+                        stdout_context.line_prefix,
+                        status="",
+                        total=module_info.module.GetNumRequirements(),
+                        visible=True,
+                    )
+
+                    # ----------------------------------------------------------------------
+                    def OnStatus(
+                        num_completed: int,
+                        num_success: int,
+                        num_error: int,
+                        num_warning: int,
+                        num_does_not_apply: int,
+                    ) -> None:
+                        progress_bar.update(
+                            progress_bar_task_id,
+                            completed=num_completed,
+                            status=_CreateStatusString(
+                                num_success,
+                                num_error,
+                                num_warning,
+                                num_does_not_apply,
+                            ),
                         )
 
-                        # ----------------------------------------------------------------------
-                        def OnStatus(
-                            num_completed: int,
-                            num_success: int,
-                            num_error: int,
-                            num_warning: int,
-                            num_does_not_apply: int,
-                        ) -> None:
-                            progress_bar.update(
-                                progress_bar_task_id,
-                                completed=num_completed,
-                                status=_CreateStatusString(
-                                    num_success,
-                                    num_error,
-                                    num_warning,
-                                    num_does_not_apply,
-                                ),
-                            )
-
-                        # ----------------------------------------------------------------------
-
-                        evaluate_results = Evaluate(module_info, OnStatus)
-
-                        assert all_results[all_results_index] is None
-                        all_results[all_results_index] = evaluate_results
-
-                        this_module_dm.result = CalcResultInfo(evaluate_results)[0]
+                    # ----------------------------------------------------------------------
+
+                    evaluate_results = Evaluate(module_info, OnStatus)
+
+                    assert all_results[all_results_index] is None
+                    all_results[all_results_index] = evaluate_results
+
+                    this_module_dm.result = CalcResultInfo(evaluate_results)[0]
 
     final_results: list[list[Module.EvaluateInfo]] = []
 
@@ -282,7 +284,7 @@ def DisplayResults(
             elif result.result == EvaluateResult.DoesNotApply:
                 num_does_not_apply += 1
             else:
-                assert False, result.result  # pragma: no cover
+                raise RuntimeError(result.result)  # pragma: no cover
 
         num_requirements = module.GetNumRequirements()
         assert num_requirements
@@ -319,7 +321,7 @@ def DisplayResults(
             elif result.result == EvaluateResult.DoesNotApply:
                 border_color = ""
             else:
-                assert False, result.result  # pragma: no cover
+                raise RuntimeError(result.result)  # pragma: no cover
 
             content: list[str] = []
 

{repoauditor-0.2.6 → repoauditor-0.2.8}/src/RepoAuditor/Impl/ParallelSequentialProcessor.py

@@ -72,7 +72,7 @@ def _Impl(
         elif execution_style == ExecutionStyle.Sequential:
             sequential.append((index, item))
         else:
-            assert False, execution_style  # pragma: no cover
+            raise RuntimeError(execution_style)  # pragma: no cover
 
     if len(parallel) == 1:
         sequential.append(parallel[0])

{repoauditor-0.2.6 → repoauditor-0.2.8}/src/RepoAuditor/Plugins/GitHub/ClassicBranchProtectionQuery.py

@@ -117,7 +117,7 @@ class ClassicBranchProtectionQuery(Query):
         self,
         module_data: dict[str, Any],
     ) -> Optional[dict[str, Any]]:
-        branch = module_data.get("branch", None)
+        branch = module_data.get("branch")
         if branch is None:
             # Get the default branch name
             response = module_data["session"].get("")

{repoauditor-0.2.6 → repoauditor-0.2.8}/src/RepoAuditor/Plugins/GitHub/Module.py

@@ -69,7 +69,7 @@ class GitHubModule(Module):
     # ----------------------------------------------------------------------
     @override
     def GenerateInitialData(self, dynamic_args: dict[str, Any]) -> Optional[dict[str, Any]]:
-        dynamic_args["session"] = _GitHubSession(dynamic_args["url"], dynamic_args.get("pat", None))
+        dynamic_args["session"] = _GitHubSession(dynamic_args["url"], dynamic_args.get("pat"))
 
         return dynamic_args
 

{repoauditor-0.2.6 → repoauditor-0.2.8}/src/RepoAuditor/Query.py

@@ -104,7 +104,7 @@ class Query(ABC):
                     status_info.num_warning += 1
                     return_code = 1
                 else:
-                    assert False, result_info.result  # pragma: no cover
+                    raise RuntimeError(result_info.result)  # pragma: no cover
 
                 status_func(*status_info.__dict__.values())
 

{repoauditor-0.2.6 → repoauditor-0.2.8}/src/RepoAuditor/__init__.py

@@ -3,4 +3,4 @@ APP_NAME = "RepoAuditor"
 # Note that this value will be overwritten by calls to `python ../../Build.py update_version` based
 # on changes observed in the git repository. The default value below will be used until the value
 # here is explicitly updated by the Continuous Integration system.
-__version__ = "0.2.6"
+__version__ = "0.2.8"