Skip to content

test(formatter): add metadata boundary corpus guards#19

Merged
MayerTim merged 1 commit into
mainfrom
test/formatter-corpus-metadata-boundaries
Jun 10, 2026
Merged

test(formatter): add metadata boundary corpus guards#19
MayerTim merged 1 commit into
mainfrom
test/formatter-corpus-metadata-boundaries

Conversation

@MayerTim

Copy link
Copy Markdown
Owner

Summary

  • Add a Watcom corpus fixture for a SQLovely metadata header immediately after a procedure signature and before BEGIN.
  • Add a Watcom corpus fixture for multiple objects with separate metadata headers in one script.
  • Preserve object-name-like SQL text inside metadata comments, string literals, and comments.
  • Preserve current formatter behavior with expected-output and idempotency checks.
  • Keep this branch test-only with no formatter implementation, metadata logic, diagnostics, extension behavior, or formatter default changes.

Validation

  • npm run test:corpus
  • npm run validate
  • npm run package:vsix

Notes

This PR fills the final small corpus coverage gap identified by the private coverage inventory. It does not change formatter implementation, metadata behavior, diagnostics behavior, formatter defaults, extension behavior, product behavior, or public documentation.

@MayerTim MayerTim merged commit 2b3a0e5 into main Jun 10, 2026
1 check passed
@MayerTim MayerTim deleted the test/formatter-corpus-metadata-boundaries branch June 10, 2026 21:19
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