Reviewers: skybrian,
Description:
Comment out an invalid test in TreeMapTest.
Please review this at http://gwt-code-reviews.appspot.com/1735805/
Affected files:
M user/test/com/google/gwt/emultest/java/util/TreeMapTest.java
Index: user/test/com/google/gwt/emultest/java/util/TreeMapTest.java
===================================================================
--- user/test/com/google/gwt/emultest/java/util/TreeMapTest.java (revision
11042)
+++ user/test/com/google/gwt/emultest/java/util/TreeMapTest.java (working
copy)
@@ -1513,6 +1513,11 @@
// The _throwsUnsupportedOperationException version of this test will
// verify that the method is not supported.
if (isRemoveSupported) {
+
+ // TODO(acleung): Post JDK7, map.put(null) will actually throw a NPE.
+ // Lets disable this for now. Once we no longer test on JDK6, we can
+ // add this back and always assert an NPE.
+ /*
Map<K, V> map;
map = createMap();
// test remove null key with map containing a single null key
@@ -1523,6 +1528,7 @@
} catch (NullPointerException e) {
assertFalse(useNullKey());
}
+ */
map = createMap();
// test remove null key with map containing a single non-null key
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
