Re: RFR: 8321616: Convert the ReadZip test to JUnit

2023-12-11 Thread Eirik Bjorsnos
On Mon, 11 Dec 2023 17:40:24 GMT, Lance Andersen wrote: > I don't have a preference whether we deal with input.jar separately, but > these tests are not that complex so I do not see a risk if they are all > converted at once, or done piece meal Thanks, I'll extend the goal of this PR to remove

Re: RFR: 8321616: Convert the ReadZip test to JUnit

2023-12-11 Thread Lance Andersen
On Mon, 11 Dec 2023 17:22:14 GMT, Eirik Bjorsnos wrote: > > One quick comment, if we are updating this test, we should look to get rid > > of input.zip > > I started going down that road, but felt uneasy about the amount of unrelated > changes in a single PR. I'd like to make efficient use of

Re: RFR: 8321616: Convert the ReadZip test to JUnit

2023-12-11 Thread Eirik Bjorsnos
On Mon, 11 Dec 2023 16:51:28 GMT, Lance Andersen wrote: > One quick comment, if we are updating this test, we should look to get rid of > input.zip I started going down that road, but felt uneasy about the amount of unrelated changes in a single PR. I'd like to make efficient use of reviewer

Re: RFR: 8321616: Convert the ReadZip test to JUnit

2023-12-11 Thread Lance Andersen
On Fri, 8 Dec 2023 20:28:20 GMT, Eirik Bjorsnos wrote: > Please review this PR which suggests we rewrite the > `../zip/ZipFile/ReadZip.java` test to JUnit. > > The current test is a single main method with a sequence of fairly unrelated > scenarios. It would benefit from a rewrite to multiple