Ignore:
Timestamp:
2015-06-03T01:27:41+02:00 (9 years ago)
Author:
Don-vip
Message:

unit tests - simplify assertions

File:
1 edited

Legend:

Unmodified
Added
Removed

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

    r7937 r8450  
    55import static org.junit.Assert.assertFalse;
    66import static org.junit.Assert.assertNotNull;
     7import static org.junit.Assert.assertNotSame;
     8import static org.junit.Assert.assertSame;
    79import static org.junit.Assert.assertTrue;
    810
    … …  
    9193        Node n2 = (Node)my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    9294        assertTrue(visitor.getConflicts().isEmpty());
    93         assertTrue(n1 != n2); // make sure we have a clone
     95        assertNotSame(n1, n2); // make sure we have a clone
    9496        assertEquals(1, n2.getId());
    9597        assertEquals(1, n2.getVersion());
    96         assertEquals(false, n2.isModified());
     98        assertFalse(n2.isModified());
    9799        assertEquals("value1", n2.get("key1"));
    98100
    99101        // merge target not modified after merging
    100         assertTrue(!n2.isModified());
     102        assertFalse(n2.isModified());
    101103    }
    102104
    … …  
    127129        Node n2 = (Node)my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    128130        assertTrue(visitor.getConflicts().isEmpty());
    129         assertTrue(n == n2); // make sure the merged node is still the original node
    130         assertTrue(n2.getDataSet() == my);
     131        assertSame(n, n2); // make sure the merged node is still the original node
     132        assertSame(n2.getDataSet(), my);
    131133        assertEquals(1, n2.getId());
    132134        assertEquals(2, n2.getVersion());
    133         assertEquals(false, n2.isModified());
     135        assertFalse(n2.isModified());
    134136        assertEquals("value1-new", n2.get("key1"));
    135137        assertEquals("value2", n2.get("key2"));
    136138
    137139        // the merge target should not be modified
    138         assertTrue(!n2.isModified());
     140        assertFalse(n2.isModified());
    139141    }
    140142
    … …  
    168170        Node n2 = (Node)my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    169171        assertEquals(1,visitor.getConflicts().size());
    170         assertTrue(n == n2);
    171         assertTrue(n1 != n2);
    172         assertTrue(n1.getDataSet() == their);
     172        assertSame(n, n2);
     173        assertNotSame(n1, n2);
     174        assertSame(n1.getDataSet(), their);
    173175    }
    174176
    … …  
    201203        Node n2 = (Node)my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    202204        assertEquals(1,visitor.getConflicts().size());
    203         assertTrue(n == n2);
    204         assertTrue(n1 != n2);
    205         assertTrue(n1.getDataSet() == their);
     205        assertSame(n, n2);
     206        assertNotSame(n1, n2);
     207        assertSame(n1.getDataSet(), their);
    206208    }
    207209
    … …  
    228230        Node n2 = (Node)my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    229231        assertEquals(0,visitor.getConflicts().size());
    230         assertEquals(true, n2.isVisible());
    231         assertTrue(n == n2);
    232         assertTrue(n.getDataSet() == my);
    233         assertTrue(n1.getDataSet() == their);
     232        assertTrue(n2.isVisible());
     233        assertSame(n, n2);
     234        assertSame(n.getDataSet(), my);
     235        assertSame(n1.getDataSet(), their);
    234236    }
    235237
    … …  
    253255
    254256        assertEquals(0, visitor.getConflicts().size());
    255         assertTrue(n.getDataSet() == my);
    256         assertTrue(n1.getDataSet() == their);
     257        assertSame(n.getDataSet(), my);
     258        assertSame(n1.getDataSet(), their);
    257259    }
    258260
    … …  
    279281        assertEquals(0,visitor.getConflicts().size());
    280282        Node n2 = (Node)my.getNodes().toArray()[0];
    281         assertTrue(n2 == n);
     283        assertSame(n2, n);
    282284        assertTrue(n2.isDeleted());
    283285    }
    … …  
    322324        assertEquals(0,visitor.getConflicts().size());
    323325        assertEquals("value1",n2.get("key1"));
    324         assertEquals(true, n1.getTimestamp().equals(n2.getTimestamp()));
     326        assertTrue(n1.getTimestamp().equals(n2.getTimestamp()));
    325327        assertEquals(theirUser,n2.getUser());
    326         assertTrue(n2 == n);
    327         assertTrue(n2 != n1);
    328         assertTrue(n2.getDataSet() == my);
     328        assertSame(n2, n);
     329        assertNotSame(n2, n1);
     330        assertSame(n2.getDataSet(), my);
    329331    }
    330332
    … …  
    356358        assertEquals(0,visitor.getConflicts().size());
    357359        assertEquals("value1",n2.get("key1"));
    358         assertEquals(true, n1.getTimestamp().equals(n2.getTimestamp()));
    359         assertEquals(false, n2.isIncomplete());
    360         assertTrue(n2 == n);
     360        assertTrue(n1.getTimestamp().equals(n2.getTimestamp()));
     361        assertFalse(n2.isIncomplete());
     362        assertSame(n2, n);
    361363    }
    362364
    … …  
    423425        assertEquals(1,merged.getNode(0).getId());
    424426        assertEquals(2,merged.getNode(1).getId());
    425         assertTrue(merged == myWay);
    426         assertTrue(merged.getDataSet() == my);
     427        assertSame(merged, myWay);
     428        assertSame(merged.getDataSet(), my);
    427429
    428430        Node mergedNode = (Node)my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    429         assertTrue(mergedNode == n1);
     431        assertSame(mergedNode, n1);
    430432        mergedNode = (Node)my.getPrimitiveById(2, OsmPrimitiveType.NODE);
    431         assertTrue(mergedNode == n2);
    432 
    433         assertTrue(!merged.isModified());
     433        assertSame(mergedNode, n2);
     434
     435        assertFalse(merged.isModified());
    434436    }
    435437
    … …  
    498500        assertEquals("value1",merged.getNode(2).get("key1"));
    499501
    500         assertTrue(merged.getNode(0) == n1);
    501         assertTrue(merged.getNode(1) != n5); // must be clone of the original node in their
    502         assertTrue(merged.getNode(2) == n2);
    503 
    504         assertTrue(!merged.isModified());  // the target wasn't modified before merging, it
    505         // mustn't be after merging
     502        assertSame(merged.getNode(0), n1);
     503        assertNotSame(merged.getNode(1), n5); // must be clone of the original node in their
     504        assertSame(merged.getNode(2), n2);
     505
     506        assertFalse(merged.isModified());  // the target wasn't modified before merging, it mustn't be after merging
    506507    }
    507508
    … …  
    613614        Way merged = (Way)my.getPrimitiveById(3, OsmPrimitiveType.WAY);
    614615        assertEquals(1,visitor.getConflicts().size());
    615         assertEquals(true, visitor.getConflicts().hasConflictForMy(myWay));
    616         assertEquals(true, visitor.getConflicts().hasConflictForTheir(theirWay));
     616        assertTrue(visitor.getConflicts().hasConflictForMy(myWay));
     617        assertTrue(visitor.getConflicts().hasConflictForTheir(theirWay));
    617618        assertEquals(myWay,merged);
    618619    }
    … …  
    669670        assertEquals(1111, merged.getUser().getId());
    670671        assertEquals(theirWay.getTimestamp(), merged.getTimestamp());
    671         assertTrue(merged == myWay);
    672         assertTrue(merged.getNode(0) == n1);
    673         assertTrue(merged.getNode(1) == n2);
    674 
    675         assertTrue(!merged.isModified());
     672        assertSame(merged, myWay);
     673        assertSame(merged.getNode(0), n1);
     674        assertSame(merged.getNode(1), n2);
     675
     676        assertFalse(merged.isModified());
    676677    }
    677678
    … …  
    723724        assertEquals(1111, merged.getUser().getId());
    724725        assertEquals(theirWay.getTimestamp(), merged.getTimestamp());
    725         assertTrue(merged == myWay);
    726         assertTrue(merged.getNode(0) == n1);
    727         assertTrue(merged.getNode(1) == n2);
    728 
    729         assertTrue(!merged.isModified());
     726        assertSame(merged, myWay);
     727        assertSame(merged.getNode(0), n1);
     728        assertSame(merged.getNode(1), n2);
     729
     730        assertFalse(merged.isModified());
    730731    }
    731732
    … …  
    830831
    831832        Node n = (Node)my.getPrimitiveById(1,OsmPrimitiveType.NODE);
    832         assertTrue(n != null);
     833        assertNotNull(n);
    833834
    834835        assertEquals(1, visitor.getConflicts().size());
    … …  
    926927        OsmPrimitive p= my.getPrimitiveById(1, OsmPrimitiveType.NODE);
    927928        assertNotNull(p);
    928         assertTrue(!p.isIncomplete());
     929        assertFalse(p.isIncomplete());
    929930        p= my.getPrimitiveById(2, OsmPrimitiveType.NODE);
    930931        assertNotNull(p);
    931         assertTrue(!p.isIncomplete());
     932        assertFalse(p.isIncomplete());
    932933        p= my.getPrimitiveById(3,OsmPrimitiveType.WAY);
    933934        assertNotNull(p);
    934         assertTrue(!p.isIncomplete());
     935        assertFalse(p.isIncomplete());
    935936
    936937        Way w = (Way)my.getPrimitiveById(3,OsmPrimitiveType.WAY);
    937938        assertNotNull(w);
    938         assertTrue(!p.isIncomplete());
     939        assertFalse(p.isIncomplete());
    939940        assertEquals(2, w.getNodesCount());
    940         assertTrue(!w.getNode(0).isIncomplete());
    941         assertTrue(!w.getNode(1).isIncomplete());
     941        assertFalse(w.getNode(0).isIncomplete());
     942        assertFalse(w.getNode(1).isIncomplete());
    942943    }
    943944
Note: See TracChangeset for help on using the changeset viewer.