Move test to CLI level and verify import pipeline errors are handled

Copilot · nathanjmcdougall · Copilot · commit 833c68c5a718 · 2025-11-05T08:04:04.000Z
Co-authored-by: nathanjmcdougall &lt;18602289+nathanjmcdougall@users.noreply.github.com&gt;
diff --git a/tests/usethis/_integrations/ci/bitbucket/test_steps.py b/tests/usethis/_integrations/ci/bitbucket/test_steps.py
@@ -648,27 +648,6 @@ def test_script_items_canonical_order_with_existing_non_canonical_items(
 """
         )
 
-    def test_import_pipeline_raises_error(self, tmp_path: Path):
-        """Test that add_bitbucket_step_in_default raises error for import pipelines."""
-        # Arrange
-        (tmp_path / "bitbucket-pipelines.yml").write_text(
-            """\
-image: atlassian/default-image:3
-pipelines:
-    default:
-        import: shared-pipeline:master:share-pipeline-1
-"""
-        )
-
-        # Act, Assert
-        with change_cwd(tmp_path), pytest.raises(UnexpectedImportPipelineError):
-            add_bitbucket_step_in_default(
-                Step(
-                    name="Greeting",
-                    script=Script(["echo 'Hello, world!'"]),
-                )
-            )
-
 
 class TestRemoveBitbucketStepFromDefault:
     def test_remove_remove_one_step(self, tmp_path: Path):
diff --git a/tests/usethis/_ui/interface/test_interface_ci.py b/tests/usethis/_ui/interface/test_interface_ci.py
@@ -135,3 +135,26 @@ def test_invalid_pyproject_toml(self, tmp_path: Path):
 
         # Assert
         assert result.exit_code == 1, result.output
+
+    def test_import_pipeline_error_handled(self, tmp_path: Path):
+        """Test that import pipeline errors are handled gracefully at CLI level."""
+        # Arrange - create a bitbucket-pipelines.yml with an import pipeline
+        (tmp_path / "bitbucket-pipelines.yml").write_text(
+            """\
+image: atlassian/default-image:3
+pipelines:
+    default:
+        import: shared-pipeline:master:share-pipeline-1
+"""
+        )
+        # Add a tool so update_bitbucket_steps gets called
+        (tmp_path / "pytest.ini").touch()
+
+        # Act
+        runner = CliRunner()
+        with change_cwd(tmp_path):
+            result = runner.invoke_safe(app, ["--backend=none"])
+
+        # Assert - error should be caught and handled, not propagate as unhandled exception
+        assert result.exit_code == 1, result.output
+        assert "import pipeline" in result.output.lower()
