search

rjkroege / edwood

Go version of Plan9 Acme Editor
Other
385 stars 34 forks source link

Enhanced framework for frame.Frame tests #415

Closed rjkroege closed 2 years ago

rjkroege commented 2 years ago

To help with #56 and other frame.Frame improvements, update the draw mock to record the commands sent to 9fans.net/go/draw. Further use this in some simple tests of frame.Insert to demonstrate that the mocks work and significantly expand test coverage of the frame unit tests.

codecov[bot] commented 2 years ago

Codecov Report

Merging #415 (8054742) into master (4ca819b) will increase coverage by 1.79%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #415      +/-   ##
==========================================
+ Coverage   53.83%   55.62%   +1.79%     
==========================================
  Files          55       55              
  Lines       10678    10678              
==========================================
+ Hits         5748     5940     +192     
+ Misses       4519     4297     -222     
- Partials      411      441      +30
Impacted Files Coverage Δ
frame/frame.go 35.71% <ø> (+35.71%) :arrow_up:
frame/util.go 55.76% <0.00%> (+5.76%) :arrow_up:
frame/select.go 9.24% <0.00%> (+9.24%) :arrow_up:
frame/draw.go 18.30% <0.00%> (+10.56%) :arrow_up:
frame/box.go 69.02% <0.00%> (+14.15%) :arrow_up:
frame/insert.go 43.17% <0.00%> (+26.43%) :arrow_up:
frame/ptofchar.go 78.08% <0.00%> (+30.13%) :arrow_up:
frame/tick.go 56.52% <0.00%> (+56.52%) :arrow_up:
frame/frameopts.go 92.30% <0.00%> (+92.30%) :arrow_up:

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 4ca819b...8054742. Read the comment docs.