algorand / algorand/go-algorand

Disabled tests in node_test.go need to be updated

オープン
#2,318 コメント 1 件 リアクション 0 件 担当者 1 名 @algonautshant が担当を希望しています GitHub で見る
SkippedTests Team Carbon-11 test
主要言語
Go
スター
1.4k
フォーク
537
平均マージ
1日 7時間
マージ済み PR(30日)
18

説明

## Summary

The following tests have been disabled some time ago, because they are "Randomly failing".
Things got worse when in https://github.com/algorand/go-algorand/pull/893 ExtendPeerList was removed, which was used by these tests.

node/node_test.go-func TestSyncingFullNode(t *testing.T) {
node/node_test.go: t.Skip("This is failing randomly again - PLEASE FIX!")

node/node_test.go-func TestInitialSync(t *testing.T) {
node/node_test.go: t.Skip("flaky TestInitialSync ")

node/node_test.go-func TestSimpleUpgrade(t *testing.T) {
node/node_test.go: t.Skip("Randomly failing: node_test.go:~330 : no block notification for account. Re-enable after agreement bug-fix pass")

## Scope/Requirements

Update the test to establish the network connections without using ExtendPeerList.
Determine why the tests are randomly failing and fix it.
Enable the tests.

## Urgency/Relative Priority

This is not urgent, since these test were disabled some time ago.
It is important to fix and enable them, since the delay causes the test to further drift away from the source code.

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

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

評価

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

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

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