Changeset 11905 in josm for trunk/test/unit/org

Timestamp:

2017-04-14T21:23:29+02:00 (7 years ago)

Author:

Don-vip

Message:

sonar - squid:S2301 - Public methods should not contain selector arguments

File:

• trunk/test/unit/org/openstreetmap/josm/gui/layer/LayerManagerTest.java (modified) (6 diffs)

trunk/test/unit/org/openstreetmap/josm/gui/layer/LayerManagerTest.java

@@ -159 +159 @@
     }
 
+    /** the tested layer manager */
     protected LayerManager layerManager;
 
@@ -397 +398 @@
 
     /**
-     * {@link LayerManager#addLayerChangeListener(LayerChangeListener, boolean)} fires fake add events
+     * {@link LayerManager#addAndFireLayerChangeListener(LayerChangeListener)} fires fake add events
      */
     @Test
@@ -406 +407 @@
         layerManager.addLayer(layer1);
         layerManager.addLayer(layer2);
-        layerManager.addLayerChangeListener(new LayerChangeListener() {
+        layerManager.addAndFireLayerChangeListener(new LayerChangeListener() {
             @Override
             public void layerRemoving(LayerRemoveEvent e) {
@@ -421 +422 @@
                 fired.add(e.getAddedLayer());
             }
-        }, true);
+        });
 
         assertEquals(Arrays.asList(layer1, layer2), fired);
@@ -452 +453 @@
 
     /**
-     * {@link LayerManager#removeLayerChangeListener(LayerChangeListener, boolean)} fires fake remove events
+     * {@link LayerManager#removeAndFireLayerChangeListener(LayerChangeListener)} fires fake remove events
      */
     @Test
@@ -477 +478 @@
             }
         };
-        layerManager.addLayerChangeListener(listener, false);
-        layerManager.removeLayerChangeListener(listener, true);
+        layerManager.addLayerChangeListener(listener);
+        layerManager.removeAndFireLayerChangeListener(listener);
 
         assertEquals(Arrays.asList(layer1, layer2), fired);