Skip to content
This repository has been archived by the owner on Oct 6, 2023. It is now read-only.

Review and add missing tests for TransactionStorage methods #597

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

max-sidorov
Copy link
Contributor

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Feb 22, 2022

Codecov Report

Merging #597 (c41b325) into master (03123c4) will increase coverage by 0.36%.
The diff coverage is 74.03%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #597      +/-   ##
==========================================
+ Coverage   61.94%   62.31%   +0.36%     
==========================================
  Files         279      278       -1     
  Lines       12921    12919       -2     
==========================================
+ Hits         8004     8050      +46     
+ Misses       4125     4084      -41     
+ Partials      792      785       -7     
Flag Coverage Δ
build-test 62.31% <74.03%> (+0.36%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
storage/transaction.go 72.89% <74.03%> (+10.65%) ⬆️
commander/new_block.go 54.30% <0.00%> (+8.06%) ⬆️
commander/workers.go 100.00% <0.00%> (+23.07%) ⬆️

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 03123c4...c41b325. Read the comment docs.

@max-sidorov max-sidorov marked this pull request as ready for review February 24, 2022 13:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants