Handle the not too uncommon case where we have an empty scene

msanatan · msanatan · commit 2ff151f7719d · 2025-11-25T16:46:27.000-04:00
diff --git a/MCPForUnity/Editor/Services/TestRunnerService.cs b/MCPForUnity/Editor/Services/TestRunnerService.cs
@@ -227,6 +227,11 @@ private static void SaveDirtyScenesIfNeeded()
                 var scene = SceneManager.GetSceneAt(i);
                 if (scene.isDirty)
                 {
+                    if (string.IsNullOrEmpty(scene.path))
+                    {
+                        McpLog.Warn($"[TestRunnerService] Skipping unsaved scene '{scene.name}': save it manually before running PlayMode tests.");
+                        continue;
+                    }
                     try
                     {
                         EditorSceneManager.SaveScene(scene);

Original file line number	Diff line number	Diff line change
`@@ -227,6 +227,11 @@ private static void SaveDirtyScenesIfNeeded()`
`227`	`227`	`var scene = SceneManager.GetSceneAt(i);`
`228`	`228`	`if (scene.isDirty)`
`229`	`229`	`{`
	`230`	`+ if (string.IsNullOrEmpty(scene.path))`
	`231`	`+ {`
	`232`	`+ McpLog.Warn($"[TestRunnerService] Skipping unsaved scene '{scene.name}': save it manually before running PlayMode tests.");`
	`233`	`+ continue;`
	`234`	`+ }`
`230`	`235`	`try`
`231`	`236`	`{`
`232`	`237`	`EditorSceneManager.SaveScene(scene);`