JSAbrahams / mamba

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

Desugar default constructor and parents arguments #117

Closed JSAbrahams closed 5 years ago

JSAbrahams commented 5 years ago

Relevant issues

Fixes #105 Fixes #53 Addresses #19

Summary

Added Tests

codecov[bot] commented 5 years ago

Codecov Report

Merging #117 into master will increase coverage by 1.39%. The diff coverage is 73.21%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #117      +/-   ##
==========================================
+ Coverage   83.42%   84.82%   +1.39%     
==========================================
  Files          70       68       -2     
  Lines        4755     4824      +69     
==========================================
+ Hits         3967     4092     +125     
+ Misses        788      732      -56
Impacted Files Coverage Ξ”
tests/desugar/types.rs 94.11% <ΓΈ> (-1.13%) :arrow_down:
tests/desugar/expression_and_statements.rs 89.85% <ΓΈ> (-0.82%) :arrow_down:
tests/output/function.rs 92.3% <ΓΈ> (+30.76%) :arrow_up:
src/desugar/mod.rs 100% <ΓΈ> (ΓΈ) :arrow_up:
src/lexer/mod.rs 97.09% <ΓΈ> (ΓΈ) :arrow_up:
src/parser/ast.rs 28.65% <0%> (+0.35%) :arrow_up:
src/parser/call.rs 86.66% <100%> (+2.24%) :arrow_up:
src/parser/_type.rs 95.03% <100%> (+2.12%) :arrow_up:
tests/parser/valid/expression_and_statement.rs 85.46% <100%> (ΓΈ) :arrow_up:
src/parser/operation.rs 97.58% <100%> (+0.01%) :arrow_up:
... and 19 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 2bce673...89a6d0b. Read the comment docs.

JSAbrahams commented 5 years ago

Probably best to only merge this after #123 so we can verify the output.