tests extension crashes when loading unless kit 105.1 update is reverted

CesiumGS · Oct 3, 2023 · fc1fb84 · fc1fb84
1 parent a61f0a8
commit fc1fb84
Show file tree

Hide file tree

Showing 4 changed files with 32 additions and 3 deletions.
diff --git a/cesiumOmniverseCppTestsExtension/include/UsdUtilTests.h b/cesiumOmniverseCppTestsExtension/include/UsdUtilTests.h
@@ -0,0 +1,3 @@
+#pragma once
+
+void run_all_UsdUtil_tests();
diff --git a/cesiumOmniverseCppTestsExtension/public/CesiumOmniverseCppTests.cpp b/cesiumOmniverseCppTestsExtension/public/CesiumOmniverseCppTests.cpp
@@ -5,9 +5,10 @@
 
 #include "CesiumOmniverseCppTests.h"
 
+#include "UsdUtilTests.h"
+
 #include "cesium/omniverse/Context.h"
 #include "cesium/omniverse/LoggerSink.h"
-#include "cesium/omniverse/UsdUtil.h"
 
 #include <carb/PluginUtils.h>
 #include <doctest/doctest.h>
@@ -66,7 +67,9 @@ class CesiumOmniverseCppTestsPlugin final : public ICesiumOmniverseCppTestsInter
 
         CESIUM_LOG_INFO("Running Cesium Omniverse Tests with stage id: {}", stage_id);
 
-        // construct a context
+        Context::instance().setStageId(stage_id);
+
+        // construct a doctest context
         doctest::Context context;
 
         // sets the context as the default one - so asserts used outside of a testing context do not crash
@@ -77,6 +80,7 @@ class CesiumOmniverseCppTestsPlugin final : public ICesiumOmniverseCppTestsInter
         context.setAssertHandler(handler);
 
         exampleTest();
+        run_all_UsdUtil_tests();
 
         CESIUM_LOG_INFO("Cesium Omniverse Tests complete");
     }

diff --git a/cesiumOmniverseCppTestsExtension/public/UsdUtilTests.cpp b/cesiumOmniverseCppTestsExtension/public/UsdUtilTests.cpp
@@ -0,0 +1,20 @@
+// #define DOCTEST_CONFIG_IMPLEMENT
+// #define DOCTEST_CONFIG_IMPLEMENTATION_IN_DLL
+// #define DOCTEST_CONFIG_SUPER_FAST_ASSERTS
+
+#include "UsdUtilTests.h"
+
+#include "cesium/omniverse/Context.h"
+#include "cesium/omniverse/LoggerSink.h"
+#include "cesium/omniverse/UsdUtil.h"
+
+#include <doctest/doctest.h>
+
+#include <iostream>
+
+void run_all_UsdUtil_tests() {
+    using namespace cesium::omniverse;
+    CESIUM_LOG_INFO("Running UsdUtil Tests...");
+    CHECK(cesium::omniverse::UsdUtil::primExists(pxr::SdfPath("/Cesium")));
+    CESIUM_LOG_INFO("UsdUtil Tests complete!");
+}
diff --git a/exts/cesium.omniverse.cpp.tests/cesium/omniverse/cpp/tests/extension.py b/exts/cesium.omniverse.cpp.tests/cesium/omniverse/cpp/tests/extension.py
@@ -1,3 +1,4 @@
+import os
 import omni.ext
 import omni.usd
 import omni.kit.ui
@@ -14,7 +15,8 @@ def on_startup(self):
         global tests_interface
         tests_interface = acquire_cesium_omniverse_tests_interface()
 
-        tests_interface.on_startup("exts/cesium.omniverse")
+        tests_interface.on_startup(os.path.join(os.path.dirname(__file__), "../../../../../cesium.omniverse"))
+        # tests_interface.on_startup("/home/melser/git/cesium-omniverse/exts/cesium.omniverse/cesium/omniverse/../../")
 
         # TODO ensure the stage has been set up before getting stage id
         stageId = omni.usd.get_context().get_stage_id()