RFR: 8339515: [TestBug] Convert web tests to JUnit 5 [v4]
Kevin Rushforth
kcr at openjdk.org
Mon Sep 16 12:58:15 UTC 2024
On Mon, 16 Sep 2024 01:23:52 GMT, Jay Bhaskar <jbhaskar at openjdk.org> wrote:
>> Successfully converted web tests from JUnit 4 to JUnit 5, ensuring all tests are fully compliant with the JUnit 5 framework.
>
> Jay Bhaskar has updated the pull request incrementally with one additional commit since the last revision:
>
> converting newly added tests to JUNIT5
You need to revert the copyright year change, since you've mistakenly removed the start year.
modules/javafx.web/src/test/java/test/com/sun/webkit/SharedBufferTest.java line 2:
> 1: /*
> 2: * Copyright (c) 2024, Oracle and/or its affiliates. All rights reserved.
You removed the starting copyright year, which is not OK. I recommend to revert all of the copyright changes. Then, if you prefer, you can either update it correctly to leave the starting year, or better still, leave it alone and let the script that @arapte runs periodically take care of it.
-------------
PR Review: https://git.openjdk.org/jfx/pull/1567#pullrequestreview-2306563017
PR Review Comment: https://git.openjdk.org/jfx/pull/1567#discussion_r1761094620
More information about the openjfx-dev
mailing list