Remove more unnecessary methods

Author: judahrand

singer/bookmarks.py

@@ -7,10 +7,6 @@
 LOGGER = get_logger()
 
 
-def write_state(state):
-    json.dump(state.to_dict(), sys.stdout, indent=2)
-
-
 class State:
     def __init__(
         self, bookmarks: Optional[Dict] = None, currently_syncing: Optional[str] = None  # pylint: disable=bad-continuation
@@ -28,11 +24,6 @@ def __eq__(self, other: Any) -> bool:
     def bookmarks(self) -> Dict:
         return self._bookmarks
 
-    @classmethod
-    def load(cls, filename: str) -> "State":
-        with open(filename) as fp:  # pylint: disable=invalid-name
-            return State.from_dict(json.load(fp))
-
     @classmethod
     def from_dict(cls, data: Dict) -> "State":
         return State(
@@ -46,9 +37,6 @@ def to_dict(self) -> Dict:
             state["currently_syncing"] = self.get_currently_syncing()
         return state
 
-    def dump(self) -> None:
-        json.dump(self.to_dict(), sys.stdout, indent=2)
-
     def _ensure_bookmark_path(self, path: Sequence) -> None:
         submap = self.bookmarks
         for path_component in path:

tests/test_bookmarks.py

@@ -1,29 +1,6 @@
 from copy import copy
 import unittest
-from singer.bookmarks import State, write_state
-
-
-class TestWriteState(unittest.TestCase):
-    def test_write_empty_state(self):
-        state = State()
-        write_state(state)
-
-    def test_write_state_with_bookmarks(self):
-        stream_id_1 = "customers"
-        bookmark_key_1 = "datetime"
-        bookmark_val_1 = 123456789
-        offset_key = "key"
-        offset_val = "fizzy water"
-
-        bookmarks = {
-            stream_id_1: {
-                bookmark_key_1: bookmark_val_1,
-                "offset": {offset_key: offset_val},
-            }
-        }
-
-        state = State(bookmarks=bookmarks, currently_syncing="customers")
-        write_state(state)
+from singer.bookmarks import State
 
 
 class TestGetBookmark(unittest.TestCase):