Ignore:
Timestamp:
2009-09-07T23:06:19+02:00 (15 years ago)
Author:
Gubaer
Message:

Had to replace DataSet:getPrimitiveById(id) with DataSet:getPrimitiveById(id,type). Primitive ids are not globally unique, only per type of primitive.
Fixed problems in unit test, available unit tests passing again.

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

    r1951 r2077  
    1414import org.openstreetmap.josm.data.osm.Node;
    1515import org.openstreetmap.josm.data.osm.OsmPrimitive;
     16import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
    1617import org.openstreetmap.josm.data.osm.Relation;
    1718import org.openstreetmap.josm.data.osm.RelationMember;
    … …  
    5051        assertEquals(2, hull.nodes.size());
    5152
    52         OsmPrimitive p = hull.getPrimitiveById(1);
     53        OsmPrimitive p = hull.getPrimitiveById(1,OsmPrimitiveType.NODE);
    5354        assertNotNull(p);
    5455        assertEquals(p.getClass(), Node.class);
    5556
    56         p = hull.getPrimitiveById(3);
     57        p = hull.getPrimitiveById(3,OsmPrimitiveType.NODE);
    5758        assertNull(p);
    5859
    … …  
    7172        Node n2 = new Node(2);
    7273        Way w1 = new Way(3);
     74        w1.incomplete = false;
    7375        w1.addNode(n1);
    7476        w1.addNode(n2);
    … …  
    8486        assertEquals(2, hull.nodes.size());
    8587
    86         OsmPrimitive p = hull.getPrimitiveById(1);
     88        OsmPrimitive p = hull.getPrimitiveById(1,OsmPrimitiveType.NODE);
    8789        assertNotNull(p);
    8890        assertEquals(p.getClass(), Node.class);
    8991
    90         p = hull.getPrimitiveById(2);
     92        p = hull.getPrimitiveById(2,OsmPrimitiveType.NODE);
    9193        assertNotNull(p);
    9294        assertEquals(p.getClass(), Node.class);
    9395
    94         p = hull.getPrimitiveById(3);
     96        p = hull.getPrimitiveById(3, OsmPrimitiveType.WAY);
    9597        assertNotNull(p);
    9698        assertEquals(p.getClass(), Way.class);
    … …  
    103105        Node n2 = new Node(2);
    104106        Way w1 = new Way(3);
     107        w1.incomplete = false;
    105108        w1.addNode(n1);
    106109        w1.addNode(n2);
    … …  
    116119        assertEquals(2, hull.nodes.size());
    117120
    118         OsmPrimitive p = hull.getPrimitiveById(1);
     121        OsmPrimitive p = hull.getPrimitiveById(1,OsmPrimitiveType.NODE);
    119122        assertNotNull(p);
    120123        assertEquals(p.getClass(), Node.class);
    121124
    122         p = hull.getPrimitiveById(2);
     125        p = hull.getPrimitiveById(2, OsmPrimitiveType.NODE);
    123126        assertNotNull(p);
    124127        assertEquals(p.getClass(), Node.class);
    125128
    126         p = hull.getPrimitiveById(3);
     129        p = hull.getPrimitiveById(3,OsmPrimitiveType.WAY);
    127130        assertNotNull(p);
    128131        assertEquals(p.getClass(), Way.class);
    … …  
    142145        assertEquals(1, hull.ways.size());
    143146
    144         OsmPrimitive p = hull.getPrimitiveById(3);
     147        OsmPrimitive p = hull.getPrimitiveById(3, OsmPrimitiveType.WAY);
    145148        assertNotNull(p);
    146149        assertEquals(p.getClass(), Way.class);
    … …  
    154157        Node n20 = new Node(20);
    155158        r1.addMember(new RelationMember("node-20",n20));
     159        r1.incomplete = false;
    156160        Way w30 = new Way(30);
     161        w30.incomplete = false;
    157162        Node n21;
    158163        w30.addNode(n21 = new Node(21));
    … …  
    177182        assertEquals(2, hull.relations.size());
    178183
    179         OsmPrimitive p = hull.getPrimitiveById(1);
     184        OsmPrimitive p = hull.getPrimitiveById(1, OsmPrimitiveType.RELATION);
    180185        assertNotNull(p);
    181186        assertEquals(p.getClass(), Relation.class);
    182187
    183         Way w = (Way)hull.getPrimitiveById(30);
     188        Way w = (Way)hull.getPrimitiveById(30,OsmPrimitiveType.WAY);
    184189        assertNotNull(w);
    185190        assertEquals(2, w.getNodesCount());
    186         Node n = (Node)hull.getPrimitiveById(21);
     191        Node n = (Node)hull.getPrimitiveById(21, OsmPrimitiveType.NODE);
    187192        assertNotNull(n);
    188193        assertTrue(w.containsNode(n));
    189194
    190         n = (Node)hull.getPrimitiveById(22);
     195        n = (Node)hull.getPrimitiveById(22,OsmPrimitiveType.NODE);
    191196        assertNotNull(n);
    192197        assertTrue(w.containsNode(n));
    193198
    194         Relation r = (Relation)hull.getPrimitiveById(40);
    195         assertNotNull(r);
    196 
    197         r = (Relation)hull.getPrimitiveById(1);
     199        Relation r = (Relation)hull.getPrimitiveById(40,OsmPrimitiveType.RELATION);
     200        assertNotNull(r);
     201
     202        r = (Relation)hull.getPrimitiveById(1, OsmPrimitiveType.RELATION);
    198203        assertNotNull(r);
    199204        assertEquals(3, r.getMembersCount());
    200         RelationMember m = new RelationMember("node-20", hull.getPrimitiveById(20));
    201         assertTrue(r.getMembers().contains(m));
    202         m = new RelationMember("way-30", hull.getPrimitiveById(30));
    203         assertTrue(r.getMembers().contains(m));
    204         m = new RelationMember("relation-40", hull.getPrimitiveById(40));
     205        RelationMember m = new RelationMember("node-20", hull.getPrimitiveById(20,OsmPrimitiveType.NODE));
     206        assertTrue(r.getMembers().contains(m));
     207        m = new RelationMember("way-30", hull.getPrimitiveById(30, OsmPrimitiveType.WAY));
     208        assertTrue(r.getMembers().contains(m));
     209        m = new RelationMember("relation-40", hull.getPrimitiveById(40, OsmPrimitiveType.RELATION));
    205210        assertTrue(r.getMembers().contains(m));
    206211    }
    … …  
    213218        r1.addMember(new RelationMember("node-20",n20));
    214219        Way w30 = new Way(30);
     220        w30.incomplete = false;
    215221        Node n21;
    216222        w30.addNode(n21 = new Node(21));
    … …  
    235241        assertEquals(2, hull.relations.size());
    236242
    237         OsmPrimitive p = hull.getPrimitiveById(1);
     243        OsmPrimitive p = hull.getPrimitiveById(1, OsmPrimitiveType.RELATION);
    238244        assertNotNull(p);
    239245        assertEquals(p.getClass(), Relation.class);
    240246
    241         Way w = (Way)hull.getPrimitiveById(30);
     247        Way w = (Way)hull.getPrimitiveById(30, OsmPrimitiveType.WAY);
    242248        assertNotNull(w);
    243249        assertTrue(w.incomplete);
    244250
    245251
    246         Node n = (Node)hull.getPrimitiveById(21);
     252        Node n = (Node)hull.getPrimitiveById(21,OsmPrimitiveType.NODE);
    247253        assertNull(n);
    248254
    249         n = (Node)hull.getPrimitiveById(22);
     255        n = (Node)hull.getPrimitiveById(22, OsmPrimitiveType.NODE);
    250256        assertNull(n);
    251257
    252         Relation r = (Relation)hull.getPrimitiveById(40);
     258        Relation r = (Relation)hull.getPrimitiveById(40, OsmPrimitiveType.RELATION);
    253259        assertNotNull(r);
    254260        assertTrue(r.incomplete);
    255261
    256         r = (Relation)hull.getPrimitiveById(1);
     262        r = (Relation)hull.getPrimitiveById(1, OsmPrimitiveType.RELATION);
    257263        assertNotNull(r);
    258264        assertEquals(3, r.getMembersCount());
    259         RelationMember m = new RelationMember("node-20", hull.getPrimitiveById(20));
    260         assertTrue(r.getMembers().contains(m));
    261         m = new RelationMember("way-30", hull.getPrimitiveById(30));
    262         assertTrue(r.getMembers().contains(m));
    263         m = new RelationMember("relation-40", hull.getPrimitiveById(40));
     265        RelationMember m = new RelationMember("node-20", hull.getPrimitiveById(20, OsmPrimitiveType.NODE));
     266        assertTrue(r.getMembers().contains(m));
     267        m = new RelationMember("way-30", hull.getPrimitiveById(30, OsmPrimitiveType.WAY));
     268        assertTrue(r.getMembers().contains(m));
     269        m = new RelationMember("relation-40", hull.getPrimitiveById(40, OsmPrimitiveType.RELATION));
    264270        assertTrue(r.getMembers().contains(m));
    265271    }
    … …  
    272278        Node n20 = new Node(new LatLon(20.0,20.0));
    273279        n20.put("name", "n20");
    274         r1.getMembers().add(new RelationMember("node-20",n20));
     280        r1.addMember(new RelationMember("node-20",n20));
     281
    275282        Way w30 = new Way();
    276283        w30.put("name", "w30");
    … …  
    285292        r40.put("name", "r40");
    286293        r1.addMember(new RelationMember("relation-40", r40));
     294
    287295        source.nodes.add(n20);
    288296        source.nodes.add(n21);
    … …  
    343351        assertEquals(1, hull.relations.size());
    344352
    345         Relation r = (Relation)hull.getPrimitiveById(1);
     353        Relation r = (Relation)hull.getPrimitiveById(1, OsmPrimitiveType.RELATION);
    346354        assertNotNull(r);
    347355        assertEquals(1, r.getMembersCount());
    … …  
    385393        assertEquals(2, hull.relations.size());
    386394
    387         r1 = (Relation)hull.getPrimitiveById(1);
     395        r1 = (Relation)hull.getPrimitiveById(1, OsmPrimitiveType.RELATION);
    388396        assertNotNull(r1);
    389         r2 = (Relation)hull.getPrimitiveById(2);
     397        r2 = (Relation)hull.getPrimitiveById(2, OsmPrimitiveType.RELATION);
    390398        assertNotNull(r2);
    391399        assertEquals(1, r1.getMembersCount());

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

    r2071 r2077  
    2424import org.openstreetmap.josm.data.osm.Node;
    2525import org.openstreetmap.josm.data.osm.OsmPrimitive;
     26import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
    2627import org.openstreetmap.josm.data.osm.Relation;
    2728import org.openstreetmap.josm.data.osm.RelationMember;
    … …  
    9798        visitor.merge();
    9899
    99         Node n2 = (Node)my.getPrimitiveById(1);
     100        Node n2 = (Node)my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    100101        assertTrue(visitor.getConflicts().isEmpty());
    101102        assertEquals(1, n2.getId());
    … …  
    133134        visitor.merge();
    134135
    135         Node n2 = (Node)my.getPrimitiveById(1);
     136        Node n2 = (Node)my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    136137        assertTrue(visitor.getConflicts().isEmpty());
    137138        assertEquals(1, n2.getId());
    … …  
    173174        visitor.merge();
    174175
    175         Node n2 = (Node)my.getPrimitiveById(1);
     176        Node n2 = (Node)my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    176177        assertEquals(1,visitor.getConflicts().size());
    177178        assertEquals(n, n2);
    … …  
    209210        visitor.merge();
    210211
    211         Node n2 = (Node)my.getPrimitiveById(1);
     212        Node n2 = (Node)my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    212213        assertEquals(1,visitor.getConflicts().size());
    213214        assertEquals(n, n2);
    … …  
    242243        visitor.merge();
    243244
    244         Node n2 = (Node)my.getPrimitiveById(1);
     245        Node n2 = (Node)my.getPrimitiveById(1,OsmPrimitiveType.NODE);
    245246        assertEquals(1,visitor.getConflicts().size());
    246247        assertEquals(true, n2.isVisible());
    … …  
    271272        visitor.merge();
    272273
    273         Node n2 = (Node)my.getPrimitiveById(1);
     274        Node n2 = (Node)my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    274275        assertEquals(0,visitor.getConflicts().size());
    275276        assertEquals(true, n2.isVisible());
    … …  
    354355        visitor.merge();
    355356
    356         Node n2 = (Node)my.getPrimitiveById(1);
     357        Node n2 = (Node)my.getPrimitiveById(1,OsmPrimitiveType.NODE);
    357358        assertEquals(0,visitor.getConflicts().size());
    358359        assertEquals(2,my.nodes.size());
    … …  
    501502        visitor.merge();
    502503
    503         Way merged = (Way)my.getPrimitiveById(3);
     504        Way merged = (Way)my.getPrimitiveById(3, OsmPrimitiveType.WAY);
    504505        assertEquals(0,visitor.getConflicts().size());
    505506        assertEquals("value1",merged.get("key1"));
    … …  
    571572        visitor.merge();
    572573
    573         Way merged = (Way)my.getPrimitiveById(3);
     574        Way merged = (Way)my.getPrimitiveById(3, OsmPrimitiveType.WAY);
    574575        assertEquals(0,visitor.getConflicts().size());
    575576        assertEquals(3,merged.getId());
    … …  
    642643        visitor.merge();
    643644
    644         Way merged = (Way)my.getPrimitiveById(3);
     645        Way merged = (Way)my.getPrimitiveById(3, OsmPrimitiveType.WAY);
    645646        assertEquals(1,visitor.getConflicts().size());
    646647        assertEquals(3,merged.getId());
    … …  
    690691        visitor.merge();
    691692
    692         Way merged = (Way)my.getPrimitiveById(3);
     693        Way merged = (Way)my.getPrimitiveById(3, OsmPrimitiveType.WAY);
    693694        assertEquals(1,visitor.getConflicts().size());
    694695        assertEquals(true, visitor.getConflicts().hasConflictForMy(myWay));
    … …  
    851852        assertEquals(0,visitor.getConflicts().size());
    852853
    853         Way myWay = (Way)my.getPrimitiveById(4);
     854        Way myWay = (Way)my.getPrimitiveById(4,OsmPrimitiveType.WAY);
    854855        assertEquals(2, myWay.getNodesCount());
    855856
    856         Node n = (Node)my.getPrimitiveById(1);
     857        Node n = (Node)my.getPrimitiveById(1,OsmPrimitiveType.NODE);
    857858        assertTrue(n != null);
    858859    }
    … …  
    906907        assertEquals(0,visitor.getConflicts().size());
    907908
    908         Relation r = (Relation)my.getPrimitiveById(4);
     909        Relation r = (Relation)my.getPrimitiveById(4,OsmPrimitiveType.RELATION);
    909910        assertEquals(2, r.getMembersCount());
    910911
    911         Node n = (Node)my.getPrimitiveById(1);
     912        Node n = (Node)my.getPrimitiveById(1,OsmPrimitiveType.NODE);
    912913        assertTrue(n != null);
    913914    }
    … …  
    942943        assertEquals(0,visitor.getConflicts().size());
    943944
    944         OsmPrimitive p= my.getPrimitiveById(1);
     945        OsmPrimitive p= my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    945946        assertNotNull(p);
    946947        assertTrue(p.incomplete);
    947         p= my.getPrimitiveById(2);
     948        p= my.getPrimitiveById(2, OsmPrimitiveType.NODE);
    948949        assertNotNull(p);
    949950        assertTrue(p.incomplete);
    950         p= my.getPrimitiveById(3);
     951        p= my.getPrimitiveById(3, OsmPrimitiveType.WAY);
    951952        assertNotNull(p);
    952953        assertTrue(p.incomplete);
    953954
    954         Way w = (Way)my.getPrimitiveById(3);
     955        Way w = (Way)my.getPrimitiveById(3, OsmPrimitiveType.WAY);
    955956        assertNotNull(w);
    956957        assertTrue(p.incomplete);
    … …  
    10051006        assertEquals(0,visitor.getConflicts().size());
    10061007
    1007         OsmPrimitive p= my.getPrimitiveById(1);
     1008        OsmPrimitive p= my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    10081009        assertNotNull(p);
    10091010        assertTrue(!p.incomplete);
    1010         p= my.getPrimitiveById(2);
     1011        p= my.getPrimitiveById(2, OsmPrimitiveType.NODE);
    10111012        assertNotNull(p);
    10121013        assertTrue(!p.incomplete);
    1013         p= my.getPrimitiveById(3);
     1014        p= my.getPrimitiveById(3,OsmPrimitiveType.WAY);
    10141015        assertNotNull(p);
    10151016        assertTrue(!p.incomplete);
    10161017
    1017         Way w = (Way)my.getPrimitiveById(3);
     1018        Way w = (Way)my.getPrimitiveById(3,OsmPrimitiveType.WAY);
    10181019        assertNotNull(w);
    10191020        assertTrue(!p.incomplete);
Note: See TracChangeset for help on using the changeset viewer.