--- jsr166/src/test/tck/ConcurrentHashMap8Test.java	2015/02/27 19:47:57	1.23
+++ jsr166/src/test/tck/ConcurrentHashMap8Test.java	2015/02/27 21:05:11	1.25
@@ -251,7 +251,7 @@ public class ConcurrentHashMap8Test exte
      */
     public void testAdd2() {
         Set full = populatedSet(3);
-        full.add(one);
+        assertFalse(full.add(one));
         assertEquals(3, full.size());
     }
 
@@ -260,7 +260,9 @@ public class ConcurrentHashMap8Test exte
      */
     public void testAdd3() {
         Set full = populatedSet(3);
-        full.add(three);
+        assertTrue(full.add(three));
+        assertTrue(full.contains(three));
+        assertFalse(full.add(three));
         assertTrue(full.contains(three));
     }
 
@@ -273,7 +275,7 @@ public class ConcurrentHashMap8Test exte
         try {
             full.add(three);
             shouldThrow();
-        } catch (UnsupportedOperationException e){}
+        } catch (UnsupportedOperationException success) {}
     }
 
     /**
@@ -285,7 +287,7 @@ public class ConcurrentHashMap8Test exte
         try {
             full.add(null);
             shouldThrow();
-        } catch (NullPointerException e){}
+        } catch (NullPointerException success) {}
     }
 
     /**