-
Notifications
You must be signed in to change notification settings - Fork 12
Add support for Java 25 and Spring Boot 4 #881
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
Changes from all commits
82b6db5
b67d1c1
5b787ca
e2b47d8
f963fe7
97b295a
c37054a
ec6c2b7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -63,7 +63,6 @@ | |
| </configuration> | ||
| </execution> | ||
| </executions> | ||
|
|
||
| </plugin> | ||
| </plugins> | ||
| </build> | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -16,14 +16,14 @@ We removed `org.jbehavesupport.core.internal.ConditionalOnMissingBean` annotatio | |
| ### Deprecated code removal | ||
| We have removed all previously deprecated code - please refer to [Deprecated.md](Deprecated.md) for more details how to migrate. | ||
|
|
||
| ### Spring 6 upgrade | ||
| We are currently using Spring 6 internally, this brings with it several other requirements: | ||
| ### Spring 7 upgrade | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I would maybe keep the old one as it is and create a new section
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I am used to |
||
| We are currently using Spring 7 internally, this brings with it several other requirements: | ||
|
|
||
| > #### Java 17 | ||
| >We now currently require Java 17 as a minimal Java version (previously 8). | ||
| >We now currently require Java 17 as a minimal Java version. | ||
| > | ||
| > #### Jakarta EE 9 | ||
| >Spring 6 internally brings with it upgrade to Jakarta EE 9, this means migration from old `javax.*` to `jakarta.*` packages, please consult guides on the Internet (and or your IDE support for migration) if needed. | ||
|
|
||
| ### Spring Boot 3 introduction | ||
| ### Spring Boot 4 introduction | ||
| We are now internally using Spring Boot - this means that ideally your main test configuration file should be annotated with both `@SpringBootConfiguration` and `@EnableAutoConfiguration`. | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -177,7 +177,7 @@ private void storeResponse(final HttpStatusCodeException e) { | |
|
|
||
| private HttpEntity createRequestEntity(ExamplesTable data) throws IOException { | ||
| if (data == null) { //return dummy | ||
| return new HttpEntity<>("", null); | ||
| return new HttpEntity<>("", HttpHeaders.EMPTY); | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Just curious - did the old null cause issues? But yeah, this new version is definitely more readable 👍
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The previous implementation is not usable anymore due to the latest changes and Intellj Idea has been screaming on me 🙂 |
||
| } | ||
| List<Triple<String, Object, String>> requestDataList = ExamplesTableUtil.convertTriple(data, NAME, DATA, ALIAS); | ||
| HttpHeaders headers = createHeaders(requestDataList); | ||
|
|
@@ -443,8 +443,7 @@ private void verifyResponseHeaders(HttpHeaders actualHeaders, ExamplesTable data | |
| if (key.startsWith(HEADER_START) && !key.equals(STATUS_HEADER)) { | ||
| String headerKey = key.substring(HEADER_START.length()); | ||
| String assertionErrorMessage = "Headers don't contain " + headerKey + "\n" + actualResponseMessage; | ||
| assertThat(actualHeaders.containsKey(headerKey)).as(assertionErrorMessage).isTrue(); | ||
|
|
||
| assertThat(actualHeaders.containsHeader(headerKey)).as(assertionErrorMessage).isTrue(); | ||
| final Verifier verifier = verifierResolver.getVerifierByName(triple.getRight(), equalsVerifier); | ||
| verifier.verify(actualHeaders.get(headerKey).get(0), triple.getMiddle()); | ||
| } | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.