Changeset 36272 in osm for applications/editors/josm/plugins/graphview

Timestamp:

2024-06-05T19:04:29+02:00 (7 months ago)

Author:

taylor.smock

Message:

Fix more plugin tests

For reverter, the test needed to be updated for newer WireMock versions.
For graphview, the assert in production code was removed and replaced with
requireNonNull for the parameters that are actually used.

File:

• applications/editors/josm/plugins/graphview/src/org/openstreetmap/josm/plugins/graphview/core/property/RoadIncline.java (modified) (2 diffs)

applications/editors/josm/plugins/graphview/src/org/openstreetmap/josm/plugins/graphview/core/property/RoadIncline.java

@@ -1 +1 @@
 // License: GPL. For details, see LICENSE file.
 package org.openstreetmap.josm.plugins.graphview.core.property;
+
+import java.util.Objects;
 
 import org.openstreetmap.josm.plugins.graphview.core.access.AccessParameters;
@@ -18 +20 @@
     public <N, W, R, M> Float evaluateW(W way, boolean forward, AccessParameters accessParameters,
             DataSource<N, W, R, M> dataSource) {
-        assert way != null && accessParameters != null && dataSource != null;
+        Objects.requireNonNull(way, "way");
+        Objects.requireNonNull(dataSource, "dataSource");
 
         TagGroup tags = dataSource.getTagsW(way);