Changeset 8450 in josm for trunk/test/unit/org/openstreetmap/josm/data/osm/OsmPrimitiveKeyHandlingTest.java

Timestamp:

2015-06-03T01:27:41+02:00 (9 years ago)

Author:

Don-vip

Message:

unit tests - simplify assertions

File:

• trunk/test/unit/org/openstreetmap/josm/data/osm/OsmPrimitiveKeyHandlingTest.java (modified) (7 diffs)

trunk/test/unit/org/openstreetmap/josm/data/osm/OsmPrimitiveKeyHandlingTest.java

@@ -2 +2 @@
 package org.openstreetmap.josm.data.osm;
 
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNull;
+import static org.junit.Assert.assertSame;
 import static org.junit.Assert.assertTrue;
 
@@ -27 +30 @@
     public void emptyNode() {
         Node n = new Node();
-        assertTrue(n.getKeys().size() == 0);
-        assertTrue(!n.hasKeys());
-        assertTrue(!n.hasKey("nosuchkey"));
+        assertSame(n.getKeys().size(), 0);
+        assertFalse(n.hasKeys());
+        assertFalse(n.hasKey("nosuchkey"));
         assertTrue(n.keySet().isEmpty());
 
@@ -43 +46 @@
         n.put("akey", "avalue");
         assertTrue(n.get("akey").equals("avalue"));
-        assertTrue(n.getKeys().size() == 1);
+        assertSame(n.getKeys().size(), 1);
 
-        assertTrue(n.keySet().size() == 1);
+        assertSame(n.keySet().size(), 1);
         assertTrue(n.keySet().contains("akey"));
     }
@@ -59 +62 @@
         assertTrue(n.get("key.1").equals("value.1"));
         assertTrue(n.get("key.2").equals("value.2"));
-        assertTrue(n.getKeys().size() == 2);
+        assertSame(n.getKeys().size(), 2);
         assertTrue(n.hasKeys());
         assertTrue(n.hasKey("key.1"));
         assertTrue(n.hasKey("key.2"));
-        assertTrue(!n.hasKey("nosuchkey"));
+        assertFalse(n.hasKey("nosuchkey"));
     }
 
@@ -75 +78 @@
         n.put("key.2", "value.2");
 
-        n.remove("nosuchkey");               // should work
-        assertTrue(n.getKeys().size() == 2); // still 2 tags ?
+        n.remove("nosuchkey");             // should work
+        assertSame(n.getKeys().size(), 2); // still 2 tags ?
 
         n.remove("key.1");
-        assertTrue(n.getKeys().size() == 1);
-        assertTrue(!n.hasKey("key.1"));
-        assertTrue(n.get("key.1") == null);
+        assertSame(n.getKeys().size(), 1);
+        assertFalse(n.hasKey("key.1"));
+        assertNull(n.get("key.1"));
         assertTrue(n.hasKey("key.2"));
         assertTrue(n.get("key.2").equals("value.2"));
 
         n.remove("key.2");
-        assertTrue(n.getKeys().size() == 0);
-        assertTrue(!n.hasKey("key.1"));
-        assertTrue(n.get("key.1") == null);
-        assertTrue(!n.hasKey("key.2"));
-        assertTrue(n.get("key.2") == null);
+        assertSame(n.getKeys().size(), 0);
+        assertFalse(n.hasKey("key.1"));
+        assertNull(n.get("key.1"));
+        assertFalse(n.hasKey("key.2"));
+        assertNull(n.get("key.2"));
     }
 
@@ -104 +107 @@
 
         n.removeAll();
-        assertTrue(n.getKeys().size() == 0);
+        assertSame(n.getKeys().size(), 0);
     }
 
@@ -141 +144 @@
         n2.put("key.2", "value.4");
 
-        assertTrue(!n1.hasEqualSemanticAttributes(n2));
+        assertFalse(n1.hasEqualSemanticAttributes(n2));
     }
 }