From b88a5909389bb2d6101e3a3756c1537e46221622 Mon Sep 17 00:00:00 2001
From: pciturri <pciturri@gfz-potsdam.de>
Date: Fri, 23 Aug 2024 15:39:13 +0200
Subject: [PATCH] refactor: Added type hints and docstrings to helper
 functions. Removed unused helper functions

---
 tests/test_plots.py | 60 ++++++++++++++++++++++-----------------------
 1 file changed, 30 insertions(+), 30 deletions(-)

diff --git a/tests/test_plots.py b/tests/test_plots.py
index df298057..2507177a 100644
--- a/tests/test_plots.py
+++ b/tests/test_plots.py
@@ -724,36 +724,36 @@ def test_plot_basemap_with_features(self, mock_get_basemap):
         self.assertIsInstance(ax, plt.Axes)
         mock_get_basemap.assert_not_called()
         self.assertTrue(ax.get_legend() is None)
-
-    @unittest.skipIf(is_github_ci(), "Skipping test in GitHub CI environment")
-    @unittest.skipIf(not is_internet_available(), "Skipping test due to no internet connection")
-    def test_plot_google_satellite(self):
-        basemap = "google-satellite"
-        ax = plot_basemap(
-            basemap=basemap,
-            extent=self.chiloe_extent,
-            coastline=True,
-            tile_depth=4,
-            show=show_plots,
-        )
-        self.assertIsInstance(ax, plt.Axes)
-        self.assertTrue(ax.get_legend() is None)
-
-    @unittest.skipIf(is_github_ci(), "Skipping test in GitHub CI environment")
-    @unittest.skipIf(not is_internet_available(), "Skipping test due to no internet connection")
-    def test_plot_esri(self):
-        basemap = "ESRI_terrain"
-
-        ax = plot_basemap(
-            basemap,
-            self.chiloe_extent,
-            coastline=True,
-            borders=True,
-            tile_depth=4,
-            show=show_plots,
-        )
-        self.assertIsInstance(ax, plt.Axes)
-        self.assertTrue(ax.get_legend() is None)
+    #
+    # @unittest.skipIf(is_github_ci(), "Skipping test in GitHub CI environment")
+    # @unittest.skipIf(not is_internet_available(), "Skipping test due to no internet connection")
+    # def test_plot_google_satellite(self):
+    #     basemap = "google-satellite"
+    #     ax = plot_basemap(
+    #         basemap=basemap,
+    #         extent=self.chiloe_extent,
+    #         coastline=True,
+    #         tile_depth=4,
+    #         show=show_plots,
+    #     )
+    #     self.assertIsInstance(ax, plt.Axes)
+    #     self.assertTrue(ax.get_legend() is None)
+    #
+    # @unittest.skipIf(is_github_ci(), "Skipping test in GitHub CI environment")
+    # @unittest.skipIf(not is_internet_available(), "Skipping test due to no internet connection")
+    # def test_plot_esri(self):
+    #     basemap = "ESRI_terrain"
+    #
+    #     ax = plot_basemap(
+    #         basemap,
+    #         self.chiloe_extent,
+    #         coastline=True,
+    #         borders=True,
+    #         tile_depth=4,
+    #         show=show_plots,
+    #     )
+    #     self.assertIsInstance(ax, plt.Axes)
+    #     self.assertTrue(ax.get_legend() is None)
 
     @patch("csep.utils.plots._get_basemap")
     def test_plot_basemap_set_global(self, mock_get_basemap):