> Please review this PR which refactors Currency tests to use JUnit. > > The most significant change occurs in `ValidateISO4217.java`. Other changes > to this file excluding the JUnit refactoring include > > - Tests are no longer dependent on each other (order of execution does not > matter) > - Testing does not occur at the same time as data generation (The data is > fully generated before any tests are executed) > - Various cleanup (dead-code, clarifying comments, more descriptive method > and var names) > > `Bug4512215.java` now renamed to `MinorUndefinedCodes` was updated to remove > redundant testing, and the file changed to focus on testing minor undefined > currency codes instead.
Justin Lu has updated the pull request incrementally with two additional commits since the last revision: - Rename Bug4512215.java as MinorUndefinedCodes.java and update goal of test file - Refactor and improve ValidateISO4217.java ------------- Changes: - all: https://git.openjdk.org/jdk/pull/14682/files - new: https://git.openjdk.org/jdk/pull/14682/files/2d6ed5a7..bff5cc6c Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=14682&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=14682&range=00-01 Stats: 496 lines in 3 files changed: 178 ins; 199 del; 119 mod Patch: https://git.openjdk.org/jdk/pull/14682.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/14682/head:pull/14682 PR: https://git.openjdk.org/jdk/pull/14682