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

Fix coverage #153

Merged
merged 6 commits into from Jul 14, 2022
Merged

Fix coverage #153

merged 6 commits into from Jul 14, 2022

Conversation

fmorency
Copy link
Contributor

No description provided.

@codecov-commenter
Copy link

Codecov Report

Merging #153 (0a7ed70) into main (daa445f) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main    #153   +/-   ##
=====================================
  Coverage   5.48%   5.48%           
=====================================
  Files         51      51           
  Lines       6741    6741           
=====================================
  Hits         370     370           
  Misses      6371    6371           

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 daa445f...0a7ed70. Read the comment docs.

@fmorency fmorency self-assigned this Jul 13, 2022
@fmorency fmorency added the bug Something isn't working label Jul 13, 2022
@github-actions
Copy link

Code Coverage Report

Coverage for this commit
63.85%
Coverage Report
FileStmtsBranchesFuncsLines
src/many-cli/src
   main.rs0.36%100%0.39%0.33%
src/many-client/src
   lib.rs100%100%100%100%
   client.rs0%100%0%0%
src/many-macros/src
   lib.rs0.40%100%1.05%0.24%
src/many/src
   lib.rs100%100%100%100%
   server.rs75.29%100%64.86%76.57%
   message.rs90.75%100%66.67%95.97%
   hsm.rs79.29%100%28.40%91.28%
   protocol.rs97.79%100%97.30%97.92%
   cbor.rs97.14%100%90%98.33%
   cose_helpers.rs93.62%100%50%96.59%
   transport.rs0%100%0%0%
   types.rs50.11%100%35.79%53.74%
src/many/src/message
   response.rs68.45%100%44.12%74.63%
   error.rs28%100%10.85%38.02%
   request.rs58.79%100%47.37%61.81%
src/many/src/protocol
   attributes.rs48.15%100%56.52%41.94%
src/many/src/server
   module.rs83.19%100%85.42%81.54%
src/many/src/server/module
   _1001_abci_frontend.rs0%100%0%0%
   _7_kvstore_commands.rs67.04%100%54.76%96.23%
   _6_ledger_commands.rs64.21%100%53.52%95.83%
   _9_account.rs47.63%100%28.10%81.54%
   _1000_abci_backend.rs0%100%0%0%
   _2_ledger.rs70.35%100%55.47%97.18%
   _4_events.rs67.89%100%54.62%96.67%
   _8_async.rs79.85%100%61.76%90.64%
   _1_blockchain.rs73.76%100%56.46%97.39%
   _3_kvstore.rs64.88%100%55.12%95.12%
   _0_base.rs68.35%100%53.74%90.21%
   _1002_idstore.rs71.68%100%57.14%97.12%
src/many/src/server/module/_1002_idstore
   store.rs75%100%70%100%
   get.rs73.68%100%68.75%100%
   types.rs71.43%100%66.67%100%
src/many/src/server/module/_2_ledger
   balance.rs72.73%100%66.67%100%
   info.rs100%100%100%100%
src/many/src/server/module/_3_kvstore
   info.rs100%100%100%100%
   get.rs72.73%100%66.67%100%
src/many/src/server/module/_4_events
   info.rs100%100%100%100%
   list.rs80%100%75%100%
src/many/src/server/module/_6_ledger_commands
   send.rs85.71%100%83.33%100%
src/many/src/server/module/_7_kvstore_commands
   delete.rs71.43%100%66.67%100%
   put.rs92.73%100%83.33%95.35%
src/many/src/server/module/_9_account
   features.rs70%100%60.87%73.68%
src/many/src/server/module/_9_account/features
   multisig.rs26.86%100%13.73%34.25%
src/many/src/transport
   http.rs0%100%0%0%
src/many/src/types
   events.rs64.38%100%59.38%65.50%
   blockchain.rs73.96%100%72.73%74.60%
   identity.rs80.06%100%73.74%81.26%
   either.rs26.96%100%28.41%26.59%
   ledger.rs48.03%100%27.55%60.90%
src/many/src/types/identity
   cose.rs69.79%100%43.40%74.82%

@fmorency fmorency marked this pull request as ready for review July 13, 2022 15:05
@fmorency fmorency requested a review from hansl July 13, 2022 15:12
@fmorency
Copy link
Contributor Author

CodeCov seems to be confused by this PR.

You can check the reporting directly on CodeCov.

Afterward, I hope things become stable.

Copy link
Contributor

@hansl hansl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you call out that you're removing GitHub actions in the commit message please?

@fmorency fmorency merged commit 30ebd04 into liftedinit:main Jul 14, 2022
@fmorency fmorency deleted the fmorency/fix-coverage branch July 14, 2022 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants