From b0389688b9c158345b3c1e819b0b90292fb6b492 Mon Sep 17 00:00:00 2001
From: Håkon Vågsether <hauk142@gmail.com>
Date: Tue, 5 Jan 2021 21:12:42 +0100
Subject: grc: Clean up test_expr_utils.py
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Håkon Vågsether <hauk142@gmail.com>
---
 grc/tests/test_expr_utils.py | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

(limited to 'grc/tests/test_expr_utils.py')

diff --git a/grc/tests/test_expr_utils.py b/grc/tests/test_expr_utils.py
index 29fa552b60..637fb34a35 100644
--- a/grc/tests/test_expr_utils.py
+++ b/grc/tests/test_expr_utils.py
@@ -8,7 +8,6 @@ id_getter = operator.itemgetter(0)
 expr_getter = operator.itemgetter(1)
 
 
-@pytest.mark.xfail(reason="core/utils/expr_utils.py:97: TypeError: '<' not supported between instances of 'NoneType' and 'str'")
 def test_simple():
     objects = [
         ['c', '2 * a + b'],
@@ -18,26 +17,24 @@ def test_simple():
     ]
 
     expected = [
-        ['a', '1'],
         ['d', '5'],
+        ['a', '1'],
         ['b', '2 * a + unknown * d'],
         ['c', '2 * a + b'],
     ]
 
-    out = expr_utils.sort_objects2(objects, id_getter, expr_getter)
+    out = expr_utils.sort_objects(objects, id_getter, expr_getter)
 
     assert out == expected
 
 
-@pytest.mark.xfail(reason="core/utils/expr_utils.py:97: TypeError: '<' not supported between instances of 'NoneType' and 'str'")
-def test_other():
+def test_circular():
     test = [
         ['c', '2 * a + b'],
         ['a', '1'],
         ['b', '2 * c + unknown'],
     ]
 
-    expr_utils.sort_objects2(test, id_getter, expr_getter, check_circular=False)
-
-    with pytest.raises(RuntimeError):
-        expr_utils.sort_objects2(test, id_getter, expr_getter)
+    # Should fail due to circular dependency
+    with pytest.raises(Exception):
+        expr_utils.sort_objects(test, id_getter, expr_getter)
-- 
cgit v1.2.3