Integrate official TOON spec tests (v1.4) and document deviations#1
Open
michal-ciechan wants to merge 9 commits into0xZunia:mainfrom
Open
Integrate official TOON spec tests (v1.4) and document deviations#1michal-ciechan wants to merge 9 commits into0xZunia:mainfrom
michal-ciechan wants to merge 9 commits into0xZunia:mainfrom
Conversation
- Import 17 JSON test fixtures from toon-format-spec repository - Add SpecTestRunner.cs for automated spec compliance testing - Update SPEC.md from v1.2 to v1.3 - 257 of 273 tests passing (94% compliance) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add comprehensive Spec Deviations section documenting 16 known test failures - Update spec version from v1.2 to v1.3 in Features - Add link to official TOON spec at toon-format/spec 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Remove JsonPropertyName attributes and rely on JsonNamingPolicy.CamelCase 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Replace SPEC.md with v1.4.0 from toon-format/spec - Update README references from v1.3 to v1.4 - Update version filter in SpecTestRunner to v1.4 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add Xunit.SkippableFact package for dynamic test skipping - Mark 19 known deviations as skipped with reference to README - Update README with correct deviation count (19) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Convert NormalizeLineEndings from private static helper to a string extension method for cleaner test syntax. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR integrates the official TOON specification tests and documents deviations from the spec. This is mainly a precursor to upgrading to newer spec versions.
Changes
SpecTestRunner.csthat automatically loads and runs tests from JSON fixturesSkip.If()referencing the READMEOpen Questions
I'm still deciding whether to:
I haven't looked into each deviation in detail yet, but I definitely think we should be using the raw test files from the spec repo to ensure compliance.
Test Results
Test plan
🤖 Generated with Claude Code