Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ci] Enable code coverage report on pr. #1904

Merged
merged 2 commits into from
Dec 30, 2020
Merged

Conversation

jolestar
Copy link
Member

No description provided.

@jolestar jolestar requested review from ssyuan and sanlee42 December 30, 2020 08:30
@codecov
Copy link

codecov bot commented Dec 30, 2020

Codecov Report

Merging #1904 (0a59952) into master (1fc7a6b) will decrease coverage by 0.18%.
The diff coverage is 14.98%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1904      +/-   ##
==========================================
- Coverage   29.67%   29.49%   -0.19%     
==========================================
  Files         431      432       +1     
  Lines       43815    44064     +249     
  Branches    20210    20311     +101     
==========================================
- Hits        13004    12997       -7     
- Misses      17191    17498     +307     
+ Partials    13620    13569      -51     
Flag Coverage Δ
unittests 29.49% <14.98%> (-0.19%) ⬇️

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

Impacted Files Coverage Δ
account/api/src/types.rs 56.52% <0.00%> (+8.40%) ⬆️
account/lib/src/account.rs 37.03% <0.00%> (ø)
chain/src/tests/test_block_chain.rs 12.45% <0.00%> (ø)
chain/src/tests/test_opened_block.rs 17.64% <0.00%> (ø)
cmd/indexer/src/es_sinker.rs 0.00% <ø> (ø)
cmd/starcoin/src/dev/sign_txn_helper.rs 14.12% <0.00%> (-1.86%) ⬇️
executor/src/account.rs 43.90% <0.00%> (ø)
miner/src/create_block_template/mod.rs 41.83% <0.00%> (ø)
rpc/api/src/account/mod.rs 29.41% <ø> (-3.93%) ⬇️
rpc/api/src/contract_api.rs 16.66% <0.00%> (-3.34%) ⬇️
... and 59 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 5fb09fb...0a59952. Read the comment docs.

@github-actions
Copy link

Benchmark for 73e9993

Click to view benchmark
Test PR Benchmark Master Benchmark %
accumulator_append 1462.4±57.96µs 1476.6±65.62µs -0.96%
block_apply/block_apply_10 392.9±3.64ms 393.8±1.90ms -0.23%
block_apply/block_apply_1000 40.8±0.19s 39.8±0.40s +2.51%
get_with_proof/db_store 43.4±0.83µs 43.7±1.48µs -0.69%
get_with_proof/mem_store 28.7±0.52µs 28.3±0.69µs +1.41%
put_and_commit/db_store/1 737.5±24.18µs 733.7±20.89µs +0.52%
put_and_commit/db_store/10 2.2±0.21ms 2.2±0.21ms 0.00%
put_and_commit/db_store/100 13.2±0.80ms 13.2±0.84ms 0.00%
put_and_commit/db_store/5 1434.3±226.25µs 1446.1±254.32µs -0.82%
put_and_commit/db_store/50 7.4±1.67ms 7.7±4.01ms -3.90%
put_and_commit/mem_store/1 108.7±6.71µs 107.7±7.14µs +0.93%
put_and_commit/mem_store/10 956.4±49.70µs 945.0±53.55µs +1.21%
put_and_commit/mem_store/100 9.4±0.76ms 9.2±0.69ms +2.17%
put_and_commit/mem_store/5 495.9±32.12µs 493.0±28.88µs +0.59%
put_and_commit/mem_store/50 4.7±0.37ms 4.7±0.36ms 0.00%
query_block/query_block_in(10)_times(100) 1291.2±8.09µs 1218.8±7.39µs +5.94%
query_block/query_block_in(10)_times(1000) 13.0±0.13ms 12.1±0.13ms +7.44%
query_block/query_block_in(10)_times(10000) 129.7±1.09ms 123.8±1.28ms +4.77%
query_block/query_block_in(1000)_times(100) 1228.5±3.92µs 1129.2±10.90µs +8.79%
query_block/query_block_in(1000)_times(1000) 12.3±0.06ms 11.4±0.09ms +7.89%
query_block/query_block_in(1000)_times(10000) 124.0±0.38ms 114.7±1.83ms +8.11%
storage_transaction 18.8±0.83ms 18.8±0.78ms 0.00%
vm/transaction_execution/1 362.1±3.33ms 361.0±2.90ms +0.30%
vm/transaction_execution/10 117.7±3.52ms 116.1±0.87ms +1.38%
vm/transaction_execution/20 109.6±1.17ms 109.7±1.27ms -0.09%
vm/transaction_execution/5 142.0±1.68ms 141.0±1.27ms +0.71%
vm/transaction_execution/50 129.4±1.51ms 129.4±1.48ms 0.00%

