#6494: Fix end cap tests

codereader · codereader · commit 4246908ef2c8 · 2026-03-08T06:33:05.000+01:00
The command binding is now opening a dialog, so use
the direct parameterised "CreatePatchPrefab endcap" command.
diff --git a/test/Patch.cpp b/test/Patch.cpp
@@ -102,7 +102,7 @@ TEST_F(PatchTest, InvertedEndCapInheritsDef2)
 {
     auto worldspawn = GlobalMapModule().findOrInsertWorldspawn();
 
-    GlobalCommandSystem().execute("PatchEndCap");
+    GlobalCommandSystem().execute("CreatePatchPrefab endcap");
 
     auto endcap = algorithm::findFirstPatch(worldspawn, [](auto&) { return true; });
     Node_setSelected(endcap, true);
@@ -125,7 +125,7 @@ TEST_F(PatchTest, InvertedEndCapInheritsDef3)
 {
     auto worldspawn = GlobalMapModule().findOrInsertWorldspawn();
 
-    GlobalCommandSystem().execute("PatchEndCap");
+    GlobalCommandSystem().execute("CreatePatchPrefab endcap");
 
     auto endcap = algorithm::findFirstPatch(worldspawn, [](auto&) { return true; });
     Node_setSelected(endcap, true);

Original file line number	Diff line number	Diff line change
`@@ -102,7 +102,7 @@ TEST_F(PatchTest, InvertedEndCapInheritsDef2)`
`102`	`102`	`{`
`103`	`103`	`auto worldspawn = GlobalMapModule().findOrInsertWorldspawn();`
`104`	`104`
`105`		`- GlobalCommandSystem().execute("PatchEndCap");`
	`105`	`+ GlobalCommandSystem().execute("CreatePatchPrefab endcap");`
`106`	`106`
`107`	`107`	`auto endcap = algorithm::findFirstPatch(worldspawn, [](auto&) { return true; });`
`108`	`108`	`Node_setSelected(endcap, true);`
`@@ -125,7 +125,7 @@ TEST_F(PatchTest, InvertedEndCapInheritsDef3)`
`125`	`125`	`{`
`126`	`126`	`auto worldspawn = GlobalMapModule().findOrInsertWorldspawn();`
`127`	`127`
`128`		`- GlobalCommandSystem().execute("PatchEndCap");`
	`128`	`+ GlobalCommandSystem().execute("CreatePatchPrefab endcap");`
`129`	`129`
`130`	`130`	`auto endcap = algorithm::findFirstPatch(worldspawn, [](auto&) { return true; });`
`131`	`131`	`Node_setSelected(endcap, true);`