JSAbrahams / mamba

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

Verify some Name functionality #243

Closed JSAbrahams closed 2 years ago

JSAbrahams commented 2 years ago

Summary

Add extra tests to test low hanging fruit of Name and AST. A lot of the type checker builds on these so best to test some extra edge cases which are not covered in any of the end-to-end tests we have currently. A lot of these end-to-end tests cover only basic language functionality, but not a lot of type unions etc.

Added Tests

codecov[bot] commented 2 years ago

Codecov Report

Merging #243 (ecd0ae8) into v0-3-1 (629ab6b) will increase coverage by 1.04%. The diff coverage is 84.09%.

Impacted file tree graph

@@            Coverage Diff             @@
##           v0-3-1     #243      +/-   ##
==========================================
+ Coverage   79.88%   80.92%   +1.04%     
==========================================
  Files          99       99              
  Lines        8731     8940     +209     
==========================================
+ Hits         6975     7235     +260     
+ Misses       1756     1705      -51     
Impacted Files Coverage Ξ”
src/parse/ast/node.rs 27.35% <34.72%> (+2.21%) :arrow_up:
src/check/constrain/constraint/expected.rs 68.42% <50.00%> (ΓΈ)
src/check/constrain/constraint/builder.rs 98.03% <100.00%> (ΓΈ)
src/check/constrain/unify/expression/substitute.rs 100.00% <100.00%> (ΓΈ)
src/check/ident.rs 96.77% <100.00%> (+30.10%) :arrow_up:
src/check/name/mod.rs 88.46% <100.00%> (+16.27%) :arrow_up:
src/parse/ast/mod.rs 67.43% <100.00%> (+7.65%) :arrow_up:
src/check/name/truename/mod.rs 82.81% <0.00%> (+1.56%) :arrow_up:
... and 2 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 629ab6b...ecd0ae8. Read the comment docs.