Added Multi-Bone skinned mesh unit tests (#440)

* Added the Unit Test

BagofBones Test and associated FBX file. Also added one line to the gitignore so it ignores the autogenerated Visual Studio Code folder so I don't have to delete it every time.

* Removed autoformat

* Test Cleanup

* Added More Unit Tests

Tests for:

+ BasicSeparateBind
+ BonesParentedInMesh
+ NullRootMultipleMeshes
+ MultipleMeshesOneHierarchy

+ LooseSkinnedJoints
+ LooseLocators
+ LooseNulls

+ LocatorsInHiearchy
+ NullsInHiearchy
+ MeshesInHierarchy

+ ImproperRootNaming
+ MultipleRoots

* TestBoneWeightExport + TestSkinnedMeshExport

Cleaned up TestMultiRoot and added the extended cases to TestBoneWeightExport  and TestSkinnedMeshExport

* Refactor

Merged the three different SkinnedMesh units tests into one and combined the various skinned mesh case Collections into SkinnedMeshCases

* Added descriptions to the various skinned character tests

* Edit of fbx descriptions

* Removed ImproperRootNaming

Author: alexismorin

.gitignore

@@ -9,6 +9,7 @@
 [Oo]ut/
 [Bb]in/
 [Dd]ebug/
+.vscode/
 
 [Aa]ssets/FbxExporters/*.zip
 

Assets/com.unity.formats.fbx.tests/ModelExporterTest.cs

@@ -551,22 +551,37 @@ private void ExportSkinnedMesh(string fileToExport, out SkinnedMeshRenderer orig
 
         public class SkinnedMeshTestDataClass
         {
-            public static System.Collections.IEnumerable TestCases1 {
-                get {
-                    // for now use this cowboy taken from the asset store as the test file
-                    // TODO: find a better/simpler test file
-                    yield return "Models/Cowboy/cowboyMidPoly(riged).fbx";
-                }
-            }
-            public static System.Collections.IEnumerable TestCases2 {
+            public static System.Collections.IEnumerable SkinnedMeshCases {
                 get {
+                    // Basic Rig with one mesh and one standard hierarchy
+                    yield return "Models/MultiRootCharacters/BasicSeparateBind.fbx";
+                    // Basic Rig with one mesh and one standard hierarchy, with the Mesh parenting the bone structure
+                    yield return "Models/MultiRootCharacters/BonesParentedInMesh.fbx";
+                    // Thigh-bone level Locators in Hierarchy that act as Handles for some of the bones
+                    yield return "Models/MultiRootCharacters/LocatorsInHierachy.fbx";
+                    // Root-Level Locators included in the fbx.
+                    yield return "Models/MultiRootCharacters/LooseLocators.fbx";
+                    // Root-Level Null Objects included in the fbx. 
+                    yield return "Models/MultiRootCharacters/LooseNulls.fbx";
+                    // Basic Rig with additional floating joints independant in hierarchy and skinned to the mesh.
+                    yield return "Models/MultiRootCharacters/LooseSkinnedJoints.fbx";
+                    // Unskinned meshes in hierarchy - eg. Sword & Shield as children of hand nodes
+                    yield return "Models/MultiRootCharacters/MeshesInHierarchy.fbx";
+                    // Rig with one mesh but two bone Hierarchies
+                    yield return "Models/MultiRootCharacters/MultipleRoots.fbx";
+                    // Rig with one standard hierarchy but skinned to multiple meshes at once, All parented under a Null Object
+                    yield return "Models/MultiRootCharacters/NullRootMultipleMeshes.fbx";
+                    // Thigh-bone level Null Objects in Hierarchy that act as Handles for some of the bones
+                    yield return "Models/MultiRootCharacters/NullsInHierarchy.fbx";
+                    // Characters from the asset store
                     yield return "Models/SimpleMan/SimpleMan.fbx";
+                    yield return "Models/Cowboy/cowboyMidPoly(riged).fbx";
                 }
             }
         }
 
-        [Test, TestCaseSource(typeof(SkinnedMeshTestDataClass), "TestCases1")]
-        public void TestSkinnedMeshExport(string fbxPath){
+        [Test, TestCaseSource(typeof(SkinnedMeshTestDataClass), "SkinnedMeshCases")]
+        public void TestSkinnedMeshes (string fbxPath) {
             fbxPath = FindPathInUnitTests (fbxPath);
             Assert.That (fbxPath, Is.Not.Null);
 
@@ -626,23 +641,7 @@ public void TestSkinnedMeshExport(string fbxPath){
                 }
             }
 
-            // TODO: find a way to compare bone weights.
-            //       The boneweights are by vertex, and duplicate vertices
-            //       are removed on export so the lists are not necessarily
-            //       the same length or order.
-            var origWeights = origMesh.boneWeights;
-            Assert.IsNotNull (origWeights);
-            var expWeights = exportedMesh.boneWeights;
-            Assert.IsNotNull (expWeights);
-        }
-
-        [Test, TestCaseSource(typeof(SkinnedMeshTestDataClass), "TestCases2")]
-        public void TestBoneWeightExport(string fbxPath){
-            fbxPath = FindPathInUnitTests (fbxPath);
-            Assert.That (fbxPath, Is.Not.Null);
-
-            SkinnedMeshRenderer originalSkinnedMesh, exportedSkinnedMesh;
-            ExportSkinnedMesh (fbxPath, out originalSkinnedMesh, out exportedSkinnedMesh);
+            // Test Bone Weights
 
             var origVerts = originalSkinnedMesh.sharedMesh.vertices;
             Assert.That (origVerts, Is.Not.Null);
@@ -699,7 +698,6 @@ public void TestBoneWeightExport(string fbxPath){
             Debug.LogWarningFormat ("Compared {0} out of a possible {1} bone weights", comparisonCount, minVertCount);
         }
 
-
         public class Vector3Comparer : IComparer<Vector3>
         {
             public int Compare(Vector3 a, Vector3 b)