Re: RFR: JDK-8317372: Refactor some NumberFormat tests to use JUnit [v3]

2023-10-12 Thread Naoto Sato
On Thu, 12 Oct 2023 20:58:21 GMT, Justin Lu wrote: >> Please review this PR which refactors a number of tests under >> `test/text/NumberFormat` to use JUnit. >> >> During the switch to JUnit, the tests had the following updates (to improve >> readability) >> - separate the test data generation

Re: RFR: JDK-8317372: Refactor some NumberFormat tests to use JUnit [v3]

2023-10-12 Thread Justin Lu
> Please review this PR which refactors a number of tests under > `test/text/NumberFormat` to use JUnit. > > During the switch to JUnit, the tests had the following updates (to improve > readability) > - separate the test data generation from the actual execution of the test > - create distinct

Re: RFR: JDK-8317372: Refactor some NumberFormat tests to use JUnit [v2]

2023-10-11 Thread Naoto Sato
On Wed, 11 Oct 2023 21:15:42 GMT, Justin Lu wrote: >> test/jdk/java/text/Format/NumberFormat/CurrencyFormat.java line 62: >> >>> 60: // currencySymbolsTest() is only ran for COMPAT >>> 61: private static final boolean isCompat = >>> 62: >>> "COMPAT".equals(System.getProperty

Re: RFR: JDK-8317372: Refactor some NumberFormat tests to use JUnit [v2]

2023-10-11 Thread Naoto Sato
On Wed, 11 Oct 2023 21:20:47 GMT, Justin Lu wrote: >> Please review this PR which refactors a number of tests under >> `test/text/NumberFormat` to use JUnit. >> >> During the switch to JUnit, the tests had the following updates (to improve >> readability) >> - separate the test data generation

Re: RFR: JDK-8317372: Refactor some NumberFormat tests to use JUnit [v2]

2023-10-11 Thread Justin Lu
> Please review this PR which refactors a number of tests under > `test/text/NumberFormat` to use JUnit. > > During the switch to JUnit, the tests had the following updates (to improve > readability) > - separate the test data generation from the actual execution of the test > - create distinct

Re: RFR: JDK-8317372: Refactor some NumberFormat tests to use JUnit [v2]

2023-10-11 Thread Justin Lu
On Mon, 9 Oct 2023 20:13:39 GMT, Naoto Sato wrote: >> Justin Lu has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Reflect review comments > > test/jdk/java/text/Format/NumberFormat/BigDecimalCompatibilityTest.java line > 162: > >> 160:

Re: RFR: JDK-8317372: Refactor some NumberFormat tests to use JUnit

2023-10-09 Thread Naoto Sato
On Tue, 3 Oct 2023 22:23:11 GMT, Justin Lu wrote: > Please review this PR which refactors a number of tests under > `test/text/NumberFormat` to use JUnit. > > During the switch to JUnit, the tests had the following updates (to improve > readability) > - separate the test data generation from t

RFR: JDK-8317372: Refactor some NumberFormat tests to use JUnit

2023-10-06 Thread Justin Lu
Please review this PR which refactors a number of tests under `test/text/NumberFormat` to use JUnit. During the switch to JUnit, the tests had the following updates (to improve readability) - separate the test data generation from the actual execution of the test - create distinct test methods s