[USDU-407] Add automated tests for Analytics

TestProject/Usd-Development/Packages/manifest.json

@@ -6,6 +6,7 @@
     "com.unity.ide.rider": "1.1.4",
     "com.unity.recorder": "2.5.5",
     "com.unity.test-framework": "1.1.24",
+    "com.unity.editor-analytics-debugger": "0.1.0",
     "com.unity.modules.ai": "1.0.0",
     "com.unity.modules.animation": "1.0.0",
     "com.unity.modules.assetbundle": "1.0.0",
@@ -39,5 +40,14 @@
   },
   "testables": [
     "com.unity.formats.usd"
+  ],
+  "scopedRegistries": [
+    {
+      "name": "Analytics Debugger",
+      "url": "https://artifactory.prd.cds.internal.unity3d.com/artifactory/api/npm/upm-candidates",
+      "scopes": [
+        "com.unity.editor-analytics-debugger"
+      ]
+    }
   ]
 }

package/com.unity.formats.usd/Common/UsdEditorAnalytics.cs

@@ -11,10 +11,12 @@
 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 // See the License for the specific language governing permissions and
 // limitations under the License.
+
 # if ENABLE_CLOUD_SERVICES_ANALYTICS && UNITY_EDITOR
 # define USE_EDITOR_ANALYTICS
 # endif
 
+using System;
 using System.Collections.Generic;
 using UnityEditor;
 using UnityEngine;
@@ -40,7 +42,11 @@ public static class UsdEditorAnalytics
         // Universal USD Analytics
         const string k_UsageEventName = "USDPackageUsage";
 
-        struct UsageAnalyticsData
+        public interface IUsdAnalyticsData
+        { }
+
+        [Serializable]
+        public struct UsageAnalyticsData : IUsdAnalyticsData
         {
             public bool InitSucceeded;
             public double TimeTakenMs;
@@ -49,7 +55,8 @@ struct UsageAnalyticsData
         // USD Import Analytics
         const string k_ImportEventName = "USDFileImport";
 
-        struct ImportAnalyticsData
+        [Serializable]
+        public struct ImportAnalyticsData : IUsdAnalyticsData
         {
             public string FileExtension;
             public double TimeTakenMs;
@@ -83,7 +90,8 @@ public struct ImportResult
         // USD Reimport Analytics
         const string k_ReimportEventName = "USDFileReimport";
 
-        struct ReimportAnalyticsData
+        [Serializable]
+        public struct ReimportAnalyticsData : IUsdAnalyticsData
         {
             public string FileExtension;
             public double TimeTakenMs;
@@ -98,7 +106,8 @@ struct ReimportAnalyticsData
         // USD Export Analytics
         const string k_ExportEventName = "USDFileExport";
 
-        struct ExportAnalyticsData
+        [Serializable]
+        public struct ExportAnalyticsData : IUsdAnalyticsData
         {
             public string FileExtension;
             public double TimeTakenMs;
@@ -109,7 +118,7 @@ struct ExportAnalyticsData
         // USD Recorder Export Analytics
         const string k_RecorderExportEventName = "USDFileRecorderExport";
 
-        struct RecorderExportAnalyticsData
+        public struct RecorderExportAnalyticsData
         {
             public string FileExtension;
             public double TimeTakenMs;

package/com.unity.formats.usd/Tests/Common/BaseFixture.cs

@@ -55,7 +55,7 @@ public string GetUSDScenePath(string usdFileName = null)
                 usdFileName = System.Guid.NewGuid().ToString();
             }
 
-            if (!usdFileName.EndsWith(".usd") && !usdFileName.EndsWith(".usda") && !usdFileName.EndsWith(".usdz"))
+            if (!usdFileName.EndsWith(".usd") && !usdFileName.EndsWith(".usda") && !usdFileName.EndsWith(".usdz") && !usdFileName.EndsWith(".usdc"))
             {
                 usdFileName += ".usda";
             }
@@ -75,6 +75,8 @@ public string GetPrefabPath(string prefabName = null, bool resource = false)
                 prefabName += ".prefab";
             }
 
+            var test = Path.Combine(ArtifactsDirectoryRelativePath, resource ? "Resources" : "", prefabName);
+            Debug.Log(test);
             return Path.Combine(ArtifactsDirectoryRelativePath, resource ? "Resources" : "", prefabName);
         }
 
@@ -106,6 +108,15 @@ public Scene CreateTestUsdScene(string fileName = "testUsd.usda")
             return scene;
         }
 
+        public Scene CreateEmptyTestUsdScene(string fileName = "testUsd.usda")
+        {
+            var dummyUsdPath = CreateTmpUsdFile(fileName);
+            var scene = ImportHelpers.InitForOpen(dummyUsdPath);
+            scene.Write("/emptyRoot", new XformSample());
+            scene.Save();
+            return scene;
+        }
+
         [SetUp]
         public void InitUSDAndArtifactsDirectory()
         {
@@ -126,7 +137,14 @@ public void CleanupTestArtifacts()
         {
             if (Directory.Exists(ArtifactsDirectoryFullPath))
             {
-                Directory.Delete(ArtifactsDirectoryFullPath, true);
+                try
+                {
+                    Directory.Delete(ArtifactsDirectoryFullPath, true);
+                }
+                catch
+                {
+                    Debug.Log("Artifact Clean up has failed - This should not happen in most cases, but even if so, the test case should not be affected.");
+                }
             }
 
             DeleteMetaFile(ArtifactsDirectoryFullPath);

package/com.unity.formats.usd/Tests/Common/Data/UsdImportTests/InvalidContent.usda

@@ -0,0 +1 @@
+invalidFile