Skip to content

Disable expect tests again#2790

Merged
tsachiherman merged 1 commit into
algorand:masterfrom
algorandskiy:pavel/expect-3
Aug 23, 2021
Merged

Disable expect tests again#2790
tsachiherman merged 1 commit into
algorand:masterfrom
algorandskiy:pavel/expect-3

Conversation

@algorandskiy
Copy link
Copy Markdown
Contributor

Summary

  • After re-enabling we started seeing failures on other e2e-go tests
    due to unsufficient resources to allow algod making new rounds
  • Possible solution is to run expect tests as a separate stage on CI
    go test ./test/e2e-go/cli/goal/expect/... -run TestGoalWithExpect

* After re-enabling we started seeing failures on other e2e-go tests
  due to unsufficient resources to allow algod making new rounds
* Possible solution is to run expect tests as a separate stage on CI
  go test ./test/e2e-go/cli/goal/expect/... -run TestGoalWithExpect
@algorandskiy algorandskiy self-assigned this Aug 23, 2021
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #2790 (6bee50a) into master (849e924) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2790   +/-   ##
=======================================
  Coverage   47.10%   47.10%           
=======================================
  Files         349      349           
  Lines       56354    56354           
=======================================
+ Hits        26545    26547    +2     
+ Misses      26836    26833    -3     
- Partials     2973     2974    +1     
Impacted Files Coverage Δ
cmd/tealdbg/debugger.go 72.86% <0.00%> (-1.01%) ⬇️
ledger/acctupdates.go 62.63% <0.00%> (+0.08%) ⬆️
network/wsPeer.go 74.65% <0.00%> (+0.27%) ⬆️
network/requestTracker.go 71.12% <0.00%> (+0.86%) ⬆️

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 849e924...6bee50a. Read the comment docs.

@tsachiherman tsachiherman merged commit 2833997 into algorand:master Aug 23, 2021
@algorandskiy algorandskiy deleted the pavel/expect-3 branch March 16, 2026 20:04
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.

4 participants