Closed JSAbrahams closed 5 years ago
Merging #40 into master will increase coverage by
3.37%
. The diff coverage is74.68%
.
@@ Coverage Diff @@
## master #40 +/- ##
==========================================
+ Coverage 61.44% 64.82% +3.37%
==========================================
Files 26 38 +12
Lines 2088 2260 +172
==========================================
+ Hits 1283 1465 +182
+ Misses 805 795 -10
Impacted Files | Coverage Ξ | |
---|---|---|
tests/lexer/lexer.rs | 100% <ΓΈ> (ΓΈ) |
|
src/parser/ast.rs | 0% <ΓΈ> (ΓΈ) |
:arrow_up: |
src/core/construct.rs | 0% <0%> (ΓΈ) |
:arrow_up: |
src/desugarer/mod.rs | 100% <100%> (ΓΈ) |
:arrow_up: |
tests/parser/collection.rs | 100% <100%> (ΓΈ) |
|
tests/parser/control_flow.rs | 100% <100%> (ΓΈ) |
|
tests/core/compound.rs | 100% <100%> (ΓΈ) |
|
tests/parser/mod.rs | 100% <100%> (ΓΈ) |
|
src/parser/control_flow_expr.rs | 87.09% <100%> (ΓΈ) |
:arrow_up: |
tests/command.rs | 96.77% <100%> (ΓΈ) |
|
... and 32 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 dd7a3a2...5e1eae3. Read the comment docs.
Relevant issues
The tests were a bit messy. Now they are slightly less messy. This works towards #32. Still need to investigate how I am going to do this.
Summary
Split up tests into smaller components. Add tests for the core. Fix some bugs or missing features in core and desugar module. Almost everything in the base language is now parsable, though we do still miss some key functionality.
Added Tests
Tests for collections with 'ofmut'. Test for the core (and by extension desugar module).
Additional Context
...