Refactor tests to reduce creation of venvs for speed

src/usethis/_core/tool.py

@@ -5,6 +5,7 @@
     remove_bitbucket_pytest_steps,
     update_bitbucket_pytest_steps,
 )
+from usethis._config import usethis_config
 from usethis._console import box_print, tick_print
 from usethis._integrations.bitbucket.steps import (
     add_bitbucket_steps_in_default,
@@ -170,9 +171,6 @@ def use_pre_commit(*, remove: bool = False) -> None:
             remove_bitbucket_steps_from_default(tool.get_bitbucket_steps())
             _add_bitbucket_linter_steps_to_default()
 
-        # Need pre-commit to be installed so we can uninstall hooks
-        add_deps_to_group(tool.dev_deps, "dev")
-
         uninstall_pre_commit_hooks()
 
         remove_pre_commit_config()
@@ -313,19 +311,20 @@ def use_requirements_txt(*, remove: bool = False) -> None:
 
         if not path.exists():
             # N.B. this is where a task runner would come in handy, to reduce duplication.
-            if not (Path.cwd() / "uv.lock").exists():
+            if not (Path.cwd() / "uv.lock").exists() and not usethis_config.frozen:
                 tick_print("Writing 'uv.lock'.")
                 call_uv_subprocess(["lock"])
 
-            tick_print("Writing 'requirements.txt'.")
-            call_uv_subprocess(
-                [
-                    "export",
-                    "--frozen",
-                    "--no-dev",
-                    "--output-file=requirements.txt",
-                ]
-            )
+            if not usethis_config.frozen:
+                tick_print("Writing 'requirements.txt'.")
+                call_uv_subprocess(
+                    [
+                        "export",
+                        "--frozen",
+                        "--no-dev",
+                        "--output-file=requirements.txt",
+                    ]
+                )
 
         if not is_pre_commit:
             _requirements_txt_instructions_basic()

src/usethis/_integrations/pre_commit/core.py

@@ -24,7 +24,7 @@ def install_pre_commit_hooks() -> None:
     in a git repo.
     """
     if usethis_config.frozen:
-        box_print("Run 'uv run pre-commit install' to register pre-commit with git.")
+        box_print("Run 'pre-commit install' to register pre-commit.")
         return
 
     tick_print("Ensuring pre-commit is installed to Git.")
@@ -47,16 +47,17 @@ def install_pre_commit_hooks() -> None:
 def uninstall_pre_commit_hooks() -> None:
     """Uninstall pre-commit hooks.
 
-    Note that this requires pre-commit to be installed. It also requires the user to be
-    in a git repo.
+    Note that this requires the user to be in a git repo.
     """
     if usethis_config.frozen:
-        box_print("Run 'uvx pre-commit uninstall' to deregister pre-commit with git.")
+        box_print(
+            "Run 'uv run --with pre-commit pre-commit uninstall' to deregister pre-commit."
+        )
         return
 
     tick_print("Ensuring pre-commit hooks are uninstalled.")
     try:
-        call_uv_subprocess(["run", "pre-commit", "uninstall"])
+        call_uv_subprocess(["run", "--with", "pre-commit", "pre-commit", "uninstall"])
     except UVSubprocessFailedError as err:
         msg = f"Failed to uninstall pre-commit hooks:\n{err}"
         raise PreCommitInstallationError(msg) from None

src/usethis/_integrations/pre_commit/hooks.py

@@ -137,7 +137,7 @@ def _get_placeholder_repo_config() -> LocalRepo:
             HookDefinition(
                 id=_PLACEHOLDER_ID,
                 name="Placeholder - add your own hooks!",
-                entry="""uv run python -c "print('hello world!')\"""",
+                entry="""uv run --frozen python -c "print('hello world!')\"""",
                 language=Language("system"),
             )
         ],

src/usethis/_integrations/uv/call.py

@@ -13,10 +13,10 @@ def call_uv_subprocess(args: list[str]) -> str:
     read_pyproject_toml_from_path.cache_clear()
 
     if usethis_config.frozen and args[0] in {
+        # Note, not "lock", for which the --frozen flags has quite a different effect
         "add",
         "remove",
         "sync",
-        "lock",
         "export",
         "tree",
         "run",

src/usethis/_integrations/uv/deps.py

@@ -2,7 +2,7 @@
 from pydantic import BaseModel, TypeAdapter
 
 from usethis._config import usethis_config
-from usethis._console import tick_print
+from usethis._console import box_print, tick_print
 from usethis._integrations.pyproject.core import (
     append_config_list,
     get_config_value,
@@ -107,6 +107,9 @@ def add_deps_to_group(deps: list[Dependency], group: str) -> None:
         f"Adding dependenc{ies} {deps_str} to the '{group}' group in 'pyproject.toml'."
     )
 
+    if usethis_config.frozen:
+        box_print(f"Install the dependenc{ies} {deps_str}.")
+
     register_default_group(group)  # Register the group before adding dependencies
 
     for dep in to_add_deps: