Closed JSAbrahams closed 5 years ago
Merging #58 into master will increase coverage by
2.83%
. The diff coverage is87.04%
.
@@ Coverage Diff @@
## master #58 +/- ##
==========================================
+ Coverage 77.26% 80.09% +2.83%
==========================================
Files 48 57 +9
Lines 3347 3788 +441
==========================================
+ Hits 2586 3034 +448
+ Misses 761 754 -7
Impacted Files | Coverage Ξ | |
---|---|---|
tests/core/collection.rs | 75% <ΓΈ> (ΓΈ) |
:arrow_up: |
src/lib.rs | 0% <ΓΈ> (ΓΈ) |
:arrow_up: |
tests/core/control_flow.rs | 75% <ΓΈ> (ΓΈ) |
:arrow_up: |
tests/core/function.rs | 66.66% <ΓΈ> (ΓΈ) |
:arrow_up: |
src/desugar/mod.rs | 100% <ΓΈ> (ΓΈ) |
|
tests/core/class.rs | 100% <ΓΈ> (ΓΈ) |
:arrow_up: |
tests/main.rs | 75% <ΓΈ> (ΓΈ) |
:arrow_up: |
tests/core/compound.rs | 100% <ΓΈ> (ΓΈ) |
:arrow_up: |
src/command/mod.rs | 79.16% <ΓΈ> (ΓΈ) |
:arrow_up: |
src/parser/ast.rs | 26.05% <0%> (ΓΈ) |
:arrow_up: |
... and 30 more |
Continue to review full report at Codecov.
Legend - Click here to learn more
Ξ = absolute <relative> (impact)
,ΓΈ = not affected
,? = missing data
Powered by Codecov. Last update 9d52cb8...0a11e2f. Read the comment docs.
Relevant issues
There were no tests for the desugar stage. It is nice to test this directly however to make sure we desugar as we intended.
Summary
Add tests for the desugar stage.
neq
was desugared asnot
andeq
instead of existingCore
neq
node.Added Tests
Additional Context
...