Merge pull request #247 from anatawa12/change-component-folder-name

chore: move every components into `Avatar Optimizer/` folder
anatawa12 · Jun 28, 2023 · 415c979 · 415c979
2 parents a2b30de + 13e5fdf
commit 415c979
Show file tree

Hide file tree

Showing 13 changed files with 47 additions and 8 deletions.
diff --git a/CHANGELOG-PRERELEASE.md b/CHANGELOG-PRERELEASE.md
@@ -10,6 +10,8 @@ The format is based on [Keep a Changelog].
 ### Added
 
 ### Changed
+- Move Components into `Avatar Optimizer` folder `#247`
+  - Previously they are `Optimizer` folder
 
 ### Deprecated
 

diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -10,6 +10,8 @@ The format is based on [Keep a Changelog].
 ### Added
 
 ### Changed
+- Move Components into `Avatar Optimizer` folder `#247`
+  - Previously they are `Optimizer` folder
 
 ### Deprecated
 

diff --git a/Runtime/ClearEndpointPosition.cs b/Runtime/ClearEndpointPosition.cs
@@ -3,7 +3,7 @@
 
 namespace Anatawa12.AvatarOptimizer
 {
-    [AddComponentMenu("Optimizer/Clear Endpoint Position")]
+    [AddComponentMenu("Avatar Optimizer/Clear Endpoint Position")]
     [RequireComponent(typeof(VRCPhysBoneBase))]
     internal class ClearEndpointPosition : AvatarTagComponent
     {

diff --git a/Runtime/FreezeBlendShape.cs b/Runtime/FreezeBlendShape.cs
@@ -3,7 +3,7 @@
 
 namespace Anatawa12.AvatarOptimizer
 {
-    [AddComponentMenu("Optimizer/Freeze BlendShapes")]
+    [AddComponentMenu("Avatar Optimizer/Freeze BlendShapes")]
     [DisallowMultipleComponent]
     internal class FreezeBlendShape : EditSkinnedMeshComponent
     {

diff --git a/Runtime/MakeChildren.cs b/Runtime/MakeChildren.cs
@@ -1,7 +1,9 @@
 using CustomLocalization4EditorExtension;
+using UnityEngine;
 
 namespace Anatawa12.AvatarOptimizer
 {
+    [AddComponentMenu("Avatar Optimizer/Make Children")]
     internal class MakeChildren : AvatarTagComponent
     {
         [CL4EELocalized("MakeChildren:prop:children")]

diff --git a/Runtime/MergeBone.cs b/Runtime/MergeBone.cs
@@ -2,7 +2,7 @@
 
 namespace Anatawa12.AvatarOptimizer
 {
-    [AddComponentMenu("Optimizer/Merge Bone")]
+    [AddComponentMenu("Avatar Optimizer/Merge Bone")]
     [DisallowMultipleComponent]
     internal class MergeBone : AvatarTagComponent
     { }

diff --git a/Runtime/MergePhysBone.cs b/Runtime/MergePhysBone.cs
@@ -8,7 +8,7 @@
 
 namespace Anatawa12.AvatarOptimizer
 {
-    [AddComponentMenu("Optimizer/Merge PhysBone")]
+    [AddComponentMenu("Avatar Optimizer/Merge PhysBone")]
     [DisallowMultipleComponent]
     [ExecuteAlways]
     internal class MergePhysBone : AvatarTagComponent, IStaticValidated

diff --git a/Runtime/MergeSkinnedMesh.cs b/Runtime/MergeSkinnedMesh.cs
@@ -4,7 +4,7 @@
 
 namespace Anatawa12.AvatarOptimizer
 {
-    [AddComponentMenu("Optimizer/Merge Skinned Mesh")]
+    [AddComponentMenu("Avatar Optimizer/Merge Skinned Mesh")]
     [RequireComponent(typeof(SkinnedMeshRenderer))]
     [DisallowMultipleComponent]
     internal class MergeSkinnedMesh : EditSkinnedMeshComponent, IStaticValidated

diff --git a/Runtime/MergeToonLitMaterial.cs b/Runtime/MergeToonLitMaterial.cs
@@ -3,7 +3,7 @@
 
 namespace Anatawa12.AvatarOptimizer
 {
-    [AddComponentMenu("Optimizer/Merge Toon Lit Material")]
+    [AddComponentMenu("Avatar Optimizer/Merge Toon Lit Material")]
     internal class MergeToonLitMaterial : EditSkinnedMeshComponent
     {
         public MergeInfo[] merges = Array.Empty<MergeInfo>();

diff --git a/Runtime/RemoveMeshInBox.cs b/Runtime/RemoveMeshInBox.cs
@@ -4,7 +4,7 @@
 
 namespace Anatawa12.AvatarOptimizer
 {
-    [AddComponentMenu("Optimizer/Remove Mesh in Box")]
+    [AddComponentMenu("Avatar Optimizer/Remove Mesh in Box")]
     [RequireComponent(typeof(SkinnedMeshRenderer))]
     [DisallowMultipleComponent]
     internal class RemoveMeshInBox : EditSkinnedMeshComponent

diff --git a/Runtime/UnusedBonesByReferencesTool.cs b/Runtime/UnusedBonesByReferencesTool.cs
@@ -3,7 +3,7 @@
 
 namespace Anatawa12.AvatarOptimizer
 {
-    [AddComponentMenu("Optimizer/UnusedBonesByReferencesTool")]
+    [AddComponentMenu("Avatar Optimizer/UnusedBonesByReferencesTool")]
     [DisallowMultipleComponent]
     internal class UnusedBonesByReferencesTool : AvatarGlobalComponent
     {

diff --git a/Test~/AddComponentMenuTest.cs b/Test~/AddComponentMenuTest.cs
@@ -0,0 +1,30 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Reflection;
+using NUnit.Framework;
+using UnityEngine;
+
+namespace Anatawa12.AvatarOptimizer.Test
+{
+    public class AddComponentMenuTest
+    {
+        [Test]
+        [TestCaseSource(nameof(ComponentTypes))]
+        public void CheckAddComponentMenuIsInAvatarOptimizer(Type type)
+        {
+            var addComponentMenu = type.GetCustomAttribute<AddComponentMenu>();
+            Assert.That(addComponentMenu, Is.Not.Null);
+            Assert.That(addComponentMenu.componentMenu, Does.StartWith("Avatar Optimizer/").Or.Empty);
+        }
+
+        static IEnumerable<Type> ComponentTypes()
+        {
+            return 
+                typeof(AvatarTagComponent).Assembly
+                .GetTypes()
+                .Where(x => x.IsClass && !x.IsAbstract)
+                .Where(x => typeof(MonoBehaviour).IsAssignableFrom(x));
+        }
+    }
+}
diff --git a/Test~/AddComponentMenuTest.cs.meta b/Test~/AddComponentMenuTest.cs.meta