SK-2527: update libraries for Java v2 SDK#275
SK-2527: update libraries for Java v2 SDK#275skyflow-himanshu merged 3 commits intorelease/26.2.2from
Conversation
|
✅ Gitleaks Findings: No secrets detected. Safe to proceed! |
|
Semgrep Findings: Issues with Error level severity are found (Error is Highest severity in Semgrep), Please resolve the issues before merging. |
|
✅ Gitleaks Findings: No secrets detected. Safe to proceed! |
|
Semgrep Findings: Issues with Error level severity are found (Error is Highest severity in Semgrep), Please resolve the issues before merging. |
There was a problem hiding this comment.
Pull request overview
Updates test tooling and build plugins to support the Java v2 SDK library refresh (removing PowerMock usage and modernizing Maven plugins).
Changes:
- Replaced PowerMock runner/annotations with
MockitoJUnitRunnerinHttpUtilityTests. - Removed PowerMock test dependencies and added
mockito-coretest dependency. - Bumped JaCoCo and Central Publishing Maven plugin versions.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/test/java/com/skyflow/utils/HttpUtilityTests.java | Migrates tests off PowerMock to Mockito’s JUnit4 runner and adjusts call sites accordingly. |
| pom.xml | Updates test dependencies (PowerMock → Mockito) and upgrades Maven plugin versions. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
✅ Gitleaks Findings: No secrets detected. Safe to proceed! |
|
Semgrep Findings: Issues with Error level severity are found (Error is Highest severity in Semgrep), Please resolve the issues before merging. |
3 participants
Why:
Update libraries for Java v2 SDK
Goal:
Removed powermock-module-junit4 and added mockito-core v4.11.0 to support Java 8.
Updated central-publishing-maven-plugin (0.4.0 to 0.10.0).
Updated jacoco-maven-plugin (0.8.7 to 0.8.14), as 0.8.15 is not yet stable.