Upgrade connect runtime and api#1600
Upgrade connect runtime and api#1600kunal sevkani (kunalmnnit) wants to merge 1 commit intomasterfrom
Conversation
|
🎉 All Contributor License Agreements have been signed. Ready to merge. |
There was a problem hiding this comment.
Pull request overview
This PR upgrades the project's Connect runtime and API dependencies, along with updating the Mockito testing framework. The changes primarily involve updating import statements to use newer Mockito APIs and adjusting the Maven configuration to support the upgraded versions.
Key changes:
- Upgraded parent version from 7.0.12 to 8.1.1
- Migrated from deprecated
MatcherstoArgumentMatchersin Mockito - Updated Mockito dependency from
mockito-alltomockito-coreversion 3.12.4
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| pom.xml | Updated parent version, migrated Mockito dependency, removed obsolete dependencies, added JDK 17+ compatibility flags |
| MSSQLDateTimeIT.java | Changed IntegrationTest import to use project-specific annotation |
| TableQuerierTest.java | Replaced deprecated Mockito Matchers with ArgumentMatchers |
| JdbcDbWriterTest.java | Replaced deprecated Mockito Matchers with ArgumentMatchers |
| DbStructureTest.java | Replaced deprecated Mockito Matchers with ArgumentMatchers and reorganized imports |
| BufferedRecordsTest.java | Replaced deprecated Mockito Matchers with ArgumentMatchers and reorganized imports |
| BaseConnectorIT.java | Reorganized imports and removed obsolete IntegrationTest import |
| OracleDatabaseDialectTest.java | Replaced deprecated Mockito Matchers with ArgumentMatchers |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
1b590a8 to
3c9b658
Compare
3c9b658 to
98cd532
Compare
|





Problem
Solution
Does this solution apply anywhere else?
If yes, where?
Test Strategy
Testing done:
Release Plan