/* * SPDX-License-Identifier: Apache-2.0 * * The OpenSearch Contributors require contributions made to * this file be licensed under the Apache-2.0 license or a * compatible open source license. */ /* * Licensed to Elasticsearch under one or more contributor * license agreements. See the NOTICE file distributed with * this work for additional information regarding copyright * ownership. Elasticsearch licenses this file to you under * the Apache License, Version 2.0 (the "License"); you may * not use this file except in compliance with the License. * You may obtain a copy of the License at * * http://www.apache.org/licenses/LICENSE-2.0 * * Unless required by applicable law or agreed to in writing, * software distributed under the License is distributed on an * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY * KIND, either express or implied. See the License for the * specific language governing permissions and limitations * under the License. */ /* * Modifications Copyright OpenSearch Contributors. See * GitHub history for details. */ package org.opensearch.common.geo.builders; import org.locationtech.jts.geom.Coordinate; import org.opensearch.common.geo.builders.ShapeBuilder.Orientation; import org.opensearch.test.geo.RandomShapeGenerator; import org.opensearch.test.geo.RandomShapeGenerator.ShapeType; import org.locationtech.spatial4j.exception.InvalidShapeException; import java.io.IOException; import static org.hamcrest.Matchers.equalTo; public class PolygonBuilderTests extends AbstractShapeBuilderTestCase { @Override protected PolygonBuilder createTestShapeBuilder() { return createRandomShape(); } @Override protected PolygonBuilder createMutation(PolygonBuilder original) throws IOException { return mutate(original); } static PolygonBuilder mutate(PolygonBuilder original) throws IOException { PolygonBuilder mutation = (PolygonBuilder) copyShape(original); return mutatePolygonBuilder(mutation); } static PolygonBuilder mutatePolygonBuilder(PolygonBuilder pb) { if (randomBoolean()) { pb = polyWithOposingOrientation(pb); } else { // change either point in shell or in random hole LineStringBuilder lineToChange; if (randomBoolean() || pb.holes().size() == 0) { lineToChange = pb.shell(); } else { lineToChange = randomFrom(pb.holes()); } Coordinate coordinate = randomFrom(lineToChange.coordinates(false)); if (randomBoolean()) { if (coordinate.x != 0.0) { coordinate.x = coordinate.x / 2; } else { coordinate.x = randomDoubleBetween(-180.0, 180.0, true); } } else { if (coordinate.y != 0.0) { coordinate.y = coordinate.y / 2; } else { coordinate.y = randomDoubleBetween(-90.0, 90.0, true); } } } return pb; } /** * Takes an input polygon and returns an identical one, only with opposing orientation setting. * This is done so we don't have to expose a setter for orientation in the actual class */ private static PolygonBuilder polyWithOposingOrientation(PolygonBuilder pb) { PolygonBuilder mutation = new PolygonBuilder( pb.shell(), pb.orientation() == Orientation.LEFT ? Orientation.RIGHT : Orientation.LEFT ); for (LineStringBuilder hole : pb.holes()) { mutation.hole(hole); } return mutation; } static PolygonBuilder createRandomShape() { PolygonBuilder pgb = (PolygonBuilder) RandomShapeGenerator.createShape(random(), ShapeType.POLYGON); if (randomBoolean()) { pgb = polyWithOposingOrientation(pgb); } return pgb; } public void testCoerceShell() { try { new PolygonBuilder( new LineStringBuilder(new CoordinatesBuilder().coordinate(0.0, 0.0).coordinate(1.0, 0.0).coordinate(1.0, 1.0).build()), Orientation.RIGHT ); fail("should raise validation exception"); } catch (IllegalArgumentException e) { assertEquals("invalid number of points in LinearRing (found [3] - must be >= 4)", e.getMessage()); } PolygonBuilder pb = new PolygonBuilder( new LineStringBuilder(new CoordinatesBuilder().coordinate(0.0, 0.0).coordinate(1.0, 0.0).coordinate(1.0, 1.0).build()), Orientation.RIGHT, true ); assertThat("Shell should have been closed via coerce", pb.shell().coordinates(false).length, equalTo(4)); } public void testCoerceHole() { PolygonBuilder pb = new PolygonBuilder( new CoordinatesBuilder().coordinate(0.0, 0.0).coordinate(2.0, 0.0).coordinate(2.0, 2.0).coordinate(0.0, 0.0) ); try { pb.hole(new LineStringBuilder(new CoordinatesBuilder().coordinate(0.0, 0.0).coordinate(1.0, 0.0).coordinate(1.0, 1.0).build())); fail("should raise validation exception"); } catch (IllegalArgumentException e) { assertEquals("invalid number of points in LinearRing (found [3] - must be >= 4)", e.getMessage()); } pb.hole( new LineStringBuilder(new CoordinatesBuilder().coordinate(0.0, 0.0).coordinate(1.0, 0.0).coordinate(1.0, 1.0).build()), true ); assertThat("hole should have been closed via coerce", pb.holes().get(0).coordinates(false).length, equalTo(4)); } public void testHoleThatIsSouthOfPolygon() { InvalidShapeException e = expectThrows(InvalidShapeException.class, () -> { PolygonBuilder pb = new PolygonBuilder(new CoordinatesBuilder().coordinate(4, 3).coordinate(3, 2).coordinate(3, 3).close()); pb.hole(new LineStringBuilder(new CoordinatesBuilder().coordinate(4, 2).coordinate(3, 1).coordinate(4, 1).close())); pb.buildS4J(); }); assertEquals("Hole lies outside shell at or near point (4.0, 1.0, NaN)", e.getMessage()); } public void testHoleThatIsNorthOfPolygon() { InvalidShapeException e = expectThrows(InvalidShapeException.class, () -> { PolygonBuilder pb = new PolygonBuilder(new CoordinatesBuilder().coordinate(3, 2).coordinate(4, 1).coordinate(3, 1).close()); pb.hole(new LineStringBuilder(new CoordinatesBuilder().coordinate(3, 3).coordinate(4, 2).coordinate(4, 3).close())); pb.buildS4J(); }); assertEquals("Hole lies outside shell at or near point (4.0, 3.0, NaN)", e.getMessage()); } public void testWidePolygonWithConfusingOrientation() { // A valid polygon that is oriented correctly (anticlockwise) but which // confounds a naive algorithm for determining its orientation leading // ES to believe that it crosses the dateline and "fixing" it in a way // that self-intersects. PolygonBuilder pb = new PolygonBuilder( new CoordinatesBuilder().coordinate(10, -20) .coordinate(100, 0) .coordinate(-100, 0) .coordinate(20, -45) .coordinate(40, -60) .close() ); pb.buildS4J(); // Should not throw an exception } public void testPolygonWithUndefinedOrientationDueToCollinearPoints() { PolygonBuilder pb = new PolygonBuilder( new CoordinatesBuilder().coordinate(0.0, 0.0).coordinate(1.0, 1.0).coordinate(-1.0, -1.0).close() ); InvalidShapeException e = expectThrows(InvalidShapeException.class, pb::buildS4J); assertEquals("Cannot determine orientation: edges adjacent to (-1.0,-1.0) coincide", e.getMessage()); } }