nastra commented on code in PR #13822:
URL: https://github.com/apache/iceberg/pull/13822#discussion_r2278448841
##########
site/docs/contribute.md:
##########
@@ -564,11 +564,10 @@ Awaitility.await("Tables were not deleted")
Please refer to the [usage
guide](https://github.com/awaitility/awaitility/wiki/Usage) of
[Awaitility](https://github.com/awaitility/awaitility) for more usage examples.
-### JUnit4 / JUnit5
-
-Iceberg currently uses a mix of JUnit4 (`org.junit` imports) and JUnit5
(`org.junit.jupiter.api` imports) tests. To allow an easier migration to JUnit5
in the future, new test classes
-that are being added to the codebase should be written purely in JUnit5 where
possible.
+### JUnit 5 / AssertJ
+Iceberg has now fully migrated to JUnit 5 (org.junit.jupiter.api imports) for
all tests. Any new test classes should be written using JUnit 5,
Review Comment:
```suggestion
Iceberg has now fully migrated to JUnit 5 (org.junit.jupiter.api imports)
for all tests. Any new test classes should be written using JUnit 5,
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]