From 58d7ce55097e01b9e3831634ea179ee1f995bb49 Mon Sep 17 00:00:00 2001
From: Matthew Mott <orbweaver3d@gmail.com>
Date: Fri, 26 Nov 2021 13:41:41 +0000
Subject: [PATCH] Add unit test for simple entity spawnarg change

Test undoing and redoing a value change to a single spawnarg.
---
 test/Entity.cpp | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/test/Entity.cpp b/test/Entity.cpp
index 77c4243181..797eba8fd0 100644
--- a/test/Entity.cpp
+++ b/test/Entity.cpp
@@ -370,6 +370,32 @@ TEST_F(EntityTest, CopySpawnargs)
     EXPECT_EQ(overlap.size(), 0);
 }
 
+TEST_F(EntityTest, UndoRedoSpawnargValueChange)
+{
+    // Create entity with initial default args. Note that we must add the node
+    // to the global map in order for undo to work.
+    auto cls = GlobalEntityClassManager().findClass("bucket_metal");
+    auto bucketNode = GlobalEntityModule().createEntity(cls);
+    scene::addNodeToContainer(bucketNode, GlobalMapModule().getRoot());
+    Entity& spawnArgs = bucketNode->getEntity();
+
+    // Make a simple value change
+    EXPECT_EQ(spawnArgs.getKeyValue("name"), "bucket_metal_1");
+    {
+        UndoableCommand cmd("changeKeyValue");
+        spawnArgs.setKeyValue("name", "another_bucket");
+    }
+
+    // Confirm we can undo this change
+    EXPECT_EQ(spawnArgs.getKeyValue("name"), "another_bucket");
+    GlobalUndoSystem().undo();
+    EXPECT_EQ(spawnArgs.getKeyValue("name"), "bucket_metal_1");
+
+    // Confirm we can redo the change
+    GlobalUndoSystem().redo();
+    EXPECT_EQ(spawnArgs.getKeyValue("name"), "another_bucket");
+}
+
 TEST_F(EntityTest, SelectEntity)
 {
     auto light = createByClassName("light");