add sanity check

Author: benvanwerkhoven

kernel_tuner/util.py

@@ -1233,7 +1233,11 @@ def convert_constraint_lambdas(restrictions):
                 new_c = unparse_constraint_lambda(lambda_ast)
                 res.append(new_c)
 
-    return list(set(res))
+    result = list(set(res))
+    if not len(result) == len(restrictions):
+        raise ValueError("An error occured when parsing restrictions. If you mix lambdas and string-based restrictions, please define the lambda first.")
+
+    return result
 
 
 def compile_restrictions(

test/test_util_functions.py

@@ -783,22 +783,20 @@ def test_convert_constraint_lambdas_illformatted():
     """ Test a number of different ways to define the restrictions.
 
     These are currently not supported but we would like to support them in the future.
-    That is why these tests exist and are expected to fail.
-
-    When there is time to fix this, we can change the test to
-        assert result == expected
-        and work on the code until the test passes.
-
-    But for now we assert result != expected
+    That is why this test expects an exception
 
     """
 
     restrictions = ["32 <= block_size_x*block_size_y <= 512",
                     lambda p: 32 <= p["block_size_x"]*p["block_size_y"] <= 1024,
                     lambda p: p["block_size_z"] < 8]
 
-    result = convert_constraint_lambdas(restrictions)
-    print(result)
     expected = ['32 <= block_size_x * block_size_y <= 1024', 'block_size_z < 8', '32 <= block_size_x*block_size_y <= 512']
 
-    assert result != expected
+    try:
+        result = convert_constraint_lambdas(restrictions)
+        print(result)
+    except ValueError:
+        pass
+
+