Sort output to fix random breakage in the test suite.
authorMichael Orlitzky <michael@orlitzky.com>
Sat, 2 Mar 2019 03:33:36 +0000 (22:33 -0500)
committerMichael Orlitzky <michael@orlitzky.com>
Sat, 2 Mar 2019 03:33:36 +0000 (22:33 -0500)
The results that we get back from DNS are returned in an arbitrary
order. Technically that's fine, because all we want to see is the
set-difference between the actual/expected results. However, it breaks
our test suite which needs to know what certain examples will
output. This commit sorts the results, so that the output is a little
bit more deterministic (subject to network issues and changes).

doc/man1/haeredes.1
src/Main.hs
test/shell/manpage-no-append-root.test

index 197b7e8eadd7f62a0b4c2397a9dc7a2acd2396a2..6ea57eb067d63a9831075b93c7297579b3ecd752 100644 (file)
@@ -143,7 +143,7 @@ get false positives.
 .I $ echo \(dqexample.com\(dq | \\\\
 .I "    haeredes \-\-no\-append\-root a.iana-servers.net b.iana-servers.net"
 Domain \(dqexample.com\(dq delegates somewhere else:
-\(dqb.iana-servers.net.\(dq \(dqa.iana-servers.net.\(dq
+\(dqa.iana-servers.net.\(dq \(dqb.iana-servers.net.\(dq
 .fi
 .IP \[bu]
 Check orlitzky.com against the expected name servers, using
index c93697a0c7d565e431201a56c7d851e10d1f4f2e..c9356bfd9318de6d96c279d0bd9dd0608fe9b332 100644 (file)
@@ -9,7 +9,7 @@ import qualified Data.ByteString.Char8 as BS (
   getContents,
   pack,
   words )
-import Data.List ( (\\), intersperse )
+import Data.List ( (\\), intersperse, sort )
 import Network.DNS (
   DNSError(NameError),
   Domain,
@@ -69,7 +69,8 @@ report _ _ (_, Left _) = return ()
 -- report the leftovers.
 report delgts normalize_function (d, Right raw_hosts) = do
   let nrml_hosts = map normalize_function raw_hosts
-  let leftovers = nrml_hosts \\ delgts
+  -- Sort the leftovers so that we can test the expected output.
+  let leftovers = sort (nrml_hosts \\ delgts)
   unless (null leftovers) $
     putStrLn $ "Domain " ++
                (show d) ++
index 8b7efce79a2602cb95cd439e992922f226b12359..f81ba8d8c7d42eedbdf4542a6ce58a2fe28bca23 100644 (file)
@@ -4,6 +4,6 @@
 # (without adding the trailing dot), then things go wrong.
 echo "example.com" | dist/build/haeredes/haeredes --no-append-root a.iana-servers.net b.iana-servers.net
 >>>
-Domain "example.com" delegates somewhere else: "b.iana-servers.net." "a.iana-servers.net."
+Domain "example.com" delegates somewhere else: "a.iana-servers.net." "b.iana-servers.net."
 >>>= 0