Skip to content

test: cover empty operation security override#4237

Open
kriptoburak wants to merge 1 commit into
swagger-api:masterfrom
kriptoburak:codex/test-security-override-empty
Open

test: cover empty operation security override#4237
kriptoburak wants to merge 1 commit into
swagger-api:masterfrom
kriptoburak:codex/test-security-override-empty

Conversation

@kriptoburak

Copy link
Copy Markdown

Description

Adds an OAS 3 authorization test for operation-level security: [] when global OAuth security exists.

Motivation and Context

OpenAPI lets an operation-level empty security array override root-level requirements. This test keeps request building from adding global authorization to explicitly public operations.

How Has This Been Tested?

  • git diff --check
  • npx --yes prettier@3.1.1 --check test/oas3/execute/authorization.js
  • Full test suite not run because this temporary clone has no node_modules and the workspace has about 1 GiB free disk.

Screenshots (if appropriate):

N/A

Types of changes

  • No code changes (changes to documentation, CI, metadata, etc)
  • Dependency changes (any modification to dependencies in package.json)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

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.

1 participant