# HG changeset patch
# User Jeroen Demeyer <jdemeyer@cage.ugent.be>
# Date 1333379973 -7200
# Node ID f7da924ee520a415895881fccc823507e288f1a2
# Parent  7bb540c65f7caa83e55b4518e50c42ee489f4d39
Fix __cmp__ tests

diff --git a/sage/combinat/root_system/type_dual.py b/sage/combinat/root_system/type_dual.py
--- a/sage/combinat/root_system/type_dual.py
+++ b/sage/combinat/root_system/type_dual.py
@@ -131,14 +131,16 @@
             sage: F4dual = CartanType(['F', 4]).dual()
             sage: cmp(F4dual, F4dual)
             0
-            sage: cmp(F4dual, B4dual)
-            1
-            sage: cmp(B4dual, F4dual)
-            -1
-            sage: cmp(B4dual, B4)
-            1
 
-        .. todo:: do we really need a cmp, or just eq?
+        Whether ``cmp()`` returns 1 or -1 doesn't matter, just check
+        that the following are non-zero::
+
+            sage: cmp(F4dual, B4dual) != 0
+            True
+            sage: cmp(B4dual, F4dual) * cmp(F4dual, B4dual) < 0
+            True
+            sage: cmp(B4dual, B4) != 0
+            True
         """
         if other.__class__ != self.__class__:
             return cmp(self.__class__, other.__class__)