Skip to content
This repository has been archived by the owner on May 11, 2024. It is now read-only.

test(all): add more tests #439

Merged
merged 12 commits into from
Oct 30, 2023
Merged

test(all): add more tests #439

merged 12 commits into from
Oct 30, 2023

Conversation

davidtaikocha
Copy link
Member

@davidtaikocha davidtaikocha commented Oct 26, 2023

Increased code coverage to 60%+

@codecov
Copy link

codecov bot commented Oct 26, 2023

Codecov Report

Merging #439 (a2c6537) into main (93b9ecf) will increase coverage by 8.16%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main     #439      +/-   ##
==========================================
+ Coverage   53.21%   61.38%   +8.16%     
==========================================
  Files          44       44              
  Lines        5160     5104      -56     
==========================================
+ Hits         2746     3133     +387     
+ Misses       2137     1660     -477     
- Partials      277      311      +34     
Files Coverage Δ
driver/state/state.go 61.97% <ø> (-0.40%) ⬇️
proposer/prover_selector/eth_fee_eoa_selector.go 68.35% <100.00%> (+68.35%) ⬆️
driver/chain_syncer/beaconsync/progress_tracker.go 69.12% <66.66%> (-3.83%) ⬇️
driver/chain_syncer/beaconsync/syncer.go 0.00% <0.00%> (ø)
prover/prover.go 51.52% <0.00%> (+0.59%) ⬆️
pkg/rpc/utils.go 25.72% <0.00%> (ø)
driver/chain_syncer/chain_syncer.go 44.57% <0.00%> (+3.00%) ⬆️
driver/state/l1_current.go 61.01% <53.84%> (+29.01%) ⬆️

... and 8 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@davidtaikocha davidtaikocha marked this pull request as ready for review October 29, 2023 13:22
@davidtaikocha davidtaikocha merged commit 9e9c7f3 into main Oct 30, 2023
4 checks passed
@davidtaikocha davidtaikocha deleted the more-a6-tests branch October 30, 2023 19:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants