GmMultiPolyIntersector for python - bug with "not" vs "is None"

_package/tests/unit_tests/geometry_tests/multi_poly_intersector_pyt.py

@@ -52,6 +52,18 @@ def _run_test(self, pt1, pt2, poly_points, polys, poly_ids, t_vals, pts, startin
                 assert_str = f'Point {idx}[{i}], : Expected {point[i]}. Got {out_point[i]}'
                 assert math.isclose(out_point[i], point[i]), assert_str
 
+    def test_init(self):
+        """Tests the initializer."""
+        with self.assertRaises(ValueError) as error:
+            grid.geometry.MultiPolyIntersector(None, [1, 2, 3])
+            assert str(error) == 'points is a required argument.'
+        with self.assertRaises(ValueError) as error:
+            grid.geometry.MultiPolyIntersector([[1, 2, 3]], None)
+            assert str(error) == 'polys is a required argument.'
+        with self.assertRaises(ValueError) as error:
+            grid.geometry.MultiPolyIntersector([[1, 2, 3]], [1, 2, 3], query='bob')
+            assert str(error) == 'query must be either "covered_by" or "intersects".'
+
     def test_traverse_line_segment_1_out_out(self):
         r"""A test.
 

_package/xms/grid/geometry/multi_poly_intersector.py

@@ -28,9 +28,9 @@ def __init__(self, points, polys, starting_id=1, query='covered_by', **kwargs) -
             **kwargs (dict): Generic keyword arguments
         """
         if 'instance' not in kwargs:
-            if not points:
-                raise ValueError('points is a required arguments.')
-            if not polys:
+            if points is None:
+                raise ValueError('points is a required argument.')
+            if polys is None:
                 raise ValueError('polys is a required argument.')
             if query not in {'covered_by', 'intersects'}:
                 raise ValueError('query must be either "covered_by" or "intersects".')