Skip to content

Commit

Permalink
Addressed code review feedback.
Browse files Browse the repository at this point in the history
  • Loading branch information
ppt-adsk committed Dec 3, 2024
1 parent 5681c15 commit 7e7268b
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion test/testUtils/mtohUtils.py
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,9 @@ def setUp(self):
self.setHdStormRenderer()

# We've just opened a new scene, so we should not be modified. Setting
# Storm should conceptually not change that status.
# Storm as the renderer should conceptually not change that status, but
# unfortunately in automated tests it does (see setHdStormRender()
# method documentation). Restore modified status to false.
cmds.file(modified=False)

@classmethod
Expand All @@ -133,6 +135,10 @@ def setHdStormRenderer(self):
cmds.modelEditor(
self.activeEditor, e=1,
rendererOverrideName=HD_STORM_OVERRIDE)
# During automated tests, tracing demonstrates that the following call
# to refresh marks the scene as modified, with the modified node being
# defaultRenderGlobals. This behavior cannot be reproduced in a
# non-automated interactive Maya.
cmds.refresh(f=1)
self.delegateId = cmds.mayaHydra(renderer=HD_STORM,
sceneDelegateId="MayaHydraSceneDelegate")
Expand Down

0 comments on commit 7e7268b

Please sign in to comment.