algorand / algorand/go-algorand
Fix broken or flaky tests
- Ngôn ngữ chính
- Go
- Star
- 1.4k
- Fork
- 537
- Merge trung bình
- 1 ngày 7 giờ
- Pull request đã merge (30 ngày)
- 18
Mô tả
## 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()
```
Hướng dẫn đóng góp
Đánh giá
Issue này chưa được đánh giá.