test: replace DuplicatePBStripping test to PreserveOutfitPB test

Assets/_ModularAvatar/EditModeTests/DuplicatePBStripping/DuplicatePBStripping.cs

@@ -1,46 +0,0 @@
-using nadena.dev.modular_avatar.core.editor;
-using NUnit.Framework;
-using VRC.SDK3.Dynamics.PhysBone.Components;
-
-namespace modular_avatar_tests.DuplicatePBStripping
-{
-    public class DuplicatePBStripping : TestBase
-    {
-        [Test]
-        public void StripsExtraPBones_withNullRootTransform()
-        {
-            var prefab = CreatePrefab("DuplicatePBStripping_nullRef.prefab");
-            AvatarProcessor.ProcessAvatar(prefab);
-
-            Assert.AreEqual(1, prefab.GetComponentsInChildren<VRCPhysBone>().Length);
-        }
-
-        [Test]
-        public void StripsExtraPBones_withExplicitRootTransform()
-        {
-            var prefab = CreatePrefab("DuplicatePBStripping_objRef.prefab");
-            AvatarProcessor.ProcessAvatar(prefab);
-
-            Assert.AreEqual(1, prefab.GetComponentsInChildren<VRCPhysBone>().Length);
-        }
-
-        [Test]
-        public void StripsExtraPBones_withSiblingRootTransform()
-        {
-            var prefab = CreatePrefab("DuplicatePBStripping_otherRef.prefab");
-            AvatarProcessor.ProcessAvatar(prefab);
-
-            Assert.AreEqual(1, prefab.GetComponentsInChildren<VRCPhysBone>().Length);
-        }
-
-        [Test]
-        public void StripsExtraPBones_notWhenTargetDiffers()
-        {
-            var prefab = CreatePrefab("DuplicatePBStripping_preserve.prefab");
-            AvatarProcessor.ProcessAvatar(prefab);
-
-            // Note that this prefab has one duplicate, one non-duplicate component
-            Assert.AreEqual(2, prefab.GetComponentsInChildren<VRCPhysBone>().Length);
-        }
-    }
-}

Assets/_ModularAvatar/EditModeTests/DuplicatePBStripping/DuplicatePBStripping.cs.meta

@@ -1,3 +0,0 @@
-fileFormatVersion: 2
-guid: f7c215421c4e48e79f30a56b59f2c174
-timeCreated: 1673875944

Assets/_ModularAvatar/EditModeTests/DuplicatePBStripping/DuplicatePBStripping_preserve.prefab.meta

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

Assets/_ModularAvatar/EditModeTests/PreserveOutfitPB/PreserveOutfitPB.cs

@@ -0,0 +1,48 @@
+using nadena.dev.modular_avatar.core.editor;
+using NUnit.Framework;
+using VRC.SDK3.Dynamics.PhysBone.Components;
+
+namespace modular_avatar_tests.PreserveOutfitPB
+{
+    public class PreserveOutfitPB : TestBase
+    {
+        [Test]
+        public void NullRootTransform()
+        {
+            var avatar = CreatePrefab("PreserveOutfitPB_NullRootTransform.prefab");
+            AvatarProcessor.ProcessAvatar(avatar);
+
+            var armature = avatar.transform.Find("Armature");
+            Assert.That(armature.childCount, Is.EqualTo(2));
+            Assert.That(armature.GetChild(0).GetComponent<VRCPhysBone>(), Is.Not.Null);
+            Assert.That(armature.GetChild(1).GetComponent<VRCPhysBone>(), Is.Not.Null);
+        }
+
+        [Test]
+        public void SelfRootTransform()
+        {
+            var avatar = CreatePrefab("PreserveOutfitPB_SelfRootTransform.prefab");
+            AvatarProcessor.ProcessAvatar(avatar);
+
+            var armature = avatar.transform.Find("Armature");
+            Assert.That(armature.childCount, Is.EqualTo(2));
+            Assert.That(armature.GetChild(0).GetComponent<VRCPhysBone>(), Is.Not.Null);
+            Assert.That(armature.GetChild(1).GetComponent<VRCPhysBone>(), Is.Not.Null);
+        }
+
+        [Test]
+        public void SiblingRootTransform()
+        {
+            var avatar = CreatePrefab("PreserveOutfitPB_SiblingRootTransform.prefab");
+            AvatarProcessor.ProcessAvatar(avatar);
+
+            var armature = avatar.transform.Find("Armature");
+            Assert.That(armature.childCount, Is.EqualTo(3));
+
+            var @ref = armature.Find("ref");
+            Assert.That(@ref.childCount, Is.EqualTo(1));
+            Assert.That(@ref.GetComponent<VRCPhysBone>(), Is.Not.Null);
+            Assert.That(@ref.GetChild(0).GetComponent<VRCPhysBone>(), Is.Not.Null);
+        }
+    }
+}

Assets/_ModularAvatar/EditModeTests/PreserveOutfitPB/PreserveOutfitPB.cs.meta

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

Assets/_ModularAvatar/EditModeTests/PreserveOutfitPB/PreserveOutfitPB_NullRootTransform.prefab

@@ -42,7 +42,7 @@ GameObject:
   - component: {fileID: 4446924385526016477}
   - component: {fileID: 2800033387965506276}
   m_Layer: 0
-  m_Name: DuplicatePBStripping_nullRef
+  m_Name: PreserveOutfitPB_NullRootTransform
   m_TagString: Untagged
   m_Icon: {fileID: 0}
   m_NavMeshLayer: 0

Assets/_ModularAvatar/EditModeTests/PreserveOutfitPB/PreserveOutfitPB_SelfRootTransform.prefab

@@ -192,7 +192,7 @@ GameObject:
   - component: {fileID: 1963453281188716255}
   - component: {fileID: 23230439543150054}
   m_Layer: 0
-  m_Name: DuplicatePBStripping_objRef
+  m_Name: PreserveOutfitPB_SelfRootTransform
   m_TagString: Untagged
   m_Icon: {fileID: 0}
   m_NavMeshLayer: 0

Assets/_ModularAvatar/EditModeTests/PreserveOutfitPB/PreserveOutfitPB_SiblingRootTransform.prefab

@@ -73,7 +73,7 @@ GameObject:
   - component: {fileID: 1967208902774019681}
   - component: {fileID: 9569865848511832}
   m_Layer: 0
-  m_Name: DuplicatePBStripping_otherRef
+  m_Name: PreserveOutfitPB_SiblingRootTransform
   m_TagString: Untagged
   m_Icon: {fileID: 0}
   m_NavMeshLayer: 0