Skip to content

CAP Java 5 und Spring boot 4 migration#602

Open
StefanHenke wants to merge 28 commits into
mainfrom
spring-boot-4
Open

CAP Java 5 und Spring boot 4 migration#602
StefanHenke wants to merge 28 commits into
mainfrom
spring-boot-4

Conversation

@StefanHenke

@StefanHenke StefanHenke commented Jan 28, 2026

Copy link
Copy Markdown
Contributor

Requires a Spring Boot 4 compatible version of CAP Java and the BTP Security Library which are not yet available on Maven Central.

@StefanHenke StefanHenke added the ☠️ do not merge ☠️ Do not merge PR label Mar 19, 2026
StefanHenke and others added 18 commits April 21, 2026 16:13
Co-authored-by: Markus Ofterdinger <markus.ofterdinger@sap.com>
Replace MockMvc with Spring Boot's RestTestClient and its native
AssertJ integration for cleaner, more readable assertions.
- Use @AutoConfigureRestTestClient instead of @AutoConfigureMockMvc
- Use RestTestClientResponse.from() with assertThat() for fluent
  status and JSON body assertions
- Remove Hamcrest dependencies in favor of AssertJ's .contains() and
  .doesNotContain() via .extractingPath().asString()
- Remove checked exceptions from test methods
@mofterdinger mofterdinger changed the title Spring boot 4 migration CAP Java 5 und Spring boot 4 migration Jun 26, 2026
@mofterdinger

Copy link
Copy Markdown
Member

@StefanHenke Hi, I've updated this PR to CAP Java 5 and I bumped also some other dependencies. The update to cds-dk 10 ist still missing. Once this is done we should merge this PR.

Comment thread srv/pom.xml
<dependency>
<groupId>org.springframework.security</groupId>
<artifactId>spring-security-test</artifactId>
<groupId>org.springframework.boot</groupId>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

are these new dependencies really required ?

@StefanHenke StefanHenke removed the ☠️ do not merge ☠️ Do not merge PR label Jun 29, 2026
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