crackerjack 0.17.3__tar.gz → 0.18.0__tar.gz

{crackerjack-0.17.3 → crackerjack-0.18.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: crackerjack
-Version: 0.17.3
+Version: 0.18.0
 Summary: Default template for PDM package
 Keywords: black,ruff,mypy,creosote,refurb
 Author-Email: lesleslie <les@wedgwoodwebworks.com>

{crackerjack-0.17.3 → crackerjack-0.18.0}/crackerjack/__main__.py

@@ -34,6 +34,7 @@ class Options(BaseModel):
     all: BumpOption | None = None
     ai_agent: bool = False
     create_pr: bool = False
+    skip_hooks: bool = False
 
     @classmethod
     @field_validator("publish", "bump", mode="before")
@@ -83,6 +84,12 @@ cli_options = {
         help="Remove docstrings, line comments, and unnecessary whitespace.",
     ),
     "test": typer.Option(False, "-t", "--test", help="Run tests."),
+    "skip_hooks": typer.Option(
+        False,
+        "-s",
+        "--skip-hooks",
+        help="Skip running pre-commit hooks (useful with -t).",
+    ),
     "all": typer.Option(
         None,
         "-a",
@@ -118,6 +125,7 @@ def main(
     bump: BumpOption | None = cli_options["bump"],
     clean: bool = cli_options["clean"],
     test: bool = cli_options["test"],
+    skip_hooks: bool = cli_options["skip_hooks"],
     create_pr: bool = cli_options["create_pr"],
     ai_agent: bool = cli_options["ai_agent"],
 ) -> None:
@@ -132,6 +140,7 @@ def main(
         bump=bump,
         clean=clean,
         test=test,
+        skip_hooks=skip_hooks,
         all=all,
         ai_agent=ai_agent,
         create_pr=create_pr,

{crackerjack-0.17.3 → crackerjack-0.18.0}/crackerjack/crackerjack.py

@@ -44,6 +44,7 @@ class OptionsProtocol(t.Protocol):
     all: t.Any | None
     ai_agent: bool = False
     create_pr: bool = False
+    skip_hooks: bool = False
 
 
 @dataclass
@@ -499,7 +500,7 @@ class Crackerjack:
                 "--no-header",  # Reduce output noise
                 "--disable-warnings",  # Disable warning capture
                 "--durations=0",  # Show slowest tests to identify potential hanging tests
-                "--timeout=300",  # 5-minute timeout for tests
+                "--timeout=60",  # 1-minute timeout for tests
             ]
         )
         return test
@@ -796,8 +797,14 @@ class Crackerjack:
         if options.clean:
             actions_performed.append("clean_project")
 
-        self.project_manager.run_pre_commit()
-        actions_performed.append("run_pre_commit")
+        if not options.skip_hooks:
+            self.project_manager.run_pre_commit()
+            actions_performed.append("run_pre_commit")
+        else:
+            self.console.print(
+                "\n[yellow]Skipping pre-commit hooks as requested[/yellow]\n"
+            )
+            actions_performed.append("skip_pre_commit")
 
         self._run_tests(options)
         if options.test:

{crackerjack-0.17.3 → crackerjack-0.18.0}/crackerjack/pyproject.toml

@@ -152,7 +152,7 @@ pythonPlatform = "Darwin"
 
 [project]
 name = "crackerjack"
-version = "0.17.2"
+version = "0.17.4"
 description = "Default template for PDM package"
 requires-python = ">=3.13"
 readme = "README.md"

{crackerjack-0.17.3 → crackerjack-0.18.0}/pyproject.toml

@@ -171,7 +171,7 @@ pythonPlatform = "Darwin"
 
 [project]
 name = "crackerjack"
-version = "0.17.3"
+version = "0.18.0"
 description = "Default template for PDM package"
 requires-python = ">=3.13"
 readme = "README.md"

{crackerjack-0.17.3 → crackerjack-0.18.0}/tests/test_crackerjack.py

@@ -40,6 +40,7 @@ class OptionsForTesting:
     all: BumpOption | None = None
     ai_agent: bool = False
     create_pr: bool = False
+    skip_hooks: bool = False
 
 
 @pytest.fixture
@@ -229,6 +230,45 @@ class TestCrackerjackProcess:
             ("Skipping config updates" in call for call in console_print_calls)
         ), "Expected 'Skipping config updates' message was not printed"
 
+    def test_process_with_skip_hooks_option(
+        self,
+        mock_execute: MagicMock,
+        mock_console_print: MagicMock,
+        mock_input: MagicMock,
+        mock_project_manager_execute: MagicMock,
+        mock_config_manager_execute: MagicMock,
+        tmp_path: Path,
+        tmp_path_package: Path,
+        create_package_dir: None,
+        options_factory: t.Callable[..., OptionsForTesting],
+    ) -> None:
+        mock_execute.return_value.returncode = 0
+        mock_project_manager_execute.return_value.returncode = 0
+        mock_config_manager_execute.return_value.returncode = 0
+        options = options_factory(test=True, skip_hooks=True)
+
+        with (
+            patch.object(Crackerjack, "_run_tests") as mock_run_tests,
+            patch.object(ProjectManager, "run_pre_commit") as mock_run_pre_commit,
+        ):
+            mock_run_tests.side_effect = lambda opts: mock_console_print(
+                "\n\nRunning tests...\n"
+            )
+            cj = Crackerjack(dry_run=True)
+            cj.process(options)
+
+            mock_run_pre_commit.assert_not_called()
+
+            mock_run_tests.assert_called_once_with(options)
+
+        console_print_calls = [str(call) for call in mock_console_print.call_args_list]
+        assert any(("Running tests" in call for call in console_print_calls)), (
+            "Expected 'Running tests' message was not printed"
+        )
+        assert any(
+            ("Skipping pre-commit hooks" in call for call in console_print_calls)
+        ), "Expected 'Skipping pre-commit hooks' message was not printed"
+
     def test_process_with_bump_option(
         self,
         mock_execute: MagicMock,

{crackerjack-0.17.3 → crackerjack-0.18.0}/tests/test_crackerjack_runner.py

@@ -24,6 +24,7 @@ class MockOptions:
         self.all = kwargs.get("all")
         self.ai_agent = kwargs.get("ai_agent", False)
         self.create_pr = kwargs.get("create_pr", False)
+        self.skip_hooks = kwargs.get("skip_hooks", False)
 
 
 def test_create_crackerjack_runner() -> None:
@@ -124,3 +125,25 @@ def test_process_with_test_option(
     with patch("builtins.input", return_value="Test commit message"):
         with patch.object(Crackerjack, "_run_tests"):
             crackerjack.process(options)
+
+
+def test_process_with_test_and_skip_hooks_options(
+    mock_crackerjack: tuple[Crackerjack, dict[str, MagicMock]],
+) -> None:
+    crackerjack, mocks = mock_crackerjack
+
+    options = MockOptions(test=True, skip_hooks=True)
+
+    crackerjack.execute_command = MagicMock(
+        return_value=MagicMock(returncode=0, stdout="All tests passed", stderr="")
+    )
+
+    with patch("builtins.input", return_value="Test commit message"):
+        with patch.object(Crackerjack, "_run_tests"):
+            crackerjack.process(options)
+
+    mocks["project_manager"].run_pre_commit.assert_not_called()
+
+    mocks["console"].print.assert_any_call(
+        "\n[yellow]Skipping pre-commit hooks as requested[/yellow]\n"
+    )

{crackerjack-0.17.3 → crackerjack-0.18.0}/tests/test_main.py

@@ -223,6 +223,7 @@ def test_create_options() -> None:
         test=True,
         all=BumpOption.minor,
         create_pr=True,
+        skip_hooks=True,
     )
     assert test_options.commit
     assert test_options.interactive
@@ -236,6 +237,7 @@ def test_create_options() -> None:
     assert test_options.test
     assert test_options.all == BumpOption.minor
     assert test_options.create_pr
+    assert test_options.skip_hooks
 
 
 def test_conflicting_options(
@@ -296,3 +298,18 @@ def test_create_pr_option(
     assert result.exit_code == 0
     options = mock_crackerjack_process.process.call_args[0][0]
     assert options.create_pr
+
+
+def test_skip_hooks_option(
+    runner: CliRunner, mock_crackerjack_process: MagicMock
+) -> None:
+    result = runner.invoke(app, ["-s"])
+    assert result.exit_code == 0
+    mock_crackerjack_process.process.assert_called_once()
+    options = mock_crackerjack_process.process.call_args[0][0]
+    assert options.skip_hooks
+    mock_crackerjack_process.process.reset_mock()
+    result = runner.invoke(app, ["--skip-hooks"])
+    assert result.exit_code == 0
+    options = mock_crackerjack_process.process.call_args[0][0]
+    assert options.skip_hooks