HYDRA-1020 : Re-enable 2 failing tests now that selection highlight o…

…n instances work (with usd 24.11)
Autodesk · Dec 9, 2024 · 72563b6 · 72563b6
1 parent 505b724
commit 72563b6
Show file tree

Hide file tree

Showing 8 changed files with 20 additions and 15 deletions.
diff --git a/test/lib/mayaUsd/render/mayaToHydra/StageVariantsTest/oneCube.png b/test/lib/mayaUsd/render/mayaToHydra/StageVariantsTest/oneCube.png
diff --git a/test/lib/mayaUsd/render/mayaToHydra/StageVariantsTest/oneCubeInstanceable.png b/test/lib/mayaUsd/render/mayaToHydra/StageVariantsTest/oneCubeInstanceable.png
diff --git a/test/lib/mayaUsd/render/mayaToHydra/StageVariantsTest/threeCubes.png b/test/lib/mayaUsd/render/mayaToHydra/StageVariantsTest/threeCubes.png
diff --git a/test/lib/mayaUsd/render/mayaToHydra/StageVariantsTest/threeCubesInstanceable.png b/test/lib/mayaUsd/render/mayaToHydra/StageVariantsTest/threeCubesInstanceable.png
diff --git a/...lib/mayaUsd/render/mayaToHydra/StageVariantsTest/threeCubesWithDisplacement.png b/...lib/mayaUsd/render/mayaToHydra/StageVariantsTest/threeCubesWithDisplacement.png
diff --git a/test/lib/mayaUsd/render/mayaToHydra/StageVariantsTest/twoCubes.png b/test/lib/mayaUsd/render/mayaToHydra/StageVariantsTest/twoCubes.png
diff --git a/test/lib/mayaUsd/render/mayaToHydra/StageVariantsTest/twoCubesInstanceable.png b/test/lib/mayaUsd/render/mayaToHydra/StageVariantsTest/twoCubesInstanceable.png
diff --git a/test/lib/mayaUsd/render/mayaToHydra/testStageVariants.py b/test/lib/mayaUsd/render/mayaToHydra/testStageVariants.py
@@ -91,8 +91,13 @@ def test_UsdStageVariants(self):
         self.assertIsNotNone(cube2Prim)
         cube3Prim = shapeStage.GetPrimAtPath('/Cubes/Geom/CubeThree')
         self.assertIsNotNone(cube3Prim)
-        cube1Prim.SetInstanceable(True)
-        self.assertTrue(cube1Prim.IsInstanceable())
+        #It is not legal in usd to set cube1 as instanceable
+        #We get the warning :
+        # Warning: The gprim at path </__Prototype_1> was directly instanced. In order to instance this prim, put the prim under an Xform, and instance the Xform parent.
+        #We have another test to check for selection highlight on instanceable prims (native instances)
+        cube1Prim.SetInstanceable(False) #don't set it as instanceable
+        self.assertFalse(cube1Prim.IsInstanceable())
+
         cube2Prim.SetInstanceable(True)
         self.assertTrue(cube2Prim.IsInstanceable())
         cube3Prim.SetInstanceable(True)
@@ -103,20 +108,20 @@ def test_UsdStageVariants(self):
         self.assertSnapshotClose("oneCubeInstanceable.png", self.IMAGE_DIFF_FAIL_THRESHOLD, self.IMAGE_DIFF_FAIL_PERCENT)
         ufeGlobalSel.clear()
 
-        # At time of writing selection highlighting of native instanced prims
-        # does not work (HYDRA-1161).  Enable the following tests when 
-        # HYDRA-1161 is fixed.
-        return
-        modVariant.SetVariantSelection('TwoCubes')
-        self.assertEqual(modVariant.GetVariantSelection(), 'TwoCubes')
-        ufeGlobalSel.append(cubesItems)
-        self.assertSnapshotClose("twoCubesInstanceable.png", self.IMAGE_DIFF_FAIL_THRESHOLD, self.IMAGE_DIFF_FAIL_PERCENT)
-        ufeGlobalSel.clear()
+        #Do these image comparisons only with usd 24.11+ where the selection highlight on instanceable prims is working
+        if self._usdVersion >= (0, 24, 11):
+            modVariant.SetVariantSelection('TwoCubes')
+            self.assertEqual(modVariant.GetVariantSelection(), 'TwoCubes')
+            ufeGlobalSel.append(cubesItems)
+            self.assertSnapshotClose("twoCubesInstanceable.png", self.IMAGE_DIFF_FAIL_THRESHOLD, self.IMAGE_DIFF_FAIL_PERCENT)
+            ufeGlobalSel.clear()
 
-        modVariant.SetVariantSelection('ThreeCubes')
-        self.assertEqual(modVariant.GetVariantSelection(), 'ThreeCubes')
-        ufeGlobalSel.append(cubesItems) 
-        self.assertSnapshotClose("threeCubesInstanceable.png", self.IMAGE_DIFF_FAIL_THRESHOLD, self.IMAGE_DIFF_FAIL_PERCENT)
+            modVariant.SetVariantSelection('ThreeCubes')
+            self.assertEqual(modVariant.GetVariantSelection(), 'ThreeCubes')
+            ufeGlobalSel.append(cubesItems) 
+            self.assertSnapshotClose("threeCubesInstanceable.png", self.IMAGE_DIFF_FAIL_THRESHOLD, self.IMAGE_DIFF_FAIL_PERCENT)
+
+        self.resetDefaultLightIntensityByUsdVersion()
 
 if __name__ == '__main__':
     fixturesUtils.runTests(globals())