Changeset 12444 in josm for trunk/test

Timestamp:

2017-07-01T12:51:44+02:00 (7 years ago)

Author:

Don-vip

Message:

refactor WMS test to integration test as it relies on external WMS server

Location:

trunk/test/unit/org/openstreetmap/josm/actions

Files:

• AddImageryLayerActionTest.java (modified) (3 diffs)
• AddImageryLayerActionTestIT.java (added)

trunk/test/unit/org/openstreetmap/josm/actions/AddImageryLayerActionTest.java

@@ -12 +12 @@
 import org.openstreetmap.josm.data.imagery.ImageryInfo;
 import org.openstreetmap.josm.gui.layer.TMSLayer;
-import org.openstreetmap.josm.gui.layer.WMSLayer;
 import org.openstreetmap.josm.testutils.JOSMTestRules;
 
@@ -43 +42 @@
 
     /**
-     * Unit test of {@link AddImageryLayerAction#actionPerformed} - Enabled cases.
+     * Unit test of {@link AddImageryLayerAction#actionPerformed} - Enabled cases for TMS.
      */
     @Test
@@ -51 +50 @@
         List<TMSLayer> tmsLayers = Main.getLayerManager().getLayersOfType(TMSLayer.class);
         assertEquals(1, tmsLayers.size());
-
-        try {
-            new AddImageryLayerAction(new ImageryInfo("wms.openstreetmap.fr", "http://wms.openstreetmap.fr/wms?",
-                    "wms_endpoint", null, null)).actionPerformed(null);
-            List<WMSLayer> wmsLayers = Main.getLayerManager().getLayersOfType(WMSLayer.class);
-            assertEquals(1, wmsLayers.size());
-
-            Main.getLayerManager().removeLayer(wmsLayers.get(0));
-        } finally {
-            Main.getLayerManager().removeLayer(tmsLayers.get(0));
-        }
+        Main.getLayerManager().removeLayer(tmsLayers.get(0));
     }