RFR: 8285405: add test and check for negative argument to HashMap::newHashMap et al

Jaikiran Pai jpai at openjdk.java.net
Wed Jun 8 06:08:17 UTC 2022


Can I please get a review of this change which addresses https://bugs.openjdk.java.net/browse/JDK-8285405?

I've added the test for `LinkedHashMap.newLinkedHashMap(int)` in the existing `test/jdk/java/util/LinkedHashMap/Basic.java` since that test has tests for various APIs of this class.

For `WeakHashMap.newWeakHashMap` and `HashMap.newHashMap`, I have created new test classes under relevant locations, since these classes already have test classes (almost) per API/feature in those locations.

-------------

Commit messages:
 - 8285405: add test and check for negative argument to HashMap::newHashMap et al

Changes: https://git.openjdk.java.net/jdk/pull/9036/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=9036&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8285405
  Stats: 189 lines in 6 files changed: 186 ins; 0 del; 3 mod
  Patch: https://git.openjdk.java.net/jdk/pull/9036.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/9036/head:pull/9036

PR: https://git.openjdk.java.net/jdk/pull/9036


More information about the core-libs-dev mailing list