Merge branch 'dev' into f-refactor-high-level-api

unittests/test_apiutils.py

@@ -186,8 +186,6 @@ def test_compare_special_properties():
             setattr(r2, set_key, 1)
 
         diff_r1, diff_r2 = compare_entities(r1, r2)
-        print(diff_r1)
-        print(diff_r2)
         assert key not in diff_r1
         assert key not in diff_r2
         assert len(diff_r1["parents"]) == 0
@@ -201,10 +199,6 @@ def test_compare_special_properties():
             setattr(r2, set_key, 2)
 
         diff_r1, diff_r2 = compare_entities(r1, r2)
-        print(r1)
-        print(r2)
-        print(diff_r1)
-        print(diff_r2)
         assert key in diff_r1
         assert key in diff_r2
         if key not in INTS:
@@ -217,6 +211,48 @@ def test_compare_special_properties():
         assert len(diff_r2["properties"]) == 0
 
 
+@pytest.mark.xfail
+def test_compare_properties():
+    p1 = db.Property()
+    p2 = db.Property()
+
+    diff_r1, diff_r2 = compare_entities(p1, p2)
+    assert len(diff_r1["parents"]) == 0
+    assert len(diff_r2["parents"]) == 0
+    assert len(diff_r1["properties"]) == 0
+    assert len(diff_r2["properties"]) == 0
+
+    p1.importance = "SUGGESTED"
+    diff_r1, diff_r2 = compare_entities(p1, p2)
+    assert len(diff_r1["parents"]) == 0
+    assert len(diff_r2["parents"]) == 0
+    assert len(diff_r1["properties"]) == 0
+    assert len(diff_r2["properties"]) == 0
+    assert "importance" in diff_r1
+    assert diff_r1["importance"] == "SUGGESTED"
+
+    # TODO: I'm not sure why it is not like this:
+    # assert diff_r2["importance"] is None
+    # ... but:
+    assert "importance" not in diff_r2
+
+    p2.importance = "SUGGESTED"
+    p1.value = 42
+    p2.value = 4
+
+    diff_r1, diff_r2 = compare_entities(p1, p2)
+    assert len(diff_r1["parents"]) == 0
+    assert len(diff_r2["parents"]) == 0
+    assert len(diff_r1["properties"]) == 0
+    assert len(diff_r2["properties"]) == 0
+
+    # Comparing values currently does not seem to be implemented:
+    assert "value" in diff_r1
+    assert diff_r1["value"] == 42
+    assert "value" in diff_r2
+    assert diff_r2["value"] == 4
+
+
 def test_copy_entities():
     r = db.Record(name="A")
     r.add_parent(name="B")