dcerpc/misc tests: asset GUID ordering in python 2 and 3
authorDouglas Bagnall <douglas.bagnall@catalyst.net.nz>
Fri, 10 Mar 2017 02:48:38 +0000 (15:48 +1300)
committerAndrew Bartlett <abartlet@samba.org>
Fri, 10 Mar 2017 06:31:11 +0000 (07:31 +0100)
Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
python/samba/tests/dcerpc/misc.py

index 7cdb0048148e91ea36b6eb11fa34c8b13ace04e5..68654a7bfd52f9104167d5a35a48a2842c911a55 100644 (file)
@@ -24,6 +24,13 @@ from samba.compat import PY3
 text1 = "76f53846-a7c2-476a-ae2c-20e2b80d7b34"
 text2 = "344edffa-330a-4b39-b96e-2c34da52e8b1"
 
+
+if not PY3:
+    # cmp() exists only in Python 2
+    def cmp(a, b):
+        return (a > b) - (a < b)
+
+
 class GUIDTests(samba.tests.TestCase):
 
     def test_str(self):
@@ -38,18 +45,15 @@ class GUIDTests(samba.tests.TestCase):
         guid1 = misc.GUID(text1)
         guid2 = misc.GUID(text2)
         self.assertFalse(guid1 == guid2)
-        if not PY3:
-            # cmp() exists only in Python 2
-            self.assertTrue(cmp(guid1, guid2) > 0)
+        self.assertGreater(guid1, guid2)
+        self.assertTrue(cmp(guid1, guid2) > 0)
 
     def test_compare_same(self):
         guid1 = misc.GUID(text1)
         guid2 = misc.GUID(text1)
         self.assertTrue(guid1 == guid2)
         self.assertEquals(guid1, guid2)
-        if not PY3:
-            # cmp() exists only in Python 2
-            self.assertEquals(0, cmp(guid1, guid2))
+        self.assertEquals(0, cmp(guid1, guid2))
 
 
 class PolicyHandleTests(samba.tests.TestCase):
@@ -66,4 +70,3 @@ class PolicyHandleTests(samba.tests.TestCase):
     def test_str(self):
         x = misc.policy_handle(text1, 42)
         self.assertEquals("%d, %s" % (42, text1), str(x))
-