Changeset 16802 in josm


Ignore:
Timestamp:
2020-07-22T09:21:48+02:00 (2 months ago)
Author:
GerdP
Message:

fix #19568: Don't complain about highway ending near other highway for railway platforms

  • treat highway node that is connected to a railway=platform like one connected to a building.
Location:
trunk
Files:
2 added
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/validation/tests/UnconnectedWays.java

    r16553 r16802  
    102102                    || n.isKeyTrue("noexit")
    103103                    || n.hasKey("entrance", "barrier")
    104                     || n.getParentWays().stream().anyMatch(Test::isBuilding);
     104                    || n.getParentWays().stream().anyMatch(p -> isBuilding(p) || p.hasTag(RAILWAY, "platform"));
    105105        }
    106106    }
  • trunk/test/unit/org/openstreetmap/josm/data/validation/tests/UnconnectedWaysTest.java

    r16618 r16802  
    119119        }
    120120    }
     121
     122    /**
     123     * Non-regression test for <a href="https://josm.openstreetmap.de/ticket/19568">Bug #19568</a>.
     124     * @throws IOException if any I/O error occurs
     125     * @throws IllegalDataException if the OSM data cannot be parsed
     126     * @throws FileNotFoundException if the data file cannot be found
     127     */
     128    @Test
     129    public void testTicket19568() throws IOException, IllegalDataException, FileNotFoundException {
     130        try (InputStream fis = TestUtils.getRegressionDataStream(19568, "data.osm")) {
     131            final DataSet ds = OsmReader.parseDataSet(fis, NullProgressMonitor.INSTANCE);
     132            MainApplication.getLayerManager().addLayer(new OsmDataLayer(ds, null, null));
     133
     134            bib.startTest(null);
     135            bib.setBeforeUpload(false);
     136            bib.visit(ds.allPrimitives());
     137            bib.endTest();
     138            assertThat(bib.getErrors(), isEmpty());
     139        }
     140    }
    121141}
Note: See TracChangeset for help on using the changeset viewer.