Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrading to spring boot 3.2.2 #67

Merged
merged 3 commits into from
Feb 26, 2024

Conversation

ryangrundy7
Copy link
Contributor

Motivation and Context

Spring Boot 3 has been out for a while and Dependabot wants us to upgrade to it. There's quite a few changes which you look up for yourselves. I listed main changes to each repo on the ticket.

What has changed

  • Move to Jakarta resources instead of Javax
  • updated postgresdialect to PostgreSQLDialect
  • localwebserver comes from the test location now instead of server
  • Use completeableFuture instead of ListenableFuture

How to test?

  • Run make build on the exception manager and uac qid service branch
  • go to the ddl repo and run UAC_QID_SERVICE_BRANCH=1288-upgrade-spring-boot-version-in-java-repos EXCEPTION_MANAGER_BRANCH=1288-upgrade-spring-boot-version-in-java-repos make dev-build . This should build the expected ddl changes for you.
  • run make build on the rest of the java services
  • Run the ATs and make sure everything works as expected.

Links

Trello

Screenshots (if appropriate):

@ryangrundy7 ryangrundy7 added the feature Adds or alters functionality label Feb 12, 2024
@@ -73,7 +73,7 @@ void processChunk() {
when(transformer.transformRow(job, jobRow, columnValidators, jobTypeProcessor.getTopic()))
.thenReturn(messageToPublish);

ListenableFuture<String> listenableFuture = mock(ListenableFuture.class);
CompletableFuture<String> listenableFuture = mock(CompletableFuture.class);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
CompletableFuture<String> listenableFuture = mock(CompletableFuture.class);
CompletableFuture<String> completableFuture = mock(CompletableFuture.class);

Copy link

codecov bot commented Feb 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.05%. Comparing base (a0d465f) to head (f4c7572).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main      #67   +/-   ##
=========================================
  Coverage     92.05%   92.05%           
  Complexity      120      120           
=========================================
  Files            35       35           
  Lines           403      403           
  Branches         16       16           
=========================================
  Hits            371      371           
  Misses           29       29           
  Partials          3        3           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ryangrundy7 ryangrundy7 merged commit 954ea91 into main Feb 26, 2024
3 checks passed
@ryangrundy7 ryangrundy7 deleted the 1288-upgrade-spring-boot-version-in-java-repos branch February 26, 2024 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Adds or alters functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants