> These are the remaining bits and pieces in order to completely remove the > JUnit Vintage Engine, and therefore JUnit 4 from JavaFX. > After that, we should either document, that JUnit5 is used (just as > information) or close > [JDK-8296284](https://bugs.openjdk.org/browse/JDK-8296284) (Since you can not > use JUnit4 anymore). > > This also removes the `org.hamcrest` dependency, which is actually only used > in one test where it can easily be replaced by a `Predicate` (instead of > `Matcher`). I'm not convinced that we should keep that dependency for just > one test. > > ~Note: I could not get the `:swt` tests to compile/test, in order to run the > tests. > I need some guidance here how to instruct `Gradle` to compile this module > (and if I need anything else? like swt?).~ > > ~Also, this probably needs an extra ticket in the JBS?~
Marius Hanl has updated the pull request incrementally with one additional commit since the last revision: Missing comma in copyright ------------- Changes: - all: https://git.openjdk.org/jfx/pull/1774/files - new: https://git.openjdk.org/jfx/pull/1774/files/dc7925f7..56ac17a2 Webrevs: - full: https://webrevs.openjdk.org/?repo=jfx&pr=1774&range=02 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=1774&range=01-02 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jfx/pull/1774.diff Fetch: git fetch https://git.openjdk.org/jfx.git pull/1774/head:pull/1774 PR: https://git.openjdk.org/jfx/pull/1774