Skip to content

e2e tests: remove unused e2e artifact#4487

Merged
winder merged 7 commits intoalgorand:masterfrom
shiqizng:ci/e2eupload
Sep 2, 2022
Merged

e2e tests: remove unused e2e artifact#4487
winder merged 7 commits intoalgorand:masterfrom
shiqizng:ci/e2eupload

Conversation

@shiqizng
Copy link
Copy Markdown
Contributor

@shiqizng shiqizng commented Aug 30, 2022

Summary

we've decided to pin indexer to rel/nightly. We can remove the artifact add in 4481.

@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 30, 2022

Codecov Report

Merging #4487 (cf570d9) into master (e83acec) will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4487      +/-   ##
==========================================
+ Coverage   55.25%   55.28%   +0.03%     
==========================================
  Files         398      398              
  Lines       50297    50297              
==========================================
+ Hits        27793    27809      +16     
+ Misses      20171    20160      -11     
+ Partials     2333     2328       -5     
Impacted Files Coverage Δ
ledger/roundlru.go 90.56% <0.00%> (-5.67%) ⬇️
ledger/blockqueue.go 87.35% <0.00%> (-1.15%) ⬇️
catchup/peerSelector.go 98.95% <0.00%> (-1.05%) ⬇️
ledger/acctupdates.go 69.59% <0.00%> (-0.60%) ⬇️
network/wsNetwork.go 64.63% <0.00%> (-0.20%) ⬇️
catchup/service.go 69.38% <0.00%> (+0.49%) ⬆️
crypto/merkletrie/node.go 93.48% <0.00%> (+1.86%) ⬆️
crypto/merkletrie/trie.go 68.61% <0.00%> (+2.18%) ⬆️
network/wsPeer.go 70.95% <0.00%> (+2.46%) ⬆️
ledger/tracker.go 78.72% <0.00%> (+4.68%) ⬆️

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

@shiqizng shiqizng changed the title e2e tests: fix e2e test data file name e2e tests: remove unused e2e artifact Aug 31, 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