[Fixes #14339] Drop support Dropbox storage manager

Author: mattiagiupponi

geonode/storage/dropbox.py

@@ -30,94 +30,10 @@
 from geonode.storage.exceptions import DataRetrieverExcepion
 from geonode.storage.manager import StorageManager
 from geonode.storage.gcs import GoogleStorageManager
-from geonode.storage.dropbox import DropboxStorageManager
 from geonode.base.populate_test_data import create_single_dataset
 from geonode.tests.base import GeoNodeBaseTestSupport
 
 
-class TestDropboxStorageManager(SimpleTestCase):
-    def setUp(self):
-        with self.settings(DROPBOX_OAUTH2_TOKEN="auth_token"):
-            self.sut = DropboxStorageManager()
-
-    @patch("geonode.storage.dropbox.DropBoxStorage.delete")
-    def test_dropbox_deleted(self, dbx):
-        """
-        Will test that the function returns the expected result
-        and that the DropBoxStorage function as been called with the expected parameters
-        """
-        dbx.return_value = None
-        output = self.sut.delete("filename")
-        self.assertIsNone(output)
-        dbx.assert_called_once_with("filename")
-
-    @patch("geonode.storage.dropbox.DropBoxStorage.exists")
-    def test_dropbox_exists(self, dbx):
-        """
-        Will test that the function returns the expected result
-        and that the DropBoxStorage function as been called with the expected parameters
-        """
-        dbx.return_value = True
-        output = self.sut.exists("filename")
-        self.assertTrue(output)
-        dbx.assert_called_once_with("filename")
-
-    @patch("geonode.storage.dropbox.DropBoxStorage.listdir")
-    def test_dropbox_listdir(self, dbx):
-        """
-        Will test that the function returns the expected result
-        and that the DropBoxStorage function as been called with the expected parameters
-        """
-        dbx.return_value = (["folder1"], ["file1", "file2"])
-        output = self.sut.listdir("Apps/")
-        self.assertTupleEqual((["folder1"], ["file1", "file2"]), output)
-        dbx.assert_called_once_with("Apps/")
-
-    @patch("geonode.storage.dropbox.DropBoxStorage._open")
-    def test_dropbox_open(self, dbx):
-        """
-        Will test that the function returns the expected result
-        and that the DropBoxStorage function as been called with the expected parameters
-        """
-        dbx.return_value = io.StringIO()
-        output = self.sut.open("name", mode="xx")
-        self.assertEqual(type(output), io.StringIO().__class__)
-        dbx.assert_called_once_with("name", "xx")
-
-    @patch("geonode.storage.dropbox.DropBoxStorage._full_path")
-    def test_dropbox_path(self, dbx):
-        """
-        Will test that the function returns the expected result
-        and that the DropBoxStorage function as been called with the expected parameters
-        """
-        dbx.return_value = "/opt/full/path/to/file"
-        output = self.sut.path("file")
-        self.assertEqual("/opt/full/path/to/file", output)
-        dbx.assert_called_once_with("file")
-
-    @patch("geonode.storage.dropbox.DropBoxStorage.save")
-    def test_dropbox_save(self, dbx):
-        """
-        Will test that the function returns the expected result
-        and that the DropBoxStorage function as been called with the expected parameters
-        """
-        dbx.return_value = "cleaned_name"
-        output = self.sut.save("file_name", "content")
-        self.assertEqual("cleaned_name", output)
-        dbx.assert_called_once_with("file_name", "content")
-
-    @patch("geonode.storage.dropbox.DropBoxStorage.size")
-    def test_dropbox_size(self, dbx):
-        """
-        Will test that the function returns the expected result
-        and that the DropBoxStorage function as been called with the expected parameters
-        """
-        dbx.return_value = 1
-        output = self.sut.size("name")
-        self.assertEqual(1, output)
-        dbx.assert_called_once_with("name")
-
-
 class TestGoogleStorageManager(SimpleTestCase):
     def setUp(self):
         self.sut = GoogleStorageManager

geonode/storage/tests.py

@@ -116,7 +116,6 @@ dependencies = [
     "django-bootstrap3-datetimepicker-2==2.8.3",
     # storage manager dependencies
     "django-storages==1.14.3",
-    "dropbox==11.36.2",
     "google-cloud-storage==2.16.0",
     "boto3==1.39.3",
     # Django Caches