fix: resolve lint violations and improve test coverage for helm-removal changes

Break long lines in kubectl_test.py to stay within 120 character limit
and remove unused OutputCallback import in operations_test.py.

Generated-By: Forge/20260416_161341_490145_52646d64

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: raballew

python/packages/jumpstarter-kubernetes/jumpstarter_kubernetes/cluster/kubectl_test.py

@@ -452,7 +452,14 @@ async def test_get_cluster_info_invalid_json(self, mock_get_contexts):
     @patch("jumpstarter_kubernetes.cluster.kubectl.get_kubectl_contexts")
     async def test_get_cluster_info_context_not_found(self, mock_get_contexts):
         mock_get_contexts.return_value = [
-            {"name": "other-context", "cluster": "other", "server": "https://other", "user": "u", "namespace": "default", "current": False}
+            {
+                "name": "other-context",
+                "cluster": "other",
+                "server": "https://other",
+                "user": "u",
+                "namespace": "default",
+                "current": False,
+            }
         ]
 
         result = await get_cluster_info("missing-context")
@@ -465,9 +472,18 @@ async def test_get_cluster_info_context_not_found(self, mock_get_contexts):
     @patch("jumpstarter_kubernetes.cluster.kubectl.get_kubectl_contexts")
     @patch("jumpstarter_kubernetes.cluster.kubectl.run_command")
     @patch("jumpstarter_kubernetes.cluster.kubectl.check_jumpstarter_installation")
-    async def test_get_cluster_info_inaccessible_cluster(self, mock_check_jumpstarter, mock_run_command, mock_get_contexts):
+    async def test_get_cluster_info_inaccessible_cluster(
+        self, mock_check_jumpstarter, mock_run_command, mock_get_contexts
+    ):
         mock_get_contexts.return_value = [
-            {"name": "test-context", "cluster": "test-cluster", "server": "https://test.example.com", "user": "test-user", "namespace": "default", "current": False}
+            {
+                "name": "test-context",
+                "cluster": "test-cluster",
+                "server": "https://test.example.com",
+                "user": "test-user",
+                "namespace": "default",
+                "current": False,
+            }
         ]
         mock_run_command.return_value = (1, "", "connection refused")
 
@@ -482,9 +498,18 @@ async def test_get_cluster_info_inaccessible_cluster(self, mock_check_jumpstarte
     @patch("jumpstarter_kubernetes.cluster.kubectl.get_kubectl_contexts")
     @patch("jumpstarter_kubernetes.cluster.kubectl.run_command")
     @patch("jumpstarter_kubernetes.cluster.kubectl.check_jumpstarter_installation")
-    async def test_get_cluster_info_version_parse_failure(self, mock_check_jumpstarter, mock_run_command, mock_get_contexts):
+    async def test_get_cluster_info_version_parse_failure(
+        self, mock_check_jumpstarter, mock_run_command, mock_get_contexts
+    ):
         mock_get_contexts.return_value = [
-            {"name": "test-context", "cluster": "test-cluster", "server": "https://test.example.com", "user": "test-user", "namespace": "default", "current": False}
+            {
+                "name": "test-context",
+                "cluster": "test-cluster",
+                "server": "https://test.example.com",
+                "user": "test-user",
+                "namespace": "default",
+                "current": False,
+            }
         ]
         mock_run_command.return_value = (0, "not json", "")
         mock_check_jumpstarter.return_value = V1Alpha1JumpstarterInstance(installed=False)
@@ -497,9 +522,18 @@ async def test_get_cluster_info_version_parse_failure(self, mock_check_jumpstart
     @pytest.mark.asyncio
     @patch("jumpstarter_kubernetes.cluster.kubectl.get_kubectl_contexts")
     @patch("jumpstarter_kubernetes.cluster.kubectl.run_command")
-    async def test_get_cluster_info_version_command_runtime_error(self, mock_run_command, mock_get_contexts):
+    async def test_get_cluster_info_version_command_runtime_error(
+        self, mock_run_command, mock_get_contexts
+    ):
         mock_get_contexts.return_value = [
-            {"name": "test-context", "cluster": "test-cluster", "server": "https://test.example.com", "user": "test-user", "namespace": "default", "current": False}
+            {
+                "name": "test-context",
+                "cluster": "test-cluster",
+                "server": "https://test.example.com",
+                "user": "test-user",
+                "namespace": "default",
+                "current": False,
+            }
         ]
         mock_run_command.side_effect = RuntimeError("command failed")
 
@@ -578,8 +612,20 @@ async def test_list_clusters_custom_parameters(self, mock_get_contexts):
     @patch("jumpstarter_kubernetes.cluster.kubectl.get_cluster_info")
     async def test_list_clusters_with_type_filter(self, mock_get_cluster_info, mock_get_contexts):
         mock_get_contexts.return_value = [
-            {"name": "kind-ctx", "cluster": "kind-cluster", "server": "https://kind", "user": "u", "current": True},
-            {"name": "remote-ctx", "cluster": "remote-cluster", "server": "https://remote", "user": "u", "current": False},
+            {
+                "name": "kind-ctx",
+                "cluster": "kind-cluster",
+                "server": "https://kind",
+                "user": "u",
+                "current": True,
+            },
+            {
+                "name": "remote-ctx",
+                "cluster": "remote-cluster",
+                "server": "https://remote",
+                "user": "u",
+                "current": False,
+            },
         ]
 
         kind_info = V1Alpha1ClusterInfo(

python/packages/jumpstarter-kubernetes/jumpstarter_kubernetes/cluster/operations_test.py

@@ -186,7 +186,6 @@ async def test_create_cluster_and_install_force_recreate_confirmed(
 
     @pytest.mark.asyncio
     async def test_create_cluster_and_install_force_recreate_cancelled(self):
-        from jumpstarter_kubernetes.callbacks import OutputCallback
         from jumpstarter_kubernetes.exceptions import ClusterOperationError
 
         class RejectingCallback: