Always use NtsGeometryServices instance for geometry creation

src/HotChocolate/Spatial/src/Types/GeoJsonResolvers.cs

@@ -39,5 +39,5 @@ public IReadOnlyCollection<double> GetBbox([Parent] Geometry geometry)
     }
 
     public int GetCrs([Parent] Geometry geometry) =>
-        geometry.SRID == 0 ? 4326 : geometry.SRID;
+        geometry.SRID is 0 or -1 ? 4326 : geometry.SRID;
 }

src/HotChocolate/Spatial/src/Types/Serialization/GeoJsonLineStringSerializer.cs

@@ -38,15 +38,11 @@ public override LineString CreateGeometry(
             throw Serializer_Parse_CoordinatesIsInvalid(type);
         }
 
-        if (crs is not null)
-        {
-            GeometryFactory factory =
-                NtsGeometryServices.Instance.CreateGeometryFactory(crs.Value);
-
-            return factory.CreateLineString(coords);
-        }
+        GeometryFactory factory = crs is null
+            ? NtsGeometryServices.Instance.CreateGeometryFactory()
+            : NtsGeometryServices.Instance.CreateGeometryFactory(crs.Value);
 
-        return new LineString(coords);
+        return factory.CreateLineString(coords);
     }
 
     public override object CreateInstance(IType type, object?[] fieldValues)

src/HotChocolate/Spatial/src/Types/Serialization/GeoJsonMultiLineStringSerializer.cs

@@ -69,15 +69,11 @@ public override MultiLineString CreateGeometry(
                 .CreateGeometry(type, parts[i], crs);
         }
 
-        if (crs is not null)
-        {
-            GeometryFactory factory =
-                NtsGeometryServices.Instance.CreateGeometryFactory(crs.Value);
-
-            return factory.CreateMultiLineString(geometries);
-        }
+        GeometryFactory factory = crs is null
+            ? NtsGeometryServices.Instance.CreateGeometryFactory()
+            : NtsGeometryServices.Instance.CreateGeometryFactory(crs.Value);
 
-        return new MultiLineString(geometries);
+        return factory.CreateMultiLineString(geometries);
     }
 
     public override object CreateInstance(IType type, object?[] fieldValues)

src/HotChocolate/Spatial/src/Types/Serialization/GeoJsonMultiPointSerializer.cs

@@ -45,14 +45,11 @@ public override MultiPoint CreateGeometry(
         goto Error;
 
 Success:
-        if (crs is not null)
-        {
-            GeometryFactory factory =
-                NtsGeometryServices.Instance.CreateGeometryFactory(crs.Value);
-            return factory.CreateMultiPoint(geometries);
-        }
-
-        return new MultiPoint(geometries);
+        GeometryFactory factory = crs is null
+            ? NtsGeometryServices.Instance.CreateGeometryFactory()
+            : NtsGeometryServices.Instance.CreateGeometryFactory(crs.Value);
+
+        return factory.CreateMultiPoint(geometries);
 
 Error:
         throw Serializer_Parse_CoordinatesIsInvalid(type);

src/HotChocolate/Spatial/src/Types/Serialization/GeoJsonMultiPolygonSerializer.cs

@@ -57,15 +57,11 @@ public override MultiPolygon CreateGeometry(
         goto Error;
 
 Success:
-        if (crs is not null)
-        {
-            GeometryFactory factory =
-                NtsGeometryServices.Instance.CreateGeometryFactory(crs.Value);
-
-            return factory.CreateMultiPolygon(geometries);
-        }
+        GeometryFactory factory = crs is null
+            ? NtsGeometryServices.Instance.CreateGeometryFactory()
+            : NtsGeometryServices.Instance.CreateGeometryFactory(crs.Value);
 
-        return new MultiPolygon(geometries);
+        return factory.CreateMultiPolygon(geometries);
 
 Error:
         throw Serializer_Parse_CoordinatesIsInvalid(type);

src/HotChocolate/Spatial/src/Types/Serialization/GeoJsonPointSerializer.cs

@@ -57,15 +57,11 @@ public override Point CreateGeometry(
             throw Serializer_Parse_CoordinatesIsInvalid(type);
         }
 
-        if (crs is not null)
-        {
-            GeometryFactory factory =
-                NtsGeometryServices.Instance.CreateGeometryFactory(crs.Value);
-
-            return factory.CreatePoint(coordinate);
-        }
+        GeometryFactory factory = crs is null
+            ? NtsGeometryServices.Instance.CreateGeometryFactory()
+            : NtsGeometryServices.Instance.CreateGeometryFactory(crs.Value);
 
-        return new Point(coordinate);
+        return factory.CreatePoint(coordinate);
     }
 
     public override object CreateInstance(IType type, object?[] fieldValues)

src/HotChocolate/Spatial/src/Types/Serialization/GeoJsonPolygonSerializer.cs

@@ -51,39 +51,22 @@ public override Polygon CreateGeometry(
             throw Serializer_Parse_CoordinatesIsInvalid(type);
         }
 
-        if (crs is not null)
-        {
-            GeometryFactory factory =
-                NtsGeometryServices.Instance.CreateGeometryFactory(crs.Value);
-
-            LinearRing ringSrid = factory.CreateLinearRing((Coordinate[])ringsCoordinates[0]);
-            LinearRing[] holes = Array.Empty<LinearRing>();
-            if (ringsCoordinates.Length > 1)
-            {
-                holes = new LinearRing[ringsCoordinates.Length - 1];
-                for (var i = 0; i < ringsCoordinates.Length - 1; i++)
-                {
-                    holes[i] = factory.CreateLinearRing((Coordinate[])ringsCoordinates[i + 1]);
-                }
-            }
+        GeometryFactory factory = crs is null
+            ? NtsGeometryServices.Instance.CreateGeometryFactory()
+            : NtsGeometryServices.Instance.CreateGeometryFactory(crs.Value);
 
-            return factory.CreatePolygon(ringSrid, holes);
-        }
-        else
+        LinearRing ringSrid = factory.CreateLinearRing((Coordinate[])ringsCoordinates[0]);
+        LinearRing[] holes = Array.Empty<LinearRing>();
+        if (ringsCoordinates.Length > 1)
         {
-            var ring = new LinearRing((Coordinate[])ringsCoordinates[0]);
-            LinearRing[] holes = Array.Empty<LinearRing>();
-            if (ringsCoordinates.Length > 1)
+            holes = new LinearRing[ringsCoordinates.Length - 1];
+            for (var i = 0; i < ringsCoordinates.Length - 1; i++)
             {
-                holes = new LinearRing[ringsCoordinates.Length - 1];
-                for (var i = 0; i < ringsCoordinates.Length - 1; i++)
-                {
-                    holes[i] = new LinearRing((Coordinate[])ringsCoordinates[i + 1]);
-                }
+                holes[i] = factory.CreateLinearRing((Coordinate[])ringsCoordinates[i + 1]);
             }
-
-            return new Polygon(ring, holes);
         }
+
+        return factory.CreatePolygon(ringSrid, holes);
     }
 
     public override object CreateInstance(IType type, object?[] fieldValues)

