Merge pull request #104 from kurobon-jp/feature/fix_delete_entity

Fix bug where scripts were not properly deleted

Author: friflo

src/ECS/Entity/Store/Extension.cs

@@ -105,7 +105,9 @@ private void RemoveEntityScript(int id)
         for (int n = 1; n <= len; n++) {
             if (scripts[n].id != id) continue;
             for (; n < len; n++) {
-                scripts[n] = scripts[n + 1];
+                var script = scripts[n + 1];
+                scripts[n] = script;
+                scriptMap[script.id] = n;
             }
             entityScriptCount   = len;
             scripts[len]        = default;

src/Tests/ECS/Entity/Test_Script.cs

@@ -369,6 +369,21 @@ public static void Test_3_Simulate_Editor() {
         test.Start();
         test.Update();
     }
+
+    [Test]
+    public static void Test_DeleteEntity()
+    {
+        var store = new EntityStore(PidType.RandomPids);
+        var entity1 = store.CreateEntity();
+        var entity2 = store.CreateEntity();
+
+        entity1.AddScript(new TestScript1());
+        entity2.AddScript(new TestScript1());
+
+        entity1.DeleteEntity();
+
+        AreEqual("id: 2  [*TestScript1]", entity2.ToString());
+    }
 }
 
 }