JSAbrahams / mamba

🐍 The Mamba programming language, because we care about safety
MIT License
85 stars 3 forks source link

Test AST's of test output #123

Closed JSAbrahams closed 5 years ago

JSAbrahams commented 5 years ago

Relevant issues

Fixes #122

Summary

Add tests to check that output ast is what we expect it to be.

Added Tests

List tests here once done

codecov[bot] commented 5 years ago

Codecov Report

Merging #123 into master will increase coverage by 0.88%. The diff coverage is 83.1%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #123      +/-   ##
==========================================
+ Coverage   82.54%   83.42%   +0.88%     
==========================================
  Files          65       70       +5     
  Lines        4565     4755     +190     
==========================================
+ Hits         3768     3967     +199     
+ Misses        797      788       -9
Impacted Files Coverage Ξ”
src/parser/expr_or_stmt.rs 100% <ΓΈ> (ΓΈ) :arrow_up:
src/parser/ast.rs 28.3% <0%> (+2.35%) :arrow_up:
src/core/construct.rs 27.41% <0%> (-0.23%) :arrow_down:
tests/parser/valid/control_flow.rs 84.44% <100%> (-0.78%) :arrow_down:
src/lexer/mod.rs 97.09% <100%> (+0.01%) :arrow_up:
src/desugar/control_flow.rs 78.12% <100%> (ΓΈ) :arrow_up:
tests/main.rs 100% <100%> (ΓΈ) :arrow_up:
tests/core/compound.rs 100% <100%> (ΓΈ) :arrow_up:
tests/parser/mod.rs 100% <100%> (ΓΈ) :arrow_up:
tests/parser/valid/compound.rs 100% <100%> (ΓΈ) :arrow_up:
... and 38 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 19481b7...ca720f0. Read the comment docs.

JSAbrahams commented 5 years ago

This PR is kind of on the large side, but I was a bit too lazy to split this up into smaller PR's.