src/HotChocolate/Spatial/test/Data.Filters.SqlServer.Tests/PostgreSqlFixture.cs

@@ -0,0 +1,9 @@
+using Squadron;
+using Xunit;
+
+namespace HotChocolate.Data.Filters.Spatial;
+
+[CollectionDefinition("Postgres")]
+public class PostgreSqlFixture : ICollectionFixture<PostgreSqlResource<PostgisConfig>>
+{
+}

src/HotChocolate/Spatial/test/Data.Filters.SqlServer.Tests/QueryableFilterVisitorContainsTests.cs

@@ -6,9 +6,9 @@
 
 namespace HotChocolate.Data.Filters.Spatial;
 
+[Collection("Postgres")]
 public class QueryableFilterVisitorContainsTests
     : SchemaCache
-    , IClassFixture<PostgreSqlResource<PostgisConfig>>
 {
     private static readonly Polygon _truePolygon = new Polygon(
         new LinearRing(new[]

src/HotChocolate/Spatial/test/Data.Filters.SqlServer.Tests/QueryableFilterVisitorDistanceTests.cs

@@ -6,9 +6,9 @@
 
 namespace HotChocolate.Data.Filters.Spatial;
 
+[Collection("Postgres")]
 public class QueryableFilterVisitorDistanceTests
     : SchemaCache
-    , IClassFixture<PostgreSqlResource<PostgisConfig>>
 {
     private static readonly Polygon _truePolygon = new Polygon(
         new LinearRing(new[]

src/HotChocolate/Spatial/test/Data.Filters.SqlServer.Tests/QueryableFilterVisitorIntersectTests.cs

@@ -6,9 +6,9 @@
 
 namespace HotChocolate.Data.Filters.Spatial;
 
+[Collection("Postgres")]
 public class QueryableFilterVisitorIntersectsTests
     : SchemaCache
-    , IClassFixture<PostgreSqlResource<PostgisConfig>>
 {
     private static readonly Polygon _truePolygon =
         new Polygon(new LinearRing(new[]

src/HotChocolate/Spatial/test/Data.Filters.SqlServer.Tests/QueryableFilterVisitorOverlapsTests.cs

@@ -6,9 +6,9 @@
 
 namespace HotChocolate.Data.Filters.Spatial;
 
+[Collection("Postgres")]
 public class QueryableFilterVisitorOverlapsTests
     : SchemaCache
-    , IClassFixture<PostgreSqlResource<PostgisConfig>>
 {
     private static readonly Polygon _truePolygon =
         new Polygon(new LinearRing(new[]

src/HotChocolate/Spatial/test/Data.Filters.SqlServer.Tests/QueryableFilterVisitorTouchesTests.cs

@@ -7,9 +7,9 @@
 
 namespace HotChocolate.Data.Filters.Spatial;
 
+[Collection("Postgres")]
 public class QueryableFilterVisitorTouchesTests
     : SchemaCache
-    , IClassFixture<PostgreSqlResource<PostgisConfig>>
 {
     private static readonly Polygon _truePolygon =
         new Polygon(new LinearRing(new[]

src/HotChocolate/Spatial/test/Data.Filters.SqlServer.Tests/QueryableFilterVisitorWithinTests.cs

@@ -6,9 +6,9 @@
 
 namespace HotChocolate.Data.Filters.Spatial;
 
+[Collection("Postgres")]
 public class QueryableFilterVisitorWithinTests
     : SchemaCache
-    , IClassFixture<PostgreSqlResource<PostgisConfig>>
 {
     private static readonly Polygon _truePolygon =
         new Polygon(new LinearRing(new[]