Changeset 2077 in josm for trunk/test


Ignore:
Timestamp:
2009-09-07T23:06:19+02:00 (14 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
Files:
6 edited

Legend:

Unmodified
Added
Removed

  • trunk/test/functional/org/openstreetmap/josm/gui/conflict/pair/properties/PropertiesMergerTest.java

    r2041 r2077  
    2626        Node my = new Node(1);
    2727        my.setCoor(new LatLon(1,1));
    28         my.deleted = true;
     28        my.setDeleted(true);
    2929
    3030        Node their = new Node(2);

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

    r2072 r2077  
    2828import org.openstreetmap.josm.data.osm.Node;
    2929import org.openstreetmap.josm.data.osm.OsmPrimitive;
     30import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
    3031import org.openstreetmap.josm.data.osm.Relation;
    3132import org.openstreetmap.josm.data.osm.RelationMember;
    … …  
    247248        while(it.hasNext()) {
    248249            Node n1 = it.next();
    249             Node n2 = (Node)ds.getPrimitiveById(n1.id);
     250            Node n2 = (Node)ds.getPrimitiveById(n1.getId(), OsmPrimitiveType.NODE);
    250251            assertNotNull(n2);
    251252            assertEquals(n2.get("name"),n2.get("name"));
    … …  
    266267        while(it.hasNext()) {
    267268            Way w1 = it.next();
    268             Way w2 = (Way)ds.getPrimitiveById(w1.id);
     269            Way w2 = (Way)ds.getPrimitiveById(w1.getId(), OsmPrimitiveType.WAY);
    269270            assertNotNull(w2);
    270271            assertEquals(w2.getNodesCount(), w1.getNodesCount());
    … …  
    286287        while(it.hasNext()) {
    287288            Relation r1 = it.next();
    288             Relation r2 = (Relation)ds.getPrimitiveById(r1.id);
     289            Relation r2 = (Relation)ds.getPrimitiveById(r1.getId(), OsmPrimitiveType.RELATION);
    289290            assertNotNull(r2);
    290291            assertEquals(r2.getMembersCount(), r1.getMembersCount());
    … …  
    306307        while(it.hasNext()) {
    307308            Node n1 = it.next();
    308             Node n2 = (Node)ds.getPrimitiveById(n1.id);
     309            Node n2 = (Node)ds.getPrimitiveById(n1.getId(), OsmPrimitiveType.NODE);
    309310            assertNotNull(n2);
    310311            assertEquals(n2.get("name"),n2.get("name"));
    … …  
    327328        while(it.hasNext()) {
    328329            Node n1 = it.next();
    329             Node n2 = (Node)ds.getPrimitiveById(n1.id);
     330            Node n2 = (Node)ds.getPrimitiveById(n1.getId(), OsmPrimitiveType.NODE);
    330331            assertNotNull(n2);
    331332            assertEquals(n2.get("name"),n2.get("name"));

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

    r2072 r2077  
    2727import org.openstreetmap.josm.data.osm.Node;
    2828import org.openstreetmap.josm.data.osm.OsmPrimitive;
     29import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
    2930import org.openstreetmap.josm.data.osm.Relation;
    3031import org.openstreetmap.josm.data.osm.RelationMember;
    … …  
    253254        assertEquals(0, referers.relations.size());
    254255        for (Way way: referers.ways) {
    255             assertEquals(w.id, way.id);
     256            assertEquals(w.getId(), way.getId());
    256257            assertEquals(false, way.incomplete);
    257258        }
    … …  
    272273        assertEquals(0, referers.relations.size());
    273274        for (Way way: referers.ways) {
    274             assertEquals(w.id, way.id);
     275            assertEquals(w.getId(), way.getId());
    275276            assertEquals(false, way.incomplete);
    276277            assertEquals(10, w.getNodesCount());
    … …  
    319320        Set<Long> expectedNodeIds = new HashSet<Long>();
    320321        for (Way way: referers.ways) {
    321             Way orig = (Way)ds.getPrimitiveById(way.id);
     322            Way orig = (Way)ds.getPrimitiveById(way.getId(), OsmPrimitiveType.WAY);
    322323            for(Node n: orig.getNodes()) {
    323                 expectedNodeIds.add(n.id);
     324                expectedNodeIds.add(n.getId());
    324325            }
    325326        }
    326327        assertEquals(expectedNodeIds.size(), referers.nodes.size());
    327328        for (Node n : referers.nodes) {
    328             assertEquals(true, expectedNodeIds.contains(n.id));
     329            assertEquals(true, expectedNodeIds.contains(n.getId()));
    329330        }
    330331
    … …  
    352353        assertNotNull(r);
    353354        assertEquals(false, r.incomplete);
    354         referringRelationsIds.add(r.id);
     355        referringRelationsIds.add(r.getId());
    355356        r = lookupRelation(referers, 7);
    356357        assertNotNull(r);
    357358        assertEquals(false, r.incomplete);
    358         referringRelationsIds.add(r.id);
     359        referringRelationsIds.add(r.getId());
    359360        r = lookupRelation(referers, 8);
    360361        assertNotNull(r);
    361362        assertEquals(false, r.incomplete);
    362         referringRelationsIds.add(r.id);
     363        referringRelationsIds.add(r.getId());
    363364        r = lookupRelation(referers, 9);
    364365        assertNotNull(r);
    365366        assertEquals(false, r.incomplete);
    366         referringRelationsIds.add(r.id);
     367        referringRelationsIds.add(r.getId());
    367368
    368369        for (Relation r1: referers.relations) {
    369             if (! referringRelationsIds.contains(r1.id)) {
     370            if (! referringRelationsIds.contains(r1.getId())) {
    370371                assertEquals(true, r1.incomplete);
    371372            }
    … …  
    378379        for (RelationMember m : lookupRelation(ds, 6).getMembers()) {
    379380            if (m.isWay()) {
    380                 expectedWayIds.add(m.getMember().id);
     381                expectedWayIds.add(m.getMember().getId());
    381382            }
    382383        }
    383384        for (RelationMember m : lookupRelation(ds, 7).getMembers()) {
    384385            if (m.isWay()) {
    385                 expectedWayIds.add(m.getMember().id);
     386                expectedWayIds.add(m.getMember().getId());
    386387            }
    387388        }
    388389        for (RelationMember m : lookupRelation(ds, 8).getMembers()) {
    389390            if (m.isWay()) {
    390                 expectedWayIds.add(m.getMember().id);
     391                expectedWayIds.add(m.getMember().getId());
    391392            }
    392393        }
    393394        for (RelationMember m : lookupRelation(ds, 9).getMembers()) {
    394395            if (m.isWay()) {
    395                 expectedWayIds.add(m.getMember().id);
     396                expectedWayIds.add(m.getMember().getId());
    396397            }
    397398        }
    … …  
    399400        assertEquals(expectedWayIds.size(), referers.ways.size());
    400401        for (Way w1 : referers.ways) {
    401             assertEquals(true, expectedWayIds.contains(w1.id));
     402            assertEquals(true, expectedWayIds.contains(w1.getId()));
    402403            assertEquals(true, w1.incomplete);
    403404        }
    … …  
    412413        if (way == null)return ret;
    413414        for (Node n: way.getNodes()) {
    414             ret.add(n.id);
     415            ret.add(n.getId());
    415416        }
    416417        return ret;
    … …  
    422423        for (RelationMember m: r.getMembers()) {
    423424            if (m.isNode()) {
    424                 ret.add(m.getMember().id);
     425                ret.add(m.getMember().getId());
    425426            } else if (m.isWay()) {
    426427                ret.addAll(getNodeIdsInWay(m.getWay()));
    … …  
    448449        assertNotNull(r);
    449450        assertEquals(false, r.incomplete);
    450         referringRelationsIds.add(r.id);
     451        referringRelationsIds.add(r.getId());
    451452        r = lookupRelation(referers, 7);
    452453        assertNotNull(r);
    453454        assertEquals(false, r.incomplete);
    454         referringRelationsIds.add(r.id);
     455        referringRelationsIds.add(r.getId());
    455456        r = lookupRelation(referers, 8);
    456457        assertNotNull(r);
    457458        assertEquals(false, r.incomplete);
    458         referringRelationsIds.add(r.id);
     459        referringRelationsIds.add(r.getId());
    459460        r = lookupRelation(referers, 9);
    460461        assertNotNull(r);
    461462        assertEquals(false, r.incomplete);
    462         referringRelationsIds.add(r.id);
     463        referringRelationsIds.add(r.getId());
    463464
    464465        // all relations are fully loaded
    … …  
    474475        for (RelationMember m : lookupRelation(ds, 6).getMembers()) {
    475476            if (m.isWay()) {
    476                 expectedWayIds.add(m.getMember().id);
     477                expectedWayIds.add(m.getMember().getId());
    477478            }
    478479        }
    479480        for (RelationMember m : lookupRelation(ds, 7).getMembers()) {
    480481            if (m.isWay()) {
    481                 expectedWayIds.add(m.getMember().id);
     482                expectedWayIds.add(m.getMember().getId());
    482483            }
    483484        }
    484485        for (RelationMember m : lookupRelation(ds, 8).getMembers()) {
    485486            if (m.isWay()) {
    486                 expectedWayIds.add(m.getMember().id);
     487                expectedWayIds.add(m.getMember().getId());
    487488            }
    488489        }
    489490        for (RelationMember m : lookupRelation(ds, 9).getMembers()) {
    490491            if (m.isWay()) {
    491                 expectedWayIds.add(m.getMember().id);
     492                expectedWayIds.add(m.getMember().getId());
    492493            }
    493494        }
    494495        for (long id : expectedWayIds) {
    495             Way w = (Way)referers.getPrimitiveById(id);
     496            Way w = (Way)referers.getPrimitiveById(id, OsmPrimitiveType.WAY);
    496497            assertNotNull(w);
    497498            assertEquals(false, w.incomplete);
    … …  
    506507        assertEquals(expectedNodeIds.size(), referers.nodes.size());
    507508        for(Node n : referers.nodes) {
    508             assertEquals(true, expectedNodeIds.contains(n.id));
     509            assertEquals(true, expectedNodeIds.contains(n.getId()));
    509510        }
    510511    }

  • 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);

  • trunk/test/unit/org/openstreetmap/josm/gui/conflict/nodes/NodeListMergeModelTest.java

    r1954 r2077  
    6565
    6666        Way myWay = new Way(1);
     67        myWay.incomplete = false;
    6768        myWay.addNode(new Node(2));
    6869        myWay.addNode(new Node(3));
    … …  
    7576
    7677        assertEquals(1, mergedNodes.size());
    77         assertEquals(2, mergedNodes.get(0).id);
     78        assertEquals(2, mergedNodes.get(0).getId());
    7879
    7980        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    … …  
    8990
    9091        Way myWay = new Way(1);
     92        myWay.incomplete = false;
    9193        myWay.addNode(new Node(2));
    9294        myWay.addNode(new Node(3));
    … …  
    102104        mergedNodes = inspectNodeList(model, "Merged");
    103105        assertEquals(2, mergedNodes.size());
    104         assertEquals(2, mergedNodes.get(0).id);
    105         assertEquals(1, mergedNodes.get(1).id);
     106        assertEquals(2, mergedNodes.get(0).getId());
     107        assertEquals(1, mergedNodes.get(1).getId());
    106108
    107109        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    … …  
    120122
    121123        Way myWay = new Way(1);
    122         myWay.addNode(new Node(2));
    123         myWay.addNode(new Node(3));
    124         Way theirWay = new Way(1);
     124        myWay.incomplete = false;
     125        myWay.addNode(new Node(2));
     126        myWay.addNode(new Node(3));
     127        Way theirWay = new Way(1);
     128
    125129
    126130        model.populate(myWay, theirWay);
    … …  
    133137        mergedNodes = inspectNodeList(model, "Merged");
    134138        assertEquals(2, mergedNodes.size());
    135         assertEquals(3, mergedNodes.get(0).id); // my node 3 at position 0
    136         assertEquals(1, mergedNodes.get(1).id); // already merged node 1 at position 1
     139        assertEquals(3, mergedNodes.get(0).getId()); // my node 3 at position 0
     140        assertEquals(1, mergedNodes.get(1).getId()); // already merged node 1 at position 1
    137141
    138142        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    … …  
    146150
    147151        Way myWay = new Way(1);
     152        myWay.incomplete = false;
    148153        myWay.addNode(new Node(2));
    149154        myWay.addNode(new Node(3));
    … …  
    160165        mergedNodes = inspectNodeList(model, "Merged");
    161166        assertEquals(3, mergedNodes.size());
    162         assertEquals(3, mergedNodes.get(0).id); // my node 3 at position 0
    163         assertEquals(4, mergedNodes.get(1).id); // my node 4 at position 1
    164         assertEquals(1, mergedNodes.get(2).id); // already merged node 1 at position 2
     167        assertEquals(3, mergedNodes.get(0).getId()); // my node 3 at position 0
     168        assertEquals(4, mergedNodes.get(1).getId()); // my node 4 at position 1
     169        assertEquals(1, mergedNodes.get(2).getId()); // already merged node 1 at position 2
    165170
    166171        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    … …  
    174179
    175180        Way myWay = new Way(1);
     181        myWay.incomplete = false;
    176182        myWay.addNode(new Node(2));
    177183        myWay.addNode(new Node(3));
    … …  
    184190
    185191        assertEquals(1, mergedNodes.size());
    186         assertEquals(2, mergedNodes.get(0).id);
     192        assertEquals(2, mergedNodes.get(0).getId());
    187193
    188194        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    … …  
    195201
    196202        Way myWay = new Way(1);
     203        myWay.incomplete = false;
    197204        myWay.addNode(new Node(2));
    198205        myWay.addNode(new Node(3));
    … …  
    208215        mergedNodes = inspectNodeList(model, "Merged");
    209216        assertEquals(2, mergedNodes.size());
    210         assertEquals(1, mergedNodes.get(0).id); // already merged node 1 at position 0
    211         assertEquals(2, mergedNodes.get(1).id); // copied node 2 at position 1
     217        assertEquals(1, mergedNodes.get(0).getId()); // already merged node 1 at position 0
     218        assertEquals(2, mergedNodes.get(1).getId()); // copied node 2 at position 1
    212219
    213220        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    … …  
    221228
    222229        Way myWay = new Way(1);
     230        myWay.incomplete = false;
    223231        myWay.addNode(new Node(2));
    224232        myWay.addNode(new Node(3));
    … …  
    234242        mergedNodes = inspectNodeList(model, "Merged");
    235243        assertEquals(2, mergedNodes.size());
    236         assertEquals(1, mergedNodes.get(0).id); // already merged node 1 at position 0
    237         assertEquals(3, mergedNodes.get(1).id); // my node 3 at position 1
     244        assertEquals(1, mergedNodes.get(0).getId()); // already merged node 1 at position 0
     245        assertEquals(3, mergedNodes.get(1).getId()); // my node 3 at position 1
    238246
    239247
    … …  
    248256
    249257        Way myWay = new Way(1);
     258        myWay.incomplete = false;
    250259        myWay.addNode(new Node(2));
    251260        myWay.addNode(new Node(3));
    … …  
    262271        mergedNodes = inspectNodeList(model, "Merged");
    263272        assertEquals(3, mergedNodes.size());
    264         assertEquals(1, mergedNodes.get(0).id); // already merged node 1 at position 0
    265         assertEquals(3, mergedNodes.get(1).id); // my node 3 at position 1
    266         assertEquals(4, mergedNodes.get(2).id); // my node 4 at position 2
     273        assertEquals(1, mergedNodes.get(0).getId()); // already merged node 1 at position 0
     274        assertEquals(3, mergedNodes.get(1).getId()); // my node 3 at position 1
     275        assertEquals(4, mergedNodes.get(2).getId()); // my node 4 at position 2
    267276
    268277
    … …  
    280289
    281290        Way myWay = new Way(1);
     291        myWay.incomplete = false;
    282292        myWay.addNode(new Node(1));
    283293        myWay.addNode(new Node(2));
    … …  
    293303
    294304        assertEquals(4, mergedNodes.size());
    295         assertEquals(10, mergedNodes.get(0).id); // already merged node
    296         assertEquals(1, mergedNodes.get(1).id);  // copied node 1 at position 1
    297         assertEquals(11, mergedNodes.get(2).id); // already merged node
    298         assertEquals(12, mergedNodes.get(3).id); // already merged node
     305        assertEquals(10, mergedNodes.get(0).getId()); // already merged node
     306        assertEquals(1, mergedNodes.get(1).getId());  // copied node 1 at position 1
     307        assertEquals(11, mergedNodes.get(2).getId()); // already merged node
     308        assertEquals(12, mergedNodes.get(3).getId()); // already merged node
    299309
    300310        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    … …  
    308318
    309319        Way myWay = new Way(1);
     320        myWay.incomplete = false;
    310321        myWay.addNode(new Node(1));
    311322        myWay.addNode(new Node(2));
    … …  
    321332
    322333        assertEquals(5, mergedNodes.size());
    323         assertEquals(1, mergedNodes.get(0).id);  // copied node 1 at position 0
    324         assertEquals(2, mergedNodes.get(1).id);  // copied node 2 at position 1
    325         assertEquals(10, mergedNodes.get(2).id); // already merged node
    326         assertEquals(11, mergedNodes.get(3).id); // already merged node
    327         assertEquals(12, mergedNodes.get(4).id); // already merged node
     334        assertEquals(1, mergedNodes.get(0).getId());  // copied node 1 at position 0
     335        assertEquals(2, mergedNodes.get(1).getId());  // copied node 2 at position 1
     336        assertEquals(10, mergedNodes.get(2).getId()); // already merged node
     337        assertEquals(11, mergedNodes.get(3).getId()); // already merged node
     338        assertEquals(12, mergedNodes.get(4).getId()); // already merged node
    328339
    329340        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    … …  
    336347
    337348        Way myWay = new Way(1);
     349        myWay.incomplete = false;
    338350        myWay.addNode(new Node(1));
    339351        myWay.addNode(new Node(2));
    … …  
    369381
    370382        Way myWay = new Way(1);
     383        myWay.incomplete = false;
    371384        myWay.addNode(new Node(1));
    372385        myWay.addNode(new Node(2));
    … …  
    382395
    383396        assertEquals(4, mergedNodes.size());
    384         assertEquals(10, mergedNodes.get(0).id); // already merged node
    385         assertEquals(11, mergedNodes.get(1).id); // already merged node
    386         assertEquals(1, mergedNodes.get(2).id);  // copied node 1 at position 2
    387         assertEquals(12, mergedNodes.get(3).id); // already merged node
     397        assertEquals(10, mergedNodes.get(0).getId()); // already merged node
     398        assertEquals(11, mergedNodes.get(1).getId()); // already merged node
     399        assertEquals(1, mergedNodes.get(2).getId());  // copied node 1 at position 2
     400        assertEquals(12, mergedNodes.get(3).getId()); // already merged node
    388401
    389402        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    … …  
    397410
    398411        Way myWay = new Way(1);
     412        myWay.incomplete = false;
    399413        myWay.addNode(new Node(1));
    400414        myWay.addNode(new Node(2));
    … …  
    411425
    412426        assertEquals(5, mergedNodes.size());
    413         assertEquals(10, mergedNodes.get(0).id); // already merged node
    414         assertEquals(11, mergedNodes.get(1).id); // already merged node
    415         assertEquals(12, mergedNodes.get(2).id); // already merged node
    416         assertEquals(1, mergedNodes.get(3).id);  // copied node 1 at position 3
    417         assertEquals(2, mergedNodes.get(4).id);  // copied node 2 at position 4
     427        assertEquals(10, mergedNodes.get(0).getId()); // already merged node
     428        assertEquals(11, mergedNodes.get(1).getId()); // already merged node
     429        assertEquals(12, mergedNodes.get(2).getId()); // already merged node
     430        assertEquals(1, mergedNodes.get(3).getId());  // copied node 1 at position 3
     431        assertEquals(2, mergedNodes.get(4).getId());  // copied node 2 at position 4
    418432
    419433
    … …  
    427441
    428442        Way myWay = new Way(1);
     443        myWay.incomplete = false;
    429444        myWay.addNode(new Node(1));
    430445        myWay.addNode(new Node(2));
    … …  
    441456
    442457        assertEquals(5, mergedNodes.size());
    443         assertEquals(10, mergedNodes.get(0).id); // already merged node
    444         assertEquals(1, mergedNodes.get(1).id);  // copied node 1 at position 1
    445         assertEquals(3, mergedNodes.get(2).id);  // copied node 3 at position 2
    446         assertEquals(11, mergedNodes.get(3).id); // already merged node
    447         assertEquals(12, mergedNodes.get(4).id); // already merged node
     458        assertEquals(10, mergedNodes.get(0).getId()); // already merged node
     459        assertEquals(1, mergedNodes.get(1).getId());  // copied node 1 at position 1
     460        assertEquals(3, mergedNodes.get(2).getId());  // copied node 3 at position 2
     461        assertEquals(11, mergedNodes.get(3).getId()); // already merged node
     462        assertEquals(12, mergedNodes.get(4).getId()); // already merged node
    448463
    449464        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    … …  
    456471
    457472        Way myWay = new Way(1);
     473        myWay.incomplete = false;
    458474        myWay.addNode(new Node(1));
    459475        myWay.addNode(new Node(2));
    … …  
    489505
    490506        Way myWay = new Way(1);
     507        myWay.incomplete = false;
    491508        myWay.addNode(new Node(1));
    492509        myWay.addNode(new Node(2));
    … …  
    502519
    503520        assertEquals(3, mergedNodes.size());
    504         assertEquals(11, mergedNodes.get(0).id);
    505         assertEquals(10, mergedNodes.get(1).id);
    506         assertEquals(12, mergedNodes.get(2).id);
     521        assertEquals(11, mergedNodes.get(0).getId());
     522        assertEquals(10, mergedNodes.get(1).getId());
     523        assertEquals(12, mergedNodes.get(2).getId());
    507524
    508525        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    … …  
    515532
    516533        Way myWay = new Way(1);
     534        myWay.incomplete = false;
    517535        myWay.addNode(new Node(1));
    518536        myWay.addNode(new Node(2));
    … …  
    530548
    531549        assertEquals(5, mergedNodes.size());
    532         assertEquals(11, mergedNodes.get(0).id);
    533         assertEquals(10, mergedNodes.get(1).id);
    534         assertEquals(12, mergedNodes.get(2).id);
    535         assertEquals(14, mergedNodes.get(3).id);
    536         assertEquals(13, mergedNodes.get(4).id);
     550        assertEquals(11, mergedNodes.get(0).getId());
     551        assertEquals(10, mergedNodes.get(1).getId());
     552        assertEquals(12, mergedNodes.get(2).getId());
     553        assertEquals(14, mergedNodes.get(3).getId());
     554        assertEquals(13, mergedNodes.get(4).getId());
    537555
    538556        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    … …  
    545563
    546564        Way myWay = new Way(1);
     565        myWay.incomplete = false;
    547566        myWay.addNode(new Node(1));
    548567        myWay.addNode(new Node(2));
    … …  
    560579
    561580        assertEquals(5, mergedNodes.size());
    562         assertEquals(11, mergedNodes.get(0).id);
    563         assertEquals(12, mergedNodes.get(1).id);
    564         assertEquals(13, mergedNodes.get(2).id);
    565         assertEquals(14, mergedNodes.get(3).id);
    566         assertEquals(10, mergedNodes.get(4).id);
     581        assertEquals(11, mergedNodes.get(0).getId());
     582        assertEquals(12, mergedNodes.get(1).getId());
     583        assertEquals(13, mergedNodes.get(2).getId());
     584        assertEquals(14, mergedNodes.get(3).getId());
     585        assertEquals(10, mergedNodes.get(4).getId());
    567586
    568587        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    … …  
    578597
    579598        Way myWay = new Way(1);
     599        myWay.incomplete = false;
    580600        myWay.addNode(new Node(1));
    581601        myWay.addNode(new Node(2));
    … …  
    591611
    592612        assertEquals(3, mergedNodes.size());
    593         assertEquals(10, mergedNodes.get(0).id);
    594         assertEquals(12, mergedNodes.get(1).id);
    595         assertEquals(11, mergedNodes.get(2).id);
     613        assertEquals(10, mergedNodes.get(0).getId());
     614        assertEquals(12, mergedNodes.get(1).getId());
     615        assertEquals(11, mergedNodes.get(2).getId());
    596616
    597617        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    … …  
    604624
    605625        Way myWay = new Way(1);
     626        myWay.incomplete = false;
    606627        myWay.addNode(new Node(1));
    607628        myWay.addNode(new Node(2));
    … …  
    619640
    620641        assertEquals(5, mergedNodes.size());
    621         assertEquals(10, mergedNodes.get(0).id);
    622         assertEquals(12, mergedNodes.get(1).id);
    623         assertEquals(11, mergedNodes.get(2).id);
    624         assertEquals(14, mergedNodes.get(3).id);
    625         assertEquals(13, mergedNodes.get(4).id);
     642        assertEquals(10, mergedNodes.get(0).getId());
     643        assertEquals(12, mergedNodes.get(1).getId());
     644        assertEquals(11, mergedNodes.get(2).getId());
     645        assertEquals(14, mergedNodes.get(3).getId());
     646        assertEquals(13, mergedNodes.get(4).getId());
    626647
    627648        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
    … …  
    634655
    635656        Way myWay = new Way(1);
     657        myWay.incomplete = false;
    636658        myWay.addNode(new Node(1));
    637659        myWay.addNode(new Node(2));
    … …  
    649671
    650672        assertEquals(5, mergedNodes.size());
    651         assertEquals(10, mergedNodes.get(0).id);
    652         assertEquals(14, mergedNodes.get(1).id);
    653         assertEquals(11, mergedNodes.get(2).id);
    654         assertEquals(12, mergedNodes.get(3).id);
    655         assertEquals(13, mergedNodes.get(4).id);
     673        assertEquals(10, mergedNodes.get(0).getId());
     674        assertEquals(14, mergedNodes.get(1).getId());
     675        assertEquals(11, mergedNodes.get(2).getId());
     676        assertEquals(12, mergedNodes.get(3).getId());
     677        assertEquals(13, mergedNodes.get(4).getId());
    656678
    657679        DefaultListSelectionModel mergedSelection = inspectListSelectionModel(model, "mergedEntriesSelectionModel");
Note: See TracChangeset for help on using the changeset viewer.