Merge pull request #27 from cedardevs/bugfix-multipolygon

Int to double coersion is not automatic, and the error is hidden in a…

Author: zebdelk

schemas-parse/src/main/groovy/org/cedar/schemas/parse/ISOParser.groovy

@@ -328,8 +328,8 @@ class ISOParser {
     else if (west > east) {
       builder = MultiPolygon.newBuilder()
       coordinates = [
-        [[[-180, south], [east, south], [east, north], [-180, north], [-180, south]]],
-        [[[west, south], [180, south], [180, north], [west, north], [west, south]]]
+        [[[-180.0, south], [east, south], [east, north], [-180.0, north], [-180.0, south]]],
+        [[[west, south], [180.0, south], [180.0, north], [west, north], [west, south]]]
         ]
     }
     else {

schemas-parse/src/test/groovy/org/cedar/schemas/parse/ISOParserSpec.groovy

@@ -1,5 +1,6 @@
 package org.cedar.schemas.parse
 
+import org.cedar.schemas.avro.psi.Discovery
 import org.cedar.schemas.avro.geojson.LineStringType
 import org.cedar.schemas.avro.geojson.PointType
 import org.cedar.schemas.avro.geojson.PolygonType
@@ -208,9 +209,11 @@ class ISOParserSpec extends Specification {
     def document = ClassLoader.systemClassLoader.getResourceAsStream("test-iso-discontinous-antimeridian-coords.xml").text
     def metadata = new XmlSlurper().parseText(document)
 
-    when:
-    def result = ISOParser.parseSpatialInfo(metadata)
+    when: // convert to map and back catches dumb problems that suck to find later (int->double coersion)
+    AvroUtils.mapToAvro(AvroUtils.avroToMap(ISOParser.parseXMLMetadataToDiscovery(document)), Discovery)
 
+    and:
+    def result = ISOParser.parseSpatialInfo(metadata)
 
     then: