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

MODWRKFLOW-46: Make compatible with spring-boot 3.4 upgrades. #135

Merged
merged 7 commits into from
Feb 11, 2025

Conversation

kaladay
Copy link
Contributor

@kaladay kaladay commented Feb 10, 2025

Resolves MODWRKFLOW-46.

The ObjectMapper bean is provided because the @MockitoSpyBean no longer instantiates when there is no bean found in the way that no available or deprecated @SpyBean does.

Appease this error:

Description:

Parameter 2 of constructor in org.folio.spring.service.TenantService required a bean of type 'org.folio.spring.liquibase.FolioSpringLiquibase' that could not be found.

Action:

Consider defining a bean of type 'org.folio.spring.liquibase.FolioSpringLiquibase' in your configuration.

Switch folio-spring-support dependency to 8.3.0-SNAPSHOT because this is needed for the Spring Boot 3.4 support. Prior versions do not work under Spring-Boot 3.4.

Update the Controlling Access to Actuator Endpoints settings.
The settings changed from enabled to accessandenabled-by-defaulttoaccess.default`.

The access values are no longer boolean and are instead one of read-only, none, and unrestricted.
I decided to use read-only for values that were previously true.

Replace @MockBean and @SpyBean.

  • @MockitoBean and MockitoSpyBean are the replacemenets to the deprecated or removed @MockBean and @SpyBean.

Address failing bean problems.

This requires the changes from Spring Module Core PR 74 for SPRNGCORE-6.

kaladay and others added 7 commits February 7, 2025 15:44
…ttings.

The settings changed from `enabled` to access` and `enabled-by-default` to `access.default`.

The `access` values are no longer boolean and are instead one of `read-only`, `none`, and `unrestricted`.
I decided to use `read-only` for values that were previously `true`.

see: https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-3.4-Release-Notes#controlling-access-to-actuator-endpoints
This is required because the `8.3.0-SNAPSHOT` is what is needed for the Spring Boot 3.4 support.
Appease this error:
```
Description:

Parameter 2 of constructor in org.folio.spring.service.TenantService required a bean of type 'org.folio.spring.liquibase.FolioSpringLiquibase' that could not be found.

Action:

Consider defining a bean of type 'org.folio.spring.liquibase.FolioSpringLiquibase' in your configuration.
```
The `ObjectMapper` bean is provided because the `@MockitoSpyBean` no longer instantiates when there is no bean found in the way that no available or deprecated `@SpyBean` does.

see: spring-projects/spring-framework#33935
@kaladay kaladay marked this pull request as ready for review February 11, 2025 16:33
@kaladay kaladay requested a review from a team February 11, 2025 18:31
@kaladay kaladay requested a review from a team February 11, 2025 18:47
@kaladay kaladay merged commit 99ddf9b into master Feb 11, 2025
4 checks passed
@kaladay kaladay deleted the MODWRKFLOW-46 branch February 11, 2025 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants