feat: upgrade Spring to v6, SpringBoot to v3 #673
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #673 +/- ##
============================================
- Coverage 88.35% 88.33% -0.03%
- Complexity 777 779 +2
============================================
Files 173 173
Lines 7078 7078
Branches 393 389 -4
============================================
- Hits 6254 6252 -2
Misses 699 699
- Partials 125 127 +2 ☔ View full report in Codecov by Sentry. |
| String userAgent = recordedRequest.getHeader("User-Agent"); | ||
|
|
||
| Assertions.assertThat(userAgent).startsWith("influxdb-client-java/7."); | ||
| String currentVersion = UserAgentInterceptor.class.getPackage().getImplementationVersion(); |
There was a problem hiding this comment.
Added a test change here. Jakub, does this work for you? Does it not defeat Utku Aydin's intent for this assertion?
karel-rehor
left a comment
There was a problem hiding this comment.
This looks OK. To check the build of this branch locally I added a change to WriteApiTest.userAgent():1004-1025. If this change does not negatively impact earlier intentions for this test, I think this PR can be merged.
Closes #649
Proposed Changes
Migrate
influxdb-springfrom Spring Boot v2 to v3, Spring to v6.Checklist
mvn testcompletes successfully