Merge pull request #1316 from CesiumGS/resolve-invalid-loader

Resolve root tile event when `Tileset` has invalid loader

Author: azrogers

CHANGES.md

@@ -1,5 +1,11 @@
 # Change Log
 
+### ? - ?
+
+##### Fixes :wrench:
+
+- `Cesium3DTilesetSelection::Tileset::getRootTileAvailableEvent` will now resolve even when a `Tileset` is constructed with invalid source parameters, instead of hanging indefinitely.
+
 ### v0.58.0 - 2026-03-02
 
 ##### Fixes :wrench:

Cesium3DTilesSelection/src/TilesetContentManager.cpp

@@ -859,6 +859,8 @@ TilesetContentManager::createFromUrl(
           pManager->_rootTileAvailablePromise.reject(
               std::runtime_error("Root tile failed to load."));
         });
+  } else {
+    pManager->_rootTileAvailablePromise.resolve();
   }
 
   return pManager;
@@ -919,6 +921,8 @@ TilesetContentManager::createFromLoaderFactory(
           pManager->_rootTileAvailablePromise.reject(
               std::runtime_error("Root tile failed to load."));
         });
+  } else {
+    pManager->_rootTileAvailablePromise.resolve();
   }
 
   return pManager;

Cesium3DTilesSelection/test/TestTilesetContentManager.cpp

@@ -877,6 +877,48 @@ TEST_CASE("Test tile state machine") {
     CHECK(!upsampledTile.isRenderContent());
     CHECK(!upsampledTile.getContent().getRenderContent());
   }
+
+  SUBCASE("Resolves root tile promise for invalid URL") {
+    // create manager
+    TilesetOptions options{};
+    options.contentOptions.generateMissingNormalsSmooth = true;
+
+    // Create an invalid Cesium ion loader that will never attempt to make a
+    // request.
+    IntrusivePointer<TilesetContentManager> pManager =
+        TilesetContentManager::createFromUrl(externals, options, std::string());
+
+    bool eventResolved = false;
+    pManager->getRootTileAvailableEvent().thenInMainThread(
+        [&eventResolved]() { eventResolved = true; });
+
+    pManager->waitUntilIdle(5000.0);
+    CHECK(eventResolved);
+    CHECK(!pManager->getRootTile());
+  }
+
+  SUBCASE("Resolves root tile promise for invalid loader") {
+    // create manager
+    TilesetOptions options{};
+    options.contentOptions.generateMissingNormalsSmooth = true;
+
+    // Create an invalid Cesium ion loader that will never attempt to make a
+    // request.
+    IntrusivePointer<TilesetContentManager> pManager =
+        TilesetContentManager::createFromCesiumIon(
+            externals,
+            options,
+            0,
+            std::string());
+
+    bool eventResolved = false;
+    pManager->getRootTileAvailableEvent().thenInMainThread(
+        [&eventResolved]() { eventResolved = true; });
+
+    pManager->waitUntilIdle(5000.0);
+    CHECK(eventResolved);
+    CHECK(!pManager->getRootTile());
+  }
 }
 
 TEST_CASE("Test the tileset content manager's post processing for gltf") {