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/functional/org/openstreetmap/josm/io
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • 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    }
Note: See TracChangeset for help on using the changeset viewer.