add unit tests

Signed-off-by: Roee Landesman <roeeland@cisco.com>

Author: ri-roee

test/core/execution/test_skypilot.py

@@ -68,6 +68,7 @@ class MockClusterNotUpError(Exception):
         "sky.core": sky_core_mock,
         "sky.skylet.job_lib": job_lib_mock,
         "sky.utils.common_utils": common_utils_mock,
+        "sky.resources": MagicMock(),
     }
 
     # Also mock the sky_exceptions module with our mock exception
@@ -141,6 +142,33 @@ def test_init_non_git_packager(self, mock_skypilot_imports):
                 packager=non_git_packager,
             )
 
+    def test_init_with_infra_and_cloud_fails(self, mock_skypilot_imports):
+        with pytest.raises(
+            AssertionError, match="Cannot specify both `infra` and `cloud` parameters."
+        ):
+            SkypilotExecutor(
+                infra="my-infra",
+                cloud="aws",
+            )
+
+    def test_init_with_infra_and_region_fails(self, mock_skypilot_imports):
+        with pytest.raises(
+            AssertionError, match="Cannot specify both `infra` and `region` parameters."
+        ):
+            SkypilotExecutor(
+                infra="my-infra",
+                region="us-west-2",
+            )
+
+    def test_init_with_infra_and_zone_fails(self, mock_skypilot_imports):
+        with pytest.raises(
+            AssertionError, match="Cannot specify both `infra` and `zone` parameters."
+        ):
+            SkypilotExecutor(
+                infra="my-infra",
+                zone="us-west-2a",
+            )
+
     def test_parse_app(self, mock_skypilot_imports):
         app_id = "app___cluster-name___task-name___123"
         cluster, task, job_id = SkypilotExecutor.parse_app(app_id)
@@ -228,6 +256,18 @@ def test_to_resources_with_none_string(self, mock_resources, mock_skypilot_impor
         assert config["cloud"] is None
         assert config["any_of"][1]["region"] is None
 
+    @patch("sky.resources.Resources")
+    def test_to_resources_with_infra_and_network_tier(self, mock_resources, mock_skypilot_imports):
+        executor = SkypilotExecutor(infra="k8s/my-context", network_tier="best")
+
+        executor.to_resources()
+
+        mock_resources.from_yaml_config.assert_called_once()
+
+        config = mock_resources.from_yaml_config.call_args[0][0]
+        assert config["infra"] == "k8s/my-context"
+        assert config["network_tier"] == "best"
+
     @patch("sky.core.status")
     @patch("sky.core.queue")
     @patch("nemo_run.core.execution.skypilot.SkypilotExecutor.parse_app")