USDSceneTest : Prefer set comparisons to sorted list comparisons

The failure messages are much clearer from set comparisons.

Author: johnhaddon

contrib/IECoreUSD/test/IECoreUSD/USDSceneTest.py

@@ -73,10 +73,9 @@ def temporaryDirectory( self ) :
 
 	def assertSetNamesEqual( self, setNames1, setNames2 ) :
 
-		# Order isn't guaranteed, so sort before comparing
 		self.assertEqual(
-			sorted( [ str( x ) for x in setNames1 ] ),
-			sorted( [ str( x ) for x in setNames2 ] )
+			{ str( x ) for x in setNames1 },
+			{ str( x ) for x in setNames2 }
 		)
 		# Duplicates are not allowed
 		self.assertEqual( len( set( setNames1 ) ), len( setNames1 ) )
@@ -2373,7 +2372,14 @@ def testCustomAttributes( self ) :
 
 		def assertExpectedAttributes( sphere ) :
 			# test expected attributes names
-			self.assertEqual( sorted( sphere.attributeNames() ), sorted( [ "user:notAConstantPrimVar", "user:notAConstantPrimVarDeprecated", "user:test", "studio:foo", "customNamespaced:testAnimated", "user:bongo", "render:test", "ai:disp_height", "ai:poly_mesh:subdiv_iterations" ] ) )
+			self.assertEqual(
+				set( sphere.attributeNames() ),
+				{
+					"user:notAConstantPrimVar", "user:notAConstantPrimVarDeprecated", "user:test",
+					"studio:foo", "customNamespaced:testAnimated", "user:bongo", "render:test",
+					"ai:disp_height", "ai:poly_mesh:subdiv_iterations"
+				}
+			)
 
 			# test incompatible primvars hasAttribute/readAttribute
 			for name in [
@@ -2412,7 +2418,7 @@ def assertAttributesValues( sphere ):
 		self.assertEqual( a.readAttribute( "user:foo", 0 ), IECore.StringData( "yellow" ) )
 
 		b = a.child( "b" )
-		self.assertEqual( sorted( b.attributeNames() ), ["render:notUserPrefixAttribute", "user:baz"] )
+		self.assertEqual( set( b.attributeNames() ), { "render:notUserPrefixAttribute", "user:baz" } )
 		for n in b.attributeNames():
 			self.assertTrue( b.hasAttribute( n ) )
 		# Make sure primvars and indices not loaded as attributes