Call constructor of SavedSearch directly in tests

getting-things-gnome · Mar 11, 2024 · 2a2ce13 · 2a2ce13
1 parent fbf6120
commit 2a2ce13
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/GTG/core/saved_searches.py b/GTG/core/saved_searches.py
@@ -46,11 +46,6 @@ def __init__(self, id: UUID, name: str, query: str) -> None:
 
         super().__init__()
 
-    # TODO probably not needed
-    @classmethod
-    def create(cls, name: str, query: str) -> 'SavedSearch':
-        return cls(uuid4(), name, query)
-
     @GObject.Property(type=str)
     def name(self) -> str:
         """Read only property."""

diff --git a/tests/core/test_saved_search.py b/tests/core/test_saved_search.py
@@ -102,8 +102,8 @@ def test_xml_load_tree(self):
     def test_xml_write_simple(self):
 
         store = SavedSearchStore()
-        search1 = store.add(SavedSearch.create('Some @tag', 'Looking for some tag'))
-        search2 = store.add(SavedSearch.create('Some @other @tag', 'Looking for more'))
+        search1 = store.add(SavedSearch(uuid4(), 'Some @tag', 'Looking for some tag'))
+        search2 = store.add(SavedSearch(uuid4(), 'Some @other @tag', 'Looking for more'))
 
         xml_root = store.to_xml()