Commit 679954a
committed
### What changes were proposed in this pull request?
This PR aims to fix `spark-operator` module to use `JUnit 5` correctly instead of the transitive `JUnit 4` dependency.
### Why are the changes needed?
`Apache Spark K8s Operator` should use `JUnit 5` consistently.
https://github.com/apache/spark-kubernetes-operator/blob/a0929a4023f7a0f9eba5765fdb47e17228d68a0e/gradle/libs.versions.toml#L26
**BEFORE**
```
$ gradle :spark-operator:dependencies | grep ' junit:junit' | wc -l
3
```
**AFTER**
```
$ gradle :spark-operator:dependencies | grep ' junit:junit' | wc -l
0
```
### Does this PR introduce _any_ user-facing change?
No, this is a test code and dependency change.
### How was this patch tested?
Pass the CIs.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #344 from dongjoon-hyun/SPARK-53712.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
1 parent a0929a4 commit 679954a
File tree
3 files changed
+5
-3
lines changed- spark-operator
- src/test/java/org/apache/spark/k8s/operator/metrics
- source
3 files changed
+5
-3
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
65 | 65 | | |
66 | 66 | | |
67 | 67 | | |
68 | | - | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
69 | 71 | | |
70 | 72 | | |
71 | 73 | | |
| |||
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
20 | 20 | | |
21 | 21 | | |
22 | 22 | | |
23 | | - | |
| 23 | + | |
24 | 24 | | |
25 | 25 | | |
26 | 26 | | |
| |||
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
19 | 19 | | |
20 | 20 | | |
21 | 21 | | |
22 | | - | |
| 22 | + | |
23 | 23 | | |
24 | 24 | | |
25 | 25 | | |
| |||
0 commit comments