On Fri, 4 Mar 2022 21:02:50 GMT, Stuart Marks <sma...@openjdk.org> wrote:
>>> This actually tests three things: 1) table is lazily allocated, 2) default >>> capacity is 16, and 3) using putAll to populate the map with 64 elements >>> results in a table size of 128. This should really be broken into three >>> separate test methods. Once they're separated, the lazy allocation test >>> should only called for HashMap and LinkedHashMap but not WeakHashMap. >> >> @stuart-marks would you mind if I break WhiteBoxResizeTest class into >> several smaller Test classes, each focus on one of the test points you said? >> If we split it into several tests, it would be more clear than sqruash into >> one test class, and we can make it parameterilized tests. > >> would you mind if I break WhiteBoxResizeTest class into several smaller Test >> classes, each focus on one of the test points you said? > > Well, separate classes wouldn't be the approach that I'd take myself. > However, I'm interested in you continuing to make progress on this, so if > you'd prefer separate classes, then go ahead. However, I reserve the right to > propose refactorings that merge things back into a single class. :-) @stuart-marks done. please find some time for review. thanks. ------------- PR: https://git.openjdk.java.net/jdk/pull/7431