com.amanotes.prefabpool 2.0.0 → 2.0.2

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## [2.0.2] - 2026-09-14
+
+### Fixed
+- Pool-group registration during prewarm updates the registration without invalidating the initialization traversal.
+
+### Removed
+- Runtime tests from the distributable UPM package; tests remain in the development project.
+
+## [2.0.1] - 2026-08-15
+
+### Fixed
+- `PrefabPool.DestroyPool` now destroys checked-out instances (not only inactive `_pool` lists). Previously live clones with `HideFlags.DontSave` and a null spawn parent survived scene unload on device because `_instanceTypeMap` was only resolved in the Editor.
+- `PrefabPool.OnDestroy` calls `DestroyPool` so scene teardown always wipes every clone the pool instantiated.
+
 ## [2.0.0] - 2026-07-22
 
 ### Added

package/Docs/PrefabPool-API.md

@@ -1,6 +1,6 @@
 # PrefabPool API Reference
 
-**Package:** `com.amanotes.prefabpool` | **Version:** 1.2.0 | **Namespace:** `Amanotes.Core`
+**Package:** `com.amanotes.prefabpool` | **Version:** 2.0.2 | **Namespace:** `Amanotes.Core`
 
 Generic object pooling system to eliminate garbage collection spikes from frequent instantiation/destruction.
 
@@ -22,6 +22,8 @@ public interface IPrefabPool
     GameObject Get(PoolGroup group, string prefabId, Vector3 localPosition = default, Transform customParent = null);
     void Return(PoolGroup group, GameObject gameObject);
     IEnumerator Initialize();                // Pre-allocate pooled objects
+    void MarkShutdown();
+    void DestroyPool();                      // Inactive + checked-out clones
 }
 ```
 

package/README.md

@@ -1,7 +1,7 @@
 # PrefabPool
 
 **Package:** `com.amanotes.prefabpool`  
-**Version:** 2.0.0
+**Version:** 2.0.2
 
 Generic object pooling system for Unity with lifecycle callbacks and material pooling support.
 
@@ -30,7 +30,7 @@ Add the package to your Unity project via `Packages/manifest.json`:
 ```json
 {
   "dependencies": {
-    "com.amanotes.prefabpool": "2.0.0"
+    "com.amanotes.prefabpool": "2.0.2"
   },
   "scopedRegistries": [
     {

package/Runtime/PrefabPool.cs

@@ -214,6 +214,10 @@ namespace Amanotes.Core
         /// when the pool's hierarchy is being deactivated.
         /// </summary>
         void MarkShutdown();
+        /// <summary>
+        /// Destroys every clone this pool instantiated (inactive lists and checked-out live instances).
+        /// </summary>
+        void DestroyPool();
     }
 
     public static class PrefabPoolExtensions
@@ -311,7 +315,10 @@ namespace Amanotes.Core
         [NonSerialized] private bool _isReady = false;
         [NonSerialized] private bool _isInitializing = false;
         [NonSerialized] private readonly Dictionary<string, PoolGroup> _groups = new Dictionary<string, PoolGroup>();
+        // Registration order for Initialize — only appended on first RegisterGroup per id.
+        [NonSerialized] private readonly List<PoolGroup> _groupOrder = new List<PoolGroup>();
         [NonSerialized] private readonly Dictionary<UnityObjectId, PoolEntry> _instanceTypeMap = new Dictionary<UnityObjectId, PoolEntry>();
+        [NonSerialized] private readonly Dictionary<UnityObjectId, GameObject> _liveInstances = new Dictionary<UnityObjectId, GameObject>();
         [NonSerialized] private readonly Dictionary<GameObject, PoolEntry> _customPool = new Dictionary<GameObject, PoolEntry>();
         
         public void Resize<T>(GameObject prefab, List<T> list, int count, Transform parent) where T: Component
@@ -359,8 +366,24 @@ namespace Amanotes.Core
         public void RegisterGroup(PoolGroup group, Transform spawnParent)
         {
             if (group == null) return;
-            
+
+            if (_groups.TryGetValue(group.id, out var existing))
+            {
+                // AttachHost re-registers tile/vfx groups from Start while Initialize is yielding.
+                // Updating spawn parent must not touch _groups — that invalidates Dictionary enumeration.
+                group.Register(this, spawnParent);
+                if (!ReferenceEquals(existing, group))
+                {
+                    _groups[group.id] = group;
+                    var index = _groupOrder.IndexOf(existing);
+                    if (index >= 0) _groupOrder[index] = group;
+                }
+                PurgeStaleInstanceMap();
+                return;
+            }
+
             _groups[group.id] = group;
+            _groupOrder.Add(group);
             group.Register(this, spawnParent);
             PurgeStaleInstanceMap();
         }
@@ -385,7 +408,9 @@ namespace Amanotes.Core
             var result = entry.Get(transform, p, localPosition);
             if (result != null)
             {
-                _instanceTypeMap[UnityObjectIdentity.GetId(result)] = entry;
+                var id = UnityObjectIdentity.GetId(result);
+                _instanceTypeMap[id] = entry;
+                _liveInstances[id] = result;
             }
             
             return result;
@@ -400,6 +425,7 @@ namespace Amanotes.Core
             {
                 entry.Return(go, transform);
                 _instanceTypeMap.Remove(instanceId);
+                _liveInstances.Remove(instanceId);
             }
         }
         
@@ -412,7 +438,11 @@ namespace Amanotes.Core
             {
                 if (UnityObjectIdentity.IdToObject(kvp.Key) == null) stale.Add(kvp.Key);
             }
-            foreach (var id in stale) _instanceTypeMap.Remove(id);
+            foreach (var id in stale)
+            {
+                _instanceTypeMap.Remove(id);
+                _liveInstances.Remove(id);
+            }
 #endif
         }
         
