fix: parity with elixir parser based on property tests#79
fix: parity with elixir parser based on property tests#79mhanberg merged 5 commits intoelixir-tools:mainfrom
Conversation
18a637e to
ccd9a5f
Compare
|
@doorgan some of the property failures in this one are related to not correctly counting new lines in some places. |
|
Yes, I'm taking a look at this one now |
ccd9a5f to
e85bdd3
Compare
They take over 40 seconds to finish otherwise
e7744e8 to
8128f41
Compare
|
Besides the metadata issues, a lot of the issues flagged by these property test were around this elixir grammar rule: % Map and structs
map_base_expr -> sub_matched_expr : '$1'.
map_base_expr -> at_op_eol map_base_expr : build_unary_op('$1', '$2').
map_base_expr -> unary_op_eol map_base_expr : build_unary_op('$1', '$2').
map_base_expr -> ellipsis_op map_base_expr : build_unary_op('$1', '$2').We were parsing the struct name as a "lone identifier" but the elixir grammar allows a whole range of very bizarre code like I also added a lot of those to |
I don't know what Oracle was supposed to mean, it just referred to the elixir parser results
🤖 I have created a release *beep* *boop* --- ## [0.3.6](v0.3.5...v0.3.6) (2026-02-16) ### Bug Fixes * parity with elixir parser based on property tests ([#79](#79)) ([8ca3746](8ca3746)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
This PR adds property tests that generate random strings in various elixir contexts.
Assumptions:
in tolerant mode
in strict mode