more tests for stuff that should not be implemented

universalcore · May 19, 2015 · 43ac31b · 43ac31b
1 parent 16ca15b
commit 43ac31b
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 0 deletions.
diff --git a/elasticgit/remote_storage.py b/elasticgit/remote_storage.py
@@ -27,6 +27,9 @@ def __init__(self, repo_url):
         self.repo_name, _, self.suffix = basename.partition('.')
 
     def mk_request(self, *args, **kwargs):
+        """
+        Mocked out in tests
+        """
         return requests.request(*args, **kwargs)
 
     def url(self, *parts):

diff --git a/elasticgit/tests/test_remote_storage.py b/elasticgit/tests/test_remote_storage.py
@@ -58,6 +58,26 @@ def test_destroy_storage(self):
         self.assertRaises(
             RemoteStorageException, self.rsm.destroy_storage)
 
+    def test_store(self):
+        self.assertRaises(
+            RemoteStorageException, self.rsm.store, TestPerson({}),
+            'commit message')
+
+    def test_store_data(self):
+        self.assertRaises(
+            RemoteStorageException, self.rsm.store_data, 'foo/bar',
+            'data', 'commit message')
+
+    def test_delete(self):
+        self.assertRaises(
+            RemoteStorageException, self.rsm.delete, TestPerson({}),
+            'commit message')
+
+    def test_delete_data(self):
+        self.assertRaises(
+            RemoteStorageException, self.rsm.delete_data, 'foo/bar',
+            'commit message')
+
     def test_iterate(self):
         with patch.object(self.rsm, 'mk_request') as mock:
             response = Response()