fix tests

Author: yurijmikhalevich

tests/tests_app/runners/test_cloud.py

@@ -1422,9 +1422,6 @@ def test_open(self, monkeypatch):
 
         mock_client.cloud_space_service_create_cloud_space.return_value = V1CloudSpace(id="cloudspace_id")
         mock_client.cloud_space_service_create_lightning_run.return_value = V1LightningRun(id="run_id")
-        mock_client.cloud_space_service_create_lightning_run_instance.return_value = Externalv1LightningappInstance(
-            id="instance_id"
-        )
 
         mock_client.cluster_service_list_clusters.return_value = V1ListClustersResponse([Externalv1Cluster(id="test")])
         cloud_backend = mock.MagicMock()
@@ -1445,9 +1442,6 @@ def test_open(self, monkeypatch):
             cloudspace_id="cloudspace_id",
             body=mock.ANY,
         )
-        mock_client.cloud_space_service_create_lightning_run_instance.assert_called_once_with(
-            project_id="test-project-id", cloudspace_id="cloudspace_id", id="run_id", body=mock.ANY
-        )
 
         assert mock_client.cloud_space_service_create_cloud_space.call_args.kwargs["body"].name == "test_space"
 
@@ -1565,10 +1559,6 @@ def test_reopen(self, monkeypatch, capsys):
             body=mock.ANY,
         )
 
-        mock_client.lightningapp_instance_service_update_lightningapp_instance_release.assert_called_once_with(
-            project_id="test-project-id", id="instance_id", body=Body4(release_id="run_id")
-        )
-
         out, _ = capsys.readouterr()
         assert "will not overwrite the files in your CloudSpace." in out
 
@@ -2012,12 +2002,11 @@ def run(self):
 
 
 @pytest.mark.parametrize(
-    "project, cloudspace_name, run_instance, user, tab, lightning_cloud_url, expected_url",
+    "project, run_instance, user, tab, lightning_cloud_url, expected_url",
     [
         # Old style
         (
             V1Membership(),
-            "any",
             Externalv1LightningappInstance(id="test-app-id"),
             V1GetUserResponse(username="tester", features=V1UserFeatures()),
             "logs",
@@ -2026,7 +2015,6 @@ def run(self):
         ),
         (
             V1Membership(),
-            "any",
             Externalv1LightningappInstance(id="test-app-id"),
             V1GetUserResponse(username="tester", features=V1UserFeatures()),
             "logs",
@@ -2036,25 +2024,58 @@ def run(self):
         # New style
         (
             V1Membership(name="tester's project"),
-            "test/app",
-            Externalv1LightningappInstance(),
+            Externalv1LightningappInstance(name="test/job"),
             V1GetUserResponse(username="tester", features=V1UserFeatures(project_selector=True)),
             "logs",
             "https://lightning.ai",
-            "https://lightning.ai/tester/tester%27s_project/apps/test~app/logs",
+            "https://lightning.ai/tester/tester%27s%20project/jobs/test%2Fjob/logs",
         ),
         (
             V1Membership(name="tester's project"),
-            "test/app",
-            Externalv1LightningappInstance(),
+            Externalv1LightningappInstance(name="test/job"),
             V1GetUserResponse(username="tester", features=V1UserFeatures(project_selector=True)),
             "logs",
+            "https://localhost:9800",
+            "https://localhost:9800/tester/tester%27s%20project/jobs/test%2Fjob/logs",
+        ),
+    ],
+)
+def test_get_app_url(monkeypatch, project, run_instance, user, tab, lightning_cloud_url, expected_url):
+    mock_client = mock.MagicMock()
+    mock_client.auth_service_get_user.return_value = user
+    cloud_backend = mock.MagicMock(client=mock_client)
+    monkeypatch.setattr(backends, "CloudBackend", mock.MagicMock(return_value=cloud_backend))
+
+    runtime = CloudRuntime()
+
+    with mock.patch(
+        "lightning.app.runners.cloud.get_lightning_cloud_url", mock.MagicMock(return_value=lightning_cloud_url)
+    ):
+        assert runtime._get_app_url(project, run_instance, tab) == expected_url
+
+
+@pytest.mark.parametrize(
+    "user, project, cloudspace_name, tab, lightning_cloud_url, expected_url",
+    [
+        (
+            V1GetUserResponse(username="tester", features=V1UserFeatures()),
+            V1Membership(name="default-project"),
+            "test/cloudspace",
+            "code",
+            "https://lightning.ai",
+            "https://lightning.ai/tester/default-project/apps/test%2Fcloudspace/code",
+        ),
+        (
+            V1GetUserResponse(username="tester", features=V1UserFeatures()),
+            V1Membership(name="Awesome Project"),
+            "The Best CloudSpace ever",
+            "web-ui",
             "http://localhost:9800",
-            "http://localhost:9800/tester/tester%27s_project/apps/test~app/logs",
+            "http://localhost:9800/tester/Awesome%20Project/apps/The%20Best%20CloudSpace%20ever/web-ui",
         ),
     ],
 )
-def test_get_app_url(monkeypatch, project, cloudspace_name, run_instance, user, tab, lightning_cloud_url, expected_url):
+def test_get_cloudspace_url(monkeypatch, user, project, cloudspace_name, tab, lightning_cloud_url, expected_url):
     mock_client = mock.MagicMock()
     mock_client.auth_service_get_user.return_value = user
     cloud_backend = mock.MagicMock(client=mock_client)
@@ -2065,4 +2086,4 @@ def test_get_app_url(monkeypatch, project, cloudspace_name, run_instance, user,
     with mock.patch(
         "lightning.app.runners.cloud.get_lightning_cloud_url", mock.MagicMock(return_value=lightning_cloud_url)
     ):
-        assert runtime._get_app_url(project, cloudspace_name, run_instance, tab) == expected_url
+        assert runtime._get_cloudspace_url(project, cloudspace_name, tab) == expected_url