Skip to content

Commit

Permalink
Improve output if no_lookup_host_duplicates fails
Browse files Browse the repository at this point in the history
If the test fails, output the offending addresses and a helpful error message.
Also slightly improve legibility of the preceding line that puts the addresses
into a HashMap.
  • Loading branch information
Munksgaard committed Sep 15, 2018
1 parent 85da245 commit 0e9d260
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/libstd/sys_common/net.rs
Expand Up @@ -622,7 +622,8 @@ mod tests {
Ok(lh) => lh,
Err(e) => panic!("couldn't resolve `localhost': {}", e)
};
let _na = lh.map(|sa| *addrs.entry(sa).or_insert(0) += 1).count();
assert!(addrs.values().filter(|&&v| v > 1).count() == 0);
for sa in lh { *addrs.entry(sa).or_insert(0) += 1; };
assert_eq!(addrs.iter().filter(|&(_, &v)| v > 1).collect::<Vec<_>>(), vec![],
"There should be no duplicate localhost entries");
}
}

0 comments on commit 0e9d260

Please sign in to comment.