Merge branch 'main' into collection

gee-community · Jan 28, 2024 · 86615e6 · 86615e6
2 parents f5a0c25 + ba22a78
commit 86615e6
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/geetools/tools/_deprecated_element.py b/geetools/tools/_deprecated_element.py
@@ -5,4 +5,4 @@
 @deprecated(version="1.0.0", reason="Don't use this method it's considered bad practice.")
 def fillNull(Element, proxy=-999):
     """Fill null values of an Element's properties with a proxy value."""
-    raise Exception("This method is deprecated. Use ee.FeatureCollection.fill() instead")
+    raise NotImplementedError("This method is deprecated. Use ee.FeatureCollection.fill() instead")
diff --git a/tests/test_deprecated.py b/tests/test_deprecated.py
@@ -23,3 +23,11 @@ class TestCollection:
     def test_enumerate(self):
         with pytest.raises(NotImplementedError):
             geetools.collection.enumerate(None)
+
+
+class TestElement:
+    """Test the methods from the deprecated_element module."""
+
+    def test_fillNull(self):
+        with pytest.raises(NotImplementedError):
+            geetools.element.fillNull(None, None)