630 dont create pyprojecttoml in usethis tool with remove

src/usethis/_core/tool.py

@@ -48,9 +48,9 @@ def use_codespell(*, remove: bool = False, how: bool = False) -> None:
         tool.print_how_to_use()
         return
 
-    ensure_pyproject_toml()
-
     if not remove:
+        ensure_pyproject_toml()
+
         if not PreCommitTool().is_used():
             tool.add_dev_deps()
             tool.update_bitbucket_steps()
@@ -74,9 +74,9 @@ def use_coverage_py(*, remove: bool = False, how: bool = False) -> None:
         tool.print_how_to_use()
         return
 
-    ensure_pyproject_toml()
-
     if not remove:
+        ensure_pyproject_toml()
+
         tool.add_test_deps()
         tool.add_configs()
         tool.print_how_to_use()
@@ -93,9 +93,9 @@ def use_deptry(*, remove: bool = False, how: bool = False) -> None:
         tool.print_how_to_use()
         return
 
-    ensure_pyproject_toml()
-
     if not remove:
+        ensure_pyproject_toml()
+
         tool.add_dev_deps()
         if PreCommitTool().is_used():
             tool.add_pre_commit_repo_configs()
@@ -118,9 +118,9 @@ def use_import_linter(*, remove: bool = False, how: bool = False) -> None:
         tool.print_how_to_use()
         return
 
-    ensure_pyproject_toml()
-
     if not remove:
+        ensure_pyproject_toml()
+
         tool.add_dev_deps()
         tool.add_configs()
         if PreCommitTool().is_used():
@@ -148,9 +148,9 @@ def use_pre_commit(*, remove: bool = False, how: bool = False) -> None:
     codespell_tool = CodespellTool()
     requirements_txt_tool = RequirementsTxtTool()
 
-    ensure_pyproject_toml()
-
     if not remove:
+        ensure_pyproject_toml()
+
         tool.add_dev_deps()
         _add_all_tools_pre_commit_configs()
 
@@ -234,9 +234,9 @@ def use_pyproject_fmt(*, remove: bool = False, how: bool = False) -> None:
         tool.print_how_to_use()
         return
 
-    ensure_pyproject_toml()
-
     if not remove:
+        ensure_pyproject_toml()
+
         if not PreCommitTool().is_used():
             tool.add_dev_deps()
             tool.update_bitbucket_steps()
@@ -260,10 +260,9 @@ def use_pyproject_toml(*, remove: bool = False, how: bool = False) -> None:
         tool.print_how_to_use()
         return
 
-    ensure_pyproject_toml()
-
     if not remove:
         ensure_pyproject_toml()
+
         ensure_pyproject_validity()
         tool.print_how_to_use()
     else:
@@ -277,11 +276,11 @@ def use_pytest(*, remove: bool = False, how: bool = False) -> None:
         tool.print_how_to_use()
         return
 
-    ensure_pyproject_toml()
-
     rule_config = tool.get_rule_config()
 
     if not remove:
+        ensure_pyproject_toml()
+
         tool.add_test_deps()
         tool.add_configs()
         if RuffTool().is_used():
@@ -318,11 +317,11 @@ def use_requirements_txt(*, remove: bool = False, how: bool = False) -> None:
         tool.print_how_to_use()
         return
 
-    ensure_pyproject_toml()
-
     path = Path.cwd() / "requirements.txt"
 
     if not remove:
+        ensure_pyproject_toml()
+
         is_pre_commit = PreCommitTool().is_used()
 
         if is_pre_commit:
@@ -368,8 +367,6 @@ def use_ruff(*, remove: bool = False, how: bool = False, minimal: bool = False)
         tool.print_how_to_use()
         return
 
-    ensure_pyproject_toml()
-
     # Only add ruff rules if the user doesn't already have a select/ignore list.
     # Otherwise, we should leave them alone.
 
