Skip to content

test: add test for date resolving for OAS 3.0 - #3366

Merged
bnasslahsen merged 1 commit into
springdoc:mainfrom
Mattias-Sehlstedt:test-date-resolving-oas-3-0
Oct 23, 2026
Merged

bnasslahsen merged 1 commit into
springdoc:mainfrom
Mattias-Sehlstedt:test-date-resolving-oas-3-0

Conversation

@Mattias-Sehlstedt

Copy link
Copy Markdown
Contributor

app243 tested date resolving for OAS 3.1, but the same did not exist for OAS 3.0.

Tied to #3351 (comment)

@Mattias-Sehlstedt
Mattias-Sehlstedt force-pushed the test-date-resolving-oas-3-0 branch 2 times, most recently from b1896b0 to ed0e69a Compare September 19, 2026 12:54
app243 tested date resolving for OAS 3.1, but the same did not exist for OAS 3.0
@Mattias-Sehlstedt
Mattias-Sehlstedt force-pushed the test-date-resolving-oas-3-0 branch from ed0e69a to 9d6cbda Compare September 19, 2026 13:02
@bnasslahsen
bnasslahsen merged commit 0da56c3 into springdoc:main Oct 23, 2026
1 check passed
bnasslahsen pushed a commit that referenced this pull request Oct 23, 2026
app243 tested date resolving for OAS 3.1 only, while the OAS 3.0 app243
tests a discriminator mapping. Restore that test for OAS 3.1 and move
date resolving to app275 for both. Drop PrimitiveType.enablePartialTime()
from app237: it is a JVM-wide switch that made the LocalTime format of
other tests depend on the test order.

Backport of gh-3366

Signed-off-by: Badr NASS LAHSEN <badr.nasslahsen@gmail.com>
@bnasslahsen

Copy link
Copy Markdown
Collaborator

Thanks @Mattias-Sehlstedt! Merged, with the date test moved to app275 since app273 is now taken on main, and backported to spring-boot-3 (126b4cc). Dropping PrimitiveType.enablePartialTime() from app237 also removed a test-order dependency: it is a JVM-wide switch, which is why app243 expected partial-time while the default for LocalTime is time-local.

@Mattias-Sehlstedt
Mattias-Sehlstedt deleted the test-date-resolving-oas-3-0 branch October 23, 2026 16:40
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.

2 participants