Skip to content

test: cover anonymous security alternatives#824

Open
kriptoburak wants to merge 1 commit into
IBM:mainfrom
kriptoburak:codex/anonymous-security-alternative
Open

test: cover anonymous security alternatives#824
kriptoburak wants to merge 1 commit into
IBM:mainfrom
kriptoburak:codex/anonymous-security-alternative

Conversation

@kriptoburak

Copy link
Copy Markdown

Summary

  • Add a ruleset regression test for operation security lists that include an anonymous alternative.
  • Keep the existing scheme usage valid while covering the empty security requirement object from OpenAPI.

Validation

  • git diff --check
  • node --check packages/ruleset/test/rules/securityschemes.test.js
  • Not run: npm ci or npm run test-ruleset because this clone has no node_modules and local disk is nearly full.

Signed-off-by: kriptoburak <kriptoburak@users.noreply.github.com>
@CLAassistant

CLAassistant commented Jul 2, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@pyrooka

pyrooka commented Jul 3, 2026

Copy link
Copy Markdown
Member

Hi @kriptoburak, thanks for your contribution. Looks like you haven't accepted the CLA yet, so please do that. Also, the tests fail so we need to fix them. Would you mind adjusting the rule to handle empty/anonym security schemes correctly?

@pyrooka pyrooka self-requested a review July 3, 2026 09:53
@pyrooka pyrooka self-assigned this Jul 3, 2026
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.

3 participants