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.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • 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.