Revert try/finally in test_run_interactive_from_environment

Assertion now passes (entrypoint=None added), so cleanup runs normally.

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

Author: tomerschwartz24

tests/test_cli.py

@@ -1365,28 +1365,26 @@ def test_run_with_env(self, skipper_runner_run_mock):
     @mock.patch("skipper.runner.run", autospec=True)
     def test_run_interactive_from_environment(self, skipper_runner_run_mock):
         os.environ["SKIPPER_INTERACTIVE"] = "True"
-        try:
-            command = ["ls", "-l"]
-            run_params = command
-            self._invoke_cli(global_params=self.global_params, subcmd="run", subcmd_params=run_params)
-            expected_fqdn_image = "build-container-image:build-container-tag"
-            skipper_runner_run_mock.assert_called_once_with(
-                command,
-                fqdn_image=expected_fqdn_image,
-                environment=[],
-                interactive=True,
-                name=None,
-                net=None,
-                publish=(),
-                volumes=None,
-                entrypoint=None,
-                workdir=None,
-                workspace=None,
-                use_cache=False,
-                env_file=(),
-            )
-        finally:
-            del os.environ["SKIPPER_INTERACTIVE"]
+        command = ["ls", "-l"]
+        run_params = command
+        self._invoke_cli(global_params=self.global_params, subcmd="run", subcmd_params=run_params)
+        expected_fqdn_image = "build-container-image:build-container-tag"
+        skipper_runner_run_mock.assert_called_once_with(
+            command,
+            fqdn_image=expected_fqdn_image,
+            environment=[],
+            interactive=True,
+            name=None,
+            net=None,
+            publish=(),
+            volumes=None,
+            entrypoint=None,
+            workdir=None,
+            workspace=None,
+            use_cache=False,
+            env_file=(),
+        )
+        del os.environ["SKIPPER_INTERACTIVE"]
 
     @mock.patch("subprocess.check_output", mock.MagicMock(autospec=True, return_value="1234567\n"))
     @mock.patch("skipper.runner.run", autospec=True)