@@ -425,26 +455,29 @@ namespace Amanotes.Core
             EditorApplication.playModeStateChanged -= HandlePlayModeChanged;
             EditorApplication.playModeStateChanged += HandlePlayModeChanged;
             #endif
-            
-            foreach (var group in _groups.Values)
-            {
-                foreach (var entry in group.entries)
-                {
-                    if (entry.prefab == null) continue;
-                    yield return entry.PreAllocate(transform);
-                }
-                
-                // Pre-allocate fallback entry if available
-                if (group.fallbackEntry != null && group.fallbackEntry.prefab != null)
-                {
-                    yield return group.fallbackEntry.PreAllocate(transform);
-                }
-            }
+
+            // Drain _groupOrder by index so groups appended during PreAllocate yields still prewarm.
+            for (var i = 0; i < _groupOrder.Count; i++)
+                yield return PrewarmGroup(_groupOrder[i]);
             
             _isInitializing = false;
             _isReady = true;
         }
 
+        IEnumerator PrewarmGroup(PoolGroup group)
+        {
+            if (group == null) yield break;
+
+            foreach (var entry in group.entries)
+            {
+                if (entry.prefab == null) continue;
+                yield return entry.PreAllocate(transform);
+            }
+
+            if (group.fallbackEntry != null && group.fallbackEntry.prefab != null)
+                yield return group.fallbackEntry.PreAllocate(transform);
+        }
+
 #if UNITY_EDITOR
         protected void HandlePlayModeChanged(PlayModeStateChange changed)
         {
@@ -454,39 +487,54 @@ namespace Amanotes.Core
             }
         }
 #endif
+
+        void OnDestroy()
+        {
+#if UNITY_EDITOR
+            EditorApplication.playModeStateChanged -= HandlePlayModeChanged;
+#endif
+            DestroyPool();
+        }
         
         public void DestroyPool()
         {
             _isDestroyed = true;
+
+            foreach (var go in _liveInstances.Values)
+                DestroyClone(go);
+            _liveInstances.Clear();
+            _instanceTypeMap.Clear();
             
-            foreach (var group in _groups.Values)
+            foreach (var group in _groupOrder)
             {
                 foreach (var entry in group.entries)
-                {
                     entry.DestroyPool();
-                }
-                
+
                 if (group.fallbackEntry != null)
-                {
                     group.fallbackEntry.DestroyPool();
-                }
             }
             _groups.Clear();
+            _groupOrder.Clear();
             
             foreach (var entry in _customPool.Values)
             {
                 entry.DestroyPool();
             }
             _customPool.Clear();
-            
+        }
+
+        static void DestroyClone(GameObject go)
+        {
+            if (go == null) return;
 #if UNITY_EDITOR
-            foreach (var kvp in _instanceTypeMap)
+            if (!Application.isPlaying)
             {
-                var obj = UnityObjectIdentity.IdToObject(kvp.Key);
-                if (obj != null) UnityObject.DestroyImmediate(obj);
+                UnityObject.DestroyImmediate(go);
+                return;
             }
 #endif
-            _instanceTypeMap.Clear();
+            UnityObject.Destroy(go);
         }
     }
-} 
+}
+ 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "com.amanotes.prefabpool",
-  "version": "2.0.0",
+  "version": "2.0.2",
   "displayName": "[Ama] Prefab Pool",
   "description": "Generic object pooling system for Unity",
   "unity": "2022.3",

package/Tests/Runtime/PrefabPool.Test.asmdef

@@ -1,11 +0,0 @@
-{
-    "name": "PrefabPool.Tests",
-    "rootNamespace": "Amanotes.Core",
-    "references": [
-        "PrefabPool",
-        "Unity.PerformanceTesting"
-    ],
-    "optionalUnityReferences": [
-        "TestAssemblies"
-    ]
-}

package/Tests/Runtime/PrefabPool.Test.asmdef.meta

@@ -1,7 +0,0 @@
-fileFormatVersion: 2
-guid: f4f4c6bd22a404f98bd5cc47335df5d8
-AssemblyDefinitionImporter:
-  externalObjects: {}
-  userData: 
-  assetBundleName: 
-  assetBundleVariant: 

package/Tests/Runtime/PrefabPoolGroupTests.cs

@@ -1,208 +0,0 @@
-using System.Collections;
-using System.Collections.Generic;
-using NUnit.Framework;
-using UnityEngine;
-using UnityEngine.TestTools;
-using UnityObject = UnityEngine.Object;
-
-namespace Amanotes.Core
-{
-    public class PrefabPoolGroupTests
-    {
-        private PrefabPool _pool;
-        private PoolGroup _groupA;
-        private PoolGroup _groupB;
-        private GameObject _prefabA;
-        private GameObject _prefabB;
-        private Transform _spawnParentA;
-        private Transform _spawnParentB;
-        
-        [SetUp]
-        public void SetUp()
-        {
-            var poolObject = new GameObject("TestPool");
-            _pool = poolObject.AddComponent<PrefabPool>();
-            
-            var spawnParentObjectA = new GameObject("SpawnParentA");
-            _spawnParentA = spawnParentObjectA.transform;
-            
-            var spawnParentObjectB = new GameObject("SpawnParentB");
-            _spawnParentB = spawnParentObjectB.transform;
-            
-            _prefabA = new GameObject("PrefabA");
-            _prefabA.AddComponent<MockPoolPrefab>();
-            
-            _prefabB = new GameObject("PrefabB");
-            _prefabB.AddComponent<MockPoolPrefab>();
-            
-            _groupA = new PoolGroup
-            {
-                id = "GroupA",
-                entries = new List<PoolEntry>
-                {
-                    new PoolEntry
-                    {
-                        id = "PrefabA",
-                        prefab = _prefabA,
-                        prewarmCount = 5
-                    }
-                }
-            };
-            
-            _groupB = new PoolGroup
-            {
-                id = "GroupB",
-                entries = new List<PoolEntry>
-                {
-                    new PoolEntry
-                    {
-                        id = "PrefabB",
-                        prefab = _prefabB,
-                        prewarmCount = 5
-                    }
-                }
-            };
-            
-            _pool.RegisterGroup(_groupA, _spawnParentA);
-            _pool.RegisterGroup(_groupB, _spawnParentB);
-        }
-        
-        [TearDown]
-        public void TearDown()
-        {
-            if (_pool != null)
-            {
-                _pool.DestroyPool();
-                UnityObject.DestroyImmediate(_pool.gameObject);
-            }
-            
-            if (_spawnParentA != null)
-            {
-                UnityObject.DestroyImmediate(_spawnParentA.gameObject);
-            }
-            
-            if (_spawnParentB != null)
-            {
-                UnityObject.DestroyImmediate(_spawnParentB.gameObject);
-            }
-            
-            if (_prefabA != null)
-            {
-                UnityObject.DestroyImmediate(_prefabA);
-            }
-            
-            if (_prefabB != null)
-            {
-                UnityObject.DestroyImmediate(_prefabB);
-            }
-            
-            var allObjects = UnityObject.FindObjectsOfType<GameObject>();
-            foreach (var obj in allObjects)
-            {
-                if (obj.hideFlags == HideFlags.DontSave)
-                {
-                    UnityObject.DestroyImmediate(obj);
-                }
-            }
-        }
-        
-        [UnityTest]
-        public IEnumerator PoolGroupIsolation_MultipleGroups_ObjectsNeverCrossBetweenGroups()
-        {
-            yield return _pool.Initialize();
-            
-            for (int i = 0; i < 100; i++)
-            {
-                var objFromA = _groupA.Get("PrefabA");
-                Assert.IsNotNull(objFromA, $"Iteration {i}: GroupA should return valid GameObject");
-                
-                var objFromB = _groupB.Get("PrefabB");
-                Assert.IsNotNull(objFromB, $"Iteration {i}: GroupB should return valid GameObject");
-                
-                Assert.AreNotSame(objFromA, objFromB,
-                    $"Iteration {i}: Objects from different groups should be different instances");
-                
-                Assert.IsTrue(objFromA.name.Contains("PrefabA"), 
-                    $"Iteration {i}: Object from GroupA should be PrefabA instance");
-                Assert.IsTrue(objFromB.name.Contains("PrefabB"), 
-                    $"Iteration {i}: Object from GroupB should be PrefabB instance");
-                
-                _groupA.Return(objFromA);
-                _groupB.Return(objFromB);
-                
-                var reusedFromA = _groupA.Get("PrefabA");
-                var reusedFromB = _groupB.Get("PrefabB");
-                
-                Assert.AreSame(objFromA, reusedFromA,
-                    $"Iteration {i}: GroupA should reuse its own objects");
-                Assert.AreSame(objFromB, reusedFromB,
-                    $"Iteration {i}: GroupB should reuse its own objects");
-                
-                Assert.IsTrue(reusedFromA.name.Contains("PrefabA"), 
-                    $"Iteration {i}: GroupA should never return PrefabB instances");
-                Assert.IsTrue(reusedFromB.name.Contains("PrefabB"), 
-                    $"Iteration {i}: GroupB should never return PrefabA instances");
-                
-                _groupA.Return(reusedFromA);
-                _groupB.Return(reusedFromB);
-            }
-        }
-        
-        [UnityTest]
-        public IEnumerator ReturnGroupRouting_ReturnToGroup_SubsequentGetRetrievesSameInstanceFromSameGroup()
-        {
-            yield return _pool.Initialize();
-            
-            for (int i = 0; i < 100; i++)
-            {
-                var objA1 = _groupA.Get("PrefabA");
-                var objA2 = _groupA.Get("PrefabA");
-                var objB1 = _groupB.Get("PrefabB");
-                var objB2 = _groupB.Get("PrefabB");
-                
-                Assert.IsNotNull(objA1, $"Iteration {i}: GroupA Get 1 should return valid GameObject");
-                Assert.IsNotNull(objA2, $"Iteration {i}: GroupA Get 2 should return valid GameObject");
-                Assert.IsNotNull(objB1, $"Iteration {i}: GroupB Get 1 should return valid GameObject");
-                Assert.IsNotNull(objB2, $"Iteration {i}: GroupB Get 2 should return valid GameObject");
-                
-                _groupA.Return(objA1);
-                _groupA.Return(objA2);
-                _groupB.Return(objB1);
-                _groupB.Return(objB2);
-                
-                var reusedA1 = _groupA.Get("PrefabA");
-                var reusedA2 = _groupA.Get("PrefabA");
-                var reusedB1 = _groupB.Get("PrefabB");
-                var reusedB2 = _groupB.Get("PrefabB");
-                
-                var reusedObjectsA = new HashSet<GameObject> { reusedA1, reusedA2 };
-                var reusedObjectsB = new HashSet<GameObject> { reusedB1, reusedB2 };
-                
-                Assert.IsTrue(reusedObjectsA.Contains(objA1),
-                    $"Iteration {i}: Object returned to GroupA should be retrieved from GroupA");
-                Assert.IsTrue(reusedObjectsA.Contains(objA2),
-                    $"Iteration {i}: Object returned to GroupA should be retrieved from GroupA");
-                
-                Assert.IsTrue(reusedObjectsB.Contains(objB1),
-                    $"Iteration {i}: Object returned to GroupB should be retrieved from GroupB");
-                Assert.IsTrue(reusedObjectsB.Contains(objB2),
-                    $"Iteration {i}: Object returned to GroupB should be retrieved from GroupB");
-                
-                Assert.IsFalse(reusedObjectsA.Contains(objB1),
-                    $"Iteration {i}: GroupA should not return objects from GroupB");
-                Assert.IsFalse(reusedObjectsA.Contains(objB2),
-                    $"Iteration {i}: GroupA should not return objects from GroupB");
-                
-                Assert.IsFalse(reusedObjectsB.Contains(objA1),
-                    $"Iteration {i}: GroupB should not return objects from GroupA");
-                Assert.IsFalse(reusedObjectsB.Contains(objA2),
-                    $"Iteration {i}: GroupB should not return objects from GroupA");
-                
-                _groupA.Return(reusedA1);
-                _groupA.Return(reusedA2);
-                _groupB.Return(reusedB1);
-                _groupB.Return(reusedB2);
-            }
-        }
-    }
-}

