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