algorand / algorand/go-algorand

Rework TestResolverWithCloudflareDNSResolution to avoid 3rd party callouts

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

説明

## Problem

`TestResolverWithCloudflareDNSResolution` makes callouts to Cloudflare. As discussed in prior PRs (e.g. https://github.com/algorand/go-algorand/pull/4941, https://github.com/algorand/go-algorand/pull/4965), we've encountered instability when CircleCI blocks outgoing calls.

It's problematic for unit tests to fail due to a 3rd party callout. The story requests reworking the test to remove the 3rd party callout (e.g. setup locally running DNS stand-in on port 53).

## Solution

- Potential Solution: Do not run DNS tests on PR tests, but rather on nightly tests

## Dependencies

## Urgency

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

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

評価

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

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

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