package/Tests/Runtime/PrefabPoolGroupTests.cs.meta

@@ -1,11 +0,0 @@
-fileFormatVersion: 2
-guid: 52ad4e4f94f0b41ff9df7f00bcbdd701
-MonoImporter:
-  externalObjects: {}
-  serializedVersion: 2
-  defaultReferences: []
-  executionOrder: 0
-  icon: {instanceID: 0}
-  userData: 
-  assetBundleName: 
-  assetBundleVariant: 

package/Tests/Runtime/PrefabPoolLifecycleTests.cs

@@ -1,253 +0,0 @@
-using System.Collections;
-using NUnit.Framework;
-using UnityEngine;
-using UnityEngine.TestTools;
-using UnityObject = UnityEngine.Object;
-
-namespace Amanotes.Core
-{
-    public class PrefabPoolLifecycleTests
-    {
-        private PrefabPool _pool;
-        private GameObject _testPrefab;
-        private PoolGroup _testGroup;
-        private Transform _spawnParent;
-        
-        [SetUp]
-        public void SetUp()
-        {
-            var poolObject = new GameObject("TestPool");
-            _pool = poolObject.AddComponent<PrefabPool>();
-            
-            var spawnParentObject = new GameObject("SpawnParent");
-            _spawnParent = spawnParentObject.transform;
-            
-            _testPrefab = new GameObject("TestPrefab");
-            _testPrefab.AddComponent<MockPoolPrefab>();
-            
-            _testGroup = new PoolGroup
-            {
-                id = "TestGroup",
-                entries = new System.Collections.Generic.List<PoolEntry>
-                {
-                    new PoolEntry
-                    {
-                        id = "TestPrefab",
-                        prefab = _testPrefab,
-                        prewarmCount = 5
-                    }
-                }
-            };
-            
-            _pool.RegisterGroup(_testGroup, _spawnParent);
-        }
-        
-        [TearDown]
-        public void TearDown()
-        {
-            if (_pool != null)
-            {
-                _pool.DestroyPool();
-                UnityObject.DestroyImmediate(_pool.gameObject);
-            }
-            
-            if (_spawnParent != null)
-            {
-                UnityObject.DestroyImmediate(_spawnParent.gameObject);
-            }
-            
-            if (_testPrefab != null)
-            {
-                UnityObject.DestroyImmediate(_testPrefab);
-            }
-            
-            var allObjects = UnityObject.FindObjectsOfType<GameObject>();
-            foreach (var obj in allObjects)
-            {
-                if (obj.hideFlags == HideFlags.DontSave)
-                {
-                    UnityObject.DestroyImmediate(obj);
-                }
-            }
-        }
-        
-        [UnityTest]
-        public IEnumerator PoolObjectRoundTrip_GetReturnGet_ReuseSameInstance()
-        {
-            yield return _pool.Initialize();
-            
-            for (int i = 0; i < 100; i++)
-            {
-                var firstGet = _testGroup.Get("TestPrefab");
-                Assert.IsNotNull(firstGet, $"Iteration {i}: First Get should return valid GameObject");
-                
-                _testGroup.Return(firstGet);
-                
-                var secondGet = _testGroup.Get("TestPrefab");
-                Assert.IsNotNull(secondGet, $"Iteration {i}: Second Get should return valid GameObject");
-                
-                Assert.AreSame(firstGet, secondGet,
-                    $"Iteration {i}: Get-Return-Get should reuse same instance (validates pooling eliminates instantiation)");
-                
-                _testGroup.Return(secondGet);
-            }
-        }
-        
-        [UnityTest]
-        public IEnumerator LifecycleCallbackInvocation_GetReturnCycle_InvokesOnGetAndOnReturnExactlyOnce()
-        {
-            yield return _pool.Initialize();
-            
-            for (int i = 0; i < 100; i++)
-            {
-                var obj = _testGroup.Get("TestPrefab");
-                Assert.IsNotNull(obj, $"Iteration {i}: Get should return valid GameObject");
-                
-                var mock = obj.GetComponent<MockPoolPrefab>();
-                Assert.IsNotNull(mock, $"Iteration {i}: GameObject should have MockPoolPrefab component");
-                
-                mock.ResetCounts();
-                
-                var objAgain = _testGroup.Get("TestPrefab");
-                var mockAgain = objAgain.GetComponent<MockPoolPrefab>();
-                mockAgain.ResetCounts();
-                
-                _testGroup.Return(obj);
-                
-                Assert.AreEqual(1, mock.returnCallCount, 
-                    $"Iteration {i}: OnReturn should be called exactly once during Return");
-                Assert.AreEqual(0, mock.getCallCount, 
-                    $"Iteration {i}: OnGet should not be called during Return");
-                
-                var reusedObj = _testGroup.Get("TestPrefab");
-                var reusedMock = reusedObj.GetComponent<MockPoolPrefab>();
-                
-                Assert.AreEqual(1, reusedMock.getCallCount, 
-                    $"Iteration {i}: OnGet should be called exactly once during Get");
-                
-                _testGroup.Return(objAgain);
-                _testGroup.Return(reusedObj);
-            }
-        }
-        
-        [UnityTest]
-        public IEnumerator PreallocationCorrectness_PrewarmCount_CreatesExactNumberOfPreallocatedObjects()
-        {
-            for (int iteration = 0; iteration < 100; iteration++)
-            {
-                var prewarmCount = UnityEngine.Random.Range(1, 20);
-                
-                var poolObject = new GameObject($"TestPool_{iteration}");
-                var pool = poolObject.AddComponent<PrefabPool>();
-                
-                var spawnParentObject = new GameObject($"SpawnParent_{iteration}");
-                var spawnParent = spawnParentObject.transform;
-                
-                var prefab = new GameObject($"Prefab_{iteration}");
-                prefab.AddComponent<MockPoolPrefab>();
-                
-                var group = new PoolGroup
-                {
-                    id = $"Group_{iteration}",
-                    entries = new System.Collections.Generic.List<PoolEntry>
-                    {
-                        new PoolEntry
-                        {
-                            id = "TestPrefab",
-                            prefab = prefab,
-                            prewarmCount = prewarmCount
-                        }
-                    }
-                };
-                
-                pool.RegisterGroup(group, spawnParent);
-                
-                yield return pool.Initialize();
-                
-                var retrievedObjects = new System.Collections.Generic.List<GameObject>();
-                
-                for (int i = 0; i < prewarmCount; i++)
-                {
-                    var obj = group.Get("TestPrefab");
-                    Assert.IsNotNull(obj, $"Iteration {iteration}: Should retrieve pre-allocated object {i}");
-                    retrievedObjects.Add(obj);
-                }
-                
-                Assert.AreEqual(prewarmCount, retrievedObjects.Count, 
-                    $"Iteration {iteration}: Should retrieve exactly {prewarmCount} pre-allocated objects");
-                
-                foreach (var obj in retrievedObjects)
-                {
-                    group.Return(obj);
-                }
-                
-                pool.DestroyPool();
-                UnityObject.DestroyImmediate(poolObject);
-                UnityObject.DestroyImmediate(spawnParentObject);
-                UnityObject.DestroyImmediate(prefab);
-            }
-        }
-        
-        [UnityTest]
-        public IEnumerator DynamicPoolExpansion_GetBeyondCapacity_SuccessfullyReturnsValidGameObjects()
-        {
-            for (int iteration = 0; iteration < 100; iteration++)
-            {
-                var initialCapacity = UnityEngine.Random.Range(2, 10);
-                var extraRequests = UnityEngine.Random.Range(5, 15);
-                
-                var poolObject = new GameObject($"TestPool_{iteration}");
-                var pool = poolObject.AddComponent<PrefabPool>();
-                
-                var spawnParentObject = new GameObject($"SpawnParent_{iteration}");
-                var spawnParent = spawnParentObject.transform;
-                
-                var prefab = new GameObject($"Prefab_{iteration}");
-                prefab.AddComponent<MockPoolPrefab>();
-                
-                var group = new PoolGroup
-                {
-                    id = $"Group_{iteration}",
-                    entries = new System.Collections.Generic.List<PoolEntry>
-                    {
-                        new PoolEntry
-                        {
-                            id = "TestPrefab",
-                            prefab = prefab,
-                            prewarmCount = initialCapacity
-                        }
-                    }
-                };
-                
-                pool.RegisterGroup(group, spawnParent);
-                
-                yield return pool.Initialize();
-                
-                var retrievedObjects = new System.Collections.Generic.List<GameObject>();
-                
-                var totalRequests = initialCapacity + extraRequests;
-                
-                for (int i = 0; i < totalRequests; i++)
-                {
-                    var obj = group.Get("TestPrefab");
-                    Assert.IsNotNull(obj, 
-                        $"Iteration {iteration}: Get beyond capacity should succeed (request {i + 1}/{totalRequests})");
-                    retrievedObjects.Add(obj);
-                }
-                
-                Assert.AreEqual(totalRequests, retrievedObjects.Count, 
-                    $"Iteration {iteration}: Should retrieve {totalRequests} objects (capacity {initialCapacity} + expansion {extraRequests})");
-                
-                foreach (var obj in retrievedObjects)
-                {
-                    group.Return(obj);
-                }
-                
-                pool.DestroyPool();
-                UnityObject.DestroyImmediate(poolObject);
-                UnityObject.DestroyImmediate(spawnParentObject);
-                UnityObject.DestroyImmediate(prefab);
-            }
-        }
-    }
-}

