Skip to content

Add compiler options parser tests (#13878)#19554

Merged
T-Gro merged 3 commits intodotnet:mainfrom
techiedesu:fsc-options-parser-tests
Apr 14, 2026
Merged

Add compiler options parser tests (#13878)#19554
T-Gro merged 3 commits intodotnet:mainfrom
techiedesu:fsc-options-parser-tests

Conversation

@techiedesu
Copy link
Copy Markdown
Contributor

@techiedesu techiedesu commented Apr 6, 2026

Summary

  • Add ParserErrors.fs — horizontal coverage of compiler options parser error handling (error codes 224, 241, 1050, 1048, 1065, 243)
  • Add UncoveredOptions.fs — smoke tests for switch, unit, and string options without dedicated test coverage

Addresses #13878

Tests only — NO_RELEASE_NOTES label is appropriate.

🤖 Generated with Claude Code

ParserErrors.fs — horizontal error handling coverage:
  missing arguments (error 224), invalid integer arguments (error 241),
  out-of-range warn level (error 1050), unrecognized target (error 1048),
  unsupported checksum algorithm (error 1065), unrecognized options (error 243).

UncoveredOptions.fs — smoke tests for options without dedicated coverage:
  switch options (+/-), unit options, valid string values,
  compilation modes, diagnostic options.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 6, 2026

✅ No release notes required

@T-Gro T-Gro added the NO_RELEASE_NOTES Label for pull requests which signals, that user opted-out of providing release notes label Apr 14, 2026
@github-project-automation github-project-automation bot moved this from New to In Progress in F# Compiler and Tooling Apr 14, 2026
@T-Gro T-Gro enabled auto-merge (squash) April 14, 2026 14:20
@T-Gro T-Gro merged commit 0b91207 into dotnet:main Apr 14, 2026
49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NO_RELEASE_NOTES Label for pull requests which signals, that user opted-out of providing release notes

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants