Skip to content

Commit

Permalink
tests(iroh-net): Remove a flaky test (#2379)
Browse files Browse the repository at this point in the history
## Description

This flakyness seems to have been successfully addressed by the
staggered DNS lookup.  Converting this test to that basically results
in the same code as the next test: test_dns_lookup_ipv4_ipv6.  So just
remove this test as it no longer provides value.

## Breaking Changes

<!-- Optional, if there are any breaking changes document them,
including how to migrate older code. -->

## Notes & open questions

<!-- Any notes, remarks or open questions you have to make about the PR.
-->

## Change checklist

- [x] Self-review.
- ~~[ ] Documentation updates if relevant.~~
- [x] Tests if relevant.
- ~~[ ] All breaking changes documented.~~
  • Loading branch information
flub authored Jun 18, 2024
1 parent 4ff1ec4 commit 8d1f691
Showing 1 changed file with 0 additions and 11 deletions.
11 changes: 0 additions & 11 deletions iroh-net/src/dns.rs
Original file line number Diff line number Diff line change
Expand Up @@ -387,17 +387,6 @@ pub(crate) mod tests {
const TIMEOUT: Duration = Duration::from_secs(5);
const STAGGERING_DELAYS: &[u64] = &[200, 300];

#[tokio::test]
#[cfg_attr(target_os = "windows", ignore = "flaky")]
async fn test_dns_lookup_basic() {
let _logging = iroh_test::logging::setup();
let resolver = default_resolver();
let res = resolver.lookup_ip(NA_RELAY_HOSTNAME).await.unwrap();
let res: Vec<_> = res.iter().collect();
assert!(!res.is_empty());
dbg!(res);
}

#[tokio::test]
async fn test_dns_lookup_ipv4_ipv6() {
let _logging = iroh_test::logging::setup();
Expand Down

0 comments on commit 8d1f691

Please sign in to comment.