Skip to content

Test operation security serialization#5224

Open
kriptoburak wants to merge 1 commit into
swagger-api:masterfrom
kriptoburak:codex/model-operation-security-alternatives-20260702
Open

Test operation security serialization#5224
kriptoburak wants to merge 1 commit into
swagger-api:masterfrom
kriptoburak:codex/model-operation-security-alternatives-20260702

Conversation

@kriptoburak

Copy link
Copy Markdown

Summary

  • Add a Swagger Core model serialization regression for operation security alternatives.
  • Assert the serialized OpenAPI keeps the anonymous requirement before the API key requirement.

Validation

  • Static whitespace check passed before commit.
  • Not run: this environment does not have a Java runtime.

@ewaostrowska ewaostrowska added the backlog label Jul 9, 2026 — with Claude
@ewaostrowska
ewaostrowska force-pushed the codex/model-operation-security-alternatives-20260702 branch from 6db8790 to c179dcd Compare July 17, 2026 10:54
@ewaostrowska
ewaostrowska force-pushed the codex/model-operation-security-alternatives-20260702 branch from c179dcd to d2c3c6d Compare July 22, 2026 13:04
@ewaostrowska

Copy link
Copy Markdown
Contributor

Thanks for the contribution @kriptoburak . Before we can determine whether this test should be added, could you please provide some additional context?

The PR is currently missing:

  • A link to the issue, user report, or regression that motivated this change.
  • A minimal example showing the observed behavior on the current master branch.
  • The actual serialized output and the expected serialized output.
  • Explanation on what regression or future change it is intended to protect against.
  • If this is intended to fix an existing problem, the corresponding production-code change. At present, the PR only adds a test.

With that information, we can assess whether this is a useful regression test, documentation of existing behavior, or whether an implementation fix is also needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants