Closed
Conversation
Lays out a two-layer testing approach (parse tests + compile tests) to replace the test suite removed in e2008b5, updated for 0.19. Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering) Co-Authored-By: Claude <noreply@anthropic.com> Co-Authored-By: Happy <yesreply@happy.engineering>
11-task plan covering cabal stanza, test runner, parse tests, compile tests, and all test .elm files. Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering) Co-Authored-By: Claude <noreply@anthropic.com> Co-Authored-By: Happy <yesreply@happy.engineering>
Add 6 parse-good .elm files covering literals, records, pattern matching, functions, types, and comments. Add 4 parse-bad .elm files covering unclosed strings, bad indentation, garbage tokens, and invalid operators. Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering) Co-Authored-By: Claude <noreply@anthropic.com> Co-Authored-By: Happy <yesreply@happy.engineering>
Add compile-good project with elm.json and 3 source files covering basic expressions, pattern matching, and type annotations. Add compile-bad project with elm.json and 3 source files covering infinite types, incomplete patterns, and type mismatches. Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering) Co-Authored-By: Claude <noreply@anthropic.com> Co-Authored-By: Happy <yesreply@happy.engineering>
Add Test-Suite elm-tests to elm.cabal with all required compiler modules and dependencies. Create Test.hs (main runner), Test/Parse.hs (parse tests using Parse.Module.fromByteString), and Test/Compile.hs (stub). All 10 parse tests pass (6 parse-good, 4 parse-bad). Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering) Co-Authored-By: Claude <noreply@anthropic.com> Co-Authored-By: Happy <yesreply@happy.engineering>
Shells out to the built elm binary for full pipeline testing. Uses System.Process with cwd to run elm make from the correct project directory. All 16 tests pass (10 parse + 6 compile). Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering) Co-Authored-By: Claude <noreply@anthropic.com> Co-Authored-By: Happy <yesreply@happy.engineering>
|
Thanks for suggesting these code changes. To set expectations:
Finally, please be patient with the core team. They are trying their best with limited resources. |
Author
|
Sorry, meant to create in my fork |
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
1 participant
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.
No description provided.