TestCase : Ensure temp directory can be deleted

GafferHQ · Jan 9, 2025 · 77d4969 · 77d4969
1 parent 49ecfd8
commit 77d4969
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/python/GafferTest/TestCase.py b/python/GafferTest/TestCase.py
@@ -115,6 +115,9 @@ def tearDown( self ) :
 		IECore.RefCounted.collectGarbage()
 
 		if self.__temporaryDirectory is not None :
+			if os.name == "nt" :
+				subprocess.check_call( [ "icacls", self.temporaryDirectory(), "/grant", "Users:(OI)(CI)(W)" ] )
+
 			for root, dirs, files in os.walk( self.temporaryDirectory() ) :
 				for fileName in [ p for p in files + dirs if not ( pathlib.Path( root ) / p ).is_symlink() ] :
 					( pathlib.Path( root ) / fileName ).chmod( stat.S_IRWXU )

diff --git a/python/GafferUSDTest/USDLayerWriterTest.py b/python/GafferUSDTest/USDLayerWriterTest.py
@@ -265,8 +265,5 @@ def testNoWritePermissions( self ) :
 		with self.assertRaisesRegex( RuntimeError, 'Failed to export layer to "{}"'.format( layerWriter["fileName"].getValue() ) ) :
 			layerWriter["task"].execute()
 
-		if os.name == "nt" :
-			subprocess.check_call( [ "icacls", self.temporaryDirectory(), "/grant", "Users:(OI)(CI)(W)" ] )
-
 if __name__ == "__main__":
 	unittest.main()