[usdUtils] Prevent ComputeAllDependenciesClient from adding invalid layers

UsdUtils_ComputeAllDependenciesClient would output invalid layers if a layer fails to open.
If the layer does not open correctly, an error will be posted

Author: nvidia-jomiller

pxr/usd/usdUtils/dependencies.cpp

@@ -17,6 +17,7 @@
 #include "pxr/usd/sdf/fileFormat.h"
 #include "pxr/usd/sdf/layerUtils.h"
 
+#include "pxr/base/tf/diagnostic.h"
 #include "pxr/base/trace/trace.h"
 
 #include <functional>
@@ -126,7 +127,12 @@ struct UsdUtils_ComputeAllDependenciesClient
             }
         }
         else if (UsdStage::IsSupportedFile(anchoredPath)) {
-            layers.insert(SdfLayer::FindOrOpen(anchoredPath));
+            SdfLayerRefPtr dependencyLayer = SdfLayer::FindOrOpen(anchoredPath);
+            if (dependencyLayer) {
+                layers.insert(dependencyLayer);
+            } else {
+                TF_RUNTIME_ERROR("Failed to open dependency layer: %s (%s)", dependency.c_str(), anchoredPath.c_str());
+            }
         }
         else {
             assets.insert(resolvedPath);

pxr/usd/usdUtils/testenv/testUsdUtilsDependencies.py

@@ -5,7 +5,7 @@
 # Licensed under the terms set forth in the LICENSE.txt file available at
 # https://openusd.org/license.
 
-from pxr import Ar, UsdUtils, Sdf, Usd
+from pxr import Ar, Tf, UsdUtils, Sdf, Usd
 from pathlib import Path
 import os
 import unittest
@@ -294,6 +294,20 @@ def test_ComputeAllDependenciesMissingExternalDep(self):
         self.assertEqual(unresolved, ["missing.png",
                                       "missing_in_package.png", 
                                       "missing_in_reference.png"])
+        
+    def test_ComputeAllDependenciesInvalidPayload(self):
+        """Tests that invalid payloads are detected and reported"""
+
+        def TestDirPath(path):
+            return os.path.normcase(
+                os.path.abspath(os.path.join(testDir, path)))
+    
+        testDir = "computeAllDependenciesInvalidPayload"
+        stagePath = "root.usda"
+
+        with self.assertRaises(Tf.ErrorException):
+            layers, assets, unresolved = \
+                UsdUtils.ComputeAllDependencies(TestDirPath(stagePath))
 
 if __name__=="__main__":
     unittest.main()

pxr/usd/usdUtils/testenv/testUsdUtilsDependencies/computeAllDependenciesInvalidPayload/invalid.usd

@@ -0,0 +1,11 @@
+#usda 1.0
+
+def "foo"
+{
+    def "invalid" (
+        prepend payload = @./invalid.usd@
+    )
+    {
+
+    }
+}