Skip to content

Fix integration tests #1265

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

Closed
wants to merge 1 commit into from

Conversation

Viicos
Copy link
Contributor

@Viicos Viicos commented Jun 4, 2025

No description provided.

@Viicos Viicos marked this pull request as draft June 4, 2025 09:25
@Viicos Viicos force-pushed the fix-integration-tests branch from 0c41e27 to 0afaacd Compare June 4, 2025 09:39
@Viicos
Copy link
Contributor Author

Viicos commented Jun 4, 2025

cc @dbanty, too many stuff breaking since https://github.com/openapi-generators/openapi-test-server was updated. Might be good to revert some changes there (on the openapi spec)?

(fyi, I've tried looking at this as it is breaking our third-party tests in Pydantic).

@Viicos Viicos closed this Jun 4, 2025
@dbanty
Copy link
Collaborator

dbanty commented Jun 4, 2025

@Viicos sorry about that! The right thing to do is to pin to a tag instead of following the main branch. The changes to the test server are to test #938 (which also has the required updates for breaking changes).

Just waiting on some final testing to merge that one.

@Viicos
Copy link
Contributor Author

Viicos commented Jun 4, 2025

Ah ok makes sense! I've opened #1266.

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