@@ -384,6 +381,8 @@ def use_ruff(*, remove: bool = False, how: bool = False, minimal: bool = False)
         rule_config = RuleConfig()
 
     if not remove:
+        ensure_pyproject_toml()
+
         tool.add_dev_deps()
         tool.add_configs()
         tool.select_rules(rule_config.get_all_selected())

tests/usethis/_core/test_core_tool.py

@@ -1,6 +1,9 @@
 import os
 import subprocess
+import unittest
+import unittest.mock
 from pathlib import Path
+from unittest.mock import MagicMock
 
 import pytest
 
@@ -14,6 +17,7 @@
     use_import_linter,
     use_pre_commit,
     use_pyproject_fmt,
+    use_pyproject_toml,
     use_pytest,
     use_requirements_txt,
     use_ruff,
@@ -257,6 +261,16 @@ def test_config_file(
             assert not err
             assert out == ("✔ Removing Codespell config from 'pyproject.toml'.\n")
 
+        def test_doesnt_add_pyproject(self, tmp_path: Path):
+            # No pyproject.toml file to begin with...
+
+            # Act
+            with change_cwd(tmp_path), files_manager():
+                use_codespell(remove=True)
+
+            # Assert
+            assert not (tmp_path / "pyproject.toml").exists()
+
 
 class TestCoverage:
     class TestAdd:
@@ -471,6 +485,20 @@ def test_pytest_integration(
                     "✔ Removing dependencies 'coverage', 'pytest-cov' from the 'test' group in \n'pyproject.toml'.\n"
                 )
 
+        def test_doesnt_add_pyproject(
+            self, tmp_path: Path, capfd: pytest.CaptureFixture[str]
+        ):
+            # No pyproject.toml file to begin with...
+
+            # Act
+            with change_cwd(tmp_path), files_manager():
+                use_coverage_py(remove=True)
+
+            # Assert
+            assert not (tmp_path / "pyproject.toml").exists()
+            out, err = capfd.readouterr()
+            assert not out
+
 
 class TestDeptry:
     class TestAdd:
@@ -695,6 +723,21 @@ def test_roundtrip(self, uv_init_dir: Path):
 """
             )
 
+        def test_doesnt_add_pyproject(
+            self, tmp_path: Path, capfd: pytest.CaptureFixture[str]
+        ):
+            # No pyproject.toml file to begin with...
+
+            # Act
+            with change_cwd(tmp_path), files_manager():
+                use_deptry(remove=True)
+
+            # Assert
+            assert not (tmp_path / "pyproject.toml").exists()
+            out, err = capfd.readouterr()
+            assert not out
+            assert not err
+
     class TestPreCommitIntegration:
         @pytest.mark.usefixtures("_vary_network_conn")
         def test_pre_commit_first(
@@ -1295,6 +1338,16 @@ def test_config_file(self, uv_init_repo_dir: Path):
             # Assert
             assert not (uv_init_repo_dir / ".importlinter").exists()
 
+        def test_doesnt_add_pyproject(self, tmp_path: Path):
+            # No pyproject.toml file to begin with...
+
+            # Act
+            with change_cwd(tmp_path), files_manager():
+                use_import_linter(remove=True)
+
+            # Assert
+            assert not (tmp_path / "pyproject.toml").exists()
+
     class TestPreCommitIntegration:
         def test_config(
             self, uv_init_repo_dir: Path, capfd: pytest.CaptureFixture[str]
@@ -1626,6 +1679,20 @@ def test_codepsell_used(
                     "☐ Run 'codespell' to run the Codespell spellchecker.\n"
                 )
 
+        def test_doesnt_add_pyproject(
+            self, uv_init_repo_dir: Path, capfd: pytest.CaptureFixture[str]
+        ):
+            # Arrange
+            # We use uv_init_repo_dir to get a git repo
+            (uv_init_repo_dir / "pyproject.toml").unlink()
+
+            # Act
+            with change_cwd(uv_init_repo_dir), files_manager():
+                use_pre_commit(remove=True)
+
+            # Assert
+            assert not (uv_init_repo_dir / "pyproject.toml").exists()
+
     class TestBitbucketCIIntegration:
         def test_prexisting(self, uv_init_repo_dir: Path):
             # Arrange
@@ -1848,6 +1915,18 @@ def test_bitbucket_integration(self, uv_init_dir: Path):
             contents = (uv_init_dir / "bitbucket-pipelines.yml").read_text()
             assert "pyproject-fmt" not in contents
 
+        def test_doesnt_add_pyproject(
+            self, tmp_path: Path, capfd: pytest.CaptureFixture[str]
+        ):
+            # No pyproject.toml file to begin with...
+
+            # Act
+            with change_cwd(tmp_path), files_manager():
+                use_pyproject_fmt(remove=True)
+
+            # Assert
+            assert not (tmp_path / "pyproject.toml").exists()
+
     class TestPreCommitIntegration:
         @pytest.mark.usefixtures("_vary_network_conn")
         def test_use_first(self, uv_init_repo_dir: Path):
@@ -1964,6 +2043,27 @@ def test_extra_warning_when_config_exists(
                 )
 
 
+class TestPyprojectTOML:
+    class TestRemove:
+        def test_doesnt_invoke_ensure_pyproject_toml(self, tmp_path: Path):
+            # Arrange
+            # Mock the ensure_pyproject_toml function to raise an error
+
+            mock = MagicMock()
+
+            # Act
+            with (
+                unittest.mock.patch("usethis._core.tool.ensure_pyproject_toml", mock),
+                change_cwd(tmp_path),
+                files_manager(),
+            ):
+                use_pyproject_toml(remove=True)
+
+            # Assert
+            assert not mock.called
+            assert not (tmp_path / "pyproject.toml").exists()
+
+
 class TestPytest:
     class TestAdd:
         @pytest.mark.usefixtures("_vary_network_conn")
@@ -2358,6 +2458,17 @@ def test_coverage_integration(
                 "☐ Run 'uv run coverage help' to see available Coverage.py commands.\n"
             )
 
+        def test_doesnt_create_pyproject_toml(self, tmp_path: Path):
+            # Arrange
+            (tmp_path / "pyproject.toml").unlink(missing_ok=True)
+
+            # Act
+            with change_cwd(tmp_path), files_manager():
+                use_pytest(remove=True)
+
+            # Assert
+            assert not (tmp_path / "pyproject.toml").exists()
+
     class TestUpdateBitbucketSteps:
         def test_new_file(self, uv_init_dir: Path, capfd: pytest.CaptureFixture[str]):
             with change_cwd(uv_init_dir), files_manager():
@@ -2748,6 +2859,17 @@ def test_roundtrip(self, uv_init_dir: Path):
 """
             )
 
+        def test_doesnt_create_pyproject_toml(self, tmp_path: Path):
+            # Arrange
+            (tmp_path / "pyproject.toml").unlink(missing_ok=True)
+
+            # Act
+            with change_cwd(tmp_path), files_manager():
+                use_ruff(remove=True)
+
+            # Assert
+            assert not (tmp_path / "pyproject.toml").exists()
+
     class TestPrecommitIntegration:
         @pytest.mark.usefixtures("_vary_network_conn")
         def test_use_first(self, uv_init_repo_dir: Path):
@@ -3045,3 +3167,11 @@ def test_roundtrip(self, tmp_path: Path):
 
             # Assert
             assert not (tmp_path / "requirements.txt").exists()
+
+        def test_doesnt_create_pyproject_toml(self, tmp_path: Path):
+            # Act
+            with change_cwd(tmp_path), files_manager():
+                use_requirements_txt(remove=True)
+
+            # Assert
+            assert not (tmp_path / "pyproject.toml").exists()