package/Tests/Runtime/PrefabPoolLifecycleTests.cs.meta

@@ -1,11 +0,0 @@
-fileFormatVersion: 2
-guid: 9c6969a47577843fb977ab5e2c7c1ad2
-MonoImporter:
-  externalObjects: {}
-  serializedVersion: 2
-  defaultReferences: []
-  executionOrder: 0
-  icon: {instanceID: 0}
-  userData: 
-  assetBundleName: 
-  assetBundleVariant: 

package/Tests/Runtime/PrefabPoolPerformanceTests.cs

@@ -1,212 +0,0 @@
-using System.Collections;
-using NUnit.Framework;
-using Unity.PerformanceTesting;
-using UnityEngine;
-using UnityEngine.TestTools;
-using UnityObject = UnityEngine.Object;
-
-namespace Amanotes.Core
-{
-    public class PrefabPoolPerformanceTests
-    {
-        private PrefabPool _pool;
-        private GameObject _testPrefab;
-        private PoolGroup _testGroup;
-        private Transform _spawnParent;
-        
-        [SetUp]
-        public void SetUp()
-        {
-            var poolObject = new GameObject("TestPool");
-            _pool = poolObject.AddComponent<PrefabPool>();
-            
-            var spawnParentObject = new GameObject("SpawnParent");
-            _spawnParent = spawnParentObject.transform;
-            
-            _testPrefab = new GameObject("TestPrefab");
-            _testPrefab.AddComponent<MockPoolPrefab>();
-            
-            _testGroup = new PoolGroup
-            {
-                id = "TestGroup",
-                entries = new System.Collections.Generic.List<PoolEntry>
-                {
-                    new PoolEntry
-                    {
-                        id = "TestPrefab",
-                        prefab = _testPrefab,
-                        prewarmCount = 10
-                    }
-                }
-            };
-            
-            _pool.RegisterGroup(_testGroup, _spawnParent);
-        }
-        
-        [TearDown]
-        public void TearDown()
-        {
-            if (_pool != null)
-            {
-                _pool.DestroyPool();
-                UnityObject.DestroyImmediate(_pool.gameObject);
-            }
-            
-            if (_spawnParent != null)
-            {
-                UnityObject.DestroyImmediate(_spawnParent.gameObject);
-            }
-            
-            if (_testPrefab != null)
-            {
-                UnityObject.DestroyImmediate(_testPrefab);
-            }
-            
-            var allObjects = UnityObject.FindObjectsOfType<GameObject>();
-            foreach (var obj in allObjects)
-            {
-                if (obj.hideFlags == HideFlags.DontSave)
-                {
-                    UnityObject.DestroyImmediate(obj);
-                }
-            }
-        }
-        
-        [UnityTest, Performance]
-        public IEnumerator Performance_GetReturnCycles()
-        {
-            yield return _pool.Initialize();
-            
-            Measure.Method(() =>
-            {
-                for (int i = 0; i < 1000; i++)
-                {
-                    var obj = _testGroup.Get("TestPrefab");
-                    _testGroup.Return(obj);
-                }
-            })
-            .WarmupCount(5)
-            .MeasurementCount(10)
-            .IterationsPerMeasurement(1)
-            .GC()
-            .Run();
-        }
-        
-        [UnityTest, Performance]
-        public IEnumerator Performance_GCAllocation()
-        {
-            yield return _pool.Initialize();
-            
-            Measure.Method(() =>
-            {
-                for (int i = 0; i < 100; i++)
-                {
-                    var obj = _testGroup.Get("TestPrefab");
-                    _testGroup.Return(obj);
-                }
-            })
-            .WarmupCount(5)
-            .MeasurementCount(10)
-            .IterationsPerMeasurement(1)
-            .GC()
-            .Run();
-        }
-        
-        [UnityTest, Performance]
-        public IEnumerator Performance_PoolExpansion()
-        {
-            yield return _pool.Initialize();
-            
-            var retrievedObjects = new System.Collections.Generic.List<GameObject>();
-            
-            for (int i = 0; i < 10; i++)
-            {
-                var obj = _testGroup.Get("TestPrefab");
-                retrievedObjects.Add(obj);
-            }
-            
-            Measure.Method(() =>
-            {
-                for (int i = 0; i < 100; i++)
-                {
-                    var obj = _testGroup.Get("TestPrefab");
-                    retrievedObjects.Add(obj);
-                }
-            })
-            .WarmupCount(2)
-            .MeasurementCount(10)
-            .IterationsPerMeasurement(1)
-            .GC()
-            .Run();
-            
-            foreach (var obj in retrievedObjects)
-            {
-                _testGroup.Return(obj);
-            }
-        }
-        
-        [UnityTest, Performance]
-        public IEnumerator Performance_ParentTransformOverhead()
-        {
-            yield return _pool.Initialize();
-            
-            var withParentTime = 0.0;
-            var withoutParentTime = 0.0;
-            
-            Measure.Method(() =>
-            {
-                for (int i = 0; i < 100; i++)
-                {
-                    var obj = _testGroup.Get("TestPrefab");
-                    _testGroup.Return(obj);
-                }
-            })
-            .WarmupCount(5)
-            .MeasurementCount(10)
-            .IterationsPerMeasurement(1)
-            .GC()
-            .Run();
-            
-            var poolObjectNoParent = new GameObject("TestPoolNoParent");
-            var poolNoParent = poolObjectNoParent.AddComponent<PrefabPool>();
-            
-            var prefabNoParent = new GameObject("TestPrefabNoParent");
-            prefabNoParent.AddComponent<MockPoolPrefab>();
-            
-            var groupNoParent = new PoolGroup
-            {
-                id = "TestGroupNoParent",
-                entries = new System.Collections.Generic.List<PoolEntry>
-                {
-                    new PoolEntry
-                    {
-                        id = "TestPrefab",
-                        prefab = prefabNoParent,
-                        prewarmCount = 10
-                    }
-                }
-            };
-            
-            poolNoParent.RegisterGroup(groupNoParent, null);
-            yield return poolNoParent.Initialize();
-            
-            Measure.Method(() =>
-            {
-                for (int i = 0; i < 100; i++)
-                {
-                    var obj = groupNoParent.Get("TestPrefab");
-                    groupNoParent.Return(obj);
-                }
-            })
-            .WarmupCount(5)
-            .MeasurementCount(10)
-            .IterationsPerMeasurement(1)
-            .GC()
-            .Run();
-            
-            poolNoParent.DestroyPool();
-            UnityObject.DestroyImmediate(poolObjectNoParent);
-            UnityObject.DestroyImmediate(prefabNoParent);
-        }
-    }
-}

