algorand / algorand/go-algorand

Fix broken or flaky tests

Đang mở
#2,367 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
Epic Team Carbon-11
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

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.