@jolestar jolestar force-pushed the feature/codecov_for_pr branch from b92fbc0 to 84134d0 Compare December 30, 2020 09:42
@github-actions
Copy link

Benchmark for 99bed6e

Click to view benchmark
Test PR Benchmark Master Benchmark %
accumulator_append 1381.0±111.15µs 1381.5±96.79µs -0.04%
block_apply/block_apply_10 345.7±1.42ms 350.3±6.91ms -1.31%
block_apply/block_apply_1000 36.5±0.17s 36.6±0.23s -0.27%
get_with_proof/db_store 43.9±0.21µs 44.0±2.51µs -0.23%
get_with_proof/mem_store 28.8±0.21µs 28.9±0.48µs -0.35%
put_and_commit/db_store/1 662.7±82.71µs 703.5±96.65µs -5.80%
put_and_commit/db_store/10 2.1±0.52ms 2.2±0.49ms -4.55%
put_and_commit/db_store/100 14.1±2.40ms 13.9±2.84ms +1.44%
put_and_commit/db_store/5 1447.3±586.48µs 1357.0±543.32µs +6.65%
put_and_commit/db_store/50 7.7±1.08ms 7.6±1.05ms +1.32%
put_and_commit/mem_store/1 108.4±7.97µs 108.5±6.71µs -0.09%
put_and_commit/mem_store/10 962.1±52.03µs 962.0±55.50µs +0.01%
put_and_commit/mem_store/100 9.4±0.91ms 9.5±0.75ms -1.05%
put_and_commit/mem_store/5 495.0±28.16µs 493.7±28.87µs +0.26%
put_and_commit/mem_store/50 4.8±0.38ms 4.8±0.37ms 0.00%
query_block/query_block_in(10)_times(100) 1238.6±13.92µs 1257.2±3.71µs -1.48%
query_block/query_block_in(10)_times(1000) 12.4±0.09ms 12.4±0.07ms 0.00%
query_block/query_block_in(10)_times(10000) 123.7±0.48ms 126.3±0.89ms -2.06%
query_block/query_block_in(1000)_times(100) 1210.1±3.95µs 1238.6±2.94µs -2.30%
query_block/query_block_in(1000)_times(1000) 12.2±0.02ms 12.3±0.11ms -0.81%
query_block/query_block_in(1000)_times(10000) 120.8±0.11ms 123.2±0.35ms -1.95%
storage_transaction 16.5±4.49ms 15.0±0.37ms +10.00%
vm/transaction_execution/1 358.5±1.21ms 357.6±1.42ms +0.25%
vm/transaction_execution/10 116.0±0.52ms 115.3±0.24ms +0.61%
vm/transaction_execution/20 109.3±0.28ms 108.6±0.10ms +0.64%
vm/transaction_execution/5 139.3±0.58ms 139.3±0.71ms 0.00%
vm/transaction_execution/50 128.3±0.22ms 127.8±1.19ms +0.39%

@jolestar jolestar merged commit 1152ca5 into master Dec 30, 2020
@jolestar jolestar deleted the feature/codecov_for_pr branch December 30, 2020 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants