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

testing: remove account balance checks #3054

Merged
merged 5 commits into from
Oct 14, 2021
Merged

testing: remove account balance checks #3054

merged 5 commits into from
Oct 14, 2021

Conversation

AlgoStephenAkiki
Copy link
Contributor

@AlgoStephenAkiki AlgoStephenAkiki commented Oct 13, 2021

Summary

Adds additional resilience to participation expiration test TestParticipationAccountsExpirationFuture.

Test Plan

This is a test.

@codecov-commenter
Copy link

codecov-commenter commented Oct 13, 2021

Codecov Report

Merging #3054 (782ba9b) into master (3572e1f) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3054   +/-   ##
=======================================
  Coverage   43.65%   43.66%           
=======================================
  Files         391      391           
  Lines       86792    86792           
=======================================
+ Hits        37893    37901    +8     
+ Misses      42872    42866    -6     
+ Partials     6027     6025    -2     
Impacted Files Coverage Δ
agreement/cryptoVerifier.go 75.73% <0.00%> (-2.21%) ⬇️
agreement/proposalManager.go 96.07% <0.00%> (-1.97%) ⬇️
network/wsNetwork.go 62.95% <0.00%> (ø)
ledger/acctupdates.go 66.53% <0.00%> (+0.09%) ⬆️
network/wsPeer.go 74.09% <0.00%> (+0.51%) ⬆️
data/transactions/verify/txn.go 44.29% <0.00%> (+0.87%) ⬆️
data/abi/abi_type.go 91.81% <0.00%> (+0.90%) ⬆️
txnsync/encodedgroupsmarshalers.go 100.00% <0.00%> (+1.12%) ⬆️

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 3572e1f...782ba9b. Read the comment docs.

tsachiherman
tsachiherman previously approved these changes Oct 13, 2021
@tsachiherman tsachiherman changed the title Bugfix/3042 remove account balance checks testing: remove account balance checks Oct 14, 2021
@tsachiherman tsachiherman merged commit 44e85be into algorand:master Oct 14, 2021
@AlgoStephenAkiki AlgoStephenAkiki deleted the bugfix/3042-remove-account-balance-checks branch October 14, 2021 14:07
cce pushed a commit to cce/go-algorand that referenced this pull request Oct 28, 2021
Summary
Adds additional resilience to participation expiration test TestParticipationAccountsExpirationFuture.

Test Plan
This is a test.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants