Ignore:
Timestamp:
2016-01-28T09:29:14+01:00 (8 years ago)
Author:
Don-vip
Message:

fix/cleanup unit tests

Location:
trunk/test/unit/org/openstreetmap/josm/actions/mapmode
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/test/unit/org/openstreetmap/josm/actions/mapmode/DrawActionTest.java

    r8949 r9661  
    6868
    6969        Main.main.addLayer(layer);
    70         assertTrue(Main.map.selectDrawTool(false));
     70        try {
     71            assertTrue(Main.map.selectDrawTool(false));
    7172
    72         Main.map.mapModeDraw.mouseReleased(new MouseEvent(
    73                 Main.map,
    74                 MouseEvent.MOUSE_RELEASED,
    75                 2000,
    76                 InputEvent.BUTTON1_MASK,
    77                 50, 0,
    78                 2, false));
     73            Main.map.mapModeDraw.mouseReleased(new MouseEvent(
     74                    Main.map,
     75                    MouseEvent.MOUSE_RELEASED,
     76                    2000,
     77                    InputEvent.BUTTON1_MASK,
     78                    50, 0,
     79                    2, false));
    7980
    80         JList<OsmPrimitive> lstPrimitives = new JList<>();
    81         OsmPrimitivRenderer renderer = new OsmPrimitivRenderer();
     81            JList<OsmPrimitive> lstPrimitives = new JList<>();
     82            OsmPrimitivRenderer renderer = new OsmPrimitivRenderer();
    8283
    83         assertEquals(3, w.getNodesCount());
    84         Collection<Node> sel = dataSet.getSelectedNodes();
    85         assertEquals(1, sel.size());
     84            assertEquals(3, w.getNodesCount());
     85            Collection<Node> sel = dataSet.getSelectedNodes();
     86            assertEquals(1, sel.size());
    8687
    87         Node n3 = sel.iterator().next();
     88            Node n3 = sel.iterator().next();
    8889
    89         assertNotNull(renderer.getListCellRendererComponent(lstPrimitives, n3, 0, false, false));
     90            assertNotNull(renderer.getListCellRendererComponent(lstPrimitives, n3, 0, false, false));
    9091
    91         Main.main.undoRedo.undo();
     92            Main.main.undoRedo.undo();
    9293
    93         assertEquals(2, w.getNodesCount());
    94         assertTrue(dataSet.getSelectedNodes().isEmpty());
     94            assertEquals(2, w.getNodesCount());
     95            assertTrue(dataSet.getSelectedNodes().isEmpty());
    9596
    96         assertNotNull(renderer.getListCellRendererComponent(lstPrimitives, n3, 0, false, false));
     97            assertNotNull(renderer.getListCellRendererComponent(lstPrimitives, n3, 0, false, false));
     98        } finally {
     99            // Ensure we clean the place before leaving, even if test fails.
     100            Main.main.removeLayer(layer);
     101        }
    97102    }
    98103}

  • trunk/test/unit/org/openstreetmap/josm/actions/mapmode/SelectActionTest.java

    r9205 r9661  
    102102        Main.pref = new PreferencesMock();
    103103        Main.main.addLayer(layer);
    104         SelectAction action = new SelectActionMock(Main.map, dataSet, layer);
    105         nodesMerged = false;
     104        try {
     105            SelectAction action = new SelectActionMock(Main.map, dataSet, layer);
     106            nodesMerged = false;
    106107
    107         action.setEnabled(true);
    108         action.putValue("active", true);
     108            action.setEnabled(true);
     109            action.putValue("active", true);
    109110
    110         MouseEvent event;
    111         event = new MouseEvent(Main.map,
    112                                MouseEvent.MOUSE_PRESSED,
    113                                0,
    114                                InputEvent.BUTTON1_MASK | InputEvent.CTRL_MASK,
    115                                100, 0,
    116                                1,
    117                                false);
    118         action.mousePressed(event);
    119         event = new MouseEvent(Main.map,
    120                                MouseEvent.MOUSE_DRAGGED,
    121                                1000,
    122                                InputEvent.BUTTON1_MASK | InputEvent.CTRL_MASK,
    123                                50, 0,
    124                                1,
    125                                false);
    126         action.mouseDragged(event);
    127         event = new MouseEvent(Main.map,
    128                                MouseEvent.MOUSE_RELEASED,
    129                                2000,
    130                                InputEvent.BUTTON1_MASK | InputEvent.CTRL_MASK,
    131                                5, 0,
    132                                1,
    133                                false);
    134         action.mouseReleased(event);
     111            MouseEvent event;
     112            event = new MouseEvent(Main.map,
     113                                   MouseEvent.MOUSE_PRESSED,
     114                                   0,
     115                                   InputEvent.BUTTON1_MASK | InputEvent.CTRL_MASK,
     116                                   100, 0,
     117                                   1,
     118                                   false);
     119            action.mousePressed(event);
     120            event = new MouseEvent(Main.map,
     121                                   MouseEvent.MOUSE_DRAGGED,
     122                                   1000,
     123                                   InputEvent.BUTTON1_MASK | InputEvent.CTRL_MASK,
     124                                   50, 0,
     125                                   1,
     126                                   false);
     127            action.mouseDragged(event);
     128            event = new MouseEvent(Main.map,
     129                                   MouseEvent.MOUSE_RELEASED,
     130                                   2000,
     131                                   InputEvent.BUTTON1_MASK | InputEvent.CTRL_MASK,
     132                                   5, 0,
     133                                   1,
     134                                   false);
     135            action.mouseReleased(event);
    135136
    136         // As result of test, we must find a 2 nodes way, from EN(0, 0) to EN(100, 0)
    137         assertTrue("Nodes are not merged", nodesMerged);
    138         assertSame(String.format("Expect exactly one way, found %d\n", dataSet.getWays().size()),
    139                    dataSet.getWays().size(), 1);
    140         Way rw = dataSet.getWays().iterator().next();
    141         assertFalse("Way shouldn't be deleted\n", rw.isDeleted());
    142         assertSame(String.format("Way shouldn't have 2 nodes, %d found\n", w.getNodesCount()),
    143                    rw.getNodesCount(), 2);
    144         Node r1 = rw.firstNode();
    145         Node r2 = rw.lastNode();
    146         if (r1.getEastNorth().east() > r2.getEastNorth().east()) {
    147             Node tmp = r1;
    148             r1 = r2;
    149             r2 = tmp;
     137            // As result of test, we must find a 2 nodes way, from EN(0, 0) to EN(100, 0)
     138            assertTrue("Nodes are not merged", nodesMerged);
     139            assertSame(String.format("Expect exactly one way, found %d\n", dataSet.getWays().size()),
     140                       dataSet.getWays().size(), 1);
     141            Way rw = dataSet.getWays().iterator().next();
     142            assertFalse("Way shouldn't be deleted\n", rw.isDeleted());
     143            assertSame(String.format("Way shouldn't have 2 nodes, %d found\n", w.getNodesCount()),
     144                       rw.getNodesCount(), 2);
     145            Node r1 = rw.firstNode();
     146            Node r2 = rw.lastNode();
     147            if (r1.getEastNorth().east() > r2.getEastNorth().east()) {
     148                Node tmp = r1;
     149                r1 = r2;
     150                r2 = tmp;
     151            }
     152            assertSame(String.format("East should be 0, found %f\n", r1.getEastNorth().east()),
     153                       Double.compare(r1.getEastNorth().east(), 0), 0);
     154            assertSame(String.format("East should be 100, found %f\n", r2.getEastNorth().east()),
     155                       Double.compare(r2.getEastNorth().east(), 100), 0);
     156        } finally {
     157            // Ensure we clean the place before leaving, even if test fails.
     158            Main.main.removeLayer(layer);
    150159        }
    151         assertSame(String.format("East should be 0, found %f\n", r1.getEastNorth().east()),
    152                    Double.compare(r1.getEastNorth().east(), 0), 0);
    153         assertSame(String.format("East should be 100, found %f\n", r2.getEastNorth().east()),
    154                    Double.compare(r2.getEastNorth().east(), 100), 0);
    155160    }
    156161}
Note: See TracChangeset for help on using the changeset viewer.