Re: RFR: 8339511: [TestBug] Convert Non parametrized base tests to JUnit 5 [v5]

2024-09-23 Thread Andy Goryachev
On Sat, 21 Sep 2024 00:54:05 GMT, Jay Bhaskar wrote: >> Successfully converted Non-parametrized base tests to JUnit 5 > > Jay Bhaskar has updated the pull request incrementally with one additional > commit since the last revision: > > restore and convert mistakenly removed tests thank you fo

Re: RFR: 8339511: [TestBug] Convert Non parametrized base tests to JUnit 5 [v5]

2024-09-21 Thread Kevin Rushforth
On Sat, 21 Sep 2024 00:54:05 GMT, Jay Bhaskar wrote: >> Successfully converted Non-parametrized base tests to JUnit 5 > > Jay Bhaskar has updated the pull request incrementally with one additional > commit since the last revision: > > restore and convert mistakenly removed tests Marked as re

Re: RFR: 8339511: [TestBug] Convert Non parametrized base tests to JUnit 5 [v4]

2024-09-21 Thread Kevin Rushforth
On Fri, 20 Sep 2024 20:08:40 GMT, Andy Goryachev wrote: >> Jay Bhaskar has updated the pull request incrementally with one additional >> commit since the last revision: >> >> one missing , formating > > looked at each line, double checking: > - before->beforeEach etc. > - exception types in a

Re: RFR: 8339511: [TestBug] Convert Non parametrized base tests to JUnit 5 [v5]

2024-09-20 Thread Jay Bhaskar
> Successfully converted Non-parametrized base tests to JUnit 5 Jay Bhaskar has updated the pull request incrementally with one additional commit since the last revision: restore and convert mistakenly removed tests - Changes: - all: https://git.openjdk.org/jfx/pull/1576/files

Re: RFR: 8339511: [TestBug] Convert Non parametrized base tests to JUnit 5 [v4]

2024-09-20 Thread Jay Bhaskar
On Fri, 20 Sep 2024 21:18:39 GMT, Kevin Rushforth wrote: > JavaBeanPropertyBuilderHelperTest I left those because these files have may assertThrows and having try catch , that required some extra care to convert, we will add then in parameterized test PR - PR Comment: https://git

Re: RFR: 8339511: [TestBug] Convert Non parametrized base tests to JUnit 5 [v4]

2024-09-20 Thread Kevin Rushforth
On Fri, 20 Sep 2024 02:36:34 GMT, Jay Bhaskar wrote: >> Successfully converted Non-parametrized base tests to JUnit 5 > > Jay Bhaskar has updated the pull request incrementally with one additional > commit since the last revision: > > one missing , formating I got curious, so I looked for al

Re: RFR: 8339511: [TestBug] Convert Non parametrized base tests to JUnit 5 [v4]

2024-09-20 Thread Kevin Rushforth
On Fri, 20 Sep 2024 20:08:40 GMT, Andy Goryachev wrote: > Could not run the junit4 query because there are still parameterized tests > left. What I did to verify this was to do a grep just on those files that changed: git diff --name-only master... | xargs grep -n -e "org.junit" -e "junit.fr

Re: RFR: 8339511: [TestBug] Convert Non parametrized base tests to JUnit 5 [v4]

2024-09-20 Thread Kevin Rushforth
On Fri, 20 Sep 2024 19:39:38 GMT, Andy Goryachev wrote: >> Jay Bhaskar has updated the pull request incrementally with one additional >> commit since the last revision: >> >> one missing , formating > > modules/javafx.base/src/test/java/test/com/sun/javafx/binding/ErrorLoggingUtiltity.java >

Re: RFR: 8339511: [TestBug] Convert Non parametrized base tests to JUnit 5 [v4]

2024-09-20 Thread Andy Goryachev
On Fri, 20 Sep 2024 02:36:34 GMT, Jay Bhaskar wrote: >> Successfully converted Non-parametrized base tests to JUnit 5 > > Jay Bhaskar has updated the pull request incrementally with one additional > commit since the last revision: > > one missing , formating looked at each line, double check

Re: RFR: 8339511: [TestBug] Convert Non parametrized base tests to JUnit 5 [v4]

2024-09-20 Thread Kevin Rushforth
On Fri, 20 Sep 2024 02:36:34 GMT, Jay Bhaskar wrote: >> Successfully converted Non-parametrized base tests to JUnit 5 > > Jay Bhaskar has updated the pull request incrementally with one additional > commit since the last revision: > > one missing , formating One remaining problem with mistak

Re: RFR: 8339511: [TestBug] Convert Non parametrized base tests to JUnit 5 [v4]

2024-09-20 Thread Kevin Rushforth
On Thu, 19 Sep 2024 21:06:40 GMT, Kevin Rushforth wrote: >> Jay Bhaskar has updated the pull request incrementally with one additional >> commit since the last revision: >> >> one missing , formating > > modules/javafx.base/src/test/java/test/com/sun/javafx/binding/ContentBindingSetTest.java

Re: RFR: 8339511: [TestBug] Convert Non parametrized base tests to JUnit 5 [v4]

2024-09-19 Thread Jay Bhaskar
> Successfully converted Non-parametrized base tests to JUnit 5 Jay Bhaskar has updated the pull request incrementally with one additional commit since the last revision: one missing , formating - Changes: - all: https://git.openjdk.org/jfx/pull/1576/files - new: https://git.

Re: RFR: 8339511: [TestBug] Convert Non parametrized base tests to JUnit 5 [v3]

2024-09-19 Thread Jay Bhaskar
On Thu, 19 Sep 2024 21:04:06 GMT, Kevin Rushforth wrote: >> Jay Bhaskar has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Modification as per review > > modules/javafx.base/src/test/java/test/com/sun/javafx/binding/ContentBindingMapTest.ja

Re: RFR: 8339511: [TestBug] Convert Non parametrized base tests to JUnit 5 [v3]

2024-09-19 Thread Jay Bhaskar
> Successfully converted Non-parametrized base tests to JUnit 5 Jay Bhaskar has updated the pull request incrementally with one additional commit since the last revision: Modification as per review - Changes: - all: https://git.openjdk.org/jfx/pull/1576/files - new: https://g

Re: RFR: 8339511: [TestBug] Convert Non parametrized base tests to JUnit 5 [v2]

2024-09-19 Thread Kevin Rushforth
On Thu, 19 Sep 2024 16:27:26 GMT, Jay Bhaskar wrote: >> Successfully converted Non-parametrized base tests to JUnit 5 > > Jay Bhaskar has updated the pull request incrementally with one additional > commit since the last revision: > > Revert "8339515: [TestBug] Convert web tests to JUnit 5" >

Re: RFR: 8339511: [TestBug] Convert Non parametrized base tests to JUnit 5 [v2]

2024-09-19 Thread Jay Bhaskar
> Successfully converted Non-parametrized base tests to JUnit 5 Jay Bhaskar has updated the pull request incrementally with one additional commit since the last revision: Revert "8339515: [TestBug] Convert web tests to JUnit 5" This reverts commit 86f73271142049a2c0162d987aee6bd0fcb2f82a.

RFR: 8339511: [TestBug] Convert Non parametrized base tests to JUnit 5

2024-09-19 Thread Jay Bhaskar
Successfully converted Non-parametrized base tests to JUnit 5 - Commit messages: - 8339511: [TestBug] Convert Non parametrized base tests to JUnit 5 - 8339515: [TestBug] Convert web tests to JUnit 5 Changes: https://git.openjdk.org/jfx/pull/1576/files Webrev: https://webrevs.open