add back GeometryCollection, initialize with srid=4326

Author: timgraham

django_mongodb_backend_gis/features.py

@@ -16,27 +16,19 @@ def django_test_expected_failures(self):
                 # SRIDs aren't populated: AssertionError: 4326 != None
                 # self.assertEqual(4326, nullcity.point.srid)
                 "gis_tests.geoapp.tests.GeoModelTest.test_proxy",
-                # GEOSException: Calling transform() with no SRID set is not supported
+                # MongoDB does not support the within lookup
                 "gis_tests.relatedapp.tests.RelatedGeoModelTest.test06_f_expressions",
                 # 'Adapter' object has no attribute 'srid'
                 "gis_tests.geoapp.test_expressions.GeoExpressionsTests.test_geometry_value_annotation",
                 # Object of type ObjectId is not JSON serializable
                 "gis_tests.geoapp.test_serializers.GeoJSONSerializerTests.test_fields_option",
-                # source and target must be of type SpatialReference
                 "gis_tests.geoapp.test_serializers.GeoJSONSerializerTests.test_geometry_field_option",
-                "gis_tests.geoapp.test_serializers.GeoJSONSerializerTests.test_id_field_option",
                 "gis_tests.geoapp.test_serializers.GeoJSONSerializerTests.test_serialization_base",
                 "gis_tests.geoapp.test_serializers.GeoJSONSerializerTests.test_srid_option",
-                # GeometryCollection not supported
+                # Crash initializing GeometryCollection: 'tuple' object is not callable
                 "gis_tests.geoapp.tests.GeoModelTest.test_geometryfield",
-                "gis_tests.geogapp.tests.GeographyTest.test05_geography_layermapping",
-                "gis_tests.layermap.tests.LayerMapTest.test_layermap_unique_multigeometry_fk",
-                "gis_tests.layermap.tests.LayerMapTest.test_null_geom_with_unique",
-                "gis_tests.layermap.tests.LayerMapTest.test_test_fid_range_step",
                 # KeyError: 'within' connection.ops.gis_operators[self.lookup_name]
                 "gis_tests.geoapp.tests.GeoModelTest.test_gis_query_as_string",
-                # Point object != Point object
-                "gis_tests.geoapp.test_expressions.GeoExpressionsTests.test_update_from_other_field",
                 # No lookups are supported (yet?)
                 "gis_tests.geoapp.tests.GeoLookupTest.test_gis_lookups_with_complex_expressions",
             }

django_mongodb_backend_gis/operations.py

@@ -73,13 +73,14 @@ def converter(value, expression, connection):  # noqa: ARG001
                 return None
             geom_class = getattr(geos, value["type"])
             if issubclass(geom_class, geos.GeometryCollection):
-                from django.db import NotSupportedError
-
-                raise NotSupportedError("GeometryCollection not supported")
-                # init_val = [
-                #     geom_class._allowed(value["coordinates"][x][0])
-                #     for x in range(len(value["coordinates"]))
-                # ]
-            return geom_class(value["coordinates"])
+                # TODO: confirm this is correct.
+                return geom_class(
+                    [
+                        geom_class._allowed(value["coordinates"][x][0])
+                        for x in range(len(value["coordinates"]))
+                    ],
+                    srid=4326,
+                )
+            return geom_class(value["coordinates"], srid=4326)
 
         return converter