Skip to content

Commit

Permalink
tests: ignores peer seed test that requires external DNS (#5187)
Browse files Browse the repository at this point in the history
Description
---
 ignores peer seed test that requires external DNS

Motivation and Context
---
This test is currently failing and was originally ignored because it depends on an external DNS to pass.

How Has This Been Tested?
---
Test ignored

<!-- Does this include a breaking change? If so, include this line as a footer -->
<!-- BREAKING CHANGE: Description what the user should do, e.g. delete a database, resync the chain -->
  • Loading branch information
sdbondi authored Feb 17, 2023
1 parent 9900d5d commit 799bae5
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion base_layer/p2p/src/peer_seeds.rs
Original file line number Diff line number Diff line change
Expand Up @@ -239,11 +239,12 @@ mod test {
use crate::{dns::mock, DEFAULT_DNS_NAME_SERVER};

#[tokio::test]
#[ignore = "Useful for developer testing but will fail unless the DNS has TXT records setup correctly."]
async fn it_returns_seeds_from_real_address() {
let mut resolver = DnsSeedResolver::connect(DEFAULT_DNS_NAME_SERVER.parse().unwrap())
.await
.unwrap();
let seeds = resolver.resolve("seeds.weatherwax.tari.com").await.unwrap();
let seeds = resolver.resolve("seeds.esmeralda.tari.com").await.unwrap();
println!("{:?}", seeds);
assert!(!seeds.is_empty());
}
Expand Down

0 comments on commit 799bae5

Please sign in to comment.