Changeset 8510 in josm for trunk/test


Ignore:
Timestamp:
2015-06-20T23:42:21+02:00 (4 years ago)
Author:
Don-vip
Message:

checkstyle: enable relevant whitespace checks and fix them

Location:
trunk/test
Files:
50 edited

Legend:

Unmodified
Added
Removed
  • trunk/test/functional/org/openstreetmap/josm/gui/conflict/pair/nodes/NodeListMergerTest.java

    r7937 r8510  
    2222        Node n1;
    2323        w1.addNode(n1 = new Node(1));
    24         for (int i=0; i < 20; i++) {
     24        for (int i = 0; i < 20; i++) {
    2525            n1.put("key" + i, "value" + i);
    2626        }
    2727        StringBuilder note = new StringBuilder();
    28         for (int i=0; i < 50; i++) {
     28        for (int i = 0; i < 50; i++) {
    2929            note.append(" A very long text ");
    3030        }
     
    4949
    5050        Way w2 = new Way();
    51         for (int i = 1; i < 200; i+=2) {
     51        for (int i = 1; i < 200; i += 2) {
    5252            w2.addNode(new Node(i));
    5353        }
     
    7272    static public void main(String args[]) {
    7373        NodeListMergerTest test = new NodeListMergerTest();
    74         test.setSize(600,600);
     74        test.setSize(600, 600);
    7575        test.setVisible(true);
    7676    }
  • trunk/test/functional/org/openstreetmap/josm/gui/conflict/pair/properties/PropertiesMergerTest.java

    r7937 r8510  
    2828    protected void populate() {
    2929        Node my = new Node(1);
    30         my.setCoor(new LatLon(1,1));
     30        my.setCoor(new LatLon(1, 1));
    3131        my.setDeleted(true);
    3232
    3333        Node their = new Node(2);
    34         their.setCoor(new LatLon(10,10));
     34        their.setCoor(new LatLon(10, 10));
    3535
    3636        merger.getModel().populate(new Conflict<OsmPrimitive>(my, their));
  • trunk/test/functional/org/openstreetmap/josm/gui/conflict/pair/relation/RelationMemberMergerTest.java

    r7937 r8510  
    5151    static public void main(String args[]) {
    5252        RelationMemberMergerTest test = new RelationMemberMergerTest();
    53         test.setSize(600,600);
     53        test.setSize(600, 600);
    5454        test.setVisible(true);
    5555    }
  • trunk/test/functional/org/openstreetmap/josm/gui/conflict/pair/tags/TagMergerTest.java

    r8509 r8510  
    2929        tagMerger.getModel().addItem(new TagMergeItem("a very long key asdfasdf asdfasdf", "a very long value asdfasdf",
    3030                "a very long value asdfasdf"));
    31         for (int i=0; i< 50; i++) {
     31        for (int i = 0; i < 50; i++) {
    3232          tagMerger.getModel().addItem(new TagMergeItem("key", "myvalue", "theirvalue"));
    3333        }
     
    3636    public static void main(String args[]) {
    3737        TagMergerTest test  = new TagMergerTest();
    38         test.setSize(600,600);
     38        test.setSize(600, 600);
    3939        test.setVisible(true);
    4040    }
  • trunk/test/functional/org/openstreetmap/josm/gui/dialogs/ConflictResolutionDialogTest.java

    r7937 r8510  
    1616
    1717    protected void build() {
    18         setSize(100,100);
     18        setSize(100, 100);
    1919        dialog = new ConflictResolutionDialog(this);
    20         dialog.setSize(600,600);
     20        dialog.setSize(600, 600);
    2121    }
    2222
  • trunk/test/functional/org/openstreetmap/josm/gui/history/HistoryBrowserTest.java

    r7937 r8510  
    2929
    3030    protected void build() {
    31         setSize(500,500);
     31        setSize(500, 500);
    3232        getContentPane().setLayout(new BorderLayout());
    3333        browser = new HistoryBrowser();
     
    4040        try {
    4141            ds = reader.parseHistory(NullProgressMonitor.INSTANCE);
    42         } catch(OsmTransferException e) {
     42        } catch (OsmTransferException e) {
    4343            Main.error(e);
    4444            return;
     
    5151     * Constructs a new {@code HistoryBrowserTest}.
    5252     */
    53     public HistoryBrowserTest(){
     53    public HistoryBrowserTest() {
    5454        build();
    5555        //populate(OsmPrimitiveType.NODE,354117);
    5656        //populate(OsmPrimitiveType.WAY,37951);
    57         populate(OsmPrimitiveType.RELATION,5055);
     57        populate(OsmPrimitiveType.RELATION, 5055);
    5858
    5959    }
  • trunk/test/functional/org/openstreetmap/josm/io/MultiFetchServerObjectReaderTest.java

    r8509 r8510  
    5757        // create a set of nodes
    5858        //
    59         for (int i=0; i< numNodes; i++) {
     59        for (int i = 0; i < numNodes; i++) {
    6060            Node n = new Node();
    61             n.setCoor(new LatLon(-36.6,47.6));
     61            n.setCoor(new LatLon(-36.6, 47.6));
    6262            n.put("name", "node-"+i);
    6363            ds.addPrimitive(n);
     
    6565        }
    6666
    67         // create a set of ways, each with a random number of
    68         // nodes
     67        // create a set of ways, each with a random number of nodes
    6968        //
    70         for (int i=0; i< numWays; i++) {
     69        for (int i = 0; i < numWays; i++) {
    7170            Way w = new Way();
    72             int numNodesInWay = 2 + (int)Math.round(Math.random() * 5);
    73             int start = (int)Math.round(Math.random() * numNodes);
    74             for (int j = 0; j < numNodesInWay;j++) {
     71            int numNodesInWay = 2 + (int) Math.round(Math.random() * 5);
     72            int start = (int) Math.round(Math.random() * numNodes);
     73            for (int j = 0; j < numNodesInWay; j++) {
    7574                int idx = (start + j) % numNodes;
    7675                Node n = nodes.get(idx);
     
    8281        }
    8382
    84         // create a set of relations each with a random number of nodes,
    85         // and ways
     83        // create a set of relations each with a random number of nodes, and ways
    8684        //
    87         for (int i=0; i< numRelations; i++) {
     85        for (int i = 0; i < numRelations; i++) {
    8886            Relation r = new Relation();
    8987            r.put("name", "relation-" +i);
    90             int numNodesInRelation = (int)Math.round(Math.random() * 10);
    91             int start = (int)Math.round(Math.random() * numNodes);
    92             for (int j = 0; j < numNodesInRelation;j++) {
     88            int numNodesInRelation = (int) Math.round(Math.random() * 10);
     89            int start = (int) Math.round(Math.random() * numNodes);
     90            for (int j = 0; j < numNodesInRelation; j++) {
    9391                int idx = (start + j) % 500;
    9492                Node n = nodes.get(idx);
    9593                r.addMember(new RelationMember("role-" + j, n));
    9694            }
    97             int numWaysInRelation = (int)Math.round(Math.random() * 10);
    98             start = (int)Math.round(Math.random() * numWays);
    99             for (int j = 0; j < numWaysInRelation;j++) {
     95            int numWaysInRelation = (int) Math.round(Math.random() * 10);
     96            start = (int) Math.round(Math.random() * numWays);
     97            for (int j = 0; j < numWaysInRelation; j++) {
    10098                int idx = (start + j) % 500;
    10199                Way w = ways.get(idx);
     
    167165                w.footer();
    168166            }
    169         } catch(IOException e) {
     167        } catch (IOException e) {
    170168            fail(MessageFormat.format("failed to open file ''{0}'' for writing", dataSetCacheOutputFile.toString()));
    171169        }
     
    191189        MultiFetchServerObjectReader reader = new MultiFetchServerObjectReader();
    192190        ArrayList<Node> nodes = new ArrayList<>(ds.getNodes());
    193         for (int i =0; i< 10; i++) {
     191        for (int i = 0; i < 10; i++) {
    194192            reader.append(nodes.get(i));
    195193        }
     
    197195        assertEquals(10, out.getNodes().size());
    198196        for (Node n1:out.getNodes()) {
    199             Node n2 = (Node)ds.getPrimitiveById(n1);
     197            Node n2 = (Node) ds.getPrimitiveById(n1);
    200198            assertNotNull(n2);
    201             assertEquals(n2.get("name"),n2.get("name"));
     199            assertEquals(n2.get("name"), n2.get("name"));
    202200        }
    203201        assertTrue(reader.getMissingPrimitives().isEmpty());
     
    207205    public void testMultiGet10Ways() throws OsmTransferException {
    208206        MultiFetchServerObjectReader reader = new MultiFetchServerObjectReader();
    209         ArrayList<Way> ways= new ArrayList<>(ds.getWays());
    210         for (int i =0; i< 10; i++) {
     207        ArrayList<Way> ways = new ArrayList<>(ds.getWays());
     208        for (int i = 0; i < 10; i++) {
    211209            reader.append(ways.get(i));
    212210        }
     
    214212        assertEquals(10, out.getWays().size());
    215213        for (Way w1: out.getWays()) {
    216             Way w2 = (Way)ds.getPrimitiveById(w1);
     214            Way w2 = (Way) ds.getPrimitiveById(w1);
    217215            assertNotNull(w2);
    218216            assertEquals(w2.getNodesCount(), w1.getNodesCount());
    219             assertEquals(w2.get("name"),w1.get("name"));
     217            assertEquals(w2.get("name"), w1.get("name"));
    220218        }
    221219        assertTrue(reader.getMissingPrimitives().isEmpty());
     
    225223    public void testMultiGet10Relations() throws OsmTransferException {
    226224        MultiFetchServerObjectReader reader = new MultiFetchServerObjectReader();
    227         ArrayList<Relation> relations= new ArrayList<>(ds.getRelations());
    228         for (int i =0; i< 10; i++) {
     225        ArrayList<Relation> relations = new ArrayList<>(ds.getRelations());
     226        for (int i = 0; i < 10; i++) {
    229227            reader.append(relations.get(i));
    230228        }
     
    232230        assertEquals(10, out.getRelations().size());
    233231        for (Relation r1: out.getRelations()) {
    234             Relation r2 = (Relation)ds.getPrimitiveById(r1);
     232            Relation r2 = (Relation) ds.getPrimitiveById(r1);
    235233            assertNotNull(r2);
    236234            assertEquals(r2.getMembersCount(), r1.getMembersCount());
    237             assertEquals(r2.get("name"),r2.get("name"));
     235            assertEquals(r2.get("name"), r2.get("name"));
    238236        }
    239237        assertTrue(reader.getMissingPrimitives().isEmpty());
     
    244242        MultiFetchServerObjectReader reader = new MultiFetchServerObjectReader();
    245243        ArrayList<Node> nodes = new ArrayList<>(ds.getNodes());
    246         for (int i =0; i< 812; i++) {
     244        for (int i = 0; i < 812; i++) {
    247245            reader.append(nodes.get(i));
    248246        }
     
    250248        assertEquals(812, out.getNodes().size());
    251249        for (Node n1:out.getNodes()) {
    252             Node n2 = (Node)ds.getPrimitiveById(n1);
     250            Node n2 = (Node) ds.getPrimitiveById(n1);
    253251            assertNotNull(n2);
    254             assertEquals(n2.get("name"),n2.get("name"));
     252            assertEquals(n2.get("name"), n2.get("name"));
    255253        }
    256254        assertTrue(reader.getMissingPrimitives().isEmpty());
     
    261259        MultiFetchServerObjectReader reader = new MultiFetchServerObjectReader();
    262260        ArrayList<Node> nodes = new ArrayList<>(ds.getNodes());
    263         for (int i =0; i< 10; i++) {
     261        for (int i = 0; i < 10; i++) {
    264262            reader.append(nodes.get(i));
    265263        }
     
    269267        assertEquals(10, out.getNodes().size());
    270268        for (Node n1:out.getNodes()) {
    271             Node n2 = (Node)ds.getPrimitiveById(n1);
     269            Node n2 = (Node) ds.getPrimitiveById(n1);
    272270            assertNotNull(n2);
    273             assertEquals(n2.get("name"),n2.get("name"));
     271            assertEquals(n2.get("name"), n2.get("name"));
    274272        }
    275273        assertFalse(reader.getMissingPrimitives().isEmpty());
  • trunk/test/functional/org/openstreetmap/josm/io/OsmServerBackreferenceReaderTest.java

    r8509 r8510  
    7373
    7474    protected static void populateTestDataSetWithNodes(DataSet ds) {
    75         for (int i=0;i<100;i++) {
     75        for (int i = 0; i < 100; i++) {
    7676            Node n = new Node();
    77             n.setCoor(new LatLon(-36.6,47.6));
     77            n.setCoor(new LatLon(-36.6, 47.6));
    7878            n.put("name", "node-"+i);
    7979            ds.addPrimitive(n);
     
    8282
    8383    protected static void populateTestDataSetWithWays(DataSet ds) {
    84         for (int i=0;i<20;i++) {
     84        for (int i = 0; i < 20; i++) {
    8585            Way w = new Way();
    86             for (int j = 0; j < 10;j++) {
     86            for (int j = 0; j < 10; j++) {
    8787                w.addNode(lookupNode(ds, i+j));
    8888            }
     
    9393
    9494    protected static void populateTestDataSetWithRelations(DataSet ds) {
    95         for (int i=0;i<10;i++) {
     95        for (int i = 0; i < 10; i++) {
    9696            Relation r = new Relation();
    9797            r.put("name", "relation-" +i);
    98             for (int j =0; j < 10; j++) {
     98            for (int j = 0; j < 10; j++) {
    9999                RelationMember member = new RelationMember("node-" + j, lookupNode(ds, i + j));
    100100                r.addMember(member);
    101101            }
    102             for (int j =0; j < 5; j++) {
     102            for (int j = 0; j < 5; j++) {
    103103                RelationMember member = new RelationMember("way-" + j, lookupWay(ds, i + j));
    104104                r.addMember(member);
    105105            }
    106106            if (i > 5) {
    107                 for (int j =0; j < 3; j++) {
     107                for (int j = 0; j < 3; j++) {
    108108                    RelationMember member = new RelationMember("relation-" + j, lookupRelation(ds, j));
    109109                    logger.info(MessageFormat.format("adding relation {0} to relation {1}", j, i));
     
    185185                w.footer();
    186186            }
    187         } catch(IOException e) {
     187        } catch (IOException e) {
    188188            fail(MessageFormat.format("failed to open file ''{0}'' for writing", dataSetCacheOutputFile.toString()));
    189189        }
     
    456456    protected static Set<Long> getNodeIdsInWay(Way way) {
    457457        HashSet<Long> ret = new HashSet<>();
    458         if (way == null)return ret;
     458        if (way == null) return ret;
    459459        for (Node n: way.getNodes()) {
    460460            ret.add(n.getId());
  • trunk/test/functional/org/openstreetmap/josm/io/UploadStrategySelectionPanelTest.java

    r7937 r8510  
    2727        getContentPane().add(uploadStrategySelectionPanel, BorderLayout.CENTER);
    2828        getContentPane().add(buildControlPanel(), BorderLayout.SOUTH);
    29         setSize(400,400);
     29        setSize(400, 400);
    3030    }
    31 
    3231
    3332    protected JPanel buildControlPanel() {
     
    4241                        try {
    4342                            n = Integer.parseInt(tf.getText());
    44                         } catch(NumberFormatException e) {
     43                        } catch (NumberFormatException e) {
    4544                            Main.error(e);
    4645                            return;
     
    6160    }
    6261
    63     public static void main(String args[]) throws OsmApiInitializationException, OsmTransferCanceledException{
     62    public static void main(String args[]) throws OsmApiInitializationException, OsmTransferCanceledException {
    6463        OsmApi.getOsmApi().initialize(NullProgressMonitor.INSTANCE);
    6564        new UploadStrategySelectionPanelTest().setVisible(true);
  • trunk/test/performance/org/openstreetmap/josm/data/osm/visitor/paint/StyledMapRendererPerformanceTest.java

    r7937 r8510  
    3535        JOSMFixture.createPerformanceTestFixture().init();
    3636        img = new BufferedImage(IMG_WIDTH, IMG_HEIGHT, BufferedImage.TYPE_3BYTE_BGR);
    37         g = (Graphics2D)img.getGraphics();
     37        g = (Graphics2D) img.getGraphics();
    3838        nc = new NavigatableComponent();
    3939        nc.setBounds(0, 0, IMG_WIDTH, IMG_HEIGHT);
     
    5454
    5555    private static void test(int iterations, DataSet ds, Bounds bounds) throws Exception {
    56         Rendering visitor = new StyledMapRenderer(g,nc,false);
     56        Rendering visitor = new StyledMapRenderer(g, nc, false);
    5757        nc.zoomTo(bounds);
    58         for (int i=0; i<iterations; i++) {
     58        for (int i = 0; i < iterations; i++) {
    5959            visitor.render(ds, true, bounds);
    6060        }
  • trunk/test/unit/org/openstreetmap/josm/JOSMFixture.java

    r8509 r8510  
    7272        } else {
    7373            File f = new File(josmHome);
    74             if (! f.exists() || ! f.canRead()) {
     74            if (!f.exists() || !f.canRead()) {
    7575                fail(MessageFormat.format(
    7676                        "property ''{0}'' points to ''{1}'' which is either not existing ({2}) or not readable ({3}). Current directory is ''{4}''.",
  • trunk/test/unit/org/openstreetmap/josm/TestUtils.java

    r8257 r8510  
    5252        System.out.println("Validating Comparable contract on array of "+array.length+" elements");
    5353        // Check each compare possibility
    54         for (int i=0; i<array.length; i++) {
     54        for (int i = 0; i < array.length; i++) {
    5555            T r1 = array[i];
    56             for (int j=i; j<array.length; j++) {
     56            for (int j = i; j < array.length; j++) {
    5757                T r2 = array[j];
    5858                int a = comparator.compare(r1, r2);
    5959                int b = comparator.compare(r2, r1);
    60                 if (i==j || a==b) {
     60                if (i == j || a == b) {
    6161                    if (a != 0 || b != 0) {
    6262                        fail(getFailMessage(r1, r2, a, b));
     
    6767                    }
    6868                }
    69                 for (int k=j; k<array.length; k++) {
     69                for (int k = j; k < array.length; k++) {
    7070                    T r3 = array[k];
    7171                    int c = comparator.compare(r1, r3);
  • trunk/test/unit/org/openstreetmap/josm/actions/CreateCircleActionTest.java

    r8450 r8510  
    113113    private static class ConstantTrafficHand implements GeoProperty<Boolean> {
    114114        boolean isLeft;
     115
    115116        ConstantTrafficHand(boolean isLeft) {
    116117            this.isLeft = isLeft;
    117118        }
     119
    118120        @Override
    119121        public Boolean get(LatLon ll) {
    120122            return isLeft;
    121123        }
     124
    122125        @Override
    123126        public Boolean get(BBox box) {
  • trunk/test/unit/org/openstreetmap/josm/actions/SplitWayActionTest.java

    r8450 r8510  
    8989        assertSame("Unselected way seems to have change during split action.",
    9090                   w1.getNodesCount(), 3);
    91         for(int i = 0; i < 3; i++) {
     91        for (int i = 0; i < 3; i++) {
    9292            assertSame("Node change in unselected way during split action.",
    9393                       w1.getNode(i), w1NodesArray[i]);
  • trunk/test/unit/org/openstreetmap/josm/actions/mapmode/SelectActionTest.java

    r8509 r8510  
    4040        public OsmDataLayer layer;
    4141        public DataSet currentDataSet;
     42
     43        public Predicate<OsmPrimitive> isSelectablePredicate =
     44                new Predicate<OsmPrimitive>() {
     45                    @Override
     46                    public boolean evaluate(OsmPrimitive prim) {
     47                        return true;
     48                    }
     49                };
     50
    4251        MapViewMock(DataSet dataSet, OsmDataLayer layer) {
    4352            super(null, null);
    4453            this.layer = layer;
    4554            this.currentDataSet = dataSet;
    46         };
     55        }
     56
    4757        @Override
    4858        public EastNorth getEastNorth(int x, int y) {
    4959            return new EastNorth(x, y);
    5060        }
     61
    5162        @Override
    5263        public void addMouseListener(MouseListener ml) {}
     64
    5365        @Override
    5466        public void removeMouseListener(MouseListener ml) {}
     67
    5568        public void addMouseMotionListener(MouseListener ml) {}
     69
    5670        public void removeMouseMotionListener(MouseListener ml) {}
     71
    5772        public void mvRepaint() {}
     73
    5874        @Override
    5975        public void setVirtualNodesEnabled(boolean enabled) {}
     76
    6077        @Override
    6178        public void setNewCursor(Cursor cursor, Object reference) {}
     79
    6280        @Override
    6381        public void setNewCursor(int cursor, Object reference) {}
     82
    6483        @Override
    6584        public boolean isActiveLayerVisible() {
    6685            return true;
    6786        }
    68         public Predicate<OsmPrimitive> isSelectablePredicate =
    69             new Predicate<OsmPrimitive>() {
    70                 @Override
    71                 public boolean evaluate(OsmPrimitive prim) {
    72                     return true;
    73                 }
    74             };
     87
    7588        @Override
    7689        public void requestClearRect() {}
     90
    7791        @Override
    7892        public Point2D getPoint2D(EastNorth p) {
    7993            return new Point2D.Double(p.getX(), p.getY());
    8094        }
     95
    8196        @Override
    8297        public void setActiveLayer(Layer layer) {}
     98
    8399        @Override
    84100        public Layer getActiveLayer() {
    85101            return layer;
    86102        }
     103
    87104        @Override
    88105        protected DataSet getCurrentDataSet() {
     
    119136            }
    120137        }
     138
    121139        @Override
    122140        public void mergeNodes(OsmDataLayer layer, Collection<Node> nodes,
  • trunk/test/unit/org/openstreetmap/josm/data/osm/APIDataSetTest.java

    r8450 r8510  
    3232        try {
    3333            apiDataSet.adjustRelationUploadOrder();
    34         } catch(CyclicUploadDependencyException e) {
     34        } catch (CyclicUploadDependencyException e) {
    3535            fail("unexpected exception:" + e);
    3636        }
     
    5858        try {
    5959            apiDataSet.adjustRelationUploadOrder();
    60         } catch(CyclicUploadDependencyException e) {
     60        } catch (CyclicUploadDependencyException e) {
    6161            fail("unexpected exception:" + e);
    6262        }
     
    9696        try {
    9797            apiDataSet.adjustRelationUploadOrder();
    98         } catch(CyclicUploadDependencyException e) {
     98        } catch (CyclicUploadDependencyException e) {
    9999            fail("unexpected exception:" + e);
    100100        }
     
    134134        try {
    135135            apiDataSet.adjustRelationUploadOrder();
    136         } catch(CyclicUploadDependencyException e) {
     136        } catch (CyclicUploadDependencyException e) {
    137137            fail("unexpected exception:" + e);
    138138        }
     
    181181        try {
    182182            apiDataSet.adjustRelationUploadOrder();
    183         } catch(CyclicUploadDependencyException e) {
     183        } catch (CyclicUploadDependencyException e) {
    184184            fail("unexpected exception:" + e);
    185185        }
     
    256256            apiDataSet.adjustRelationUploadOrder();
    257257            fail("expected cyclic upload dependency exception not thrown");
    258         } catch(CyclicUploadDependencyException e) {
     258        } catch (CyclicUploadDependencyException e) {
    259259            System.out.println(e);
    260260        }
  • trunk/test/unit/org/openstreetmap/josm/data/osm/ChangesetCacheTest.groovy

    r7938 r8510  
    22package org.openstreetmap.josm.data.osm;
    33
    4 import java.net.InetAddress.Cache;
    5 import org.junit.Test
     4import static org.junit.Assert.*
     5
     6import org.junit.Test
     7
     8class ChangesetCacheTest {
     9
     10    @Test
     11    public void test_Constructor() {
     12        ChangesetCache cache = ChangesetCache.getInstance()
     13        assert cache != null
     14    }
     15
     16    @Test
     17    public void test_addAndRemoveListeners() {
     18        ChangesetCache cache = ChangesetCache.getInstance()
     19        cache.clear()
     20
     21        // should work
     22        cache.addChangesetCacheListener null
     23
     24        // should work
     25        def listener = new ChangesetCacheListener() {
     26            public void changesetCacheUpdated(ChangesetCacheEvent event) {}
     27        }
     28        cache.addChangesetCacheListener listener
     29        // adding a second time - should work too
     30        cache.addChangesetCacheListener listener
     31        assert cache.@listeners.size() == 1 // ... but only added once
     32
     33        cache.removeChangesetCacheListener null
     34
     35        cache.removeChangesetCacheListener listener
     36        assert cache.@listeners.size() == 0
     37    }
     38
     39    @Test
     40    public void update_get_remove_cycle() {
     41        ChangesetCache cache = ChangesetCache.getInstance()
     42        cache.clear()
     43
     44        cache.update new Changeset(1)
     45        assert cache.size() == 1
     46        assert cache.get(1) != null
     47        assert cache.get(1).id == 1
     48        cache.remove(1)
     49        assert cache.size() == 0
     50    }
     51
     52    @Test
     53    public void updateTwice() {
     54        ChangesetCache cache = ChangesetCache.getInstance()
     55        cache.clear()
     56
     57        Changeset cs = new Changeset(1)
     58        cs.incomplete = false
     59        cs.put "key1", "value1"
     60        cs.open = true
     61        cache.update cs
     62
     63        Changeset cs2 = new Changeset(cs)
     64        assert cs2 != null
     65        cs2.put "key2", "value2"
     66        cs2.open = false
     67        cache.update(cs2)
     68
     69        assert cache.size() == 1
     70        assert cache.get(1) != null
     71
     72        cs = cache.get(1)
     73        assert cs.get("key1") == "value1"
     74        assert cs.get("key2") == "value2"
     75        assert !cs.open
     76    }
    677
    778
    8 import static org.junit.Assert.*;
     79    @Test
     80    public void contains() {
     81        ChangesetCache cache = ChangesetCache.getInstance()
     82        cache.@listeners.clear()
     83        cache.clear()
    984
    10 class ChangesetCacheTest {
    11        
    12         @Test
    13         public void test_Constructor() {
    14             ChangesetCache cache = ChangesetCache.getInstance()
    15                 assert cache != null
    16         }
    17        
    18         @Test
    19         public void test_addAndRemoveListeners() {
    20                 ChangesetCache cache = ChangesetCache.getInstance()
    21                 cache.clear()
    22                
    23                 // should work
    24                 cache.addChangesetCacheListener null
    25                
    26                 // should work
    27                 def listener = new ChangesetCacheListener() {
    28                         public void changesetCacheUpdated(ChangesetCacheEvent event) {}
    29                 }               
    30                 cache.addChangesetCacheListener listener
    31                 // adding a second time - should work too
    32                 cache.addChangesetCacheListener listener
    33                 assert cache.@listeners.size() == 1 // ... but only added once
    34                
    35                 cache.removeChangesetCacheListener null
    36                
    37                 cache.removeChangesetCacheListener listener
    38                 assert cache.@listeners.size() == 0
    39         }
    40        
    41         @Test
    42         public void update_get_remove_cycle() {
    43                 ChangesetCache cache = ChangesetCache.getInstance()
    44                 cache.clear()
    45                
    46                 cache.update new Changeset(1)
    47                 assert cache.size() == 1
    48                 assert cache.get(1) != null
    49                 assert cache.get(1).id == 1
    50                 cache.remove(1)
    51                 assert cache.size() == 0
    52         }
    53        
    54         @Test
    55         public void updateTwice() {
    56                 ChangesetCache cache = ChangesetCache.getInstance()
    57                 cache.clear()
    5885
    59                 Changeset cs = new Changeset(1)
    60                 cs.incomplete = false
    61                 cs.put "key1", "value1"
    62                 cs.open = true
    63                 cache.update cs
    64                
    65                 Changeset cs2 = new Changeset(cs)
    66                 assert cs2 != null
    67                 cs2.put "key2", "value2"
    68                 cs2.open = false
    69                 cache.update(cs2)
    70                
    71                 assert cache.size() == 1
    72                 assert cache.get(1) != null
    73                
    74                 cs = cache.get(1)
    75                 assert cs.get("key1") == "value1"
    76                 assert cs.get("key2") == "value2"
    77                 assert !cs.open         
    78         }
    79        
    80        
    81         @Test
    82         public void contains() {
    83                 ChangesetCache cache = ChangesetCache.getInstance()
    84                 cache.@listeners.clear()
    85                 cache.clear()
    86                
    87                
    88                 Changeset cs = new Changeset(1)
    89                 cache.update cs
    90                
    91                 assert cache.contains(1)
    92                 assert cache.contains(cs)
    93                 assert cache.contains(new Changeset(cs))
    94                
    95                 assert ! cache.contains(2)
    96                 assert ! cache.contains(new Changeset(2))
    97                 assert ! cache.contains(null)
    98         }
    99        
    100         @Test
    101         public void fireingEvents_AddAChangeset() {
    102                 ChangesetCache cache = ChangesetCache.getInstance()
    103                 cache.clear()
    104                 cache.@listeners.clear()
    105                
    106                 // should work
    107                 def listener = new ChangesetCacheListener() {
    108                                         public void changesetCacheUpdated(ChangesetCacheEvent event) {
    109                                             assert event != null
    110                                                 assert event.getAddedChangesets().size() == 1
    111                                                 assert event.getRemovedChangesets().empty
    112                                                 assert event.getUpdatedChangesets().empty
    113                                                 assert event.getSource() == cache
    114                                         }
    115                 }   
    116                 cache.addChangesetCacheListener listener
    117                 cache.update(new Changeset(1))
    118                 cache.removeChangesetCacheListener listener
    119         }
    120        
    121         @Test
    122         public void fireingEvents_UpdateChangeset() {
    123                 ChangesetCache cache = ChangesetCache.getInstance()
    124                 cache.clear()
    125                 cache.@listeners.clear()
    126                
    127                 // should work
    128                 def listener = new ChangesetCacheListener() {
    129                                         public void changesetCacheUpdated(ChangesetCacheEvent event) {
    130                                                 assert event != null
    131                                                 assert event.getAddedChangesets().empty
    132                                                 assert event.getRemovedChangesets().empty
    133                                                 assert event.getUpdatedChangesets().size() == 1
    134                                                 assert event.getSource() == cache
    135                                         }
    136                                 }   
    137                 cache.update(new Changeset(1))
    138                
    139                 cache.addChangesetCacheListener listener
    140                 cache.update(new Changeset(1))
    141                 cache.removeChangesetCacheListener listener
    142         }
    143        
    144         @Test
    145         public void fireingEvents_RemoveChangeset() {
    146                 ChangesetCache cache = ChangesetCache.getInstance()
    147                 cache.clear()
    148                 cache.@listeners.clear()
    149                
    150                 // should work
    151                 def listener = new ChangesetCacheListener() {
    152                                         public void changesetCacheUpdated(ChangesetCacheEvent event) {
    153                                                 assert event != null
    154                                                 assert event.getAddedChangesets().empty
    155                                                 assert event.getRemovedChangesets().size() == 1
    156                                                 assert event.getUpdatedChangesets().empty
    157                                                 assert event.getSource() == cache
    158                                         }
    159                                 }   
    160                 cache.update(new Changeset(1))
    161                
    162                 cache.addChangesetCacheListener listener
    163                 cache.remove 1
    164                 cache.removeChangesetCacheListener listener
    165         }
    166        
     86        Changeset cs = new Changeset(1)
     87        cache.update cs
     88
     89        assert cache.contains(1)
     90        assert cache.contains(cs)
     91        assert cache.contains(new Changeset(cs))
     92
     93        assert ! cache.contains(2)
     94        assert ! cache.contains(new Changeset(2))
     95        assert ! cache.contains(null)
     96    }
     97
     98    @Test
     99    public void fireingEvents_AddAChangeset() {
     100        ChangesetCache cache = ChangesetCache.getInstance()
     101        cache.clear()
     102        cache.@listeners.clear()
     103
     104        // should work
     105        def listener = new ChangesetCacheListener() {
     106                    public void changesetCacheUpdated(ChangesetCacheEvent event) {
     107                        assert event != null
     108                        assert event.getAddedChangesets().size() == 1
     109                        assert event.getRemovedChangesets().empty
     110                        assert event.getUpdatedChangesets().empty
     111                        assert event.getSource() == cache
     112                    }
     113        }
     114        cache.addChangesetCacheListener listener
     115        cache.update(new Changeset(1))
     116        cache.removeChangesetCacheListener listener
     117    }
     118
     119    @Test
     120    public void fireingEvents_UpdateChangeset() {
     121        ChangesetCache cache = ChangesetCache.getInstance()
     122        cache.clear()
     123        cache.@listeners.clear()
     124
     125        // should work
     126        def listener = new ChangesetCacheListener() {
     127                    public void changesetCacheUpdated(ChangesetCacheEvent event) {
     128                        assert event != null
     129                        assert event.getAddedChangesets().empty
     130                        assert event.getRemovedChangesets().empty
     131                        assert event.getUpdatedChangesets().size() == 1
     132                        assert event.getSource() == cache
     133                    }
     134                }
     135        cache.update(new Changeset(1))
     136
     137        cache.addChangesetCacheListener listener
     138        cache.update(new Changeset(1))
     139        cache.removeChangesetCacheListener listener
     140    }
     141
     142    @Test
     143    public void fireingEvents_RemoveChangeset() {
     144        ChangesetCache cache = ChangesetCache.getInstance()
     145        cache.clear()
     146        cache.@listeners.clear()
     147
     148        // should work
     149        def listener = new ChangesetCacheListener() {
     150                    public void changesetCacheUpdated(ChangesetCacheEvent event) {
     151                        assert event != null
     152                        assert event.getAddedChangesets().empty
     153                        assert event.getRemovedChangesets().size() == 1
     154                        assert event.getUpdatedChangesets().empty
     155                        assert event.getSource() == cache
     156                    }
     157                }
     158        cache.update(new Changeset(1))
     159
     160        cache.addChangesetCacheListener listener
     161        cache.remove 1
     162        cache.removeChangesetCacheListener listener
     163    }
    167164}
  • trunk/test/unit/org/openstreetmap/josm/data/osm/DataSetMergerTest.java

    r8450 r8510  
    7575    @Test
    7676    public void nodeSimple_IdenticalNoConflict() {
    77         Node n = new Node(new LatLon(0,0));
    78         n.setOsmId(1,1);
     77        Node n = new Node(new LatLon(0, 0));
     78        n.setOsmId(1, 1);
    7979        n.setModified(false);
    8080        n.put("key1", "value1");
    8181        my.addPrimitive(n);
    8282
    83         Node n1 = new Node(new LatLon(0,0));
    84         n1.setOsmId(1,1);
     83        Node n1 = new Node(new LatLon(0, 0));
     84        n1.setOsmId(1, 1);
    8585        n1.setModified(false);
    8686        n1.put("key1", "value1");
     
    8888
    8989
    90         DataSetMerger visitor = new DataSetMerger(my,their);
    91         visitor.merge();
    92 
    93         Node n2 = (Node)my.getPrimitiveById(1, OsmPrimitiveType.NODE);
     90        DataSetMerger visitor = new DataSetMerger(my, their);
     91        visitor.merge();
     92
     93        Node n2 = (Node) my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    9494        assertTrue(visitor.getConflicts().isEmpty());
    9595        assertNotSame(n1, n2); // make sure we have a clone
     
    110110    @Test
    111111    public void nodeSimple_locallyUnmodifiedNoConflict() {
    112         Node n = new Node(new LatLon(0,0));
    113         n.setOsmId(1,1);
     112        Node n = new Node(new LatLon(0, 0));
     113        n.setOsmId(1, 1);
    114114        n.setModified(false);
    115115        n.put("key1", "value1");
    116116        my.addPrimitive(n);
    117117
    118         Node n1 = new Node(new LatLon(0,0));
    119         n1.setOsmId(1,2);
     118        Node n1 = new Node(new LatLon(0, 0));
     119        n1.setOsmId(1, 2);
    120120        n1.setModified(false);
    121121        n1.put("key1", "value1-new");
     
    124124
    125125
    126         DataSetMerger visitor = new DataSetMerger(my,their);
    127         visitor.merge();
    128 
    129         Node n2 = (Node)my.getPrimitiveById(1, OsmPrimitiveType.NODE);
     126        DataSetMerger visitor = new DataSetMerger(my, their);
     127        visitor.merge();
     128
     129        Node n2 = (Node) my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    130130        assertTrue(visitor.getConflicts().isEmpty());
    131131        assertSame(n, n2); // make sure the merged node is still the original node
     
    150150    @Test
    151151    public void nodeSimple_TagConflict() {
    152         Node n = new Node(new LatLon(0,0));
    153         n.setOsmId(1,1);
     152        Node n = new Node(new LatLon(0, 0));
     153        n.setOsmId(1, 1);
    154154        n.setModified(true);
    155155        n.put("key1", "value1");
     
    157157        my.addPrimitive(n);
    158158
    159         Node n1 = new Node(new LatLon(0,0));
    160         n1.setOsmId(1,2);
     159        Node n1 = new Node(new LatLon(0, 0));
     160        n1.setOsmId(1, 2);
    161161        n1.setModified(false);
    162162        n1.put("key1", "value1-new");
     
    165165
    166166
    167         DataSetMerger visitor = new DataSetMerger(my,their);
    168         visitor.merge();
    169 
    170         Node n2 = (Node)my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    171         assertEquals(1,visitor.getConflicts().size());
     167        DataSetMerger visitor = new DataSetMerger(my, their);
     168        visitor.merge();
     169
     170        Node n2 = (Node) my.getPrimitiveById(1, OsmPrimitiveType.NODE);
     171        assertEquals(1, visitor.getConflicts().size());
    172172        assertSame(n, n2);
    173173        assertNotSame(n1, n2);
     
    185185    public void nodeSimple_DeleteConflict() {
    186186        Node n = new Node(1, 1);
    187         n.setCoor(new LatLon(0,0));
     187        n.setCoor(new LatLon(0, 0));
    188188        n.setDeleted(true);
    189189        n.put("key1", "value1");
    190190        my.addPrimitive(n);
    191191
    192         Node n1 = new Node(new LatLon(0,0));
    193         n1.setOsmId(1,2);
     192        Node n1 = new Node(new LatLon(0, 0));
     193        n1.setOsmId(1, 2);
    194194        n1.setModified(false);
    195195        n1.put("key1", "value1-new");
     
    198198
    199199
    200         DataSetMerger visitor = new DataSetMerger(my,their);
    201         visitor.merge();
    202 
    203         Node n2 = (Node)my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    204         assertEquals(1,visitor.getConflicts().size());
     200        DataSetMerger visitor = new DataSetMerger(my, their);
     201        visitor.merge();
     202
     203        Node n2 = (Node) my.getPrimitiveById(1, OsmPrimitiveType.NODE);
     204        assertEquals(1, visitor.getConflicts().size());
    205205        assertSame(n, n2);
    206206        assertNotSame(n1, n2);
     
    215215    @Test
    216216    public void nodeSimple_DeleteConflict_2() {
    217         Node n = new Node(new LatLon(0,0));
    218         n.setOsmId(1,1);
     217        Node n = new Node(new LatLon(0, 0));
     218        n.setOsmId(1, 1);
    219219        n.setDeleted(true);
    220220        my.addPrimitive(n);
    221221
    222         Node n1 = new Node(new LatLon(0,0));
    223         n1.setOsmId(1,1);
    224         their.addPrimitive(n1);
    225 
    226 
    227         DataSetMerger visitor = new DataSetMerger(my,their);
    228         visitor.merge();
    229 
    230         Node n2 = (Node)my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    231         assertEquals(0,visitor.getConflicts().size());
     222        Node n1 = new Node(new LatLon(0, 0));
     223        n1.setOsmId(1, 1);
     224        their.addPrimitive(n1);
     225
     226
     227        DataSetMerger visitor = new DataSetMerger(my, their);
     228        visitor.merge();
     229
     230        Node n2 = (Node) my.getPrimitiveById(1, OsmPrimitiveType.NODE);
     231        assertEquals(0, visitor.getConflicts().size());
    232232        assertTrue(n2.isVisible());
    233233        assertSame(n, n2);
     
    243243    @Test
    244244    public void nodeSimple_DeleteConflict_3() {
    245         Node n = new Node(new LatLon(1,1));
     245        Node n = new Node(new LatLon(1, 1));
    246246        n.setDeleted(true);
    247247        my.addPrimitive(n);
    248248
    249         Node n1 = new Node(new LatLon(1,1));
    250         their.addPrimitive(n1);
    251 
    252 
    253         DataSetMerger visitor = new DataSetMerger(my,their);
     249        Node n1 = new Node(new LatLon(1, 1));
     250        their.addPrimitive(n1);
     251
     252
     253        DataSetMerger visitor = new DataSetMerger(my, their);
    254254        visitor.merge();
    255255
     
    267267    @Test
    268268    public void nodeSimple_DeleteConflict_4() {
    269         Node n = new Node(new LatLon(1,1));
     269        Node n = new Node(new LatLon(1, 1));
    270270        n.setDeleted(true);
    271271        my.addPrimitive(n);
    272272
    273         Node n1 = new Node(new LatLon(1,1));
     273        Node n1 = new Node(new LatLon(1, 1));
    274274        n1.setDeleted(true);
    275275        their.addPrimitive(n1);
    276276
    277277
    278         DataSetMerger visitor = new DataSetMerger(my,their);
    279         visitor.merge();
    280 
    281         assertEquals(0,visitor.getConflicts().size());
    282         Node n2 = (Node)my.getNodes().toArray()[0];
     278        DataSetMerger visitor = new DataSetMerger(my, their);
     279        visitor.merge();
     280
     281        assertEquals(0, visitor.getConflicts().size());
     282        Node n2 = (Node) my.getNodes().toArray()[0];
    283283        assertSame(n2, n);
    284284        assertTrue(n2.isDeleted());
     
    301301
    302302        Node n = new Node();
    303         n.setCoor(new LatLon(0,0));
     303        n.setCoor(new LatLon(0, 0));
    304304        n.put("key1", "value1");
    305305        n.setUser(myUser);
     
    309309
    310310        Node n1 = new Node();
    311         n1.setCoor(new LatLon(0,0));
     311        n1.setCoor(new LatLon(0, 0));
    312312        n1.put("key1", "value1");
    313313        cal.add(Calendar.HOUR, 1);
     
    318318
    319319
    320         DataSetMerger visitor = new DataSetMerger(my,their);
     320        DataSetMerger visitor = new DataSetMerger(my, their);
    321321        visitor.merge();
    322322
    323323        Node n2 = my.getNodes().iterator().next();
    324         assertEquals(0,visitor.getConflicts().size());
    325         assertEquals("value1",n2.get("key1"));
     324        assertEquals(0, visitor.getConflicts().size());
     325        assertEquals("value1", n2.get("key1"));
    326326        assertTrue(n1.getTimestamp().equals(n2.getTimestamp()));
    327         assertEquals(theirUser,n2.getUser());
     327        assertEquals(theirUser, n2.getUser());
    328328        assertSame(n2, n);
    329329        assertNotSame(n2, n1);
     
    344344
    345345        Node n1 = new Node();
    346         n1.setCoor(new LatLon(0,0));
    347         n1.setOsmId(1,1);
     346        n1.setCoor(new LatLon(0, 0));
     347        n1.setOsmId(1, 1);
    348348        n1.put("key1", "value1");
    349349        Date timestamp = new Date();
     
    352352
    353353
    354         DataSetMerger visitor = new DataSetMerger(my,their);
     354        DataSetMerger visitor = new DataSetMerger(my, their);
    355355        visitor.merge();
    356356
    357357        Node n2 = my.getNodes().iterator().next();
    358         assertEquals(0,visitor.getConflicts().size());
    359         assertEquals("value1",n2.get("key1"));
     358        assertEquals(0, visitor.getConflicts().size());
     359        assertEquals("value1", n2.get("key1"));
    360360        assertTrue(n1.getTimestamp().equals(n2.getTimestamp()));
    361361        assertFalse(n2.isIncomplete());
     
    376376
    377377        Node n1 = new Node();
    378         n1.setCoor(new LatLon(0,0));
    379         n1.setOsmId(1,1);
     378        n1.setCoor(new LatLon(0, 0));
     379        n1.setOsmId(1, 1);
    380380        my.addPrimitive(n1);
    381381
    382382        Node n2 = new Node();
    383         n2.setCoor(new LatLon(0,0));
    384         n2.setOsmId(2,1);
     383        n2.setCoor(new LatLon(0, 0));
     384        n2.setOsmId(2, 1);
    385385
    386386        my.addPrimitive(n2);
    387387
    388388        Way myWay = new Way();
    389         myWay.setOsmId(3,1);
     389        myWay.setOsmId(3, 1);
    390390        myWay.put("key1", "value1");
    391391        myWay.addNode(n1);
     
    395395        // -- the source data set
    396396
    397         Node n3 = new Node(new LatLon(0,0));
    398         n3.setOsmId(1,1);
     397        Node n3 = new Node(new LatLon(0, 0));
     398        n3.setOsmId(1, 1);
    399399        their.addPrimitive(n3);
    400400
    401         Node n4 = new Node(new LatLon(1,1));
    402         n4.setOsmId(2,1);
     401        Node n4 = new Node(new LatLon(1, 1));
     402        n4.setOsmId(2, 1);
    403403        their.addPrimitive(n4);
    404404
    405405        Way theirWay = new Way();
    406         theirWay.setOsmId(3,2);
     406        theirWay.setOsmId(3, 2);
    407407        theirWay.put("key1", "value1");
    408408        theirWay.put("key2", "value2");
     
    412412
    413413
    414         DataSetMerger visitor = new DataSetMerger(my,their);
     414        DataSetMerger visitor = new DataSetMerger(my, their);
    415415        visitor.merge();
    416416
    417417        // -- tests
    418         Way merged = (Way)my.getPrimitiveById(3, OsmPrimitiveType.WAY);
    419         assertEquals(0,visitor.getConflicts().size());
    420         assertEquals("value1",merged.get("key1"));
    421         assertEquals("value2",merged.get("key2"));
    422         assertEquals(3,merged.getId());
    423         assertEquals(2,merged.getVersion());
    424         assertEquals(2,merged.getNodesCount());
    425         assertEquals(1,merged.getNode(0).getId());
    426         assertEquals(2,merged.getNode(1).getId());
     418        Way merged = (Way) my.getPrimitiveById(3, OsmPrimitiveType.WAY);
     419        assertEquals(0, visitor.getConflicts().size());
     420        assertEquals("value1", merged.get("key1"));
     421        assertEquals("value2", merged.get("key2"));
     422        assertEquals(3, merged.getId());
     423        assertEquals(2, merged.getVersion());
     424        assertEquals(2, merged.getNodesCount());
     425        assertEquals(1, merged.getNode(0).getId());
     426        assertEquals(2, merged.getNode(1).getId());
    427427        assertSame(merged, myWay);
    428428        assertSame(merged.getDataSet(), my);
    429429
    430         Node mergedNode = (Node)my.getPrimitiveById(1, OsmPrimitiveType.NODE);
     430        Node mergedNode = (Node) my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    431431        assertSame(mergedNode, n1);
    432         mergedNode = (Node)my.getPrimitiveById(2, OsmPrimitiveType.NODE);
     432        mergedNode = (Node) my.getPrimitiveById(2, OsmPrimitiveType.NODE);
    433433        assertSame(mergedNode, n2);
    434434
     
    448448        // -- my data set
    449449
    450         Node n1 = new Node(new LatLon(0,0));
    451         n1.setOsmId(1,1);
     450        Node n1 = new Node(new LatLon(0, 0));
     451        n1.setOsmId(1, 1);
    452452        my.addPrimitive(n1);
    453453
    454         Node n2 = new Node(new LatLon(1,1));
    455         n2.setOsmId(2,1);
     454        Node n2 = new Node(new LatLon(1, 1));
     455        n2.setOsmId(2, 1);
    456456        my.addPrimitive(n2);
    457457
    458458        Way myWay = new Way();
    459         myWay.setOsmId(3,1);
     459        myWay.setOsmId(3, 1);
    460460        myWay.addNode(n1);
    461461        myWay.addNode(n2);
     
    464464        // --- their data set
    465465
    466         Node n3 = new Node(new LatLon(0,0));
    467         n3.setOsmId(1,1);
     466        Node n3 = new Node(new LatLon(0, 0));
     467        n3.setOsmId(1, 1);
    468468        their.addPrimitive(n3);
    469469
    470         Node n5 = new Node(new LatLon(1,1));
    471         n5.setOsmId(4,1);
     470        Node n5 = new Node(new LatLon(1, 1));
     471        n5.setOsmId(4, 1);
    472472
    473473        their.addPrimitive(n5);
    474474
    475         Node n4 = new Node(new LatLon(2,2));
    476         n4.setOsmId(2,2);
     475        Node n4 = new Node(new LatLon(2, 2));
     476        n4.setOsmId(2, 2);
    477477        n4.put("key1", "value1");
    478478        their.addPrimitive(n4);
     
    480480
    481481        Way theirWay = new Way();
    482         theirWay.setOsmId(3,2);
     482        theirWay.setOsmId(3, 2);
    483483        theirWay.addNode(n3);
    484484        theirWay.addNode(n5); // insert a node
     
    486486        their.addPrimitive(theirWay);
    487487
    488         DataSetMerger visitor = new DataSetMerger(my,their);
     488        DataSetMerger visitor = new DataSetMerger(my, their);
    489489        visitor.merge();
    490490
    491491        // -- tests
    492         Way merged = (Way)my.getPrimitiveById(3, OsmPrimitiveType.WAY);
    493         assertEquals(0,visitor.getConflicts().size());
    494         assertEquals(3,merged.getId());
    495         assertEquals(2,merged.getVersion());
    496         assertEquals(3,merged.getNodesCount());
    497         assertEquals(1,merged.getNode(0).getId());
    498         assertEquals(4,merged.getNode(1).getId());
    499         assertEquals(2,merged.getNode(2).getId());
    500         assertEquals("value1",merged.getNode(2).get("key1"));
     492        Way merged = (Way) my.getPrimitiveById(3, OsmPrimitiveType.WAY);
     493        assertEquals(0, visitor.getConflicts().size());
     494        assertEquals(3, merged.getId());
     495        assertEquals(2, merged.getVersion());
     496        assertEquals(3, merged.getNodesCount());
     497        assertEquals(1, merged.getNode(0).getId());
     498        assertEquals(4, merged.getNode(1).getId());
     499        assertEquals(2, merged.getNode(2).getId());
     500        assertEquals("value1", merged.getNode(2).get("key1"));
    501501
    502502        assertSame(merged.getNode(0), n1);
     
    518518        // -- the target dataset
    519519
    520         Node n1 = new Node(new LatLon(0,0));
    521         n1.setOsmId(1,1);
     520        Node n1 = new Node(new LatLon(0, 0));
     521        n1.setOsmId(1, 1);
    522522        my.addPrimitive(n1);
    523523
    524         Node n2 = new Node(new LatLon(1,1));
    525         n2.setOsmId(2,1);
     524        Node n2 = new Node(new LatLon(1, 1));
     525        n2.setOsmId(2, 1);
    526526        my.addPrimitive(n2);
    527527
    528528        Way myWay = new Way();
    529         myWay.setOsmId(3,1);
     529        myWay.setOsmId(3, 1);
    530530
    531531        myWay.addNode(n1);
     
    537537        // -- the source dataset
    538538
    539         Node n3 = new Node(new LatLon(0,0));
    540         n3.setOsmId(1,1);
     539        Node n3 = new Node(new LatLon(0, 0));
     540        n3.setOsmId(1, 1);
    541541        their.addPrimitive(n3);
    542542
    543         Node n5 = new Node(new LatLon(1,1));
    544         n5.setOsmId(4,1);
     543        Node n5 = new Node(new LatLon(1, 1));
     544        n5.setOsmId(4, 1);
    545545        their.addPrimitive(n5);
    546546
    547         Node n4 = new Node(new LatLon(2,2));
    548         n4.setOsmId(2,1);
     547        Node n4 = new Node(new LatLon(2, 2));
     548        n4.setOsmId(2, 1);
    549549        n4.put("key1", "value1");
    550550        their.addPrimitive(n4);
    551551
    552 
    553552        Way theirWay = new Way();
    554         theirWay.setOsmId(3,2);
     553        theirWay.setOsmId(3, 2);
    555554
    556555        theirWay.addNode(n3);
     
    563562        visitor.merge();
    564563
    565         Way merged = (Way)my.getPrimitiveById(3, OsmPrimitiveType.WAY);
    566         assertEquals(1,visitor.getConflicts().size());
    567         assertEquals(3,merged.getId());
    568         assertEquals(1,merged.getVersion());
    569         assertEquals(2,merged.getNodesCount());
    570         assertEquals(1,merged.getNode(0).getId());
    571         assertEquals(2,merged.getNode(1).getId());
    572         assertEquals("value1",merged.get("key1"));
    573     }
    574 
     564        Way merged = (Way) my.getPrimitiveById(3, OsmPrimitiveType.WAY);
     565        assertEquals(1, visitor.getConflicts().size());
     566        assertEquals(3, merged.getId());
     567        assertEquals(1, merged.getVersion());
     568        assertEquals(2, merged.getNodesCount());
     569        assertEquals(1, merged.getNode(0).getId());
     570        assertEquals(2, merged.getNode(1).getId());
     571        assertEquals("value1", merged.get("key1"));
     572    }
    575573
    576574    /**
     
    583581    public void waySimple_TheirVersionNotVisibleMyIsModified() {
    584582
    585 
    586         Node mn1 = new Node(new LatLon(0,0));
    587         mn1.setOsmId(1,1);
     583        Node mn1 = new Node(new LatLon(0, 0));
     584        mn1.setOsmId(1, 1);
    588585        my.addPrimitive(mn1);
    589586
    590         Node mn2 = new Node(new LatLon(1,1));
    591         mn2.setOsmId(2,1);
     587        Node mn2 = new Node(new LatLon(1, 1));
     588        mn2.setOsmId(2, 1);
    592589        my.addPrimitive(mn2);
    593590
    594591        Way myWay = new Way();
    595         myWay.setOsmId(3,1);
     592        myWay.setOsmId(3, 1);
    596593        myWay.addNode(mn1);
    597594        myWay.addNode(mn2);
     
    599596        my.addPrimitive(myWay);
    600597
    601 
    602598        Way theirWay = new Way();
    603         theirWay.setOsmId(3,2);
     599        theirWay.setOsmId(3, 2);
    604600        theirWay.setVisible(false);
    605601        /* Invisible objects fetched from the server should be marked as "deleted".
     
    609605        their.addPrimitive(theirWay);
    610606
    611         DataSetMerger visitor = new DataSetMerger(my,their);
    612         visitor.merge();
    613 
    614         Way merged = (Way)my.getPrimitiveById(3, OsmPrimitiveType.WAY);
    615         assertEquals(1,visitor.getConflicts().size());
     607        DataSetMerger visitor = new DataSetMerger(my, their);
     608        visitor.merge();
     609
     610        Way merged = (Way) my.getPrimitiveById(3, OsmPrimitiveType.WAY);
     611        assertEquals(1, visitor.getConflicts().size());
    616612        assertTrue(visitor.getConflicts().hasConflictForMy(myWay));
    617613        assertTrue(visitor.getConflicts().hasConflictForTheir(theirWay));
    618         assertEquals(myWay,merged);
     614        assertEquals(myWay, merged);
    619615    }
    620616
     
    630626        // -- my data set
    631627
    632         Node n1 = new Node(new LatLon(0,0));
    633         n1.setOsmId(1,1);
     628        Node n1 = new Node(new LatLon(0, 0));
     629        n1.setOsmId(1, 1);
    634630        my.addPrimitive(n1);
    635631
    636         Node n2 = new Node(new LatLon(1,1));
    637         n2.setOsmId(2,1);
     632        Node n2 = new Node(new LatLon(1, 1));
     633        n2.setOsmId(2, 1);
    638634        my.addPrimitive(n2);
    639635
     
    645641        // -- their data set
    646642
    647         Node n3 = new Node(new LatLon(0,0));
    648         n3.setOsmId(1,1);
     643        Node n3 = new Node(new LatLon(0, 0));
     644        n3.setOsmId(1, 1);
    649645        their.addPrimitive(n3);
    650646
    651         Node n4 = new Node(new LatLon(1,1));
    652         n4.setOsmId(2,1);
     647        Node n4 = new Node(new LatLon(1, 1));
     648        n4.setOsmId(2, 1);
    653649        their.addPrimitive(n4);
    654650
     
    661657        their.addPrimitive(theirWay);
    662658
    663         DataSetMerger visitor = new DataSetMerger(my,their);
     659        DataSetMerger visitor = new DataSetMerger(my, their);
    664660        visitor.merge();
    665661
    666662        // -- tests
    667         Way merged = (Way)my.getWays().toArray()[0];
    668         assertEquals(0,visitor.getConflicts().size());
     663        Way merged = (Way) my.getWays().toArray()[0];
     664        assertEquals(0, visitor.getConflicts().size());
    669665        assertEquals("their", merged.getUser().getName());
    670666        assertEquals(1111, merged.getUser().getId());
     
    688684        // -- my data set
    689685
    690         Node n1 = new Node(new LatLon(0,0));
     686        Node n1 = new Node(new LatLon(0, 0));
    691687        my.addPrimitive(n1);
    692688
    693         Node n2 = new Node(new LatLon(1,1));
     689        Node n2 = new Node(new LatLon(1, 1));
    694690        my.addPrimitive(n2);
    695691
     
    701697        // -- their data set
    702698
    703         Node n3 = new Node(new LatLon(0,0));
     699        Node n3 = new Node(new LatLon(0, 0));
    704700        their.addPrimitive(n3);
    705701
    706         Node n4 = new Node(new LatLon(1,1));
     702        Node n4 = new Node(new LatLon(1, 1));
    707703        their.addPrimitive(n4);
    708704
     
    715711        their.addPrimitive(theirWay);
    716712
    717         DataSetMerger visitor = new DataSetMerger(my,their);
     713        DataSetMerger visitor = new DataSetMerger(my, their);
    718714        visitor.merge();
    719715
    720716        // -- tests
    721         Way merged = (Way)my.getWays().toArray()[0];
    722         assertEquals(0,visitor.getConflicts().size());
     717        Way merged = (Way) my.getWays().toArray()[0];
     718        assertEquals(0, visitor.getConflicts().size());
    723719        assertEquals("their", merged.getUser().getName());
    724720        assertEquals(1111, merged.getUser().getId());
     
    731727    }
    732728
    733 
    734729    /**
    735730     * My dataset includes a deleted node.
     
    745740        // -- my dataset
    746741
    747         Node mn1 = new Node(new LatLon(0,0));
    748         mn1.setOsmId(1,1);
     742        Node mn1 = new Node(new LatLon(0, 0));
     743        mn1.setOsmId(1, 1);
    749744        mn1.setDeleted(true);
    750745        my.addPrimitive(mn1);
    751746
    752 
    753         Node tn1 = new Node(new LatLon(0,0));
    754         tn1.setOsmId(1,1);
     747        Node tn1 = new Node(new LatLon(0, 0));
     748        tn1.setOsmId(1, 1);
    755749        their.addPrimitive(tn1);
    756750
    757         Node tn2 = new Node(new LatLon(1,1));
    758         tn2.setOsmId(2,1);
     751        Node tn2 = new Node(new LatLon(1, 1));
     752        tn2.setOsmId(2, 1);
    759753        their.addPrimitive(tn2);
    760754
    761         Node tn3 = new Node(new LatLon(2,2));
    762         tn3.setOsmId(3,1);
     755        Node tn3 = new Node(new LatLon(2, 2));
     756        tn3.setOsmId(3, 1);
    763757        their.addPrimitive(tn3);
    764758
    765759        // -- their data set
    766760        Way theirWay = new Way();
    767         theirWay.setOsmId(4,1);
     761        theirWay.setOsmId(4, 1);
    768762        theirWay.addNode(tn1);
    769763        theirWay.addNode(tn2);
     
    773767        their.addPrimitive(theirWay);
    774768
    775         DataSetMerger visitor = new DataSetMerger(my,their);
     769        DataSetMerger visitor = new DataSetMerger(my, their);
    776770        visitor.merge();
    777771
     
    779773        assertTrue(visitor.getConflicts().get(0).isMyDeleted());
    780774
    781         Way myWay = (Way)my.getPrimitiveById(4, OsmPrimitiveType.WAY);
     775        Way myWay = (Way) my.getPrimitiveById(4, OsmPrimitiveType.WAY);
    782776        assertEquals(3, myWay.getNodesCount());
    783777
    784         Node n = (Node)my.getPrimitiveById(1,OsmPrimitiveType.NODE);
     778        Node n = (Node) my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    785779        assertTrue(myWay.getNodes().contains(n));
    786780
     
    799793    public void relationComplex_mergingADeletedNode() {
    800794
    801 
    802         Node mn1 = new Node(new LatLon(0,0));
    803         mn1.setOsmId(1,1);
     795        Node mn1 = new Node(new LatLon(0, 0));
     796        mn1.setOsmId(1, 1);
    804797        mn1.setDeleted(true);
    805798        my.addPrimitive(mn1);
    806799
    807800
    808         Node tn1 = new Node(new LatLon(0,0));
    809         tn1.setOsmId(1,1);
     801        Node tn1 = new Node(new LatLon(0, 0));
     802        tn1.setOsmId(1, 1);
    810803        their.addPrimitive(tn1);
    811804
    812         Node tn2 = new Node(new LatLon(1,1));
    813         tn2.setOsmId(2,1);
     805        Node tn2 = new Node(new LatLon(1, 1));
     806        tn2.setOsmId(2, 1);
    814807        their.addPrimitive(tn2);
    815808
    816         Node tn3 = new Node(new LatLon(2,2));
    817         tn3.setOsmId(3,1);
     809        Node tn3 = new Node(new LatLon(2, 2));
     810        tn3.setOsmId(3, 1);
    818811        their.addPrimitive(tn3);
    819812
    820813
    821814        Relation theirRelation = new Relation();
    822         theirRelation.setOsmId(4,1);
     815        theirRelation.setOsmId(4, 1);
    823816
    824817        theirRelation.addMember(new RelationMember("", tn1));
     
    827820        their.addPrimitive(theirRelation);
    828821
    829         DataSetMerger visitor = new DataSetMerger(my,their);
    830         visitor.merge();
    831 
    832         Node n = (Node)my.getPrimitiveById(1,OsmPrimitiveType.NODE);
     822        DataSetMerger visitor = new DataSetMerger(my, their);
     823        visitor.merge();
     824
     825        Node n = (Node) my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    833826        assertNotNull(n);
    834827
     
    837830        assertTrue(visitor.getConflicts().get(0).isMyDeleted());
    838831
    839         Relation r = (Relation)my.getPrimitiveById(4,OsmPrimitiveType.RELATION);
     832        Relation r = (Relation) my.getPrimitiveById(4, OsmPrimitiveType.RELATION);
    840833        assertEquals(3, r.getMembersCount());
    841834
     
    858851
    859852        Way w3 = new Way(3);
    860         w3.setNodes(Arrays.asList(n1,n2));
     853        w3.setNodes(Arrays.asList(n1, n2));
    861854        their.addPrimitive(w3);
    862855        assertTrue(w3.isIncomplete());
    863856
    864 
    865         DataSetMerger visitor = new DataSetMerger(my, their);
    866         visitor.merge();
    867 
    868         assertEquals(0,visitor.getConflicts().size());
    869 
    870         OsmPrimitive p= my.getPrimitiveById(1, OsmPrimitiveType.NODE);
     857        DataSetMerger visitor = new DataSetMerger(my, their);
     858        visitor.merge();
     859
     860        assertEquals(0, visitor.getConflicts().size());
     861
     862        OsmPrimitive p = my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    871863        assertNotNull(p);
    872864        assertTrue(p.isIncomplete());
    873         p= my.getPrimitiveById(2, OsmPrimitiveType.NODE);
     865        p = my.getPrimitiveById(2, OsmPrimitiveType.NODE);
    874866        assertNotNull(p);
    875867        assertTrue(p.isIncomplete());
    876         p= my.getPrimitiveById(3, OsmPrimitiveType.WAY);
     868        p = my.getPrimitiveById(3, OsmPrimitiveType.WAY);
    877869        assertNotNull(p);
    878870        assertTrue(p.isIncomplete());
    879871
    880         Way w = (Way)my.getPrimitiveById(3, OsmPrimitiveType.WAY);
     872        Way w = (Way) my.getPrimitiveById(3, OsmPrimitiveType.WAY);
    881873        assertNotNull(w);
    882874        assertTrue(p.isIncomplete());
     
    904896        // an incomplete way with two incomplete nodes
    905897        Way w3 = new Way(3);
    906         w3.setNodes(Arrays.asList(n1,n2));
     898        w3.setNodes(Arrays.asList(n1, n2));
    907899        their.addPrimitive(w3);
    908900
    909 
    910         Node n4 = new Node(new LatLon(0,0));
    911         n4.setOsmId(1,1);
     901        Node n4 = new Node(new LatLon(0, 0));
     902        n4.setOsmId(1, 1);
    912903        my.addPrimitive(n4);
    913904
    914         Node n5 = new Node(new LatLon(1,1));
    915         n5.setOsmId(2,1);
     905        Node n5 = new Node(new LatLon(1, 1));
     906        n5.setOsmId(2, 1);
    916907        my.addPrimitive(n5);
    917908
    918909        Way w6 = new Way(3, 1);
    919         w6.setNodes(Arrays.asList(n4,n5));
     910        w6.setNodes(Arrays.asList(n4, n5));
    920911        my.addPrimitive(w6);
    921912
    922         DataSetMerger visitor = new DataSetMerger(my,their);
    923         visitor.merge();
    924 
    925         assertEquals(0,visitor.getConflicts().size());
    926 
    927         OsmPrimitive p= my.getPrimitiveById(1, OsmPrimitiveType.NODE);
     913        DataSetMerger visitor = new DataSetMerger(my, their);
     914        visitor.merge();
     915
     916        assertEquals(0, visitor.getConflicts().size());
     917
     918        OsmPrimitive p = my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    928919        assertNotNull(p);
    929920        assertFalse(p.isIncomplete());
    930         p= my.getPrimitiveById(2, OsmPrimitiveType.NODE);
     921        p = my.getPrimitiveById(2, OsmPrimitiveType.NODE);
    931922        assertNotNull(p);
    932923        assertFalse(p.isIncomplete());
    933         p= my.getPrimitiveById(3,OsmPrimitiveType.WAY);
     924        p = my.getPrimitiveById(3, OsmPrimitiveType.WAY);
    934925        assertNotNull(p);
    935926        assertFalse(p.isIncomplete());
    936927
    937         Way w = (Way)my.getPrimitiveById(3,OsmPrimitiveType.WAY);
     928        Way w = (Way) my.getPrimitiveById(3, OsmPrimitiveType.WAY);
    938929        assertNotNull(w);
    939930        assertFalse(p.isIncomplete());
     
    954945        // an complete node
    955946        Node n1 = new Node(1, 1);
    956         n1.setCoor(new LatLon(1,1));
     947        n1.setCoor(new LatLon(1, 1));
    957948        their.addPrimitive(n1);
    958949
    959950        // another complete node
    960951        Node n2 = new Node(2, 1);
    961         n2.setCoor(new LatLon(2,2));
     952        n2.setCoor(new LatLon(2, 2));
    962953        their.addPrimitive(n2);
    963954
     
    980971
    981972        // -- test it
    982         assertEquals(0,visitor.getConflicts().size());
    983 
    984         Node n = (Node)my.getPrimitiveById(1, OsmPrimitiveType.NODE);
     973        assertEquals(0, visitor.getConflicts().size());
     974
     975        Node n = (Node) my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    985976        assertNotNull(n);
    986977        assertFalse(n.isIncomplete());
    987978
    988         n = (Node)my.getPrimitiveById(2, OsmPrimitiveType.NODE);
     979        n = (Node) my.getPrimitiveById(2, OsmPrimitiveType.NODE);
    989980        assertNotNull(n);
    990981        assertFalse(n.isIncomplete());
    991982
    992         Way w = (Way)my.getPrimitiveById(3, OsmPrimitiveType.WAY);
     983        Way w = (Way) my.getPrimitiveById(3, OsmPrimitiveType.WAY);
    993984        assertNotNull(w);
    994985        assertFalse(w.hasIncompleteNodes());
  • trunk/test/unit/org/openstreetmap/josm/data/osm/FilterTest.java

    r8509 r8510  
    6464    @Test
    6565    public void filter_test() throws ParseError, IllegalDataException, IOException {
    66         for (int i : new int [] {1,2,3, 11,12,13,14, 15}) {
     66        for (int i : new int[] {1, 2, 3, 11, 12, 13, 14, 15}) {
    6767            DataSet ds;
    6868            try (InputStream is = new FileInputStream("data_nodist/filterTests.osm")) {
  • trunk/test/unit/org/openstreetmap/josm/data/osm/OsmPrimitiveKeyHandlingTest.java

    r8450 r8510  
    117117    public void hasEqualSemanticAttributes() {
    118118        Node n1 = new Node(1);
    119         n1.setCoor(new LatLon(0,0));
     119        n1.setCoor(new LatLon(0, 0));
    120120        n1.put("key.1", "value.1");
    121121        n1.put("key.2", "value.2");
    122122
    123123        Node n2 = new Node(1);
    124         n2.setCoor(new LatLon(0,0));
     124        n2.setCoor(new LatLon(0, 0));
    125125        n2.put("key.2", "value.2");
    126126        n2.put("key.1", "value.1");
     
    135135    public void hasEqualSemanticAttributes_2() {
    136136        Node n1 = new Node(1);
    137         n1.setCoor(new LatLon(0,0));
     137        n1.setCoor(new LatLon(0, 0));
    138138        n1.put("key.1", "value.1");
    139139        n1.put("key.2", "value.3");
    140140
    141141        Node n2 = new Node(1);
    142         n2.setCoor(new LatLon(0,0));
     142        n2.setCoor(new LatLon(0, 0));
    143143        n2.put("key.1", "value.1");
    144144        n2.put("key.2", "value.4");
  • trunk/test/unit/org/openstreetmap/josm/data/osm/OsmPrimitiveTest.java

    r7937 r8510  
    143143    }
    144144
    145     @Test(expected=DataIntegrityProblemException.class)
     145    @Test(expected = DataIntegrityProblemException.class)
    146146    public void checkMustBeInDatasate() {
    147147        Node n = new Node();
    148148        n.getReferrers();
    149149    }
    150 
    151150}
  • trunk/test/unit/org/openstreetmap/josm/data/osm/RelationTest.java

    r7937 r8510  
    2020    }
    2121
    22     @Test(expected=NullPointerException.class)
     22    @Test(expected = NullPointerException.class)
    2323    public void createNewRelation() {
    2424        new Relation(null);
  • trunk/test/unit/org/openstreetmap/josm/data/osm/history/HistoryNodeTest.java

    r8450 r8510  
    2424                4l,
    2525                d,
    26                 new LatLon(0,0)
     26                new LatLon(0, 0)
    2727                );
    2828
     
    4646                4,
    4747                d,
    48                 new LatLon(0,0)
     48                new LatLon(0, 0)
    4949                );
    5050
  • trunk/test/unit/org/openstreetmap/josm/data/osm/history/HistoryWayTest.java

    r8450 r8510  
    7171            way.getNodeId(1);
    7272            fail("expected expection of type " + IndexOutOfBoundsException.class.toString());
    73         } catch(IndexOutOfBoundsException e) {
     73        } catch (IndexOutOfBoundsException e) {
    7474            // OK
    7575        }
  • trunk/test/unit/org/openstreetmap/josm/data/osm/visitor/MergeSourceBuildingVisitorTest.java

    r7937 r8510  
    4545        DataSet source = new DataSet();
    4646        Node n1 = new Node(1);
    47         Node n2 = new Node(new LatLon(10.0,10.0));
    48         n2.put("name","n2");
     47        Node n2 = new Node(new LatLon(10.0, 10.0));
     48        n2.put("name", "n2");
    4949        Node n3 = new Node(3);
    50         Node n4 = new Node(new LatLon(20.0,20.0));
    51         n4.put("name","n4");
     50        Node n4 = new Node(new LatLon(20.0, 20.0));
     51        n4.put("name", "n4");
    5252        source.addPrimitive(n1);
    5353        source.addPrimitive(n2);
    5454        source.addPrimitive(n3);
    5555        source.addPrimitive(n4);
    56         source.setSelected(n1,n2);
     56        source.setSelected(n1, n2);
    5757
    5858        MergeSourceBuildingVisitor builder = new MergeSourceBuildingVisitor(source);
     
    6161        assertEquals(2, hull.getNodes().size());
    6262
    63         OsmPrimitive p = hull.getPrimitiveById(1,OsmPrimitiveType.NODE);
     63        OsmPrimitive p = hull.getPrimitiveById(1, OsmPrimitiveType.NODE);
    6464        assertNotNull(p);
    6565        assertEquals(p.getClass(), Node.class);
    6666
    67         p = hull.getPrimitiveById(3,OsmPrimitiveType.NODE);
     67        p = hull.getPrimitiveById(3, OsmPrimitiveType.NODE);
    6868        assertNull(p);
    6969
     
    7474        assertNull(p);
    7575    }
    76 
    7776
    7877    @Test
     
    9594        assertEquals(2, hull.getNodes().size());
    9695
    97         OsmPrimitive p = hull.getPrimitiveById(1,OsmPrimitiveType.NODE);
     96        OsmPrimitive p = hull.getPrimitiveById(1, OsmPrimitiveType.NODE);
    9897        assertNotNull(p);
    9998        assertEquals(p.getClass(), Node.class);
    10099
    101         p = hull.getPrimitiveById(2,OsmPrimitiveType.NODE);
     100        p = hull.getPrimitiveById(2, OsmPrimitiveType.NODE);
    102101        assertNotNull(p);
    103102        assertEquals(p.getClass(), Node.class);
     
    119118        source.addPrimitive(n2);
    120119        source.addPrimitive(w1);
    121         source.setSelected(w1,n1,n2);
     120        source.setSelected(w1, n1, n2);
    122121
    123122        MergeSourceBuildingVisitor builder = new MergeSourceBuildingVisitor(source);
     
    127126        assertEquals(2, hull.getNodes().size());
    128127
    129         OsmPrimitive p = hull.getPrimitiveById(1,OsmPrimitiveType.NODE);
     128        OsmPrimitive p = hull.getPrimitiveById(1, OsmPrimitiveType.NODE);
    130129        assertNotNull(p);
    131130        assertEquals(p.getClass(), Node.class);
     
    135134        assertEquals(p.getClass(), Node.class);
    136135
    137         p = hull.getPrimitiveById(3,OsmPrimitiveType.WAY);
     136        p = hull.getPrimitiveById(3, OsmPrimitiveType.WAY);
    138137        assertNotNull(p);
    139138        assertEquals(p.getClass(), Way.class);
     
    164163        Node n20 = new Node(20, 1);
    165164        n20.setCoor(new LatLon(0, 0));
    166         r1.addMember(new RelationMember("node-20",n20));
     165        r1.addMember(new RelationMember("node-20", n20));
    167166        Way w30 = new Way(30, 1);
    168167        Node n21  = new Node(21);
     
    170169        Node n22 = new Node(22);
    171170        w30.addNode(n22);
    172         r1.addMember(new RelationMember("way-30",w30));
     171        r1.addMember(new RelationMember("way-30", w30));
    173172        Relation r40 = new Relation(40);
    174173        r1.addMember(new RelationMember("relation-40", r40));
     
    179178        source.addPrimitive(r40);
    180179        source.addPrimitive(r1);
    181         source.setSelected(r1,n20,w30,r40);
     180        source.setSelected(r1, n20, w30, r40);
    182181
    183182        MergeSourceBuildingVisitor builder = new MergeSourceBuildingVisitor(source);
     
    192191        assertEquals(p.getClass(), Relation.class);
    193192
    194         Way w = (Way)hull.getPrimitiveById(30,OsmPrimitiveType.WAY);
     193        Way w = (Way) hull.getPrimitiveById(30, OsmPrimitiveType.WAY);
    195194        assertNotNull(w);
    196195        assertEquals(2, w.getNodesCount());
    197         Node n = (Node)hull.getPrimitiveById(21, OsmPrimitiveType.NODE);
     196        Node n = (Node) hull.getPrimitiveById(21, OsmPrimitiveType.NODE);
    198197        assertNotNull(n);
    199198        assertTrue(w.containsNode(n));
    200199
    201         n = (Node)hull.getPrimitiveById(22,OsmPrimitiveType.NODE);
     200        n = (Node) hull.getPrimitiveById(22, OsmPrimitiveType.NODE);
    202201        assertNotNull(n);
    203202        assertTrue(w.containsNode(n));
    204203
    205         Relation r = (Relation)hull.getPrimitiveById(40,OsmPrimitiveType.RELATION);
    206         assertNotNull(r);
    207 
    208         r = (Relation)hull.getPrimitiveById(1, OsmPrimitiveType.RELATION);
     204        Relation r = (Relation) hull.getPrimitiveById(40, OsmPrimitiveType.RELATION);
     205        assertNotNull(r);
     206
     207        r = (Relation) hull.getPrimitiveById(1, OsmPrimitiveType.RELATION);
    209208        assertNotNull(r);
    210209        assertEquals(3, r.getMembersCount());
    211         RelationMember m = new RelationMember("node-20", hull.getPrimitiveById(20,OsmPrimitiveType.NODE));
     210        RelationMember m = new RelationMember("node-20", hull.getPrimitiveById(20, OsmPrimitiveType.NODE));
    212211        assertTrue(r.getMembers().contains(m));
    213212        m = new RelationMember("way-30", hull.getPrimitiveById(30, OsmPrimitiveType.WAY));
     
    222221        Relation r1 = new Relation(1, 1);
    223222        Node n20 = new Node(20);
    224         r1.addMember(new RelationMember("node-20",n20));
     223        r1.addMember(new RelationMember("node-20", n20));
    225224        Way w30 = new Way(30, 1);
    226225        Node n21;
     
    228227        Node n22;
    229228        w30.addNode(n22 = new Node(22));
    230         r1.addMember(new RelationMember("way-30",w30));
     229        r1.addMember(new RelationMember("way-30", w30));
    231230        Relation r40 = new Relation(40);
    232231        r1.addMember(new RelationMember("relation-40", r40));
     
    250249        assertEquals(p.getClass(), Relation.class);
    251250
    252         Way w = (Way)hull.getPrimitiveById(30, OsmPrimitiveType.WAY);
     251        Way w = (Way) hull.getPrimitiveById(30, OsmPrimitiveType.WAY);
    253252        assertNotNull(w);
    254253        assertTrue(w.isIncomplete());
    255254
    256255
    257         Node n = (Node)hull.getPrimitiveById(21,OsmPrimitiveType.NODE);
     256        Node n = (Node) hull.getPrimitiveById(21, OsmPrimitiveType.NODE);
    258257        assertNull(n);
    259258
    260         n = (Node)hull.getPrimitiveById(22, OsmPrimitiveType.NODE);
     259        n = (Node) hull.getPrimitiveById(22, OsmPrimitiveType.NODE);
    261260        assertNull(n);
    262261
    263         Relation r = (Relation)hull.getPrimitiveById(40, OsmPrimitiveType.RELATION);
     262        Relation r = (Relation) hull.getPrimitiveById(40, OsmPrimitiveType.RELATION);
    264263        assertNotNull(r);
    265264        assertTrue(r.isIncomplete());
    266265
    267         r = (Relation)hull.getPrimitiveById(1, OsmPrimitiveType.RELATION);
     266        r = (Relation) hull.getPrimitiveById(1, OsmPrimitiveType.RELATION);
    268267        assertNotNull(r);
    269268        assertEquals(3, r.getMembersCount());
     
    281280        Relation r1 = new Relation();
    282281        r1.put("name", "r1");
    283         Node n20 = new Node(new LatLon(20.0,20.0));
     282        Node n20 = new Node(new LatLon(20.0, 20.0));
    284283        n20.put("name", "n20");
    285         r1.addMember(new RelationMember("node-20",n20));
     284        r1.addMember(new RelationMember("node-20", n20));
    286285
    287286        Way w30 = new Way();
    288287        w30.put("name", "w30");
    289288        Node n21;
    290         w30.addNode(n21 = new Node(new LatLon(21.0,21.0)));
    291         n21.put("name","n21");
     289        w30.addNode(n21 = new Node(new LatLon(21.0, 21.0)));
     290        n21.put("name", "n21");
    292291        Node n22;
    293         w30.addNode(n22 = new Node(new LatLon(22.0,22.0)));
    294         n22.put("name","n22");
    295         r1.addMember(new RelationMember("way-30",w30));
     292        w30.addNode(n22 = new Node(new LatLon(22.0, 22.0)));
     293        n22.put("name", "n22");
     294        r1.addMember(new RelationMember("way-30", w30));
    296295        Relation r40 = new Relation();
    297296        r40.put("name", "r40");
     
    317316        assertEquals(p.getClass(), Relation.class);
    318317
    319         Way w = (Way)lookupByName(hull.getWays(), "w30");
     318        Way w = (Way) lookupByName(hull.getWays(), "w30");
    320319        assertNotNull(w);
    321320        assertEquals(2, w.getNodesCount());
    322321
    323         Node n = (Node)lookupByName(hull.getNodes(), "n21");
     322        Node n = (Node) lookupByName(hull.getNodes(), "n21");
    324323        assertNotNull(n);
    325324        assertTrue(w.containsNode(n));
    326325
    327         n = (Node)lookupByName(hull.getNodes(), "n22");
     326        n = (Node) lookupByName(hull.getNodes(), "n22");
    328327        assertNotNull(n);
    329328        assertTrue(w.containsNode(n));
    330329
    331         Relation r = (Relation)lookupByName(hull.getRelations(), "r40");
    332         assertNotNull(r);
    333 
    334         r = (Relation)lookupByName(hull.getRelations(), "r1");
     330        Relation r = (Relation) lookupByName(hull.getRelations(), "r40");
     331        assertNotNull(r);
     332
     333        r = (Relation) lookupByName(hull.getRelations(), "r1");
    335334        assertNotNull(r);
    336335        assertEquals(3, r.getMembersCount());
     
    347346        DataSet source = new DataSet();
    348347        Relation r1 = new Relation(1, 1);
    349         r1.addMember(new RelationMember("relation-1",r1));
     348        r1.addMember(new RelationMember("relation-1", r1));
    350349        source.addPrimitive(r1);
    351350        source.setSelected(r1);
     
    356355        assertEquals(1, hull.getRelations().size());
    357356
    358         Relation r = (Relation)hull.getPrimitiveById(1, OsmPrimitiveType.RELATION);
     357        Relation r = (Relation) hull.getPrimitiveById(1, OsmPrimitiveType.RELATION);
    359358        assertNotNull(r);
    360359        assertEquals(1, r.getMembersCount());
    361         assertTrue(r.getMembers().contains(new RelationMember("relation-1",r)));
     360        assertTrue(r.getMembers().contains(new RelationMember("relation-1", r)));
    362361    }
    363362
     
    367366        Relation r1 = new Relation();
    368367        r1.put("name", "r1");
    369         r1.addMember(new RelationMember("relation-1",r1));
     368        r1.addMember(new RelationMember("relation-1", r1));
    370369        source.addPrimitive(r1);
    371370        source.setSelected(r1);
     
    376375        assertEquals(1, hull.getRelations().size());
    377376
    378         Relation r = (Relation)lookupByName(hull.getRelations(), "r1");
     377        Relation r = (Relation) lookupByName(hull.getRelations(), "r1");
    379378        assertNotNull(r);
    380379        assertEquals(1, r.getMembersCount());
    381         assertTrue(r.getMembers().contains(new RelationMember("relation-1",r)));
     380        assertTrue(r.getMembers().contains(new RelationMember("relation-1", r)));
    382381    }
    383382
     
    389388        Relation r2 = new Relation(2, 3);
    390389        source.addPrimitive(r2);
    391         r1.addMember(new RelationMember("relation-2",r2));
    392         r2.addMember(new RelationMember("relation-1",r1));
    393         source.setSelected(r1,r2);
     390        r1.addMember(new RelationMember("relation-2", r2));
     391        r2.addMember(new RelationMember("relation-1", r1));
     392        source.setSelected(r1, r2);
    394393
    395394        MergeSourceBuildingVisitor builder = new MergeSourceBuildingVisitor(source);
     
    398397        assertEquals(2, hull.getRelations().size());
    399398
    400         r1 = (Relation)hull.getPrimitiveById(1, OsmPrimitiveType.RELATION);
     399        r1 = (Relation) hull.getPrimitiveById(1, OsmPrimitiveType.RELATION);
    401400        assertNotNull(r1);
    402         r2 = (Relation)hull.getPrimitiveById(2, OsmPrimitiveType.RELATION);
     401        r2 = (Relation) hull.getPrimitiveById(2, OsmPrimitiveType.RELATION);
    403402        assertNotNull(r2);
    404403        assertEquals(1, r1.getMembersCount());
    405         assertTrue(r1.getMembers().contains(new RelationMember("relation-2",r2)));
     404        assertTrue(r1.getMembers().contains(new RelationMember("relation-2", r2)));
    406405        assertEquals(1, r2.getMembersCount());
    407         assertTrue(r2.getMembers().contains(new RelationMember("relation-1",r1)));
     406        assertTrue(r2.getMembers().contains(new RelationMember("relation-1", r1)));
    408407    }
    409408}
  • trunk/test/unit/org/openstreetmap/josm/data/projection/EllipsoidTest.java

    r6881 r8510  
    2020        double maxErrLat = 0, maxErrLon = 0;
    2121        Ellipsoid ellips = Ellipsoid.WGS84;
    22         for (int num=0; num<1000; ++num) {
     22        for (int num = 0; num < 1000; ++num) {
    2323
    2424            double lat = r.nextDouble() * 180.0 - 90.0;
    2525            double lon = r.nextDouble() * 360.0 - 180.0;
    26             LatLon ll = new LatLon(lat,lon);
     26            LatLon ll = new LatLon(lat, lon);
    2727
    28             for (int i=0; i<1000; ++i) {
     28            for (int i = 0; i < 1000; ++i) {
    2929                double[] cart = ellips.latLon2Cart(ll);
    3030                ll = ellips.cart2LatLon(cart);
    3131
    32                 if (!(Math.abs(lat - ll.lat())<EPSILON && Math.abs(lon - ll.lon())<EPSILON)) {
     32                if (!(Math.abs(lat - ll.lat()) < EPSILON && Math.abs(lon - ll.lon()) < EPSILON)) {
    3333                    String error = String.format("point: %s iterations: %s current: %s errorLat: %s errorLon %s",
    3434                            new LatLon(lat, lon), i, ll, Math.abs(lat - ll.lat()), Math.abs(lon - ll.lon()));
  • trunk/test/unit/org/openstreetmap/josm/data/projection/ProjectionRegressionTest.java

    r8509 r8510  
    6565            prevData = readData();
    6666        }
    67         Map<String,TestData> prevCodesMap = new HashMap<>();
     67        Map<String, TestData> prevCodesMap = new HashMap<>();
    6868        for (TestData data : prevData) {
    6969            prevCodesMap.put(data.code, data);
     
    116116                TestData next = new TestData();
    117117
    118                 Pair<Double,Double> ll = readLine("ll", in.readLine());
    119                 Pair<Double,Double> en = readLine("en", in.readLine());
    120                 Pair<Double,Double> ll2 = readLine("ll2", in.readLine());
     118                Pair<Double, Double> ll = readLine("ll", in.readLine());
     119                Pair<Double, Double> en = readLine("en", in.readLine());
     120                Pair<Double, Double> ll2 = readLine("ll2", in.readLine());
    121121
    122122                next.code = line;
     
    131131    }
    132132
    133     private static Pair<Double,Double> readLine(String expectedName, String input) {
     133    private static Pair<Double, Double> readLine(String expectedName, String input) {
    134134        String[] fields = input.trim().split("[ ]+");
    135135        if (fields.length != 3) throw new AssertionError();
  • trunk/test/unit/org/openstreetmap/josm/data/projection/ProjectionTest.java

    r6881 r8510  
    2828        }
    2929
    30         for (int i=0; i<=3; ++i) {
     30        for (int i = 0; i <= 3; ++i) {
    3131            testProj(Projections.getProjectionByCode("EPSG:"+Integer.toString(27561+i))); // Lambert 4 Zones France
    3232        }
    3333
    34         for (int i=0; i<=4; ++i) {
     34        for (int i = 0; i <= 4; ++i) {
    3535            testProj(Projections.getProjectionByCode("EPSG:"+Integer.toString(2176+i))); // PUWG Poland
    3636        }
     
    3838        testProj(Projections.getProjectionByCode("EPSG:21781")); // Swiss grid
    3939
    40         for (int i=0; i<=60; ++i) {
     40        for (int i = 0; i <= 60; ++i) {
    4141            testProj(Projections.getProjectionByCode("EPSG:"+Integer.toString(32601+i))); // UTM North
    4242            testProj(Projections.getProjectionByCode("EPSG:"+Integer.toString(32701+i))); // UTM South
     
    4444
    4545        if (!"yes".equals(System.getProperty("suppressPermanentFailure"))) {
    46             for (int i=0; i<=4; ++i) {
     46            for (int i = 0; i <= 4; ++i) {
    4747                testProj(Projections.getProjectionByCode("EPSG:"+Integer.toString(2969+i))); // UTM France DOM
    4848            }
    4949        }
    5050
    51         for (int i=0; i<=8; ++i) {
     51        for (int i = 0; i <= 8; ++i) {
    5252            testProj(Projections.getProjectionByCode("EPSG:"+Integer.toString(3942+i))); // Lambert CC9 Zones France
    5353        }
     
    6565
    6666            text += String.format("*** %s %s%n", p.toString(), p.toCode());
    67             for (int num=0; num < 1000; ++num) {
     67            for (int num = 0; num < 1000; ++num) {
    6868
    6969                double lat = rand.nextDouble() * (b.getMax().lat() - b.getMin().lat()) + b.getMin().lat();
     
    7272                LatLon ll = new LatLon(lat, lon);
    7373
    74                 for (int i=0; i<10; ++i) {
     74                for (int i = 0; i < 10; ++i) {
    7575                    EastNorth en = p.latlon2eastNorth(ll);
    7676                    ll = p.eastNorth2latlon(en);
  • trunk/test/unit/org/openstreetmap/josm/data/projection/SwissGridTest.java

    r8450 r8510  
    2828     */
    2929    ProjData[] data = {
    30             new ProjData("Zimmerwald",      d(7,27,54.983506), d(46,52,37.540562), 947.149, 602030.680, 191775.030, 897.915),
    31             new ProjData("Chrischona",      d(7,40,6.983077), d(47,34, 1.385301), 504.935,  617306.300, 268507.300, 456.064),
    32             new ProjData("Pfaender",        d(9,47,3.697723), d(47,30,55.172797), 1089.372, 776668.105, 265372.681, 1042.624),
    33             new ProjData("La Givrine",      d(6,6,7.326361), d(46,27,14.690021), 1258.274,  497313.292, 145625.438, 1207.434),
    34             new ProjData("Monte Generoso",  d(9,1,16.389053), d(45,55,45.438020), 1685.027, 722758.810, 87649.670, 1636.600) };
     30            new ProjData("Zimmerwald",      d(7, 27, 54.983506), d(46, 52, 37.540562), 947.149, 602030.680, 191775.030, 897.915),
     31            new ProjData("Chrischona",      d(7, 40, 6.983077), d(47, 34, 1.385301), 504.935,  617306.300, 268507.300, 456.064),
     32            new ProjData("Pfaender",        d(9, 47, 3.697723), d(47, 30, 55.172797), 1089.372, 776668.105, 265372.681, 1042.624),
     33            new ProjData("La Givrine",      d(6, 6, 7.326361), d(46, 27, 14.690021), 1258.274,  497313.292, 145625.438, 1207.434),
     34            new ProjData("Monte Generoso",  d(9, 1, 16.389053), d(45, 55, 45.438020), 1685.027, 722758.810, 87649.670, 1636.600) };
    3535
    3636    private double d(double deg, double min, double sec) {
     
    4242        public LatLon ll;
    4343        public EastNorth en;
     44
    4445        public ProjData(String name, double lon, double lat, double h1, double x, double y, double h2) {
    4546            this.name = name;
     
    128129    }
    129130
    130 
    131131    @Test
    132132    public void b_eastNorth2latlon_test() {
  • trunk/test/unit/org/openstreetmap/josm/data/validation/tests/MultipolygonTestTest.java

    r7937 r8510  
    4242        nodes.add(new Node(new LatLon(0, 2)));
    4343
    44         Way w = (Way)OsmUtils.createPrimitive("way "+tags);
     44        Way w = (Way) OsmUtils.createPrimitive("way "+tags);
    4545        w.setNodes(nodes);
    4646        return w;
  • trunk/test/unit/org/openstreetmap/josm/gui/JosmUserIdentityManagerTest.groovy

    r7938 r8510  
    1212class JosmUserIdentityManagerTest {
    1313
    14         final shouldFail = new GroovyTestCase().&shouldFail
    15 
    16         @BeforeClass
    17         public static void initTestCase() {
    18             JOSMFixture.createUnitTestFixture().init()
    19         }
    20 
    21         @Test
    22         public void test_SingletonAccess() {
    23 
    24                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    25 
    26                 // created ?
    27                 assert im != null
    28 
    29                 // registered as listener ?
    30                 assert Main.pref.@listeners.contains(im)
    31 
    32                 JosmUserIdentityManager im2 = JosmUserIdentityManager.getInstance()
    33 
    34                 // only one instance
    35                 assert im == im2
    36         }
    37 
    38         @Test
    39         public void test_setAnonymouse() {
    40                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    41 
    42                 im.setPartiallyIdentified "test"
    43                 im.setAnonymous()
    44 
    45                 assert im.isAnonymous()
    46                 assert ! im.isPartiallyIdentified()
    47                 assert ! im.isFullyIdentified()
    48 
    49                 assert im.getUserId() == 0
    50                 assert im.getUserName() == null
    51                 assert im.getUserInfo() == null
    52         }
    53 
    54         @Test
    55         public void test_setPartiallyIdentified() {
    56                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    57 
    58                 im.setPartiallyIdentified "test"
    59 
    60                 shouldFail(IllegalArgumentException) {
    61                     im.setPartiallyIdentified null
    62                 }
    63 
    64                 shouldFail(IllegalArgumentException) {
    65                         im.setPartiallyIdentified ""
    66                 }
    67 
    68                 shouldFail(IllegalArgumentException) {
    69                         im.setPartiallyIdentified "  \t  "
    70                 }
    71 
    72                 im.setPartiallyIdentified "test"
    73 
    74                 assert ! im.isAnonymous()
    75                 assert im.isPartiallyIdentified()
    76                 assert ! im.isFullyIdentified()
    77 
    78                 assert im.getUserId() == 0
    79                 assert im.getUserName() == "test"
    80                 assert im.getUserInfo() == null
    81         }
    82 
    83 
    84         @Test
    85         public void test_setFullyIdentified() {
    86                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    87 
    88                 UserInfo userInfo = new UserInfo(id: 1, description: "a description")
    89 
    90                 im.setFullyIdentified "test", userInfo
    91 
    92                 shouldFail(IllegalArgumentException) {
    93                         im.setFullyIdentified null, userInfo
    94                 }
    95                 shouldFail(IllegalArgumentException) {
    96                         im.setFullyIdentified "", userInfo
    97                 }
    98                 shouldFail(IllegalArgumentException) {
    99                         im.setFullyIdentified " \t ", userInfo
    100                 }
    101                 shouldFail(IllegalArgumentException) {
    102                         im.setFullyIdentified "test", null
    103                 }
    104 
    105                 im.setFullyIdentified "test", userInfo
    106 
    107                 assert ! im.isAnonymous()
    108                 assert ! im.isPartiallyIdentified()
    109                 assert im.isFullyIdentified()
    110 
    111                 assert im.getUserId() == 1
    112                 assert im.getUserName() == "test"
    113                 assert im.getUserInfo() == userInfo
    114         }
    115 
    116         /**
    117          * Preferences include neither an url nor a user name => we have
    118          * an anonymous user
    119          */
    120         @Test
    121         public void initFromPreferences_1() {
    122                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    123 
    124                 // reset it
    125                 im.@userName = null
    126                 im.@userInfo = null
    127 
    128                 Main.pref.put "osm-server.url", null
    129                 Main.pref.put "osm-server.username", null
    130 
    131                 im.initFromPreferences()
    132 
    133                 assert im.isAnonymous()
    134         }
    135 
    136         /**
    137          * Preferences include neither an url nor a user name => we have
    138          * an annoymous user
    139          */
    140         @Test
    141         public void initFromPreferences_2() {
    142                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    143 
    144                 // reset it
    145                 im.@userName = null
    146                 im.@userInfo = null
    147 
    148                 // for this test we disable the listener
    149                 Main.pref.removePreferenceChangeListener im
    150 
    151                 Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
    152                 Main.pref.put "osm-server.username", null
    153 
    154                 im.initFromPreferences()
    155 
    156                 assert im.isAnonymous()
    157         }
    158 
    159         /**
    160          * Preferences include an user name => we have a partially identified user
    161          */
    162         @Test
    163         public void initFromPreferences_3() {
    164                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     14    final shouldFail = new GroovyTestCase().&shouldFail
     15
     16    @BeforeClass
     17    public static void initTestCase() {
     18        JOSMFixture.createUnitTestFixture().init()
     19    }
     20
     21    @Test
     22    public void test_SingletonAccess() {
     23
     24        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     25
     26        // created ?
     27        assert im != null
     28
     29        // registered as listener ?
     30        assert Main.pref.@listeners.contains(im)
     31
     32        JosmUserIdentityManager im2 = JosmUserIdentityManager.getInstance()
     33
     34        // only one instance
     35        assert im == im2
     36    }
     37
     38    @Test
     39    public void test_setAnonymouse() {
     40        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     41
     42        im.setPartiallyIdentified "test"
     43        im.setAnonymous()
     44
     45        assert im.isAnonymous()
     46        assert ! im.isPartiallyIdentified()
     47        assert ! im.isFullyIdentified()
     48
     49        assert im.getUserId() == 0
     50        assert im.getUserName() == null
     51        assert im.getUserInfo() == null
     52    }
     53
     54    @Test
     55    public void test_setPartiallyIdentified() {
     56        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     57
     58        im.setPartiallyIdentified "test"
     59
     60        shouldFail(IllegalArgumentException) {
     61            im.setPartiallyIdentified null
     62        }
     63
     64        shouldFail(IllegalArgumentException) {
     65            im.setPartiallyIdentified ""
     66        }
     67
     68        shouldFail(IllegalArgumentException) {
     69            im.setPartiallyIdentified "  \t  "
     70        }
     71
     72        im.setPartiallyIdentified "test"
     73
     74        assert ! im.isAnonymous()
     75        assert im.isPartiallyIdentified()
     76        assert ! im.isFullyIdentified()
     77
     78        assert im.getUserId() == 0
     79        assert im.getUserName() == "test"
     80        assert im.getUserInfo() == null
     81    }
     82
     83
     84    @Test
     85    public void test_setFullyIdentified() {
     86        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     87
     88        UserInfo userInfo = new UserInfo(id: 1, description: "a description")
     89
     90        im.setFullyIdentified "test", userInfo
     91
     92        shouldFail(IllegalArgumentException) {
     93            im.setFullyIdentified null, userInfo
     94        }
     95        shouldFail(IllegalArgumentException) {
     96            im.setFullyIdentified "", userInfo
     97        }
     98        shouldFail(IllegalArgumentException) {
     99            im.setFullyIdentified " \t ", userInfo
     100        }
     101        shouldFail(IllegalArgumentException) {
     102            im.setFullyIdentified "test", null
     103        }
     104
     105        im.setFullyIdentified "test", userInfo
     106
     107        assert ! im.isAnonymous()
     108        assert ! im.isPartiallyIdentified()
     109        assert im.isFullyIdentified()
     110
     111        assert im.getUserId() == 1
     112        assert im.getUserName() == "test"
     113        assert im.getUserInfo() == userInfo
     114    }
     115
     116    /**
     117     * Preferences include neither an url nor a user name => we have
     118     * an anonymous user
     119     */
     120    @Test
     121    public void initFromPreferences_1() {
     122        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     123
     124        // reset it
     125        im.@userName = null
     126        im.@userInfo = null
     127
     128        Main.pref.put "osm-server.url", null
     129        Main.pref.put "osm-server.username", null
     130
     131        im.initFromPreferences()
     132
     133        assert im.isAnonymous()
     134    }
     135
     136    /**
     137     * Preferences include neither an url nor a user name => we have
     138     * an annoymous user
     139     */
     140    @Test
     141    public void initFromPreferences_2() {
     142        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     143
     144        // reset it
     145        im.@userName = null
     146        im.@userInfo = null
    165147
    166148        // for this test we disable the listener
    167                 Main.pref.removePreferenceChangeListener im
    168 
    169                 // reset it
    170                 im.@userName = null
    171                 im.@userInfo = null
    172 
    173                 Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
    174                 Main.pref.put "osm-server.username", "test"
    175 
    176                 im.initFromPreferences()
    177 
    178                 assert im.isPartiallyIdentified()
    179         }
    180 
    181         /**
    182          * Preferences include an user name which is different from the current
    183          * user name and we are currently fully identifed => josm user becomes
    184          * partially identified
    185          */
    186         @Test
    187         public void initFromPreferences_4() {
    188                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     149        Main.pref.removePreferenceChangeListener im
     150
     151        Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
     152        Main.pref.put "osm-server.username", null
     153
     154        im.initFromPreferences()
     155
     156        assert im.isAnonymous()
     157    }
     158
     159    /**
     160     * Preferences include an user name => we have a partially identified user
     161     */
     162    @Test
     163    public void initFromPreferences_3() {
     164        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    189165
    190166        // for this test we disable the listener
    191                 Main.pref.removePreferenceChangeListener im
    192 
    193                 im.setFullyIdentified "test1", new UserInfo(id: 1)
    194 
    195                 Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
    196                 Main.pref.put "osm-server.username", "test2"
    197 
    198                 im.initFromPreferences()
    199 
    200                 assert im.isPartiallyIdentified()
    201         }
    202 
    203         /**
    204          * Preferences include an user name which is the same as the current
    205          * user name and we are currently fully identifed => josm user remains
    206          * fully identified
    207          */
    208         @Test
    209         public void initFromPreferences_5() {
    210                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     167        Main.pref.removePreferenceChangeListener im
     168
     169        // reset it
     170        im.@userName = null
     171        im.@userInfo = null
     172
     173        Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
     174        Main.pref.put "osm-server.username", "test"
     175
     176        im.initFromPreferences()
     177
     178        assert im.isPartiallyIdentified()
     179    }
     180
     181    /**
     182     * Preferences include an user name which is different from the current
     183     * user name and we are currently fully identifed => josm user becomes
     184     * partially identified
     185     */
     186    @Test
     187    public void initFromPreferences_4() {
     188        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    211189
    212190        // for this test we disable the listener
    213                 Main.pref.removePreferenceChangeListener im
    214 
    215                 im.setFullyIdentified "test1", new UserInfo(id: 1)
    216 
    217                 Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
    218                 Main.pref.put "osm-server.username", "test1"
    219 
    220                 im.initFromPreferences()
    221 
    222                 assert im.isFullyIdentified()
    223         }
    224 
    225         @Test
    226         public void apiUrlChanged() {
    227                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    228 
    229                 // make sure im is a preference change listener
    230                 Main.pref.addPreferenceChangeListener im
    231 
    232                 // reset it
    233                 im.@userName = null
    234                 im.@userInfo = null
    235 
    236                 Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
    237                 assert im.isAnonymous()
     191        Main.pref.removePreferenceChangeListener im
     192
     193        im.setFullyIdentified "test1", new UserInfo(id: 1)
     194
     195        Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
     196        Main.pref.put "osm-server.username", "test2"
     197
     198        im.initFromPreferences()
     199
     200        assert im.isPartiallyIdentified()
     201    }
     202
     203    /**
     204     * Preferences include an user name which is the same as the current
     205     * user name and we are currently fully identifed => josm user remains
     206     * fully identified
     207     */
     208    @Test
     209    public void initFromPreferences_5() {
     210        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     211
     212        // for this test we disable the listener
     213        Main.pref.removePreferenceChangeListener im
     214
     215        im.setFullyIdentified "test1", new UserInfo(id: 1)
     216
     217        Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
     218        Main.pref.put "osm-server.username", "test1"
     219
     220        im.initFromPreferences()
     221
     222        assert im.isFullyIdentified()
     223    }
     224
     225    @Test
     226    public void apiUrlChanged() {
     227        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     228
     229        // make sure im is a preference change listener
     230        Main.pref.addPreferenceChangeListener im
     231
     232        // reset it
     233        im.@userName = null
     234        im.@userInfo = null
     235
     236        Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
     237        assert im.isAnonymous()
    238238
    239239         Main.pref.put "osm-server.url", null
    240240         assert im.isAnonymous()
    241241
    242                 // reset it
    243                 im.@userName = "test"
    244                 im.@userInfo = null
    245 
    246                 Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
    247                 assert im.isPartiallyIdentified()
    248                 assert im.getUserName() == "test"
    249 
    250                 Main.pref.put "osm-server.url", null
    251                 assert im.isAnonymous()
    252 
    253                 // reset it
    254                 im.@userName = "test"
    255                 im.@userInfo = new UserInfo(id:1)
    256 
    257                 Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
    258                 assert im.isPartiallyIdentified()
    259                 assert im.getUserName() == "test"
    260 
    261                 // reset it
    262                 im.@userName = "test"
    263                 im.@userInfo = new UserInfo(id:1)
    264 
    265 
    266                 Main.pref.put "osm-server.url", null
    267                 assert im.isAnonymous()
    268         }
    269 
    270         @Test
    271         public void userNameChanged() {
    272                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
    273 
    274                 // make sure im is a preference change listener
    275                 Main.pref.addPreferenceChangeListener im
    276 
    277                 // reset it
    278                 im.@userName = null
    279                 im.@userInfo = null
    280 
    281                 Main.pref.put "osm-server.username", "test"
    282                 assert im.isPartiallyIdentified()
    283                 assert im.getUserName() == "test"
    284 
    285                 Main.pref.put "osm-server.username", null
    286                 assert im.isAnonymous()
    287 
    288                 // reset it
    289                 im.@userName = "test1"
    290                 im.@userInfo = null
    291 
    292                 Main.pref.put "osm-server.username", "test2"
    293                 assert im.isPartiallyIdentified()
    294                 assert im.getUserName() == "test2"
    295 
    296                 Main.pref.put "osm-server.username", null
    297                 assert im.isAnonymous()
    298 
    299                 // reset it
    300                 im.@userName = "test1"
    301                 im.@userInfo = new UserInfo(id:1)
    302 
    303                 Main.pref.put "osm-server.username", "test2"
    304                 assert im.isPartiallyIdentified()
    305                 assert im.getUserName() == "test2"
    306 
    307                 // reset it
    308                 im.@userName = "test1"
    309                 im.@userInfo = new UserInfo(id:1)
    310 
    311 
    312                 Main.pref.put "osm-server.username", null
    313                 assert im.isAnonymous()
    314         }
     242        // reset it
     243        im.@userName = "test"
     244        im.@userInfo = null
     245
     246        Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
     247        assert im.isPartiallyIdentified()
     248        assert im.getUserName() == "test"
     249
     250        Main.pref.put "osm-server.url", null
     251        assert im.isAnonymous()
     252
     253        // reset it
     254        im.@userName = "test"
     255        im.@userInfo = new UserInfo(id:1)
     256
     257        Main.pref.put "osm-server.url", "http://api.openstreetmap.org"
     258        assert im.isPartiallyIdentified()
     259        assert im.getUserName() == "test"
     260
     261        // reset it
     262        im.@userName = "test"
     263        im.@userInfo = new UserInfo(id:1)
     264
     265
     266        Main.pref.put "osm-server.url", null
     267        assert im.isAnonymous()
     268    }
     269
     270    @Test
     271    public void userNameChanged() {
     272        JosmUserIdentityManager im = JosmUserIdentityManager.getInstance()
     273
     274        // make sure im is a preference change listener
     275        Main.pref.addPreferenceChangeListener im
     276
     277        // reset it
     278        im.@userName = null
     279        im.@userInfo = null
     280
     281        Main.pref.put "osm-server.username", "test"
     282        assert im.isPartiallyIdentified()
     283        assert im.getUserName() == "test"
     284
     285        Main.pref.put "osm-server.username", null
     286        assert im.isAnonymous()
     287
     288        // reset it
     289        im.@userName = "test1"
     290        im.@userInfo = null
     291
     292        Main.pref.put "osm-server.username", "test2"
     293        assert im.isPartiallyIdentified()
     294        assert im.getUserName() == "test2"
     295
     296        Main.pref.put "osm-server.username", null
     297        assert im.isAnonymous()
     298
     299        // reset it
     300        im.@userName = "test1"
     301        im.@userInfo = new UserInfo(id:1)
     302
     303        Main.pref.put "osm-server.username", "test2"
     304        assert im.isPartiallyIdentified()
     305        assert im.getUserName() == "test2"
     306
     307        // reset it
     308        im.@userName = "test1"
     309        im.@userInfo = new UserInfo(id:1)
     310
     311
     312        Main.pref.put "osm-server.username", null
     313        assert im.isAnonymous()
     314    }
    315315}
  • trunk/test/unit/org/openstreetmap/josm/gui/conflict/nodes/NodeListMergeModelTest.java

    r8450 r8510  
    5252    protected void ensureSelected(DefaultListSelectionModel model, Object... idx) {
    5353        if (idx == null) return;
    54         for (int i=0; i < idx.length; i++) {
     54        for (int i = 0; i < idx.length; i++) {
    5555            if (idx[i] instanceof Integer) {
    56                 int j = (Integer)idx[i];
     56                int j = (Integer) idx[i];
    5757                assertTrue("expected row " + j + " to be selected", model.isSelectedIndex(j));
    5858                break;
    5959            }
    6060            try {
    61                 int rows[] = (int[])idx[i];
     61                int rows[] = (int[]) idx[i];
    6262                if (rows.length != 2) {
    6363                    fail("illegal selection range. Either null or not length 2: " + Arrays.toString(rows));
     
    6969                    assertTrue("expected row " + j + " to be selected", model.isSelectedIndex(j));
    7070                }
    71             } catch(ClassCastException e) {
     71            } catch (ClassCastException e) {
    7272                fail("illegal selection range:" + idx[i]);
    7373            }
     
    9595        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    9696        ensureSelected(mergedSelection, 0);
    97 
    98     }
    99 
     97    }
    10098
    10199    @Test
     
    123121        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    124122        ensureSelected(mergedSelection, 0);
    125 
    126123    }
    127124
     
    167164        mergedNodes.add(new Node(1));
    168165
    169         model.copyMyToTop(new int[]{1,2}); // copy node 3 and 4
     166        model.copyMyToTop(new int[]{1, 2}); // copy node 3 and 4
    170167
    171168        mergedNodes = inspectNodeList(model, "Merged");
     
    176173
    177174        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    178         ensureSelected(mergedSelection, 0,1); // first two rows selected
    179     }
    180 
     175        ensureSelected(mergedSelection, 0, 1); // first two rows selected
     176    }
    181177
    182178    @Test
     
    269265        mergedNodes.add(new Node(1));
    270266
    271         model.copyMyToEnd(new int[]{1,2}); // copy node 3 and 4
     267        model.copyMyToEnd(new int[]{1, 2}); // copy node 3 and 4
    272268
    273269        mergedNodes = inspectNodeList(model, "Merged");
     
    279275
    280276        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    281         ensureSelected(mergedSelection, 1,2); // last two rows selected
     277        ensureSelected(mergedSelection, 1, 2); // last two rows selected
    282278    }
    283279
     
    313309    }
    314310
    315 
    316311    @Test
    317312    public void test_copyMyNodesBeforeCurrent_2() {
     
    329324        mergedNodes.add(new Node(12));
    330325
    331         model.copyMyBeforeCurrent(new int[]{0,1}, 0);
     326        model.copyMyBeforeCurrent(new int[]{0, 1}, 0);
    332327
    333328        assertEquals(5, mergedNodes.size());
     
    339334
    340335        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    341         ensureSelected(mergedSelection, 0,1); // position 0 and 1 selected
     336        ensureSelected(mergedSelection, 0, 1); // position 0 and 1 selected
    342337    }
    343338
     
    358353
    359354        try {
    360             model.copyMyBeforeCurrent(new int[]{0,1}, -1);
     355            model.copyMyBeforeCurrent(new int[]{0, 1}, -1);
    361356            fail("expected IllegalArgumentException");
    362         } catch(IllegalArgumentException e) {
     357        } catch (IllegalArgumentException e) {
    363358            // OK
    364359        }
    365360
    366361        try {
    367             model.copyMyBeforeCurrent(new int[]{0,1}, 4);
     362            model.copyMyBeforeCurrent(new int[]{0, 1}, 4);
    368363            fail("expected IllegalArgumentException");
    369         } catch(IllegalArgumentException e) {
     364        } catch (IllegalArgumentException e) {
    370365            // OK
    371366        }
     
    402397    }
    403398
    404 
    405399    @Test
    406400    public void test_copyMyNodesAfterCurrent_2() {
     
    419413        mergedNodes.add(new Node(12));
    420414
    421         model.copyMyAfterCurrent(new int[]{0,1}, 2);
     415        model.copyMyAfterCurrent(new int[]{0, 1}, 2);
    422416
    423417        assertEquals(5, mergedNodes.size());
     
    428422        assertEquals(2, mergedNodes.get(4).getId());  // copied node 2 at position 4
    429423
    430 
    431         DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    432         ensureSelected(mergedSelection, 3,4); // position 3,4 selected
     424        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
     425        ensureSelected(mergedSelection, 3, 4); // position 3,4 selected
    433426    }
    434427
     
    449442        mergedNodes.add(new Node(12));
    450443
    451         model.copyMyAfterCurrent(new int[]{0,2}, 0);
     444        model.copyMyAfterCurrent(new int[]{0, 2}, 0);
    452445
    453446        assertEquals(5, mergedNodes.size());
     
    459452
    460453        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    461         ensureSelected(mergedSelection, 1,2); // position 1,2 selected
     454        ensureSelected(mergedSelection, 1, 2); // position 1,2 selected
    462455    }
    463456
     
    509502        mergedNodes.add(new Node(14));
    510503
    511         model.moveUpMerged(new int[]{1,4});
     504        model.moveUpMerged(new int[]{1, 4});
    512505
    513506        assertEquals(5, mergedNodes.size());
     
    519512
    520513        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    521         ensureSelected(mergedSelection, 0,3); // position 0 and 3 selecte0
     514        ensureSelected(mergedSelection, 0, 3); // position 0 and 3 selecte0
    522515    }
    523516
     
    539532        mergedNodes.add(new Node(14));
    540533
    541         model.moveUpMerged(new int[]{1,2,3,4});
     534        model.moveUpMerged(new int[]{1, 2, 3, 4});
    542535
    543536        assertEquals(5, mergedNodes.size());
     
    549542
    550543        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    551         ensureSelected(mergedSelection, 0,1,2,3);
     544        ensureSelected(mergedSelection, 0, 1, 2, 3);
    552545    }
    553546
     
    598591        mergedNodes.add(new Node(14));
    599592
    600         model.moveDownMerged(new int[]{1,3});
     593        model.moveDownMerged(new int[]{1, 3});
    601594
    602595        assertEquals(5, mergedNodes.size());
     
    608601
    609602        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    610         ensureSelected(mergedSelection, 2,4);
     603        ensureSelected(mergedSelection, 2, 4);
    611604    }
    612605
     
    628621        mergedNodes.add(new Node(14));
    629622
    630         model.moveDownMerged(new int[]{1,2,3});
     623        model.moveDownMerged(new int[]{1, 2, 3});
    631624
    632625        assertEquals(5, mergedNodes.size());
     
    638631
    639632        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    640         ensureSelected(mergedSelection, 2,3,4);
     633        ensureSelected(mergedSelection, 2, 3, 4);
    641634    }
    642635
     
    712705            }
    713706        }
     707
    714708        MyListener listener = new MyListener();
    715709        model.addPropertyChangeListener(listener);
     
    722716        assertEquals(!oldValue, listener.events.get(0).getNewValue());
    723717    }
    724 
    725718}
  • trunk/test/unit/org/openstreetmap/josm/gui/conflict/properties/PropertiesMergeModelTest.java

    r7937 r8510  
    9494        assertFalse(model.hasCoordConflict());
    9595
    96         n1.setCoor(new LatLon(1,1));
     96        n1.setCoor(new LatLon(1, 1));
    9797        populate(n1, n2);
    9898        assertTrue(model.hasCoordConflict());
     
    100100
    101101        n1.cloneFrom(new Node(1));
    102         n2.setCoor(new LatLon(2,2));
     102        n2.setCoor(new LatLon(2, 2));
    103103        populate(n1, n2);
    104104        assertTrue(model.hasCoordConflict());
    105105
    106         n1.setCoor(new LatLon(1,1));
    107         n2.setCoor(new LatLon(2,2));
     106        n1.setCoor(new LatLon(1, 1));
     107        n2.setCoor(new LatLon(2, 2));
    108108        populate(n1, n2);
    109109        assertTrue(model.hasCoordConflict());
  • trunk/test/unit/org/openstreetmap/josm/gui/conflict/tags/TagMergeItemTest.java

    r8509 r8510  
    6868        assertEquals("key", item.getKey());
    6969        assertNull(item.getMyTagValue());
    70         assertEquals("theirvalue",item.getTheirTagValue());
     70        assertEquals("theirvalue", item.getTheirTagValue());
    7171        assertEquals(MergeDecisionType.UNDECIDED, item.getMergeDecision());
    7272    }
    73 
    7473
    7574    @Test
     
    8685            item.decide(null);
    8786            fail("expected IllegalArgumentException not thrown");
    88         } catch(IllegalArgumentException e) {
     87        } catch (IllegalArgumentException e) {
    8988            // OK
    9089        }
     
    104103        item.applyToMyPrimitive(n1);
    105104        assertEquals("myvalue", n1.get("key"));
    106 
    107105    }
    108106
     
    133131            item.applyToMyPrimitive(n1);
    134132            fail("expected IllegalStateException");
    135         } catch(IllegalStateException e) {
     133        } catch (IllegalStateException e) {
    136134            // OK
    137135        }
     
    145143            item.applyToMyPrimitive(null);
    146144            fail("expected IllegalArgumentException");
    147         } catch(IllegalArgumentException e) {
     145        } catch (IllegalArgumentException e) {
    148146            // OK
    149147        }
  • trunk/test/unit/org/openstreetmap/josm/gui/conflict/tags/TagMergeModelTest.java

    r8391 r8510  
    210210
    211211        TagMergeModel model = new TagMergeModel();
    212         for (int i=0; i < 10; i++) {
     212        for (int i = 0; i < 10; i++) {
    213213            model.addItem(new TagMergeItem("key-" + i, "myvalue-" + i, "theirvalue-" +i));
    214214        }
     
    222222
    223223        model.decide(new int[] {0, 3, 5}, MergeDecisionType.KEEP_MINE);
    224         for (int i = 0; i< 10; i++) {
     224        for (int i = 0; i < 10; i++) {
    225225            TagMergeItem item = list.get(i);
    226226            if (i == 0 || i == 3 || i == 5) {
  • trunk/test/unit/org/openstreetmap/josm/gui/dialogs/relation/sort/RelationSorterTest.java

    r7937 r8510  
    4343    private String[] getNames(List<RelationMember> members) {
    4444        String[] result = new String[members.size()];
    45         for (int i=0; i<result.length; i++) {
     45        for (int i = 0; i < result.length; i++) {
    4646            result[i] = members.get(i).getMember().get("name");
    4747        }
  • trunk/test/unit/org/openstreetmap/josm/gui/dialogs/relation/sort/WayConnectionTypeCalculatorTest.java

    r7937 r8510  
    4444    private String getConnections(List<WayConnectionType> connections) {
    4545        String[] result = new String[connections.size()];
    46         for (int i=0; i<result.length; i++) {
     46        for (int i = 0; i < result.length; i++) {
    4747            WayConnectionType wc = connections.get(i);
    4848
  • trunk/test/unit/org/openstreetmap/josm/gui/preferences/ToolbarPreferencesTest.java

    r8509 r8510  
    4444    private void checkAction(ActionDefinition a, Object... params) {
    4545        Map<String, Object> expected = new HashMap<>();
    46         for (int i=0; i<params.length; i+=2) {
    47             expected.put((String)params[i], params[i+1]);
     46        for (int i = 0; i < params.length; i += 2) {
     47            expected.put((String) params[i], params[i+1]);
    4848        }
    4949        Assert.assertEquals(expected, a.getParameters());
  • trunk/test/unit/org/openstreetmap/josm/gui/tagging/TaggingPresetReaderTest.java

    r7937 r8510  
    6767        String presetfile = "resource://data/defaultpresets.xml";
    6868        final Collection<TaggingPreset> presets = TaggingPresetReader.readAll(presetfile, true);
    69         Assert.assertTrue("Default presets are empty", presets.size()>0);
     69        Assert.assertTrue("Default presets are empty", presets.size() > 0);
    7070    }
    7171}
  • trunk/test/unit/org/openstreetmap/josm/io/ChangesetQueryUrlParserTest.groovy

    r7938 r8510  
    22package org.openstreetmap.josm.io;
    33
    4 
    5 import java.util.Calendar;
    6 import java.util.GregorianCalendar;
    7 import java.util.TimeZone;
    8 
    9 import org.junit.Test
    10 import org.openstreetmap.josm.io.ChangesetQuery.ChangesetQueryUrlException;
    11 import org.openstreetmap.josm.io.ChangesetQuery.ChangesetQueryUrlParser;
    12 
    13 
    14 import static org.junit.Assert.*;
     4import static org.junit.Assert.*
     5
     6import org.junit.Test
     7import org.openstreetmap.josm.io.ChangesetQuery.ChangesetQueryUrlException
     8import org.openstreetmap.josm.io.ChangesetQuery.ChangesetQueryUrlParser
    159
    1610class ChangesetQueryUrlParserTest {
    17         final shouldFail = new GroovyTestCase().&shouldFail
    18        
    19         @Test
    20         public void test_constructor() {
    21             ChangesetQueryUrlParser parser = new ChangesetQueryUrlParser();
    22         }
    23        
    24         @Test
    25         public void test_parse_basic() {
    26                 ChangesetQueryUrlParser parser = new ChangesetQueryUrlParser();
    27                
    28                 // OK
    29                 parser.parse ""
    30                
    31                 // should be OK
    32                 ChangesetQuery q = parser.parse(null)
    33                 assert q != null
    34                
    35                 // should be OK
    36                 q = parser.parse("")
    37                 assert q != null
    38         }
    39        
    40         @Test
    41         public void test_uid() {
    42                 ChangesetQueryUrlParser parser = new ChangesetQueryUrlParser();
    43                 def ChangesetQuery q
    44                
    45                 // OK
    46                 q = parser.parse("uid=1234")
    47                 assert q != null
    48                
    49                 shouldFail(ChangesetQueryUrlException) {
    50                         q = parser.parse("uid=0")                               
    51                 }
    52                
    53                 shouldFail(ChangesetQueryUrlException) {
    54                         q = parser.parse("uid=-1")               
    55                 }
    56                
    57                 shouldFail(ChangesetQueryUrlException) {
    58                         q = parser.parse("uid=abc")               
    59                 }
    60         }
    61        
    62         @Test
    63         public void test_display_name() {
    64                 ChangesetQueryUrlParser parser = new ChangesetQueryUrlParser();
    65                 def ChangesetQuery q
    66                
    67                 // OK
    68                 q = parser.parse("display_name=abcd")
    69                 assert q != null
    70                 assert q.@userName == "abcd"
    71         }
    72        
    73        
    74         @Test
    75         public void test_open() {
    76                 ChangesetQueryUrlParser parser = new ChangesetQueryUrlParser();
    77                 def ChangesetQuery q
    78                
    79                 // OK
    80                 q = parser.parse("open=true")
    81                 assert q != null
    82                 assert q.@open == true
    83                
    84                 // OK
    85                 q = parser.parse("open=false")
    86                 assert q != null
    87                 assert q.@open == false
    88                
    89                 // illegal value for open
    90                 shouldFail(ChangesetQueryUrlException) {
    91                         q = parser.parse("open=abcd")               
    92                 }               
    93         }
    94        
    95         @Test
    96         public void test_closed() {
    97                 ChangesetQueryUrlParser parser = new ChangesetQueryUrlParser();
    98                 def ChangesetQuery q
    99                
    100                 // OK
    101                 q = parser.parse("closed=true")
    102                 assert q != null
    103                 assert q.@closed == true
    104                
    105                 // OK
    106                 q = parser.parse("closed=false")
    107                 assert q != null
    108                 assert q.@closed == false
    109                
    110                 // illegal value for open
    111                 shouldFail(ChangesetQueryUrlException) {
    112                         q = parser.parse("closed=abcd")               
    113                 }           
    114         }
    115        
    116        
    117         @Test
    118         public void test_uid_and_display_name() {
    119                 ChangesetQueryUrlParser parser = new ChangesetQueryUrlParser();
    120                 def ChangesetQuery q
    121                
    122                 // we can't have both an uid and a display name
    123                 shouldFail(ChangesetQueryUrlException) {
    124                         q = parser.parse("uid=1&display_name=abcd")               
    125                 }               
    126         }
    127        
    128         @Test
    129         public void test_time() {
    130                 ChangesetQueryUrlParser parser = new ChangesetQueryUrlParser();
    131                 def ChangesetQuery q
    132                
    133                 // OK
    134                 q = parser.parse("time=2009-12-25T10:00:00Z")
    135                 assert q != null
    136                 assert q.@closedAfter != null   
    137                 Calendar cal = new GregorianCalendar(TimeZone.getTimeZone("GMT+0"));
    138                 cal.setTime(q.@closedAfter);
    139                 assert cal.get(Calendar.YEAR) == 2009
    140                 assert cal.get(Calendar.MONTH) == 11 // calendar is 0-based
    141                 assert cal.get(Calendar.DAY_OF_MONTH) == 25
    142                 assert cal.get(Calendar.HOUR_OF_DAY) == 10
    143                 assert cal.get(Calendar.MINUTE) == 0
    144                 assert cal.get(Calendar.SECOND) == 0
    145                
    146                 // OK
    147                 q = parser.parse("time=2009-12-25T10:00:00Z,2009-11-25T10:00:00Z")
    148                 assert q!= null
    149                 assert q.@closedAfter != null
    150                 assert q.@createdBefore != null
    151                
    152                 // should fail
    153                 shouldFail(ChangesetQueryUrlException) {
    154                         q = parser.parse("time=asdf")               
    155                 }               
    156         }
    157        
    158         @Test
    159         public void test_bbox() {
    160                 ChangesetQueryUrlParser parser = new ChangesetQueryUrlParser();
    161                 def ChangesetQuery q
    162                
    163                 // OK
    164                 q = parser.parse("bbox=-1,-1,1,1")
    165                 assert q != null
    166                 assert q.@bounds != null
    167                
    168                 // should fail
    169                 shouldFail(ChangesetQueryUrlException) {
    170                         q = parser.parse("bbox=-91,-1,1,1")               
    171                 }           
    172                                
    173                 // should fail
    174                 shouldFail(ChangesetQueryUrlException) {
    175                         q = parser.parse("bbox=-1,-181,1,1")               
    176                 }           
    177                
    178                 // should fail
    179                 shouldFail(ChangesetQueryUrlException) {
    180                         q = parser.parse("bbox=-1,-1,91,1")               
    181                 }           
    182                 // should fail
    183                 shouldFail(ChangesetQueryUrlException) {
    184                         q = parser.parse("bbox=-1,-1,1,181")               
    185                 } 
    186                 // should fail
    187                 shouldFail(ChangesetQueryUrlException) {
    188                         q = parser.parse("bbox=-1,-1,1")               
    189                 } 
    190         }
     11    final shouldFail = new GroovyTestCase().&shouldFail
     12
     13    @Test
     14    public void test_constructor() {
     15        ChangesetQueryUrlParser parser = new ChangesetQueryUrlParser();
     16    }
     17
     18    @Test
     19    public void test_parse_basic() {
     20        ChangesetQueryUrlParser parser = new ChangesetQueryUrlParser();
     21
     22        // OK
     23        parser.parse ""
     24
     25        // should be OK
     26        ChangesetQuery q = parser.parse(null)
     27        assert q != null
     28
     29        // should be OK
     30        q = parser.parse("")
     31        assert q != null
     32    }
     33
     34    @Test
     35    public void test_uid() {
     36        ChangesetQueryUrlParser parser = new ChangesetQueryUrlParser();
     37        def ChangesetQuery q
     38
     39        // OK
     40        q = parser.parse("uid=1234")
     41        assert q != null
     42
     43        shouldFail(ChangesetQueryUrlException) {
     44            q = parser.parse("uid=0")
     45        }
     46
     47        shouldFail(ChangesetQueryUrlException) {
     48            q = parser.parse("uid=-1")
     49        }
     50
     51        shouldFail(ChangesetQueryUrlException) {
     52            q = parser.parse("uid=abc")
     53        }
     54    }
     55
     56    @Test
     57    public void test_display_name() {
     58        ChangesetQueryUrlParser parser = new ChangesetQueryUrlParser();
     59        def ChangesetQuery q
     60
     61        // OK
     62        q = parser.parse("display_name=abcd")
     63        assert q != null
     64        assert q.@userName == "abcd"
     65    }
     66
     67
     68    @Test
     69    public void test_open() {
     70        ChangesetQueryUrlParser parser = new ChangesetQueryUrlParser();
     71        def ChangesetQuery q
     72
     73        // OK
     74        q = parser.parse("open=true")
     75        assert q != null
     76        assert q.@open == true
     77
     78        // OK
     79        q = parser.parse("open=false")
     80        assert q != null
     81        assert q.@open == false
     82
     83        // illegal value for open
     84        shouldFail(ChangesetQueryUrlException) {
     85            q = parser.parse("open=abcd")
     86        }
     87    }
     88
     89    @Test
     90    public void test_closed() {
     91        ChangesetQueryUrlParser parser = new ChangesetQueryUrlParser();
     92        def ChangesetQuery q
     93
     94        // OK
     95        q = parser.parse("closed=true")
     96        assert q != null
     97        assert q.@closed == true
     98
     99        // OK
     100        q = parser.parse("closed=false")
     101        assert q != null
     102        assert q.@closed == false
     103
     104        // illegal value for open
     105        shouldFail(ChangesetQueryUrlException) {
     106            q = parser.parse("closed=abcd")
     107        }
     108    }
     109
     110
     111    @Test
     112    public void test_uid_and_display_name() {
     113        ChangesetQueryUrlParser parser = new ChangesetQueryUrlParser();
     114        def ChangesetQuery q
     115
     116        // we can't have both an uid and a display name
     117        shouldFail(ChangesetQueryUrlException) {
     118            q = parser.parse("uid=1&display_name=abcd")
     119        }
     120    }
     121
     122    @Test
     123    public void test_time() {
     124        ChangesetQueryUrlParser parser = new ChangesetQueryUrlParser();
     125        def ChangesetQuery q
     126
     127        // OK
     128        q = parser.parse("time=2009-12-25T10:00:00Z")
     129        assert q != null
     130        assert q.@closedAfter != null
     131        Calendar cal = new GregorianCalendar(TimeZone.getTimeZone("GMT+0"));
     132        cal.setTime(q.@closedAfter);
     133        assert cal.get(Calendar.YEAR) == 2009
     134        assert cal.get(Calendar.MONTH) == 11 // calendar is 0-based
     135        assert cal.get(Calendar.DAY_OF_MONTH) == 25
     136        assert cal.get(Calendar.HOUR_OF_DAY) == 10
     137        assert cal.get(Calendar.MINUTE) == 0
     138        assert cal.get(Calendar.SECOND) == 0
     139
     140        // OK
     141        q = parser.parse("time=2009-12-25T10:00:00Z,2009-11-25T10:00:00Z")
     142        assert q!= null
     143        assert q.@closedAfter != null
     144        assert q.@createdBefore != null
     145
     146        // should fail
     147        shouldFail(ChangesetQueryUrlException) {
     148            q = parser.parse("time=asdf")
     149        }
     150    }
     151
     152    @Test
     153    public void test_bbox() {
     154        ChangesetQueryUrlParser parser = new ChangesetQueryUrlParser();
     155        def ChangesetQuery q
     156
     157        // OK
     158        q = parser.parse("bbox=-1,-1,1,1")
     159        assert q != null
     160        assert q.@bounds != null
     161
     162        // should fail
     163        shouldFail(ChangesetQueryUrlException) {
     164            q = parser.parse("bbox=-91,-1,1,1")
     165        }
     166
     167        // should fail
     168        shouldFail(ChangesetQueryUrlException) {
     169            q = parser.parse("bbox=-1,-181,1,1")
     170        }
     171
     172        // should fail
     173        shouldFail(ChangesetQueryUrlException) {
     174            q = parser.parse("bbox=-1,-1,91,1")
     175        }
     176        // should fail
     177        shouldFail(ChangesetQueryUrlException) {
     178            q = parser.parse("bbox=-1,-1,1,181")
     179        }
     180        // should fail
     181        shouldFail(ChangesetQueryUrlException) {
     182            q = parser.parse("bbox=-1,-1,1")
     183        }
     184    }
    191185
    192186    @Test
  • trunk/test/unit/org/openstreetmap/josm/io/DiffResultProcessorTest.groovy

    r8351 r8510  
    1717class DiffResultProcessorTest {
    1818
     19    @Test
     20    public void testConstructor() {
     21        Node n = new Node(1)
     22        // these calls should not fail
     23        //
     24        def DiffResultProcessor processor  = new DiffResultProcessor(null)
     25        processor  = new DiffResultProcessor([])
     26        processor  = new DiffResultProcessor([n])
     27    }
    1928
    20         @Test
    21         public void testConstructor() {
    22                 Node n = new Node(1)
    23                 // these calls should not fail
    24                 //
    25                 def DiffResultProcessor processor  = new DiffResultProcessor(null)
    26                 processor  = new DiffResultProcessor([])
    27                 processor  = new DiffResultProcessor([n])
    28         }
     29    @Test
     30    public void testParse_NOK_Cases() {
     31        def DiffResultProcessor processor  = new DiffResultProcessor([])
    2932
    30         @Test
    31         public void testParse_NOK_Cases() {
    32                 def DiffResultProcessor processor  = new DiffResultProcessor([])
     33        final shouldFail = new GroovyTestCase().&shouldFail
    3334
    34                 final shouldFail = new GroovyTestCase().&shouldFail
     35        shouldFail(IllegalArgumentException) {
     36            processor.parse null, NullProgressMonitor.INSTANCE
     37        }
    3538
    36                 shouldFail(IllegalArgumentException) {
    37                     processor.parse null, NullProgressMonitor.INSTANCE
    38                 }
     39        shouldFail(XmlParsingException) {
     40            processor.parse "", NullProgressMonitor.INSTANCE
     41        }
    3942
    40                 shouldFail(XmlParsingException) {
    41                         processor.parse "", NullProgressMonitor.INSTANCE
    42                 }
     43        shouldFail(XmlParsingException) {
     44            processor.parse "<x></x>", NullProgressMonitor.INSTANCE
     45        }
     46    }
    4347
    44                 shouldFail(XmlParsingException) {
    45                         processor.parse "<x></x>", NullProgressMonitor.INSTANCE
    46                 }
    47         }
    48 
    49         @Test
    50         public void testParse_OK_Cases() {
    51                 def DiffResultProcessor processor  = new DiffResultProcessor([])
    52                 String doc = """\
    53                 <diffResult version="0.6" generator="Test Data">
    54                     <node old_id="-1" new_id="1" new_version="1"/>
    55                     <way old_id="-2" new_id="2" new_version="1"/>
    56                     <relation old_id="-3" new_id="3" new_version="1"/>
    57                 </diffResult>
    58                 """
    59 
    60                 processor.parse doc, null
    61                 assert processor.@diffResults.size() == 3
    62                 SimplePrimitiveId id = new SimplePrimitiveId(-1, OsmPrimitiveType.NODE)
    63                 def entry = processor.@diffResults[id]
    64                 assert entry != null
    65                 assert entry.newId == 1
    66                 assert entry.newVersion == 1
    67 
    68                 id = new SimplePrimitiveId(-2, OsmPrimitiveType.WAY)
    69                 entry = processor.@diffResults[id]
    70                 assert entry != null
    71                 assert entry.newId == 2
    72                 assert entry.newVersion == 1
    73 
    74                 id = new SimplePrimitiveId(-3, OsmPrimitiveType.RELATION)
    75                 entry = processor.@diffResults[id]
    76                 assert entry != null
    77                 assert entry.newId == 3
    78                 assert entry.newVersion == 1
    79         }
    80 
    81         @Test
    82         public void testPostProcess_Invocation_Variants() {
    83                 def DiffResultProcessor processor  = new DiffResultProcessor([])
    84                 String doc = """\
     48    @Test
     49    public void testParse_OK_Cases() {
     50        def DiffResultProcessor processor  = new DiffResultProcessor([])
     51        String doc = """\
    8552        <diffResult version="0.6" generator="Test Data">
    8653            <node old_id="-1" new_id="1" new_version="1"/>
     
    9057        """
    9158
    92                 processor.parse doc, null
     59        processor.parse doc, null
     60        assert processor.@diffResults.size() == 3
     61        SimplePrimitiveId id = new SimplePrimitiveId(-1, OsmPrimitiveType.NODE)
     62        def entry = processor.@diffResults[id]
     63        assert entry != null
     64        assert entry.newId == 1
     65        assert entry.newVersion == 1
    9366
    94                 // should all be ok
    95                 //
    96                 processor.postProcess null, null
    97                 processor.postProcess null, NullProgressMonitor.INSTANCE
    98                 processor.postProcess new Changeset(1), null
    99                 processor.postProcess new Changeset(1), NullProgressMonitor.INSTANCE
    100         }
     67        id = new SimplePrimitiveId(-2, OsmPrimitiveType.WAY)
     68        entry = processor.@diffResults[id]
     69        assert entry != null
     70        assert entry.newId == 2
     71        assert entry.newVersion == 1
    10172
    102         @Test
    103         public void testPostProcess_OK() {
     73        id = new SimplePrimitiveId(-3, OsmPrimitiveType.RELATION)
     74        entry = processor.@diffResults[id]
     75        assert entry != null
     76        assert entry.newId == 3
     77        assert entry.newVersion == 1
     78    }
    10479
    105                 Node n = new Node()
    106                 Way w = new Way()
    107                 Relation r = new Relation()
     80    @Test
     81    public void testPostProcess_Invocation_Variants() {
     82        def DiffResultProcessor processor  = new DiffResultProcessor([])
     83        String doc = """\
     84        <diffResult version="0.6" generator="Test Data">
     85            <node old_id="-1" new_id="1" new_version="1"/>
     86            <way old_id="-2" new_id="2" new_version="1"/>
     87            <relation old_id="-3" new_id="3" new_version="1"/>
     88        </diffResult>
     89        """
    10890
    109                 String doc = """\
    110                 <diffResult version="0.6" generator="Test Data">
    111                     <node old_id="${n.uniqueId}" new_id="1" new_version="10"/>
    112                     <way old_id="${w.uniqueId}" new_id="2" new_version="11"/>
    113                     <relation old_id="${r.uniqueId}" new_id="3" new_version="12"/>
    114                 </diffResult>
    115                 """
     91        processor.parse doc, null
    11692
    117                 def DiffResultProcessor processor  = new DiffResultProcessor([n,w,r])
    118                 processor.parse doc, null
    119                 def processed = processor.postProcess(new Changeset(5), null)
    120                 assert processed.size() == 3
    121                 n = processed.find {it.uniqueId == 1}
    122                 assert n.changesetId == 5
    123                 assert n.version == 10
     93        // should all be ok
     94        //
     95        processor.postProcess null, null
     96        processor.postProcess null, NullProgressMonitor.INSTANCE
     97        processor.postProcess new Changeset(1), null
     98        processor.postProcess new Changeset(1), NullProgressMonitor.INSTANCE
     99    }
    124100
    125                 w = processed.find {it.uniqueId == 2}
    126                 assert w.changesetId == 5
    127                 assert w.version == 11
     101    @Test
     102    public void testPostProcess_OK() {
    128103
    129                 r = processed.find {it.uniqueId == 3}
    130                 assert r.changesetId == 5
    131                 assert r.version == 12
    132         }
     104        Node n = new Node()
     105        Way w = new Way()
     106        Relation r = new Relation()
    133107
    134         @Test
    135         public void testPostProcess_ForCreatedElement() {
     108        String doc = """\
     109            <diffResult version="0.6" generator="Test Data">
     110                <node old_id="${n.uniqueId}" new_id="1" new_version="10"/>
     111                <way old_id="${w.uniqueId}" new_id="2" new_version="11"/>
     112                <relation old_id="${r.uniqueId}" new_id="3" new_version="12"/>
     113            </diffResult>
     114            """
    136115
    137                 Node n = new Node()
    138                 String doc = """\
     116        def DiffResultProcessor processor  = new DiffResultProcessor([n,w,r])
     117        processor.parse doc, null
     118        def processed = processor.postProcess(new Changeset(5), null)
     119        assert processed.size() == 3
     120        n = processed.find {it.uniqueId == 1}
     121        assert n.changesetId == 5
     122        assert n.version == 10
     123
     124        w = processed.find {it.uniqueId == 2}
     125        assert w.changesetId == 5
     126        assert w.version == 11
     127
     128        r = processed.find {it.uniqueId == 3}
     129        assert r.changesetId == 5
     130        assert r.version == 12
     131    }
     132
     133    @Test
     134    public void testPostProcess_ForCreatedElement() {
     135
     136        Node n = new Node()
     137        String doc = """\
    139138            <diffResult version="0.6" generator="Test Data">
    140139                <node old_id="${n.uniqueId}" new_id="1" new_version="1"/>
     
    142141            """
    143142
    144                 def DiffResultProcessor processor  = new DiffResultProcessor([n])
    145                 processor.parse doc, null
    146                 def processed = processor.postProcess(new Changeset(5), null)
    147                 assert processed.size() == 1
    148                 n = processed.find {it.uniqueId == 1}
    149                 assert n.changesetId == 5
    150                 assert n.version == 1
    151         }
     143        def DiffResultProcessor processor  = new DiffResultProcessor([n])
     144        processor.parse doc, null
     145        def processed = processor.postProcess(new Changeset(5), null)
     146        assert processed.size() == 1
     147        n = processed.find {it.uniqueId == 1}
     148        assert n.changesetId == 5
     149        assert n.version == 1
     150    }
    152151
    153         @Test
    154         public void testPostProcess_ForModifiedElement() {
     152    @Test
     153    public void testPostProcess_ForModifiedElement() {
    155154
    156                 Node n = new Node(1)
    157                 n.coor = new LatLon(1,1)
    158                 n.setOsmId 1, 2
    159                 n.modified = true
    160                 String doc = """\
     155        Node n = new Node(1)
     156        n.coor = new LatLon(1,1)
     157        n.setOsmId 1, 2
     158        n.modified = true
     159        String doc = """\
    161160            <diffResult version="0.6" generator="Test Data">
    162161                <node old_id="${n.uniqueId}" new_id="${n.uniqueId}" new_version="3"/>
     
    164163            """
    165164
    166                 def DiffResultProcessor processor  = new DiffResultProcessor([n])
    167                 processor.parse doc, null
    168                 def processed = processor.postProcess(new Changeset(5), null)
    169                 assert processed.size() == 1
    170                 n = processed.find {it.uniqueId == 1}
    171                 assert n.changesetId == 5
    172                 assert n.version == 3
    173         }
     165        def DiffResultProcessor processor  = new DiffResultProcessor([n])
     166        processor.parse doc, null
     167        def processed = processor.postProcess(new Changeset(5), null)
     168        assert processed.size() == 1
     169        n = processed.find {it.uniqueId == 1}
     170        assert n.changesetId == 5
     171        assert n.version == 3
     172    }
    174173
    175         @Test
    176         public void testPostProcess_ForDeletedElement() {
     174    @Test
     175    public void testPostProcess_ForDeletedElement() {
    177176
    178                 Node n = new Node(1)
    179                 n.coor = new LatLon(1,1)
    180                 n.setOsmId 1, 2
    181                 n.deleted = true
    182                 String doc = """\
     177        Node n = new Node(1)
     178        n.coor = new LatLon(1,1)
     179        n.setOsmId 1, 2
     180        n.deleted = true
     181        String doc = """\
    183182            <diffResult version="0.6" generator="Test Data">
    184183                <node old_id="${n.uniqueId}"/>
     
    186185            """
    187186
    188                 def DiffResultProcessor processor  = new DiffResultProcessor([n])
    189                 processor.parse doc, null
    190                 def processed = processor.postProcess(new Changeset(5), null)
    191                 assert processed.size() == 1
    192                 n = processed.find {it.uniqueId == 1}
    193                 assert n.changesetId == 5
    194                 assert n.version == 2
    195         }
     187        def DiffResultProcessor processor  = new DiffResultProcessor([n])
     188        processor.parse doc, null
     189        def processed = processor.postProcess(new Changeset(5), null)
     190        assert processed.size() == 1
     191        n = processed.find {it.uniqueId == 1}
     192        assert n.changesetId == 5
     193        assert n.version == 2
     194    }
    196195}
  • trunk/test/unit/org/openstreetmap/josm/io/OsmChangeBuilderTest.groovy

    r7938 r8510  
    1010class OsmChangeBuilderTest {
    1111
    12         /**
    13         * Test various constructor invocations
    14         */
    15         @Test
    16         public void testConstructor() {
    17                 def Changeset cs = new Changeset(1)
    18                 // should not fail
    19                 OsmChangeBuilder builder = new OsmChangeBuilder(cs)
    20 
    21                 // should not fail either - null allowed
    22                 builder = new OsmChangeBuilder(null)
    23 
    24                 // should not fail
    25                 builder = new OsmChangeBuilder(cs, "0.5")
    26 
    27                 builder = new OsmChangeBuilder(cs, null)
    28 
    29                 builder = new OsmChangeBuilder(null, null)
    30         }
    31 
    32         /**
    33         * Test the sequence of method calls. Should throw IllegalStateException if
    34         * the protocol start(),append()*, finish() is violated.
    35         */
    36         @Test
    37         public void testSequenceOfMethodCalls() {
    38                 def Changeset cs = new Changeset(1)
    39                 OsmChangeBuilder builder = new OsmChangeBuilder(cs)
    40 
    41                 final shouldFail = new GroovyTestCase().&shouldFail
    42 
    43                 // should be OK
    44                 builder.start()
    45                 Node n = new Node(new LatLon(0,0))
    46                 builder.append n
    47                 builder.finish()
    48 
    49                 shouldFail(IllegalStateException) {
    50                         builder = new OsmChangeBuilder(cs)
    51                         builder.append n
    52                 }
    53 
    54                 shouldFail(IllegalStateException) {
    55                         builder = new OsmChangeBuilder(cs)
    56                         builder.append([n])
    57                 }
    58 
    59                 shouldFail(IllegalStateException) {
    60                         builder = new OsmChangeBuilder(cs)
    61                         builder.finish()
    62                 }
    63 
    64                 shouldFail(IllegalStateException) {
    65                         builder = new OsmChangeBuilder(cs)
    66                         builder.start()
    67                         builder.start()
    68                 }
    69         }
    70 
    71         @Test
    72         public void testDocumentWithNewNode() {
    73                 def Changeset cs = new Changeset(1)
    74                 OsmChangeBuilder builder = new OsmChangeBuilder(cs)
    75                 Node n = new Node(new LatLon(0,0))
    76 
    77                 builder.start()
    78                 builder.append n
    79                 builder.finish()
    80 
    81                 def doc = new XmlParser().parseText(builder.document)
    82                 assert doc.@version == "0.6"
    83                 assert doc.@generator == "JOSM"
    84                 assert doc.name() == "osmChange"
    85                 assert doc.children().size() == 1
    86                 def create = doc.create
    87                 assert create != null
    88 
    89                 assert create.size() == 1
    90                 def nodes = create[0].node
    91                 assert nodes.size() == 1
    92                 def node = nodes[0]
    93                 assert node.@id == n.uniqueId.toString()
    94                 assert node.@lat != null
    95                 assert node.@lon != null
    96                 assert node.@changeset == cs.id.toString()
    97         }
    98 
    99         /**
    100         * Test building a document with a modified node
    101         */
    102         @Test
    103         public void testDocumentWithModifiedNode() {
    104                 def Changeset cs = new Changeset(1)
    105                 OsmChangeBuilder builder = new OsmChangeBuilder(cs)
    106                 Node n = new Node(1)
    107                 n.coor = new LatLon(0,0)
    108                 n.incomplete = false
    109                 n.modified = true
    110 
    111                 builder.start()
    112                 builder.append n
    113                 builder.finish()
    114 
    115                 def doc = new XmlParser().parseText(builder.document)
    116                 assert doc.@version == "0.6"
    117                 assert doc.@generator == "JOSM"
    118                 assert doc.name() == "osmChange"
    119                 assert doc.children().size() == 1
    120                 def modify = doc.modify
    121                 assert modify != null
    122 
    123                 assert modify.size() == 1
    124                 def nodes = modify[0].node
    125                 assert nodes.size() == 1
    126                 def node = nodes[0]
    127                 assert node.@id == n.uniqueId.toString()
    128                 assert node.@lat != null
    129                 assert node.@lon != null
    130                 assert node.@changeset == cs.id.toString()
    131         }
    132 
    133         /**
    134         * Test building a document with a deleted node
    135         */
    136         @Test
    137         public void testDocumentWithDeletedNode() {
    138                 def Changeset cs = new Changeset(1)
    139                 OsmChangeBuilder builder = new OsmChangeBuilder(cs)
    140                 Node n = new Node(1)
    141                 n.coor = new LatLon(0,0)
    142                 n.incomplete = false
    143                 n.deleted = true
    144 
    145                 builder.start()
    146                 builder.append n
    147                 builder.finish()
    148 
    149                 def doc = new XmlParser().parseText(builder.document)
    150                 assert doc.@version == "0.6"
    151                 assert doc.@generator == "JOSM"
    152                 assert doc.name() == "osmChange"
    153                 assert doc.children().size() == 1
    154                 def delete = doc.delete
    155                 assert delete != null
    156 
    157                 assert delete.size() == 1
    158                 def nodes = delete[0].node
    159                 assert nodes.size() == 1
    160                 def node = nodes[0]
    161                 assert node.@id == n.uniqueId.toString()
    162                 assert node.@lat == null
    163                 assert node.@lon == null
    164                 assert node.@changeset == cs.id.toString()
    165         }
    166 
    167         /**
    168         * Test building a mixed document.
    169         *
    170         */
    171         @Test
    172         public void testMixed() {
    173                 def Changeset cs = new Changeset(1)
    174                 OsmChangeBuilder builder = new OsmChangeBuilder(cs)
    175                 Node n1 = new Node(1)
    176                 n1.coor = new LatLon(0,0)
    177                 n1.incomplete = false
    178                 n1.deleted = true
    179 
    180                 Node n2 = new Node(new LatLon(0,0))
    181 
    182                 Node n3 = new Node(2)
    183                 n3.coor = new LatLon(0,0)
    184                 n3.incomplete = false
    185                 n3.modified = true
    186 
    187                 builder.start()
    188                 builder.append([n1,n2,n3])
    189                 builder.finish()
    190 
    191                 def doc = new XmlParser().parseText(builder.document)
    192 
    193                 assert doc.children().size() == 3
    194                 assert doc.children()[0].name() == "delete"
    195                 assert doc.children()[1].name() == "create"
    196                 assert doc.children()[2].name() == "modify"
    197 
    198                 def node = doc.children()[0].node[0]
    199                 assert node.@id == n1.uniqueId.toString()
    200 
    201                 node = doc.children()[1].node[0]
    202                 assert node.@id == n2.uniqueId.toString()
    203 
    204                 node = doc.children()[2].node[0]
    205                 assert node.@id == n3.uniqueId.toString()
    206         }
     12    /**
     13    * Test various constructor invocations
     14    */
     15    @Test
     16    public void testConstructor() {
     17        def Changeset cs = new Changeset(1)
     18        // should not fail
     19        OsmChangeBuilder builder = new OsmChangeBuilder(cs)
     20
     21        // should not fail either - null allowed
     22        builder = new OsmChangeBuilder(null)
     23
     24        // should not fail
     25        builder = new OsmChangeBuilder(cs, "0.5")
     26
     27        builder = new OsmChangeBuilder(cs, null)
     28
     29        builder = new OsmChangeBuilder(null, null)
     30    }
     31
     32    /**
     33    * Test the sequence of method calls. Should throw IllegalStateException if
     34    * the protocol start(),append()*, finish() is violated.
     35    */
     36    @Test
     37    public void testSequenceOfMethodCalls() {
     38        def Changeset cs = new Changeset(1)
     39        OsmChangeBuilder builder = new OsmChangeBuilder(cs)
     40
     41        final shouldFail = new GroovyTestCase().&shouldFail
     42
     43        // should be OK
     44        builder.start()
     45        Node n = new Node(new LatLon(0,0))
     46        builder.append n
     47        builder.finish()
     48
     49        shouldFail(IllegalStateException) {
     50            builder = new OsmChangeBuilder(cs)
     51            builder.append n
     52        }
     53
     54        shouldFail(IllegalStateException) {
     55            builder = new OsmChangeBuilder(cs)
     56            builder.append([n])
     57        }
     58
     59        shouldFail(IllegalStateException) {
     60            builder = new OsmChangeBuilder(cs)
     61            builder.finish()
     62        }
     63
     64        shouldFail(IllegalStateException) {
     65            builder = new OsmChangeBuilder(cs)
     66            builder.start()
     67            builder.start()
     68        }
     69    }
     70
     71    @Test
     72    public void testDocumentWithNewNode() {
     73        def Changeset cs = new Changeset(1)
     74        OsmChangeBuilder builder = new OsmChangeBuilder(cs)
     75        Node n = new Node(new LatLon(0,0))
     76
     77        builder.start()
     78        builder.append n
     79        builder.finish()
     80
     81        def doc = new XmlParser().parseText(builder.document)
     82        assert doc.@version == "0.6"
     83        assert doc.@generator == "JOSM"
     84        assert doc.name() == "osmChange"
     85        assert doc.children().size() == 1
     86        def create = doc.create
     87        assert create != null
     88
     89        assert create.size() == 1
     90        def nodes = create[0].node
     91        assert nodes.size() == 1
     92        def node = nodes[0]
     93        assert node.@id == n.uniqueId.toString()
     94        assert node.@lat != null
     95        assert node.@lon != null
     96        assert node.@changeset == cs.id.toString()
     97    }
     98
     99    /**
     100    * Test building a document with a modified node
     101    */
     102    @Test
     103    public void testDocumentWithModifiedNode() {
     104        def Changeset cs = new Changeset(1)
     105        OsmChangeBuilder builder = new OsmChangeBuilder(cs)
     106        Node n = new Node(1)
     107        n.coor = new LatLon(0,0)
     108        n.incomplete = false
     109        n.modified = true
     110
     111        builder.start()
     112        builder.append n
     113        builder.finish()
     114
     115        def doc = new XmlParser().parseText(builder.document)
     116        assert doc.@version == "0.6"
     117        assert doc.@generator == "JOSM"
     118        assert doc.name() == "osmChange"
     119        assert doc.children().size() == 1
     120        def modify = doc.modify
     121        assert modify != null
     122
     123        assert modify.size() == 1
     124        def nodes = modify[0].node
     125        assert nodes.size() == 1
     126        def node = nodes[0]
     127        assert node.@id == n.uniqueId.toString()
     128        assert node.@lat != null
     129        assert node.@lon != null
     130        assert node.@changeset == cs.id.toString()
     131    }
     132
     133    /**
     134    * Test building a document with a deleted node
     135    */
     136    @Test
     137    public void testDocumentWithDeletedNode() {
     138        def Changeset cs = new Changeset(1)
     139        OsmChangeBuilder builder = new OsmChangeBuilder(cs)
     140        Node n = new Node(1)
     141        n.coor = new LatLon(0,0)
     142        n.incomplete = false
     143        n.deleted = true
     144
     145        builder.start()
     146        builder.append n
     147        builder.finish()
     148
     149        def doc = new XmlParser().parseText(builder.document)
     150        assert doc.@version == "0.6"
     151        assert doc.@generator == "JOSM"
     152        assert doc.name() == "osmChange"
     153        assert doc.children().size() == 1
     154        def delete = doc.delete
     155        assert delete != null
     156
     157        assert delete.size() == 1
     158        def nodes = delete[0].node
     159        assert nodes.size() == 1
     160        def node = nodes[0]
     161        assert node.@id == n.uniqueId.toString()
     162        assert node.@lat == null
     163        assert node.@lon == null
     164        assert node.@changeset == cs.id.toString()
     165    }
     166
     167    /**
     168    * Test building a mixed document.
     169    *
     170    */
     171    @Test
     172    public void testMixed() {
     173        def Changeset cs = new Changeset(1)
     174        OsmChangeBuilder builder = new OsmChangeBuilder(cs)
     175        Node n1 = new Node(1)
     176        n1.coor = new LatLon(0,0)
     177        n1.incomplete = false
     178        n1.deleted = true
     179
     180        Node n2 = new Node(new LatLon(0,0))
     181
     182        Node n3 = new Node(2)
     183        n3.coor = new LatLon(0,0)
     184        n3.incomplete = false
     185        n3.modified = true
     186
     187        builder.start()
     188        builder.append([n1,n2,n3])
     189        builder.finish()
     190
     191        def doc = new XmlParser().parseText(builder.document)
     192
     193        assert doc.children().size() == 3
     194        assert doc.children()[0].name() == "delete"
     195        assert doc.children()[1].name() == "create"
     196        assert doc.children()[2].name() == "modify"
     197
     198        def node = doc.children()[0].node[0]
     199        assert node.@id == n1.uniqueId.toString()
     200
     201        node = doc.children()[1].node[0]
     202        assert node.@id == n2.uniqueId.toString()
     203
     204        node = doc.children()[2].node[0]
     205        assert node.@id == n3.uniqueId.toString()
     206    }
    207207}
  • trunk/test/unit/org/openstreetmap/josm/io/OsmChangesetContentParserTest.groovy

    r7938 r8510  
    22package org.openstreetmap.josm.io;
    33
     4import java.nio.charset.StandardCharsets
     5
    46import org.junit.Test
    5 
    6 import java.nio.charset.StandardCharsets
    7 
    8 import org.openstreetmap.josm.data.osm.ChangesetDataSet;
    9 import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
    10 import org.openstreetmap.josm.data.osm.SimplePrimitiveId;
    11 import org.openstreetmap.josm.data.osm.ChangesetDataSet.ChangesetModificationType;
    12 import org.openstreetmap.josm.data.osm.history.HistoryOsmPrimitive;
    13 import org.openstreetmap.josm.data.osm.history.HistoryRelation;
    14 import org.openstreetmap.josm.data.osm.history.HistoryWay;
    15 import org.openstreetmap.josm.gui.progress.NullProgressMonitor;
    16 
     7import org.openstreetmap.josm.data.osm.ChangesetDataSet
     8import org.openstreetmap.josm.data.osm.OsmPrimitiveType
     9import org.openstreetmap.josm.data.osm.SimplePrimitiveId
     10import org.openstreetmap.josm.data.osm.ChangesetDataSet.ChangesetModificationType
     11import org.openstreetmap.josm.data.osm.history.HistoryOsmPrimitive
     12import org.openstreetmap.josm.data.osm.history.HistoryRelation
     13import org.openstreetmap.josm.data.osm.history.HistoryWay
     14import org.openstreetmap.josm.gui.progress.NullProgressMonitor
    1715
    1816class OsmChangesetContentParserTest {
    19         final shouldFail = new GroovyTestCase().&shouldFail
    20 
    21         @Test
    22         public void test_Constructor() {
    23             OsmChangesetContentParser parser
    24                
    25                 // should be OK
    26                 parser = new OsmChangesetContentParser(new ByteArrayInputStream("".bytes))
     17    final shouldFail = new GroovyTestCase().&shouldFail
     18
     19    @Test
     20    public void test_Constructor() {
     21        OsmChangesetContentParser parser
     22
     23        // should be OK
     24        parser = new OsmChangesetContentParser(new ByteArrayInputStream("".bytes))
    2725
    2826        shouldFail(IllegalArgumentException) {
     
    3331            parser = new OsmChangesetContentParser((InputStream) null)
    3432        }
    35         }
    36        
    37        
    38         @Test
    39         public void test_parse_arguments() {
    40                 OsmChangesetContentParser parser
    41                
    42                 def String doc = """
    43                     <osmChange version="0.6" generator="OpenStreetMap server">
    44                     </osmChange>
    45                 """
    46                
    47                 // should be OK
    48                 parser = new OsmChangesetContentParser(new ByteArrayInputStream(doc.getBytes(StandardCharsets.UTF_8)))
    49                 parser.parse null
    50                
    51                 // should be OK
    52                 parser = new OsmChangesetContentParser(new ByteArrayInputStream(doc.getBytes(StandardCharsets.UTF_8)))
    53                 parser.parse NullProgressMonitor.INSTANCE
    54                
    55                 // should be OK
    56                 parser = new OsmChangesetContentParser(doc)     
    57                 parser.parse null               
    58         }       
    59        
    60         /**
    61          * A simple changeset content document with one created node
    62          *
    63         */
    64         @Test
    65         public void test_OK_OneCreatedNode() {
    66                 OsmChangesetContentParser parser
    67                
    68                 def String doc = """
    69             <osmChange version="0.6" generator="OpenStreetMap server">
    70                       <create>
    71                         <node id="1" version="1" visible="true" changeset="1" lat="1.0" lon="1.0" timestamp="2009-12-22" />
    72                       </create>
    73             </osmChange>
    74         """
    75                
    76                 // should be OK
    77                 parser = new OsmChangesetContentParser(doc)
    78                 ChangesetDataSet ds = parser.parse()
    79 
    80         assert ds.size() == 1           
    81                 HistoryOsmPrimitive p = ds.getPrimitive(new SimplePrimitiveId(1, OsmPrimitiveType.NODE));
    82                 assert p != null
    83                 assert p.getId() == 1
    84                 assert p.getVersion() == 1
    85                 assert p.getChangesetId() == 1
    86                 assert p.getTimestamp() != null
    87                 assert ds.getModificationType(p.getPrimitiveId()) == ChangesetModificationType.CREATED
    88                 assert ds.isCreated(p.getPrimitiveId())
    89         }
    90        
    91         /**
    92          * A simple changeset content document with one updated node
    93          *
    94         */
    95         @Test
    96         public void test_OK_OneUpdatedNode() {
    97                 OsmChangesetContentParser parser
    98                
    99                 def String doc = """
     33    }
     34
     35
     36    @Test
     37    public void test_parse_arguments() {
     38        OsmChangesetContentParser parser
     39
     40        def String doc = """
     41            <osmChange version="0.6" generator="OpenStreetMap server">
     42            </osmChange>
     43        """
     44
     45        // should be OK
     46        parser = new OsmChangesetContentParser(new ByteArrayInputStream(doc.getBytes(StandardCharsets.UTF_8)))
     47        parser.parse null
     48
     49        // should be OK
     50        parser = new OsmChangesetContentParser(new ByteArrayInputStream(doc.getBytes(StandardCharsets.UTF_8)))
     51        parser.parse NullProgressMonitor.INSTANCE
     52
     53        // should be OK
     54        parser = new OsmChangesetContentParser(doc)
     55        parser.parse null
     56    }
     57
     58    /**
     59     * A simple changeset content document with one created node
     60     *
     61    */
     62    @Test
     63    public void test_OK_OneCreatedNode() {
     64        OsmChangesetContentParser parser
     65
     66        def String doc = """
     67            <osmChange version="0.6" generator="OpenStreetMap server">
     68              <create>
     69                <node id="1" version="1" visible="true" changeset="1" lat="1.0" lon="1.0" timestamp="2009-12-22" />
     70              </create>
     71            </osmChange>
     72        """
     73
     74        // should be OK
     75        parser = new OsmChangesetContentParser(doc)
     76        ChangesetDataSet ds = parser.parse()
     77
     78        assert ds.size() == 1
     79        HistoryOsmPrimitive p = ds.getPrimitive(new SimplePrimitiveId(1, OsmPrimitiveType.NODE));
     80        assert p != null
     81        assert p.getId() == 1
     82        assert p.getVersion() == 1
     83        assert p.getChangesetId() == 1
     84        assert p.getTimestamp() != null
     85        assert ds.getModificationType(p.getPrimitiveId()) == ChangesetModificationType.CREATED
     86        assert ds.isCreated(p.getPrimitiveId())
     87    }
     88
     89    /**
     90     * A simple changeset content document with one updated node
     91     *
     92    */
     93    @Test
     94    public void test_OK_OneUpdatedNode() {
     95        OsmChangesetContentParser parser
     96
     97        def String doc = """
    10098            <osmChange version="0.6" generator="OpenStreetMap server">
    10199              <modify>
     
    104102            </osmChange>
    105103        """
    106                
    107                 // should be OK
    108                 parser = new OsmChangesetContentParser(doc)
    109                 ChangesetDataSet ds = parser.parse()
    110                
    111                 assert ds.size() == 1       
    112                 HistoryOsmPrimitive p = ds.getPrimitive(new SimplePrimitiveId(1, OsmPrimitiveType.NODE));
    113                 assert p != null
    114                 assert p.getId() == 1
    115                 assert p.getVersion() == 1
    116                 assert p.getChangesetId() == 1
    117                 assert p.getTimestamp() != null
    118                 assert ds.getModificationType(p.getPrimitiveId()) == ChangesetModificationType.UPDATED
    119                 assert ds.isUpdated(p.getPrimitiveId())
    120         }
    121        
    122         /**
    123          * A simple changeset content document with one deleted node
    124          *
    125         */
    126         @Test
    127         public void test_OK_OneDeletedNode() {
    128                 OsmChangesetContentParser parser
    129                
    130                 def String doc = """
     104
     105        // should be OK
     106        parser = new OsmChangesetContentParser(doc)
     107        ChangesetDataSet ds = parser.parse()
     108
     109        assert ds.size() == 1
     110        HistoryOsmPrimitive p = ds.getPrimitive(new SimplePrimitiveId(1, OsmPrimitiveType.NODE));
     111        assert p != null
     112        assert p.getId() == 1
     113        assert p.getVersion() == 1
     114        assert p.getChangesetId() == 1
     115        assert p.getTimestamp() != null
     116        assert ds.getModificationType(p.getPrimitiveId()) == ChangesetModificationType.UPDATED
     117        assert ds.isUpdated(p.getPrimitiveId())
     118    }
     119
     120    /**
     121     * A simple changeset content document with one deleted node
     122     *
     123    */
     124    @Test
     125    public void test_OK_OneDeletedNode() {
     126        OsmChangesetContentParser parser
     127
     128        def String doc = """
    131129            <osmChange version="0.6" generator="OpenStreetMap server">
    132130              <delete>
     
    135133            </osmChange>
    136134        """
    137                
    138                 // should be OK
    139                 parser = new OsmChangesetContentParser(doc)
    140                 ChangesetDataSet ds = parser.parse()
    141                
    142                 assert ds.size() == 1       
    143                 HistoryOsmPrimitive p = ds.getPrimitive(new SimplePrimitiveId(1, OsmPrimitiveType.NODE));
    144                 assert p != null
    145                 assert p.getId() == 1
    146                 assert p.getVersion() == 1
    147                 assert p.getChangesetId() == 1
    148                 assert p.getTimestamp() != null
    149                 assert ds.getModificationType(p.getPrimitiveId()) == ChangesetModificationType.DELETED
    150                 assert ds.isDeleted(p.getPrimitiveId())
    151         }
    152        
    153         /**
    154          * A more complex test with a document including nodes, ways, and relations.
    155          *
    156         */
    157         @Test
    158         public void test_OK_ComplexTestCase() {
    159                 OsmChangesetContentParser parser
    160                
    161                 def String doc = """
     135
     136        // should be OK
     137        parser = new OsmChangesetContentParser(doc)
     138        ChangesetDataSet ds = parser.parse()
     139
     140        assert ds.size() == 1
     141        HistoryOsmPrimitive p = ds.getPrimitive(new SimplePrimitiveId(1, OsmPrimitiveType.NODE));
     142        assert p != null
     143        assert p.getId() == 1
     144        assert p.getVersion() == 1
     145        assert p.getChangesetId() == 1
     146        assert p.getTimestamp() != null
     147        assert ds.getModificationType(p.getPrimitiveId()) == ChangesetModificationType.DELETED
     148        assert ds.isDeleted(p.getPrimitiveId())
     149    }
     150
     151    /**
     152     * A more complex test with a document including nodes, ways, and relations.
     153     *
     154    */
     155    @Test
     156    public void test_OK_ComplexTestCase() {
     157        OsmChangesetContentParser parser
     158
     159        def String doc = """
    162160            <osmChange version="0.6" generator="OpenStreetMap server">
    163161              <create>
    164162                <node id="1" version="1" visible="true" changeset="1" lat="1.0" lon="1.0" timestamp="2009-12-22">
    165                           <tag k="a.key" v="a.value" />
    166                         </node>
     163                  <tag k="a.key" v="a.value" />
     164                </node>
    167165              </create>
    168166              <modify>
    169167               <way id="2" version="2" visible="true" changeset="1" timestamp="2009-12-22">
    170                           <nd ref="21"/>
    171                           <nd ref="22"/>
    172                        </way>
     168                  <nd ref="21"/>
     169                  <nd ref="22"/>
     170               </way>
    173171             </modify>
    174                      <delete>
     172             <delete>
    175173                <relation id="3" version="3" visible="true" changeset="1" timestamp="2009-12-22" />
    176174              </delete>
    177175            </osmChange>
    178176        """
    179                
    180                 // should be OK
    181                 parser = new OsmChangesetContentParser(doc)
    182                 ChangesetDataSet ds = parser.parse()
    183                
    184                 assert ds.size() == 3       
    185                
    186                 HistoryOsmPrimitive p = ds.getPrimitive(new SimplePrimitiveId(1, OsmPrimitiveType.NODE));
    187                 assert p != null
    188                 assert p.getId() == 1
    189                 assert p.getVersion() == 1
    190                 assert p.getChangesetId() == 1
    191                 assert p.getTimestamp() != null
    192                 assert ds.getModificationType(p.getPrimitiveId()) == ChangesetModificationType.CREATED
    193                 assert ds.isCreated(p.getPrimitiveId())
    194                 assert p.get("a.key") == "a.value"
    195                
    196                 HistoryWay w = (HistoryWay)ds.getPrimitive(new SimplePrimitiveId(2, OsmPrimitiveType.WAY));
    197                 assert w != null
    198                 assert w.getId() == 2
    199                 assert w.getVersion() == 2
    200                 assert w.getChangesetId() == 1
    201                 assert w.getTimestamp() != null
    202                 assert ds.getModificationType(w.getPrimitiveId()) == ChangesetModificationType.UPDATED
    203                 assert ds.isUpdated(w.getPrimitiveId())
    204                 assert w.getNumNodes() == 2
    205                 assert w.getNodes() == [21,22]
    206                
    207                 HistoryRelation r = (HistoryRelation)ds.getPrimitive(new SimplePrimitiveId(3, OsmPrimitiveType.RELATION));
    208                 assert r != null
    209                 assert r.getId() == 3
    210                 assert r.getVersion() == 3
    211                 assert r.getChangesetId() == 1
    212                 assert r.getTimestamp() != null
    213                 assert ds.getModificationType(r.getPrimitiveId()) == ChangesetModificationType.DELETED
    214                 assert ds.isDeleted(r.getPrimitiveId())
    215         }
     177
     178        // should be OK
     179        parser = new OsmChangesetContentParser(doc)
     180        ChangesetDataSet ds = parser.parse()
     181
     182        assert ds.size() == 3
     183
     184        HistoryOsmPrimitive p = ds.getPrimitive(new SimplePrimitiveId(1, OsmPrimitiveType.NODE));
     185        assert p != null
     186        assert p.getId() == 1
     187        assert p.getVersion() == 1
     188        assert p.getChangesetId() == 1
     189        assert p.getTimestamp() != null
     190        assert ds.getModificationType(p.getPrimitiveId()) == ChangesetModificationType.CREATED
     191        assert ds.isCreated(p.getPrimitiveId())
     192        assert p.get("a.key") == "a.value"
     193
     194        HistoryWay w = (HistoryWay)ds.getPrimitive(new SimplePrimitiveId(2, OsmPrimitiveType.WAY));
     195        assert w != null
     196        assert w.getId() == 2
     197        assert w.getVersion() == 2
     198        assert w.getChangesetId() == 1
     199        assert w.getTimestamp() != null
     200        assert ds.getModificationType(w.getPrimitiveId()) == ChangesetModificationType.UPDATED
     201        assert ds.isUpdated(w.getPrimitiveId())
     202        assert w.getNumNodes() == 2
     203        assert w.getNodes() == [21,22]
     204
     205        HistoryRelation r = (HistoryRelation)ds.getPrimitive(new SimplePrimitiveId(3, OsmPrimitiveType.RELATION));
     206        assert r != null
     207        assert r.getId() == 3
     208        assert r.getVersion() == 3
     209        assert r.getChangesetId() == 1
     210        assert r.getTimestamp() != null
     211        assert ds.getModificationType(r.getPrimitiveId()) == ChangesetModificationType.DELETED
     212        assert ds.isDeleted(r.getPrimitiveId())
     213    }
    216214}
  • trunk/test/unit/org/openstreetmap/josm/io/ParseWithChangesetReaderTest.groovy

    r7938 r8510  
    2020    }
    2121
    22         /**
    23         * A new node with a changeset id. Ignore it.
    24         */
    25         @Test
    26         public void test_1() {
    27                 String doc = """\
     22    /**
     23    * A new node with a changeset id. Ignore it.
     24    */
     25    @Test
     26    public void test_1() {
     27        String doc = """\
    2828         <osm version="0.6">
    2929        <node id="-1" lat="0.0" lon="0.0" changeset="1">
     
    3333        """
    3434
    35                 DataSet ds = getDataSet(doc)
    36         Node n = ds.nodes.find {it.get("external-id") == "-1"}
    37                 assert n != null
    38                 assert n.changesetId == 0
    39         }
    40 
    41         /**
    42         * A new node with an invalid changeset id. Ignore it.
    43         */
    44         @Test
    45         public void test_11() {
    46                 String doc = """\
     35        DataSet ds = getDataSet(doc)
     36        Node n = ds.nodes.find {it.get("external-id") == "-1"}
     37        assert n != null
     38        assert n.changesetId == 0
     39    }
     40
     41    /**
     42    * A new node with an invalid changeset id. Ignore it.
     43    */
     44    @Test
     45    public void test_11() {
     46        String doc = """\
    4747         <osm version="0.6">
    4848        <node id="-1" lat="0.0" lon="0.0" changeset="0">
    49                     <tag k="external-id" v="-1"/>
    50                 </node>
    51         </osm>
    52         """
    53 
    54         DataSet ds = getDataSet(doc)
    55                 Node n = ds.nodes.find {it.get("external-id") == "-1"}
    56                 assert n != null
    57                 assert n.changesetId == 0
    58         }
    59 
    60         /**
    61         * A new node with an invalid changeset id. Ignore it.
    62         */
    63         @Test
    64         public void test_12() {
    65                 String doc = """\
     49            <tag k="external-id" v="-1"/>
     50        </node>
     51        </osm>
     52        """
     53
     54        DataSet ds = getDataSet(doc)
     55        Node n = ds.nodes.find {it.get("external-id") == "-1"}
     56        assert n != null
     57        assert n.changesetId == 0
     58    }
     59
     60    /**
     61    * A new node with an invalid changeset id. Ignore it.
     62    */
     63    @Test
     64    public void test_12() {
     65        String doc = """\
    6666         <osm version="0.6">
    6767        <node id="-1" lat="0.0" lon="0.0" changeset="-1">
    68                     <tag k="external-id" v="-1"/>
    69         </node>
    70         </osm>
    71         """
    72 
    73         DataSet ds = getDataSet(doc)
    74                 Node n = ds.nodes.find {it.get("external-id") == "-1"}
    75                 assert n != null
    76                 assert n.changesetId == 0
    77         }
    78 
    79         /**
    80         * A new node with an invalid changeset id. Ignore it.
    81         */
    82         @Test
    83         public void test_13() {
    84                 String doc = """\
     68            <tag k="external-id" v="-1"/>
     69        </node>
     70        </osm>
     71        """
     72
     73        DataSet ds = getDataSet(doc)
     74        Node n = ds.nodes.find {it.get("external-id") == "-1"}
     75        assert n != null
     76        assert n.changesetId == 0
     77    }
     78
     79    /**
     80    * A new node with an invalid changeset id. Ignore it.
     81    */
     82    @Test
     83    public void test_13() {
     84        String doc = """\
    8585         <osm version="0.6">
    8686        <node id="-1" lat="0.0" lon="0.0" changeset="aaa">
    87                     <tag k="external-id" v="-1"/>
    88         </node>
    89         </osm>
    90         """
    91 
    92         DataSet ds = getDataSet(doc)
    93                 Node n = ds.nodes.find {it.get("external-id") == "-1"}
    94                 assert n != null
    95                 assert n.changesetId == 0
    96         }
    97 
    98         /**
    99         * A new node with a missing changeset id. That's fine. The changeset id
    100         * is reset to 0.
    101         */
    102         @Test
    103         public void test_14() {
    104                 String doc = """\
     87            <tag k="external-id" v="-1"/>
     88        </node>
     89        </osm>
     90        """
     91
     92        DataSet ds = getDataSet(doc)
     93        Node n = ds.nodes.find {it.get("external-id") == "-1"}
     94        assert n != null
     95        assert n.changesetId == 0
     96    }
     97
     98    /**
     99    * A new node with a missing changeset id. That's fine. The changeset id
     100    * is reset to 0.
     101    */
     102    @Test
     103    public void test_14() {
     104        String doc = """\
    105105         <osm version="0.6">
    106106        <node id="-1" lat="0.0" lon="0.0" >
    107                     <tag k="external-id" v="-1"/>
    108         </node>
    109         </osm>
    110         """
    111 
    112         DataSet ds = getDataSet(doc)
    113                 Node n = ds.nodes.find {it.get("external-id") == "-1"}
    114                 assert n != null
    115                 assert n.changesetId == 0
    116         }
    117 
    118 
    119         /**
    120         * An existing node with a missing changeset id. That's fine. The changeset id
    121         * is reset to 0.
    122         */
    123         @Test
    124         public void test_2() {
    125                 String doc = """\
     107            <tag k="external-id" v="-1"/>
     108        </node>
     109        </osm>
     110        """
     111
     112        DataSet ds = getDataSet(doc)
     113        Node n = ds.nodes.find {it.get("external-id") == "-1"}
     114        assert n != null
     115        assert n.changesetId == 0
     116    }
     117
     118
     119    /**
     120    * An existing node with a missing changeset id. That's fine. The changeset id
     121    * is reset to 0.
     122    */
     123    @Test
     124    public void test_2() {
     125        String doc = """\
    126126         <osm version="0.6">
    127127        <node id="1" lat="0.0" lon="0.0" version="1"/>
     
    130130
    131131        DataSet ds = getDataSet(doc)
    132                 Node n = ds.getPrimitiveById(1, OsmPrimitiveType.NODE)
    133                 assert n != null
    134                 assert n.uniqueId == 1
    135                 assert n.changesetId == 0
    136         }
    137 
    138         /**
     132        Node n = ds.getPrimitiveById(1, OsmPrimitiveType.NODE)
     133        assert n != null
     134        assert n.uniqueId == 1
     135        assert n.changesetId == 0
     136    }
     137
     138    /**
    139139     * An existing node with a valid changeset id id. That's fine. The changeset id
    140140     * is applied.
    141141     */
    142         @Test
    143         public void test_3() {
    144                 String doc = """\
     142    @Test
     143    public void test_3() {
     144        String doc = """\
    145145         <osm version="0.6">
    146146        <node id="1" lat="0.0" lon="0.0" version="1" changeset="4"/>
     
    149149
    150150        DataSet ds = getDataSet(doc)
    151                 Node n = ds.getPrimitiveById(1, OsmPrimitiveType.NODE)
    152                 assert n != null
    153                 assert n.uniqueId == 1
    154                 assert n.changesetId == 4
    155         }
    156 
    157         /**
    158         * An existing node with an invalid changeset id. That's a problem. An exception
    159         * is thrown.
    160         */
    161         @Test
    162         public void test_4() {
    163                 String doc = """\
     151        Node n = ds.getPrimitiveById(1, OsmPrimitiveType.NODE)
     152        assert n != null
     153        assert n.uniqueId == 1
     154        assert n.changesetId == 4
     155    }
     156
     157    /**
     158    * An existing node with an invalid changeset id. That's a problem. An exception
     159    * is thrown.
     160    */
     161    @Test
     162    public void test_4() {
     163        String doc = """\
    164164         <osm version="0.6">
    165165        <node id="1" lat="0.0" lon="0.0" version="1" changeset="-1"/>
     
    167167        """
    168168
    169                 final shouldFail = new GroovyTestCase().&shouldFail
    170 
    171                 shouldFail(IllegalDataException) {
     169        final shouldFail = new GroovyTestCase().&shouldFail
     170
     171        shouldFail(IllegalDataException) {
    172172            DataSet ds = getDataSet(doc)
    173                 }
    174         }
    175         /**
    176         * An existing node with an invalid changeset id. That's a problem. An exception
    177         * is thrown.
    178         */
    179         @Test
    180         public void test_5() {
    181                 String doc = """\
     173        }
     174    }
     175    /**
     176    * An existing node with an invalid changeset id. That's a problem. An exception
     177    * is thrown.
     178    */
     179    @Test
     180    public void test_5() {
     181        String doc = """\
    182182         <osm version="0.6">
    183183        <node id="1" lat="0.0" lon="0.0" version="1" changeset="0"/>
     
    185185        """
    186186
    187                 final shouldFail = new GroovyTestCase().&shouldFail
    188 
    189                 shouldFail(IllegalDataException) {
     187        final shouldFail = new GroovyTestCase().&shouldFail
     188
     189        shouldFail(IllegalDataException) {
    190190            DataSet ds = getDataSet(doc)
    191                 }
    192         }
    193         /**
    194         * An existing node with an invalid changeset id. That's a problem. An exception
    195         * is thrown.
    196         */
    197         @Test
    198         public void test_6() {
    199                 String doc = """\
    200                  <osm version="0.6">
    201                 <node id="1" lat="0.0" lon="0.0" version="1" changeset="abc"/>
    202                 </osm>
    203                 """
    204 
    205                 final shouldFail = new GroovyTestCase().&shouldFail
    206 
    207                 shouldFail(IllegalDataException) {
     191        }
     192    }
     193    /**
     194    * An existing node with an invalid changeset id. That's a problem. An exception
     195    * is thrown.
     196    */
     197    @Test
     198    public void test_6() {
     199        String doc = """\
     200             <osm version="0.6">
     201            <node id="1" lat="0.0" lon="0.0" version="1" changeset="abc"/>
     202            </osm>
     203            """
     204
     205        final shouldFail = new GroovyTestCase().&shouldFail
     206
     207        shouldFail(IllegalDataException) {
    208208            DataSet ds = getDataSet(doc)
    209                 }
    210         }
     209        }
     210    }
    211211}
  • trunk/test/unit/org/openstreetmap/josm/io/remotecontrol/RemoteControlTest.java

    r7937 r8510  
    7272                    return null;
    7373                }
     74
    7475                public void checkClientTrusted(X509Certificate[] certs, String authType) {
    7576                }
     77
    7678                public void checkServerTrusted(X509Certificate[] certs, String authType) {
    7779                }
     
    137139            try (BufferedReader in = new BufferedReader(new InputStreamReader(is, StandardCharsets.UTF_8))) {
    138140                String s;
    139                 while((s = in.readLine()) != null) {
     141                while ((s = in.readLine()) != null) {
    140142                    responseBody.append(s);
    141143                    responseBody.append("\n");
  • trunk/test/unit/org/openstreetmap/josm/io/remotecontrol/handler/RequestHandlerTest.java

    r8509 r8510  
    4343    }
    4444
    45 
    4645    @Test
    4746    public void testRequestParameter1() {
  • trunk/test/unit/org/openstreetmap/josm/io/session/SessionWriterTest.java

    r8338 r8510  
    3333            supe