Ignore:
Timestamp:
2009-08-05T08:19:02+02:00 (15 years ago)
Author:
jttt
Message:

Way refactoring - finish replacing Way.nodes with the new api

Location:
trunk/test/unit/org/openstreetmap/josm/data/osm/visitor
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/test/unit/org/openstreetmap/josm/data/osm/visitor/MergeSourceBuildingVisitorTest.java

    r1890 r1910  
    22package org.openstreetmap.josm.data.osm.visitor;
    33
     4import static org.junit.Assert.assertEquals;
     5import static org.junit.Assert.assertNotNull;
     6import static org.junit.Assert.assertNull;
     7import static org.junit.Assert.assertTrue;
     8
    49import java.util.Collection;
    510
    6 import org.junit.Ignore;
    711import org.junit.Test;
    812import org.openstreetmap.josm.data.coor.LatLon;
    … …  
    1317import org.openstreetmap.josm.data.osm.RelationMember;
    1418import org.openstreetmap.josm.data.osm.Way;
    15 
    16 import static org.junit.Assert.*;
    1719
    1820public class MergeSourceBuildingVisitorTest {
    … …  
    6971        Node n2 = new Node(2);
    7072        Way w1 = new Way(3);
    71         w1.nodes.add(n1);
    72         w1.nodes.add(n2);
     73        w1.addNode(n1);
     74        w1.addNode(n2);
    7375        source.nodes.add(n1);
    7476        source.nodes.add(n2);
    … …  
    101103        Node n2 = new Node(2);
    102104        Way w1 = new Way(3);
    103         w1.nodes.add(n1);
    104         w1.nodes.add(n2);
     105        w1.addNode(n1);
     106        w1.addNode(n2);
    105107        source.nodes.add(n1);
    106108        source.nodes.add(n2);
    … …  
    154156        Way w30 = new Way(30);
    155157        Node n21;
    156         w30.nodes.add(n21 = new Node(21));
     158        w30.addNode(n21 = new Node(21));
    157159        Node n22;
    158         w30.nodes.add(n22 = new Node(22));
     160        w30.addNode(n22 = new Node(22));
    159161        r1.members.add(new RelationMember("way-30",w30));
    160162        Relation r40 = new Relation(40);
    … …  
    181183        Way w = (Way)hull.getPrimitiveById(30);
    182184        assertNotNull(w);
    183         assertEquals(2, w.nodes.size());
     185        assertEquals(2, w.getNodesCount());
    184186        Node n = (Node)hull.getPrimitiveById(21);
    185187        assertNotNull(n);
    186         assertTrue(w.nodes.contains(n));
     188        assertTrue(w.getNodes().contains(n));
    187189
    188190        n = (Node)hull.getPrimitiveById(22);
    189191        assertNotNull(n);
    190         assertTrue(w.nodes.contains(n));
     192        assertTrue(w.getNodes().contains(n));
    191193
    192194        Relation r = (Relation)hull.getPrimitiveById(40);
    … …  
    212214        Way w30 = new Way(30);
    213215        Node n21;
    214         w30.nodes.add(n21 = new Node(21));
     216        w30.addNode(n21 = new Node(21));
    215217        Node n22;
    216         w30.nodes.add(n22 = new Node(22));
     218        w30.addNode(n22 = new Node(22));
    217219        r1.members.add(new RelationMember("way-30",w30));
    218220        Relation r40 = new Relation(40);
    … …  
    274276        w30.put("name", "w30");
    275277        Node n21;
    276         w30.nodes.add(n21 = new Node(new LatLon(21.0,21.0)));
     278        w30.addNode(n21 = new Node(new LatLon(21.0,21.0)));
    277279        n21.put("name","n21");
    278280        Node n22;
    279         w30.nodes.add(n22 = new Node(new LatLon(22.0,22.0)));
     281        w30.addNode(n22 = new Node(new LatLon(22.0,22.0)));
    280282        n22.put("name","n22");
    281283        r1.members.add(new RelationMember("way-30",w30));
    … …  
    304306        Way w = (Way)lookupByName(hull.ways, "w30");
    305307        assertNotNull(w);
    306         assertEquals(2, w.nodes.size());
     308        assertEquals(2, w.getNodesCount());
    307309
    308310        Node n = (Node)lookupByName(hull.nodes, "n21");
    309311        assertNotNull(n);
    310         assertTrue(w.nodes.contains(n));
     312        assertTrue(w.getNodes().contains(n));
    311313
    312314        n = (Node)lookupByName(hull.nodes, "n22");
    313315        assertNotNull(n);
    314         assertTrue(w.nodes.contains(n));
     316        assertTrue(w.getNodes().contains(n));
    315317
    316318        Relation r = (Relation)lookupByName(hull.relations, "r40");

  • trunk/test/unit/org/openstreetmap/josm/data/osm/visitor/MergeVisitorTest.java

    r1813 r1910  
    489489        myWay.version = 1;
    490490        myWay.put("key1", "value1");
    491         myWay.nodes.add(n1);
    492         myWay.nodes.add(n2);
     491        myWay.addNode(n1);
     492        myWay.addNode(n2);
    493493        my.addPrimitive(myWay);
    494494
    … …  
    511511        theirWay.put("key1", "value1");
    512512        theirWay.put("key2", "value2");
    513         theirWay.nodes.add(n3);
    514         theirWay.nodes.add(n4);
     513        theirWay.addNode(n3);
     514        theirWay.addNode(n4);
    515515        their.addPrimitive(theirWay);
    516516
    … …  
    525525        assertEquals(3,merged.id);
    526526        assertEquals(2,merged.version);
    527         assertEquals(2,merged.nodes.size());
    528         assertEquals(1,merged.nodes.get(0).id);
    529         assertEquals(2,merged.nodes.get(1).id);
     527        assertEquals(2,merged.getNodesCount());
     528        assertEquals(1,merged.getNode(0).id);
     529        assertEquals(2,merged.getNode(1).id);
    530530
    531531    }
    … …  
    557557        myWay.id = 3;
    558558        myWay.version = 1;
    559         myWay.nodes.add(n1);
    560         myWay.nodes.add(n2);
     559        myWay.addNode(n1);
     560        myWay.addNode(n2);
    561561        my.addPrimitive(myWay);
    562562
    … …  
    586586        theirWay.id = 3;
    587587        theirWay.version = 2;
    588         theirWay.nodes.add(n3);
    589         theirWay.nodes.add(n5); // insert a node
    590         theirWay.nodes.add(n4); // this one is updated
     588        theirWay.addNode(n3);
     589        theirWay.addNode(n5); // insert a node
     590        theirWay.addNode(n4); // this one is updated
    591591        their.addPrimitive(theirWay);
    592592
    … …  
    599599        assertEquals(3,merged.id);
    600600        assertEquals(2,merged.version);
    601         assertEquals(3,merged.nodes.size());
    602         assertEquals(1,merged.nodes.get(0).id);
    603         assertEquals(4,merged.nodes.get(1).id);
    604         assertEquals(2,merged.nodes.get(2).id);
    605         assertEquals("value1",merged.nodes.get(2).get("key1"));
     601        assertEquals(3,merged.getNodesCount());
     602        assertEquals(1,merged.getNode(0).id);
     603        assertEquals(4,merged.getNode(1).id);
     604        assertEquals(2,merged.getNode(2).id);
     605        assertEquals("value1",merged.getNode(2).get("key1"));
    606606    }
    607607
    … …  
    631631        myWay.id = 3;
    632632        myWay.version = 1;
    633         myWay.nodes.add(n1);
    634         myWay.nodes.add(n2);
     633        myWay.addNode(n1);
     634        myWay.addNode(n2);
    635635        myWay.modified = true;
    636636        myWay.put("key1", "value1");
    … …  
    662662        theirWay.id = 3;
    663663        theirWay.version = 2;
    664         theirWay.nodes.add(n3);
    665         theirWay.nodes.add(n5); // insert a node
    666         theirWay.nodes.add(n4); // this one is updated
     664        theirWay.addNode(n3);
     665        theirWay.addNode(n5); // insert a node
     666        theirWay.addNode(n4); // this one is updated
    667667        their.addPrimitive(theirWay);
    668668
    … …  
    675675        assertEquals(3,merged.id);
    676676        assertEquals(1,merged.version);
    677         assertEquals(2,merged.nodes.size());
    678         assertEquals(1,merged.nodes.get(0).id);
    679         assertEquals(2,merged.nodes.get(1).id);
     677        assertEquals(2,merged.getNodesCount());
     678        assertEquals(1,merged.getNode(0).id);
     679        assertEquals(2,merged.getNode(1).id);
    680680        assertEquals("value1",merged.get("key1"));
    681681    }
    … …  
    707707        myWay.id = 3;
    708708        myWay.version = 1;
    709         myWay.nodes.add(n1);
    710         myWay.nodes.add(n2);
     709        myWay.addNode(n1);
     710        myWay.addNode(n2);
    711711        my.addPrimitive(myWay);
    712712
    … …  
    757757        myWay.id = 0;
    758758        myWay.version = -1;
    759         myWay.nodes.add(n1);
    760         myWay.nodes.add(n2);
     759        myWay.addNode(n1);
     760        myWay.addNode(n2);
    761761        my.addPrimitive(myWay);
    762762
    … …  
    777777        theirWay.id = 0;
    778778        theirWay.version = -1;
    779         theirWay.nodes.add(n3);
    780         theirWay.nodes.add(n4);
     779        theirWay.addNode(n3);
     780        theirWay.addNode(n4);
    781781        theirWay.user = User.get("their");
    782782        theirWay.user.uid = "1111";
    … …  
    819819        myWay.id = 0;
    820820        myWay.version = -1;
    821         myWay.nodes.add(n1);
    822         myWay.nodes.add(n2);
     821        myWay.addNode(n1);
     822        myWay.addNode(n2);
    823823        my.addPrimitive(myWay);
    824824
    … …  
    839839        theirWay.id = 0;
    840840        theirWay.version = -1;
    841         theirWay.nodes.add(n3);
    842         theirWay.nodes.add(n4);
     841        theirWay.addNode(n3);
     842        theirWay.addNode(n4);
    843843        theirWay.user = User.get("their");
    844844        theirWay.user.uid = "1111";
    … …  
    899899        theirWay.id = 4;
    900900        theirWay.version = 1;
    901         theirWay.nodes.add(n3);
    902         theirWay.nodes.add(n4);
    903         theirWay.nodes.add(n5);
     901        theirWay.addNode(n3);
     902        theirWay.addNode(n4);
     903        theirWay.addNode(n5);
    904904        theirWay.user = User.get("their");
    905905        theirWay.user.uid = "1111";
    … …  
    913913
    914914        Way myWay = (Way)my.getPrimitiveById(4);
    915         assertEquals(2, myWay.nodes.size());
     915        assertEquals(2, myWay.getNodesCount());
    916916
    917917        Node n = (Node)my.getPrimitiveById(1);
Note: See TracChangeset for help on using the changeset viewer.