package/Tests/Runtime/PrefabPoolPerformanceTests.cs.meta

@@ -1,11 +0,0 @@
-fileFormatVersion: 2
-guid: 16aa39925a01643c3894a56145c56a21
-MonoImporter:
-  externalObjects: {}
-  serializedVersion: 2
-  defaultReferences: []
-  executionOrder: 0
-  icon: {instanceID: 0}
-  userData: 
-  assetBundleName: 
-  assetBundleVariant: 

package/Tests/Runtime/PrefabPoolTestsSetup.cs

@@ -1,14 +0,0 @@
-using NUnit.Framework;
-using Amanotes.Core;
-
-namespace Amanotes.Core
-{
-    public class PrefabPoolTestsSetup
-    {
-        [Test]
-        public void TestInfrastructureVerification()
-        {
-            Assert.IsTrue(true, "PrefabPool test infrastructure is working");
-        }
-    }
-}

package/Tests/Runtime/PrefabPoolTestsSetup.cs.meta

@@ -1,11 +0,0 @@
-fileFormatVersion: 2
-guid: e61f89af128b84b429ac96d86bd0554f
-MonoImporter:
-  externalObjects: {}
-  serializedVersion: 2
-  defaultReferences: []
-  executionOrder: 0
-  icon: {instanceID: 0}
-  userData: 
-  assetBundleName: 
-  assetBundleVariant: 

