Skip to content

test: cover optional security fixture#30

Open
kriptoburak wants to merge 1 commit into
parallelworks:canaryfrom
kriptoburak:codex/optional-security-fixture
Open

test: cover optional security fixture#30
kriptoburak wants to merge 1 commit into
parallelworks:canaryfrom
kriptoburak:codex/optional-security-fixture

Conversation

@kriptoburak

Copy link
Copy Markdown

Summary

  • Add an anonymous security alternative to the petstore fixture.
  • Keep analyzer coverage for collecting bearer and API key schemes when anonymous access is allowed.

Validation

  • go test ./internal/analyzer -run TestAnalyzeSecuritySchemes_Petstore
  • git diff --check

@giraffesyo

Copy link
Copy Markdown
Member

@kriptoburak thanks for the PR. I only saw the testdata addition, was there a test file you meant to commit alongside it?

The generator currently only reads components.securitySchemes and operation-level security, the document-level security array this line touches isn't consumed anywhere yet.

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