Skip to content

ledger: fix TestCatchpointReadDatabaseOverflowSingleAccount#4579

Merged
algorandskiy merged 1 commit intoalgorand:masterfrom
nicholasguo:nguo/fixcptest
Sep 23, 2022
Merged

ledger: fix TestCatchpointReadDatabaseOverflowSingleAccount#4579
algorandskiy merged 1 commit intoalgorand:masterfrom
nicholasguo:nguo/fixcptest

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Sep 23, 2022

Summary

Fixed the TestCatchpointReadDatabaseOverflowSingleAccount test. It was failing due to the same reason as the TestCatchpointReadDatabaseOverflowAccounts test. The fix is exactly the same as #4480.

Test Plan

This is a test.

@ghost ghost self-assigned this Sep 23, 2022
@ghost ghost added the Team Carbon-11 label Sep 23, 2022
@ghost ghost requested a review from algorandskiy September 23, 2022 17:32
@ghost ghost added the Bug-Fix label Sep 23, 2022
@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 23, 2022

Codecov Report

Merging #4579 (baad408) into master (42b9533) will increase coverage by 0.29%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4579      +/-   ##
==========================================
+ Coverage   53.86%   54.15%   +0.29%     
==========================================
  Files         401      401              
  Lines       51642    51642              
==========================================
+ Hits        27817    27967     +150     
+ Misses      21456    21328     -128     
+ Partials     2369     2347      -22     
Impacted Files Coverage Δ
ledger/tracker.go 74.89% <0.00%> (ø)
network/wsNetwork.go 64.63% <0.00%> (ø)
data/transactions/logic/assembler.go 85.96% <0.00%> (+0.14%) ⬆️
ledger/accountdb.go 73.03% <0.00%> (+0.15%) ⬆️
data/transactions/logic/eval.go 89.31% <0.00%> (+0.42%) ⬆️
ledger/acctupdates.go 69.59% <0.00%> (+0.44%) ⬆️
data/account/participationRegistry.go 78.50% <0.00%> (+0.48%) ⬆️
cmd/tealdbg/cdtState.go 85.15% <0.00%> (+0.51%) ⬆️
ledger/acctonline.go 78.64% <0.00%> (+0.52%) ⬆️
cmd/tealdbg/debugger.go 73.49% <0.00%> (+0.80%) ⬆️
... and 22 more

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

@algorandskiy algorandskiy merged commit 1d95756 into algorand:master Sep 23, 2022
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.

2 participants