algorand / algorand/go-algorand

Fix broken or flaky tests

オープン
#2,367 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
Epic Team Carbon-11
主要言語
Go
スター
1.4k
フォーク
537
平均マージ
1日 7時間
マージ済み PR(30日)
18

説明

## Overview

We should address the amount of test-debt we have accumulated over time.
Running a quick `grep` on go-algorand shows that we are skipping the tests below. Let's prioritize and address the tests that we can.

## Tests Skipped
```
./crypto/merklesignature/kats_test.go: t.Skip("Skipping; GEN_KATS not set")
./crypto/stateproof/coinGenerator_test.go: t.Skip("Skipping; GEN_KATS not set")
./ledger/ledger_test.go: t.Skip() // for manual runs only
./ledger/ledger_test.go: t.Skip("skip flaky test")
./ledger/tracker_test.go: t.Skip("For manual run when touching ledger locking")
./ledger/eval/prefetcher/prefetcher_test.go: t.Skip()
./ledger/catchpointtracker_test.go: t.Skip("This test is too slow on ARM and causes CI builds to time out")
./ledger/catchpointtracker_test.go: t.Skip("This test is too slow on ARM and causes CI builds to time out")
./ledger/catchpointtracker_test.go: t.Skip("This test is too slow on CI executors: cannot repack catchpoint")
./tools/network/dnssec/dnssec_test.go: t.Skip() // skip real network tests in autotest
./tools/network/dnssec/dnssec_test.go: t.Skip() // skip real network tests in autotest
./tools/network/resolveController_test.go: t.Skip() // skip real network tests in autotest
./tools/x-repo-types/xrt_test.go: t.Skip(tc.skipReason)
./test/framework/fixtures/fixture.go: st.t.Skip(args...)
./test/framework/fixtures/expectFixture.go: t.Skip("Expect tests disabled by environment variables.")
./test/netperf-go/puppeteer/puppeteer_test.go: t.Skip()
./test/netperf-go/puppeteer/promMetricFetcher_test.go: t.Skip()
./test/e2e-go/features/devmode/devmode_test.go: t.Skip()
./test/e2e-go/features/followernode/syncDeltas_test.go: t.Skip()
./test/e2e-go/features/followernode/syncRestart_test.go: t.Skip()
./test/e2e-go/features/stateproofs/stateproofs_test.go: t.Skip()
./test/e2e-go/features/stateproofs/stateproofs_test.go: t.Skip()
./test/e2e-go/features/stateproofs/stateproofs_test.go: t.Skip()
./test/e2e-go/features/stateproofs/stateproofs_test.go: t.Skip()
./test/e2e-go/features/stateproofs/stateproofs_test.go: t.Skip()
./test/e2e-go/features/teal/compile_test.go: t.Skip()
./test/e2e-go/features/transactions/asset_test.go: t.Skip()
./test/e2e-go/features/transactions/asset_test.go: t.Skip()
./test/e2e-go/features/partitionRecovery/partitionRecovery_test.go: t.Skip()
./test/e2e-go/features/partitionRecovery/partitionRecovery_test.go: t.Skip()
./test/e2e-go/features/partitionRecovery/partitionRecovery_test.go: t.Skip()
./test/e2e-go/features/partitionRecovery/partitionRecovery_test.go: t.Skip()
./test/e2e-go/features/partitionRecovery/partitionRecovery_test.go: t.Skip()
./test/e2e-go/features/catchup/stateproofsCatchup_test.go: t.Skip()
./test/e2e-go/features/catchup/stateproofsCatchup_test.go: t.Skip()
./test/e2e-go/features/catchup/stateproofsCatchup_test.go: t.Skip()
./test/e2e-go/features/catchup/catchpointCatchup_test.go: t.Skip()
./test/e2e-go/features/catchup/catchpointCatchup_test.go: t.Skip()
./test/e2e-go/features/catchup/catchpointCatchup_test.go: t.Skip()
./test/e2e-go/features/catchup/catchpointCatchup_test.go: t.Skip()
./test/e2e-go/features/catchup/catchpointCatchup_test.go: t.Skip()
./test/e2e-go/features/catchup/basicCatchup_test.go: t.Skip()
./test/e2e-go/features/catchup/basicCatchup_test.go: t.Skip()
./test/e2e-go/features/catchup/basicCatchup_test.go: t.Skip()
./test/e2e-go/features/participation/participationRewards_test.go: t.Skip()
./test/e2e-go/features/participation/participationRewards_test.go: t.Skip()
./test/e2e-go/features/participation/onlineOfflineParticipation_test.go: t.Skip()
./test/e2e-go/features/participation/accountParticipationTransitions_test.go: t.Skip()
./test/e2e-go/features/participation/participationExpiration_test.go: t.Skip()
./test/e2e-go/features/participation/participationExpiration_test.go: t.Skip()
./test/e2e-go/restAPI/restClient_test.go: t.Skip("new FIFO pool does not have prioritization")
./test/e2e-go/upgrades/application_support_test.go: t.Skip("Test platform is too slow for this test")
./util/db/dbutil_test.go: t.Skip("skipped as part of short test suite")
./util/db/dbutil_test.go: t.Skip("skipped as part of short test suite")
./network/wsNetwork_test.go: t.Skip("flaky test")
./network/wsNetwork_test.go: t.Skip() // todo - update this test to reflect the new implementation.
./network/wsNetwork_test.go: t.Skip("Skipping; IDLETEST not set")
./network/wsNetwork_test.go: t.Skip("Flaky on CIRCLECI")
./network/requestTracker_test.go: t.Skip()
./network/requestTracker_test.go: t.Skip()
./agreement/fuzzer/tests_test.go: t.Skip()
./agreement/fuzzer/tests_test.go: t.Skip()
./agreement/fuzzer/tests_test.go: t.Skip()
./agreement/fuzzer/tests_test.go: t.Skip()
./agreement/fuzzer/tests_test.go: t.Skip()
./agreement/fuzzer/tests_test.go: t.Skip()
./agreement/service_test.go: // t.Skip("Skipping agreement integration test")
./agreement/service_test.go: // t.Skip("Skipping agreement integration test")
./agreement/service_test.go: // t.Skip("Skipping agreement integration test")
./agreement/service_test.go: t.Skip("Skipping agreement integration test")
./agreement/service_test.go: t.Skip("Skipping agreement integration test")
./agreement/service_test.go: t.Skip("Skipping flaky agreement integration test")
./agreement/service_test.go: t.Skip("Skipping agreement integration test")
./agreement/service_test.go: t.Skip("Skipping agreement integration test")
./agreement/service_test.go: // t.Skip("Skipping agreement integration test")
./agreement/service_test.go: t.Skip("Skipping agreement integration test")
./agreement/service_test.go: t.Skip("Skipping agreement integration test")
./agreement/gossip/networkFull_test.go: t.Skip()
./agreement/gossip/networkFull_test.go: t.Skip()
./agreement/cryptoVerifier_test.go: t.Skip("Test is flaky")
./protocol/codec_tester.go: t.Skip()
./gen/generate_test.go: t.Skip() // skip in auto-test mode
./gen/generate_test.go: t.Skip() // skip in auto-test mode
./node/node_test.go: t.Skip("Flaky in nightly test environment")
./node/node_test.go: t.Skip("Test takes ~25 seconds.")
./node/node_test.go: t.Skip("Test is too heavy for amd64 builder running in parallel with other packages")
./node/node_test.go: t.Skip("Flaky in nightly test environment.")
./data/transactions/logic/debugger_eval_test.go: t.Skip("this must be run manually")
./data/account/participationRegistry_test.go: t.Skip()
```

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。