[Java] Update Unit Tests for Adapter Module #42030
Comments
11 tasks
12 tasks
lidavidm
pushed a commit
that referenced
this issue
Jun 14, 2024
### Rationale for this change
Update package from JUnit 4(`org.junit`) to JUnit 5(`org.junit.jupiter`).
### What changes are included in this PR?
- `avro` and `jdbc` module
- [x] Replacing `org.junit` with `org.junit.jupiter.api`.
- [x] Updating `Assertions.assertXXX` to `assertXXX` using static imports.
- [x] Updating annotations such as `@ Before`, `@ After`.
- `@ Before` -> `@ BeforeEach`
- `@ After` -> `@ AfterEach`
- `@ Test` -> `@ Test` with `org.junit.jupiter`
- `@ ClassRule` -> `@ TempDir` and `@ BeforeAll`
- [x] Updating `Parameterized` test
- [x] Doing self review for avro
- [x] Dealing with `java.io.IOException: Failed to delete temp directory` on Windows with JDK 11
- [x] Exploring a more effective structure for `ParameterizedTest` in JDBC tests.
- [x] Doing self review for jdbc
- `orc` module
- [x] Reviewing the build method
- [x] Updating annotations such as `@ BeforeAll`, `@ Rule`, `@ TemporaryFolder`
- [x] Doing self review
### Are these changes tested?
Yes, existing tests have passed.
### Are there any user-facing changes?
No.
* GitHub Issue: #42030
Authored-by: Hyunseok Seo <[email protected]>
Signed-off-by: David Li <[email protected]>
|
Issue resolved by pull request 42038 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the enhancement requested
This is a sub-issue of the issue below.
org.junit.jupiter.api#41680The unit tests should use the
org.junit.jupiter.apipackage. This issue deals with theadaptermodule.Component(s)
Java
The text was updated successfully, but these errors were encountered: