From 009e2c07625fb5002def297f126ed6df81b91b37 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Henrik=20tom=20W=C3=B6rden?= <h.tomwoerden@indiscale.com>
Date: Mon, 13 Jan 2025 15:26:31 +0100
Subject: [PATCH] TST: mock more unittests
---
unittests/test_crawler.py | 8 ++++++--
unittests/test_sync_graph.py | 12 ++++++++----
2 files changed, 14 insertions(+), 6 deletions(-)
diff --git a/unittests/test_crawler.py b/unittests/test_crawler.py
index 7b19e9a..def00e4 100644
--- a/unittests/test_crawler.py
+++ b/unittests/test_crawler.py
@@ -371,7 +371,8 @@ def test_split_into_inserts_and_updates_with_circ(crawler_mocked_identifiable_re
with pytest.raises(RuntimeError):
crawler._split_into_inserts_and_updates(st)
-
+@patch("caoscrawler.identifiable_adapters._retrieve_RecordType",
+ new=Mock(side_effect=lambda id, name: db.RecordType(id=id,name=name)))
def test_split_into_inserts_and_updates_with_complex(crawler_mocked_identifiable_retrieve):
crawler = crawler_mocked_identifiable_retrieve
# A
@@ -583,7 +584,8 @@ def test_split_into_inserts_and_updates_diff_backref(crawler_mocked_for_backref_
assert len(update) == 2
assert len(insert) == 1
-
+@patch("caoscrawler.identifiable_adapters._retrieve_RecordType",
+ new=Mock(side_effect=lambda id, name: db.RecordType(id=id,name=name)))
def test_replace_entities_with_ids():
crawler = Crawler()
a = (db.Record().add_parent("B").add_property("A", 12345)
@@ -595,6 +597,7 @@ def test_replace_entities_with_ids():
assert a.get_property("B").value == 12345
assert a.get_property("C").value == [12345, 233324]
+
@patch("caoscrawler.identifiable_adapters._retrieve_RecordType",
new=Mock(side_effect=lambda id, name: db.RecordType(id=id,name=name)))
@patch("caoscrawler.crawl.cached_get_entity_by",
@@ -621,6 +624,7 @@ def test_synchronization_no_commit(upmock, insmock):
assert len(ins) == 1
assert len(ups) == 1
+
@patch("caoscrawler.identifiable_adapters._retrieve_RecordType",
new=Mock(side_effect=lambda id, name: db.RecordType(id=id,name=name)))
@patch("caoscrawler.crawl.cached_get_entity_by",
diff --git a/unittests/test_sync_graph.py b/unittests/test_sync_graph.py
index 3c838d5..85979a5 100644
--- a/unittests/test_sync_graph.py
+++ b/unittests/test_sync_graph.py
@@ -384,7 +384,8 @@ def test_backward_references_backref():
st = SyncGraph(ent_list, ident_adapter)
assert st.nodes[1] in st.backward_references_backref[id(st.nodes[0])]
-
+@patch("caoscrawler.identifiable_adapters._retrieve_RecordType",
+ new=Mock(side_effect=lambda id, name: db.RecordType(id=id,name=name)))
@patch("caoscrawler.identifiable_adapters.get_children_of_rt",
new=Mock(side_effect=lambda x: [x]))
def test_set_id_of_node(simple_adapter):
@@ -573,7 +574,8 @@ def test_merging(simple_adapter):
assert len(st.nodes) == 4
assert len(st.unchecked) == 0
-
+@patch("caoscrawler.identifiable_adapters._retrieve_RecordType",
+ new=Mock(side_effect=lambda id, name: db.RecordType(id=id,name=name)))
def test_update_of_reference_values(simple_adapter):
# multiple nodes are merged including one that is referenced
# assure that this still leads to the value of the property of the referencing node to be
@@ -596,7 +598,8 @@ def test_update_of_reference_values(simple_adapter):
b_prop = st.nodes[1].properties[0].value
assert b_prop.id == 101
-
+@patch("caoscrawler.identifiable_adapters._retrieve_RecordType",
+ new=Mock(side_effect=lambda id, name: db.RecordType(id=id,name=name)))
def test_ignoring_irrelevant_references(simple_adapter):
# make sure that a circle of references is no problem if one references is not identifying
b = db.Record(name='b').add_parent("RT5")
@@ -657,7 +660,8 @@ def test_set_each_scalar_value():
_set_each_scalar_value(a, lambda x: x == 42, lambda x: None)
assert a.properties[0].value is None
-
+@patch("caoscrawler.identifiable_adapters._retrieve_RecordType",
+ new=Mock(side_effect=lambda id, name: db.RecordType(id=id,name=name)))
@patch("caoscrawler.identifiable_adapters.cached_query",
new=Mock(side_effect=mock_cached_only_rt_allow_empty))
def test_merge_referenced_by():
--
GitLab