diff --git a/unittests/test_apiutils.py b/unittests/test_apiutils.py index 1df392a3f66dff3e2c760359bd61eb7f4ed32823..43ab8107183f16bf8df1d0ea8e447b378bcf8123 100644 --- a/unittests/test_apiutils.py +++ b/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")