package/Tests/Runtime/TestHelpers/MockPoolPrefab.cs

@@ -1,26 +0,0 @@
-using UnityEngine;
-
-namespace Amanotes.Core
-{
-    public class MockPoolPrefab : MonoBehaviour, IPoolPrefab
-    {
-        public int getCallCount;
-        public int returnCallCount;
-        
-        public void OnGet()
-        {
-            getCallCount++;
-        }
-        
-        public void OnReturn()
-        {
-            returnCallCount++;
-        }
-        
-        public void ResetCounts()
-        {
-            getCallCount = 0;
-            returnCallCount = 0;
-        }
-    }
-}

package/Tests/Runtime/TestHelpers/MockPoolPrefab.cs.meta

@@ -1,11 +0,0 @@
-fileFormatVersion: 2
-guid: 8e7402f0e0f2440338429ebbb480ffc3
-MonoImporter:
-  externalObjects: {}
-  serializedVersion: 2
-  defaultReferences: []
-  executionOrder: 0
-  icon: {instanceID: 0}
-  userData: 
-  assetBundleName: 
-  assetBundleVariant: 

package/Tests/Runtime/TestHelpers.meta

@@ -1,8 +0,0 @@
-fileFormatVersion: 2
-guid: 2ce9db2584cc4444f930d1ab257afcaf
-folderAsset: yes
-DefaultImporter:
-  externalObjects: {}
-  userData: 
-  assetBundleName: 
-  assetBundleVariant: 

package/Tests/Runtime.meta

@@ -1,8 +0,0 @@
-fileFormatVersion: 2
-guid: f4d670095ee40448b93f6f5dbc2c07e6
-folderAsset: yes
-DefaultImporter:
-  externalObjects: {}
-  userData: 
-  assetBundleName: 
-  assetBundleVariant: 

package/Tests.meta

@@ -1,8 +0,0 @@
-fileFormatVersion: 2
-guid: 518067fd8d1ac4415b0468af746cce4a
-folderAsset: yes
-DefaultImporter:
-  externalObjects: {}
-  userData: 
-  assetBundleName: 
-  assetBundleVariant: