JuliaLang / JuliaSyntax.jl

The Julia compiler frontend
Other
274 stars 33 forks source link

Improve test coverge of source_files, syntax_tree #204

Closed timholy closed 1 year ago

timholy commented 1 year ago

These two files are particularly affected by recent and upcoming changes (e.g., #193). This adds a bit more coverage as a guard against breakage.

timholy commented 1 year ago

Overall coverage is quite good, but on main these two files are on the lower side (though still not too bad): https://app.codecov.io/gh/JuliaLang/JuliaSyntax.jl/tree/main/src

codecov[bot] commented 1 year ago

Codecov Report

Merging #204 (e51eca0) into main (1429841) will increase coverage by 1.93%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #204      +/-   ##
==========================================
+ Coverage   92.90%   94.83%   +1.93%     
==========================================
  Files          15       15              
  Lines        3776     3776              
==========================================
+ Hits         3508     3581      +73     
+ Misses        268      195      -73     
Impacted Files Coverage Δ
src/syntax_tree.jl 94.73% <100.00%> (+28.65%) :arrow_up:
src/source_files.jl 85.71% <0.00%> (+19.64%) :arrow_up:
src/utils.jl 76.56% <0.00%> (+20.31%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.