Changeset 2381 in josm for trunk/test


Ignore:
Timestamp:
2009-11-02T07:51:28+01:00 (15 years ago)
Author:
jttt
Message:

Change most occurrences of Dataset.nodes/ways/relations with getNodes()/../.. or addPrimitive

Location:
trunk/test
Files:
5 edited

Legend:

Unmodified
Added
Removed

  • trunk/test/functional/org/openstreetmap/josm/io/MultiFetchServerObjectReaderTest.java

    r2199 r2381  
    3535import org.openstreetmap.josm.data.projection.Mercator;
    3636import org.openstreetmap.josm.gui.progress.NullProgressMonitor;
    37 import org.xml.sax.SAXException;
    3837
    3938public class MultiFetchServerObjectReaderTest {
    … …  
    124123        logger.info("creating data set on the server ...");
    125124        ArrayList<OsmPrimitive> primitives = new ArrayList<OsmPrimitive>();
    126         primitives.addAll(testDataSet.nodes);
    127         primitives.addAll(testDataSet.ways);
    128         primitives.addAll(testDataSet.relations);
     125        primitives.addAll(testDataSet.getNodes());
     126        primitives.addAll(testDataSet.getWays());
     127        primitives.addAll(testDataSet.getRelations());
    129128
    130129        OsmServerWriter writer = new OsmServerWriter();
    … …  
    247246        }
    248247        DataSet out = reader.parseOsm(NullProgressMonitor.INSTANCE);
    249         assertEquals(10, out.nodes.size());
     248        assertEquals(10, out.getNodes().size());
    250249        Iterator<Node> it = out.nodes.iterator();
    251250        while(it.hasNext()) {
    … …  
    261260    public void testMultiGet10Ways() throws OsmTransferException {
    262261        MultiFetchServerObjectReader reader = new MultiFetchServerObjectReader();
    263         ArrayList<Way> ways= new ArrayList<Way>(ds.ways);
     262        ArrayList<Way> ways= new ArrayList<Way>(ds.getWays());
    264263        for (int i =0; i< 10; i++) {
    265264            reader.append(ways.get(i));
    266265        }
    267266        DataSet out = reader.parseOsm(NullProgressMonitor.INSTANCE);
    268         assertEquals(10, out.ways.size());
    269         Iterator<Way> it = out.ways.iterator();
    270         while(it.hasNext()) {
    271             Way w1 = it.next();
     267        assertEquals(10, out.getWays().size());
     268        for (Way w1: out.getWays()) {
    272269            Way w2 = (Way)ds.getPrimitiveById(w1.getId(), OsmPrimitiveType.WAY);
    273270            assertNotNull(w2);
    … …  
    281278    public void testMultiGet10Relations() throws OsmTransferException {
    282279        MultiFetchServerObjectReader reader = new MultiFetchServerObjectReader();
    283         ArrayList<Relation> relations= new ArrayList<Relation>(ds.relations);
     280        ArrayList<Relation> relations= new ArrayList<Relation>(ds.getRelations());
    284281        for (int i =0; i< 10; i++) {
    285282            reader.append(relations.get(i));
    286283        }
    287284        DataSet out = reader.parseOsm(NullProgressMonitor.INSTANCE);
    288         assertEquals(10, out.relations.size());
    289         Iterator<Relation> it = out.relations.iterator();
    290         while(it.hasNext()) {
    291             Relation r1 = it.next();
     285        assertEquals(10, out.getRelations().size());
     286        for (Relation r1: out.getRelations()) {
    292287            Relation r2 = (Relation)ds.getPrimitiveById(r1.getId(), OsmPrimitiveType.RELATION);
    293288            assertNotNull(r2);
    … …  
    301296    public void testMultiGet800Nodes() throws OsmTransferException {
    302297        MultiFetchServerObjectReader reader = new MultiFetchServerObjectReader();
    303         ArrayList<Node> nodes = new ArrayList<Node>(ds.nodes);
     298        ArrayList<Node> nodes = new ArrayList<Node>(ds.getNodes());
    304299        for (int i =0; i< 812; i++) {
    305300            reader.append(nodes.get(i));
    306301        }
    307302        DataSet out = reader.parseOsm(NullProgressMonitor.INSTANCE);
    308         assertEquals(812, out.nodes.size());
     303        assertEquals(812, out.getNodes().size());
    309304        Iterator<Node> it = out.nodes.iterator();
    310305        while(it.hasNext()) {
    … …  
    320315    public void multiGetWithNonExistingNode() throws OsmTransferException {
    321316        MultiFetchServerObjectReader reader = new MultiFetchServerObjectReader();
    322         ArrayList<Node> nodes = new ArrayList<Node>(ds.nodes);
     317        ArrayList<Node> nodes = new ArrayList<Node>(ds.getNodes());
    323318        for (int i =0; i< 10; i++) {
    324319            reader.append(nodes.get(i));
    … …  
    327322        reader.append(n); // doesn't exist
    328323        DataSet out = reader.parseOsm(NullProgressMonitor.INSTANCE);
    329         assertEquals(10, out.nodes.size());
    330         Iterator<Node> it = out.nodes.iterator();
    331         while(it.hasNext()) {
    332             Node n1 = it.next();
     324        assertEquals(10, out.getNodes().size());
     325        for (Node n1:out.getNodes()) {
    333326            Node n2 = (Node)ds.getPrimitiveById(n1.getId(), OsmPrimitiveType.NODE);
    334327            assertNotNull(n2);

  • trunk/test/functional/org/openstreetmap/josm/io/OsmServerBackreferenceReaderTest.java

    r2199 r2381  
    3434import org.openstreetmap.josm.data.projection.Mercator;
    3535import org.openstreetmap.josm.gui.progress.NullProgressMonitor;
    36 import org.xml.sax.SAXException;
    3736
    3837public class OsmServerBackreferenceReaderTest {
    … …  
    4039
    4140    protected static Node lookupNode(DataSet ds, int i) {
    42         for (Node n: ds.nodes) {
     41        for (Node n : ds.getNodes()) {
    4342            if (("node-" + i).equals(n.get("name"))) return n;
    4443        }
    … …  
    4847
    4948    protected static Way lookupWay(DataSet ds, int i) {
    50         for (Way w: ds.ways) {
     49        for (Way w : ds.getWays()) {
    5150            if (("way-" + i).equals(w.get("name"))) return w;
    5251        }
    … …  
    5554
    5655    protected static Relation lookupRelation(DataSet ds, int i) {
    57         for (Relation r: ds.relations) {
     56        for (Relation r : ds.getRelations()) {
    5857            if (("relation-" + i).equals(r.get("name"))) return r;
    5958        }
    … …  
    103102                }
    104103            }
    105             ds.relations.add(r);
     104            ds.addPrimitive(r);
    106105        }
    107106    }
    … …  
    127126        logger.info("creating data set on the server ...");
    128127        ArrayList<OsmPrimitive> primitives = new ArrayList<OsmPrimitive>();
    129         primitives.addAll(ds.nodes);
    130         primitives.addAll(ds.ways);
    131         primitives.addAll(ds.relations);
     128        primitives.addAll(ds.getNodes());
     129        primitives.addAll(ds.getWays());
     130        primitives.addAll(ds.getRelations());
    132131        OsmServerWriter writer = new OsmServerWriter();
    133132        Changeset cs  = new Changeset();
    … …  
    253252        reader.setReadFull(false);
    254253        DataSet referers = reader.parseOsm(NullProgressMonitor.INSTANCE);
    255         assertEquals(10, referers.nodes.size());
    256         assertEquals(1, referers.ways.size());
    257         assertEquals(0, referers.relations.size());
    258         for (Way way: referers.ways) {
     254        assertEquals(10, referers.getNodes().size());
     255        assertEquals(1, referers.getWays().size());
     256        assertEquals(0, referers.getRelations().size());
     257        for (Way way : referers.getWays()) {
    259258            assertEquals(w.getId(), way.getId());
    260259            assertEquals(false, way.incomplete);
    … …  
    272271        reader.setReadFull(true);
    273272        DataSet referers = reader.parseOsm(NullProgressMonitor.INSTANCE);
    274         assertEquals(10, referers.nodes.size());
    275         assertEquals(1, referers.ways.size());
    276         assertEquals(0, referers.relations.size());
    277         for (Way way: referers.ways) {
     273        assertEquals(10, referers.getNodes().size());
     274        assertEquals(1, referers.getWays().size());
     275        assertEquals(0, referers.getRelations().size());
     276        for (Way way : referers.getWays()) {
    278277            assertEquals(w.getId(), way.getId());
    279278            assertEquals(false, way.incomplete);
    … …  
    292291        reader.setReadFull(false);
    293292        DataSet referers = reader.parseOsm(NullProgressMonitor.INSTANCE);
    294         assertEquals(0, referers.nodes.size()); // no nodes loaded
    295         assertEquals(6, referers.ways.size());  // 6 ways referred by two relations
    296         for (Way w1: referers.ways) {
     293        assertEquals(0, referers.getNodes().size()); // no nodes loaded
     294        assertEquals(6, referers.getWays().size());  // 6 ways referred by two relations
     295        for (Way w1 : referers.getWays()) {
    297296            assertEquals(true, w1.incomplete);
    298297        }
    299         assertEquals(2, referers.relations.size());  // two relations referring to w
     298        assertEquals(2, referers.getRelations().size());  // two relations referring to w
    300299
    301300        Relation r = lookupRelation(referers, 0);
    … …  
    316315        reader.setReadFull(true);
    317316        DataSet referers = reader.parseOsm(NullProgressMonitor.INSTANCE);
    318         assertEquals(6, referers.ways.size());  // 6 ways referred by two relations
    319         for (Way w1: referers.ways) {
     317        assertEquals(6, referers.getWays().size());  // 6 ways referred by two relations
     318        for (Way w1 : referers.getWays()) {
    320319            assertEquals(false, w1.incomplete);
    321320        }
    322         assertEquals(2, referers.relations.size());  // two relations referring to
     321        assertEquals(2, referers.getRelations().size());  // two relations referring to
    323322        Set<Long> expectedNodeIds = new HashSet<Long>();
    324         for (Way way: referers.ways) {
    325             Way orig = (Way)ds.getPrimitiveById(way.getId(), OsmPrimitiveType.WAY);
    326             for(Node n: orig.getNodes()) {
     323        for (Way way : referers.getWays()) {
     324            Way orig = (Way) ds.getPrimitiveById(way.getId(), OsmPrimitiveType.WAY);
     325            for (Node n : orig.getNodes()) {
    327326                expectedNodeIds.add(n.getId());
    328327            }
    329328        }
    330         assertEquals(expectedNodeIds.size(), referers.nodes.size());
    331         for (Node n : referers.nodes) {
     329        assertEquals(expectedNodeIds.size(), referers.getNodes().size());
     330        for (Node n : referers.getNodes()) {
    332331            assertEquals(true, expectedNodeIds.contains(n.getId()));
    333332        }
    … …  
    342341    @Test
    343342    public void testBackrefrenceForRelation() throws OsmTransferException {
    344         Relation r = lookupRelation(ds,1);
     343        Relation r = lookupRelation(ds, 1);
    345344        assertNotNull(r);
    346345        // way with name "relation-1" is referred to by four relations:
    … …  
    370369        referringRelationsIds.add(r.getId());
    371370
    372         for (Relation r1: referers.relations) {
    373             if (! referringRelationsIds.contains(r1.getId())) {
     371        for (Relation r1 : referers.getRelations()) {
     372            if (!referringRelationsIds.contains(r1.getId())) {
    374373                assertEquals(true, r1.incomplete);
    375374            }
    … …  
    401400        }
    402401
    403         assertEquals(expectedWayIds.size(), referers.ways.size());
    404         for (Way w1 : referers.ways) {
     402        assertEquals(expectedWayIds.size(), referers.getWays().size());
     403        for (Way w1 : referers.getWays()) {
    405404            assertEquals(true, expectedWayIds.contains(w1.getId()));
    406405            assertEquals(true, w1.incomplete);
    … …  
    409408        // make sure we didn't read any nodes
    410409        //
    411         assertEquals(0, referers.nodes.size());
     410        assertEquals(0, referers.getNodes().size());
    412411    }
    413412
    … …  
    438437    @Test
    439438    public void testBackrefrenceForRelation_Full() throws OsmTransferException {
    440         Relation r = lookupRelation(ds,1);
     439        Relation r = lookupRelation(ds, 1);
    441440        assertNotNull(r);
    442441        // way with name "relation-1" is referred to by four relations:
    … …  
    468467        // all relations are fully loaded
    469468        //
    470         for (Relation r1: referers.relations) {
     469        for (Relation r1 : referers.getRelations()) {
    471470            assertEquals(false, r1.incomplete);
    472471        }
    … …  
    497496        }
    498497        for (long id : expectedWayIds) {
    499             Way w = (Way)referers.getPrimitiveById(id, OsmPrimitiveType.WAY);
     498            Way w = (Way) referers.getPrimitiveById(id, OsmPrimitiveType.WAY);
    500499            assertNotNull(w);
    501500            assertEquals(false, w.incomplete);
    … …  
    503502
    504503        Set<Long> expectedNodeIds = new HashSet<Long>();
    505         for(int i=6; i< 10;i++) {
     504        for (int i = 6; i < 10; i++) {
    506505            Relation r1 = lookupRelation(ds, i);
    507506            expectedNodeIds.addAll(getNodeIdsInRelation(r1));
    508507        }
    509508
    510         assertEquals(expectedNodeIds.size(), referers.nodes.size());
    511         for(Node n : referers.nodes) {
     509        assertEquals(expectedNodeIds.size(), referers.getNodes().size());
     510        for (Node n : referers.getNodes()) {
    512511            assertEquals(true, expectedNodeIds.contains(n.getId()));
    513512        }

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

    r2168 r2381  
    2020        r.incomplete = false;
    2121        DataSet ds = new DataSet();
    22         ds.relations.add(r);
     22        ds.addPrimitive(r);
    2323
    2424        APIDataSet apiDataSet = new APIDataSet();
    … …  
    4848
    4949        DataSet ds = new DataSet();
    50         ds.relations.add(r1);
    51         ds.relations.add(r2);
     50        ds.addPrimitive(r1);
     51        ds.addPrimitive(r2);
    5252
    5353        APIDataSet apiDataSet = new APIDataSet();
    … …  
    8888
    8989        DataSet ds = new DataSet();
    90         ds.relations.add(r1);
    91         ds.relations.add(r2);
    92         ds.relations.add(r3);
    93         ds.relations.add(r4);
     90        ds.addPrimitive(r1);
     91        ds.addPrimitive(r2);
     92        ds.addPrimitive(r3);
     93        ds.addPrimitive(r4);
    9494
    9595        APIDataSet apiDataSet = new APIDataSet();
    … …  
    130130
    131131        DataSet ds = new DataSet();
    132         ds.relations.add(r1);
    133         ds.relations.add(r2);
    134         ds.relations.add(r3);
     132        ds.addPrimitive(r1);
     133        ds.addPrimitive(r2);
     134        ds.addPrimitive(r3);
    135135
    136136        APIDataSet apiDataSet = new APIDataSet();
    … …  
    167167
    168168        DataSet ds = new DataSet();
    169         ds.relations.add(r1);
    170         ds.relations.add(r2);
    171         ds.relations.add(r3);
     169        ds.addPrimitive(r1);
     170        ds.addPrimitive(r2);
     171        ds.addPrimitive(r3);
    172172
    173173        APIDataSet apiDataSet = new APIDataSet();

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

    r2077 r2381  
    4040        Node n4 = new Node(new LatLon(20.0,20.0));
    4141        n4.put("name","n4");
    42         source.nodes.add(n1);
    43         source.nodes.add(n2);
    44         source.nodes.add(n3);
    45         source.nodes.add(n4);
     42        source.addPrimitive(n1);
     43        source.addPrimitive(n2);
     44        source.addPrimitive(n3);
     45        source.addPrimitive(n4);
    4646        source.setSelected(n1,n2);
    4747
    … …  
    4949        DataSet hull = builder.build();
    5050        assertNotNull(hull);
    51         assertEquals(2, hull.nodes.size());
     51        assertEquals(2, hull.getNodes().size());
    5252
    5353        OsmPrimitive p = hull.getPrimitiveById(1,OsmPrimitiveType.NODE);
    … …  
    5858        assertNull(p);
    5959
    60         p = lookupByName(hull.nodes, "n2");
    61         assertNotNull(p);
    62 
    63         p = lookupByName(hull.nodes, "n4");
     60        p = lookupByName(hull.getNodes(), "n2");
     61        assertNotNull(p);
     62
     63        p = lookupByName(hull.getNodes(), "n4");
    6464        assertNull(p);
    6565    }
    … …  
    7575        w1.addNode(n1);
    7676        w1.addNode(n2);
    77         source.nodes.add(n1);
    78         source.nodes.add(n2);
    79         source.ways.add(w1);
     77        source.addPrimitive(n1);
     78        source.addPrimitive(n2);
     79        source.addPrimitive(w1);
    8080        source.setSelected(w1);
    8181
    … …  
    8383        DataSet hull = builder.build();
    8484        assertNotNull(hull);
    85         assertEquals(1, hull.ways.size());
    86         assertEquals(2, hull.nodes.size());
     85        assertEquals(1, hull.getWays().size());
     86        assertEquals(2, hull.getNodes().size());
    8787
    8888        OsmPrimitive p = hull.getPrimitiveById(1,OsmPrimitiveType.NODE);
    … …  
    108108        w1.addNode(n1);
    109109        w1.addNode(n2);
    110         source.nodes.add(n1);
    111         source.nodes.add(n2);
    112         source.ways.add(w1);
     110        source.addPrimitive(n1);
     111        source.addPrimitive(n2);
     112        source.addPrimitive(w1);
    113113        source.setSelected(w1,n1,n2);
    114114
    … …  
    116116        DataSet hull = builder.build();
    117117        assertNotNull(hull);
    118         assertEquals(1, hull.ways.size());
    119         assertEquals(2, hull.nodes.size());
     118        assertEquals(1, hull.getWays().size());
     119        assertEquals(2, hull.getNodes().size());
    120120
    121121        OsmPrimitive p = hull.getPrimitiveById(1,OsmPrimitiveType.NODE);
    … …  
    137137        Way w1 = new Way(3);
    138138        w1.incomplete = true;
    139         source.ways.add(w1);
     139        source.addPrimitive(w1);
    140140        source.setSelected(w1);
    141141
    … …  
    143143        DataSet hull = builder.build();
    144144        assertNotNull(hull);
    145         assertEquals(1, hull.ways.size());
     145        assertEquals(1, hull.getWays().size());
    146146
    147147        OsmPrimitive p = hull.getPrimitiveById(3, OsmPrimitiveType.WAY);
    … …  
    167167        Relation r40 = new Relation(40);
    168168        r1.addMember(new RelationMember("relation-40", r40));
    169         source.nodes.add(n20);
    170         source.nodes.add(n21);
    171         source.nodes.add(n22);
    172         source.ways.add(w30);
    173         source.relations.add(r1);
    174         source.relations.add(r40);
     169        source.addPrimitive(n20);
     170        source.addPrimitive(n21);
     171        source.addPrimitive(n22);
     172        source.addPrimitive(w30);
     173        source.addPrimitive(r1);
     174        source.addPrimitive(r40);
    175175        source.setSelected(r1,n20,w30,r40);
    176176
    … …  
    178178        DataSet hull = builder.build();
    179179        assertNotNull(hull);
    180         assertEquals(1, hull.ways.size());
    181         assertEquals(3, hull.nodes.size());
    182         assertEquals(2, hull.relations.size());
     180        assertEquals(1, hull.getWays().size());
     181        assertEquals(3, hull.getNodes().size());
     182        assertEquals(2, hull.getRelations().size());
    183183
    184184        OsmPrimitive p = hull.getPrimitiveById(1, OsmPrimitiveType.RELATION);
    … …  
    226226        Relation r40 = new Relation(40);
    227227        r1.addMember(new RelationMember("relation-40", r40));
    228         source.nodes.add(n20);
    229         source.nodes.add(n21);
    230         source.nodes.add(n22);
    231         source.ways.add(w30);
    232         source.relations.add(r1);
    233         source.relations.add(r40);
     228        source.addPrimitive(n20);
     229        source.addPrimitive(n21);
     230        source.addPrimitive(n22);
     231        source.addPrimitive(w30);
     232        source.addPrimitive(r1);
     233        source.addPrimitive(r40);
    234234        source.setSelected(r1);
    235235
    … …  
    237237        DataSet hull = builder.build();
    238238        assertNotNull(hull);
    239         assertEquals(1, hull.ways.size());
    240         assertEquals(1, hull.nodes.size());
    241         assertEquals(2, hull.relations.size());
     239        assertEquals(1, hull.getWays().size());
     240        assertEquals(1, hull.getNodes().size());
     241        assertEquals(2, hull.getRelations().size());
    242242
    243243        OsmPrimitive p = hull.getPrimitiveById(1, OsmPrimitiveType.RELATION);
    … …  
    293293        r1.addMember(new RelationMember("relation-40", r40));
    294294
    295         source.nodes.add(n20);
    296         source.nodes.add(n21);
    297         source.nodes.add(n22);
    298         source.ways.add(w30);
    299         source.relations.add(r1);
    300         source.relations.add(r40);
     295        source.addPrimitive(n20);
     296        source.addPrimitive(n21);
     297        source.addPrimitive(n22);
     298        source.addPrimitive(w30);
     299        source.addPrimitive(r1);
     300        source.addPrimitive(r40);
    301301        source.setSelected(r1);
    302302
    … …  
    304304        DataSet hull = builder.build();
    305305        assertNotNull(hull);
    306         assertEquals(1, hull.ways.size());
    307         assertEquals(3, hull.nodes.size());
    308         assertEquals(2, hull.relations.size());
    309 
    310         OsmPrimitive p = lookupByName(hull.relations, "r1");
     306        assertEquals(1, hull.getWays().size());
     307        assertEquals(3, hull.getNodes().size());
     308        assertEquals(2, hull.getRelations().size());
     309
     310        OsmPrimitive p = lookupByName(hull.getRelations(), "r1");
    311311        assertNotNull(p);
    312312        assertEquals(p.getClass(), Relation.class);
    313313
    314         Way w = (Way)lookupByName(hull.ways, "w30");
     314        Way w = (Way)lookupByName(hull.getWays(), "w30");
    315315        assertNotNull(w);
    316316        assertEquals(2, w.getNodesCount());
    317317
    318         Node n = (Node)lookupByName(hull.nodes, "n21");
     318        Node n = (Node)lookupByName(hull.getNodes(), "n21");
    319319        assertNotNull(n);
    320320        assertTrue(w.containsNode(n));
    321321
    322         n = (Node)lookupByName(hull.nodes, "n22");
     322        n = (Node)lookupByName(hull.getNodes(), "n22");
    323323        assertNotNull(n);
    324324        assertTrue(w.containsNode(n));
    325325
    326         Relation r = (Relation)lookupByName(hull.relations, "r40");
    327         assertNotNull(r);
    328 
    329         r = (Relation)lookupByName(hull.relations, "r1");
     326        Relation r = (Relation)lookupByName(hull.getRelations(), "r40");
     327        assertNotNull(r);
     328
     329        r = (Relation)lookupByName(hull.getRelations(), "r1");
    330330        assertNotNull(r);
    331331        assertEquals(3, r.getMembersCount());
    332         RelationMember m = new RelationMember("node-20", lookupByName(hull.nodes, "n20"));
     332        RelationMember m = new RelationMember("node-20", lookupByName(hull.getNodes(), "n20"));
    333333        assertTrue(r.getMembers().contains(m));
    334334        m = new RelationMember("way-30", lookupByName(hull.ways, "w30"));
    … …  
    343343        Relation r1 = new Relation(1);
    344344        r1.addMember(new RelationMember("relation-1",r1));
    345         source.relations.add(r1);
     345        source.addPrimitive(r1);
    346346        source.setSelected(r1);
    347347
    … …  
    349349        DataSet hull = builder.build();
    350350        assertNotNull(hull);
    351         assertEquals(1, hull.relations.size());
     351        assertEquals(1, hull.getRelations().size());
    352352
    353353        Relation r = (Relation)hull.getPrimitiveById(1, OsmPrimitiveType.RELATION);
    … …  
    363363        r1.put("name", "r1");
    364364        r1.addMember(new RelationMember("relation-1",r1));
    365         source.relations.add(r1);
     365        source.addPrimitive(r1);
    366366        source.setSelected(r1);
    367367
    … …  
    369369        DataSet hull = builder.build();
    370370        assertNotNull(hull);
    371         assertEquals(1, hull.relations.size());
    372 
    373         Relation r = (Relation)lookupByName(hull.relations, "r1");
     371        assertEquals(1, hull.getRelations().size());
     372
     373        Relation r = (Relation)lookupByName(hull.getRelations(), "r1");
    374374        assertNotNull(r);
    375375        assertEquals(1, r.getMembersCount());
    … …  
    384384        r1.addMember(new RelationMember("relation-2",r2));
    385385        r2.addMember(new RelationMember("relation-1",r1));
    386         source.relations.add(r1);
    387         source.relations.add(r2);
     386        source.addPrimitive(r1);
     387        source.addPrimitive(r2);
    388388        source.setSelected(r1,r2);
    389389
    … …  
    391391        DataSet hull = builder.build();
    392392        assertNotNull(hull);
    393         assertEquals(2, hull.relations.size());
     393        assertEquals(2, hull.getRelations().size());
    394394
    395395        r1 = (Relation)hull.getPrimitiveById(1, OsmPrimitiveType.RELATION);

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

    r2292 r2381  
    88
    99import java.io.File;
    10 import java.net.URLEncoder;
    1110import java.text.MessageFormat;
    1211import java.util.Arrays;
    … …  
    358357        Node n2 = (Node)my.getPrimitiveById(1,OsmPrimitiveType.NODE);
    359358        assertEquals(0,visitor.getConflicts().size());
    360         assertEquals(2,my.nodes.size());
     359        assertEquals(2, my.getNodes().size());
    361360        assertEquals(n,n2);
    362361    }
    … …  
    402401        visitor.merge();
    403402
    404         Node n2 = my.nodes.iterator().next();
     403        Node n2 = my.getNodes().iterator().next();
    405404        assertEquals(0,visitor.getConflicts().size());
    406405        assertEquals("value1",n2.get("key1"));
    … …  
    440439        visitor.merge();
    441440
    442         Node n2 = my.nodes.iterator().next();
     441        Node n2 = my.getNodes().iterator().next();
    443442        assertEquals(0,visitor.getConflicts().size());
    444443        assertEquals("value1",n2.get("key1"));
Note: See TracChangeset for help on using the changeset viewer.