Testsuite: strip trailing space at -bt output
authorHeiko Schlittermann (HS12-RIPE) <hs@schlittermann.de>
Fri, 29 Nov 2019 09:38:47 +0000 (17:38 +0800)
committerHeiko Schlittermann (HS12-RIPE) <hs@schlittermann.de>
Fri, 29 Nov 2019 15:46:29 +0000 (16:46 +0100)
Depending on the length of the ip address, Exim adds
spaces to -bt output, to make all lines the same length (why? Maybe to
add port= or mx= later in a nicely aligned manner)

compare:
    host 127.0.0.1     [127.0.0.1]
    host 10.0.0.1      [10.0.0.1]-

vs:
    host 127.0.0.1     [127.0.0.1]--
    host 169.16.16.16  [169.16.16.10]

test/runtest
test/stdout/0190
test/stdout/0350
test/stdout/0430
test/stdout/1006
test/stdout/1009
test/stdout/4950

index 358e2bab8d54420b79cbca0d1ca60ed8a4a8feab..be35e157fd87dc6a1676ea47a83ec8def1aaab11 100755 (executable)
@@ -800,6 +800,12 @@ RESET_AFTER_EXTRA_LINE_READ:
   # Also, the length of space at the end of the host line is dependent
   # on the length of the longest line, so strip it also on otherwise
   # un-rewritten lines like localhost
+  #
+  # host 127.0.0.1     [127.0.0.1]
+  # host 10.0.0.1      [10.0.0.1]-
+  #
+  # host 127.0.0.1     [127.0.0.1]--
+  # host 169.16.16.16  [169.16.16.10]
 
   s/^\s+host\s(\S+)\s+(\S+)/  host $1 $2/;
   s/^\s+(host\s\S+\s\S+)\s+(port=.*)/  host $1 $2/;
@@ -811,7 +817,7 @@ RESET_AFTER_EXTRA_LINE_READ:
   s/(^|\W)\K\Q$parm_ipv6_stripped\E/ip6:ip6:ip6:ip6:ip6:ip6:ip6:ip6/g;
   s/\b\Q$parm_ipv4r\E\b/ip4-reverse/g;
   s/(^|\W)\K\Q$parm_ipv6r\E/ip6-reverse/g;
-  s/^(\s+host\s\S+\s+\[\S+\]) +$/$1 /;
+  s/^\s+host\s\S+\s+\[\S+\]\K +$//;     # strip, not collapse the trailing whitespace
 
 
   # ======== Test network IP addresses ========
index a88a851b0227dacd31310590861680b54df5f882..756f7f851d49db0c1af9f6e18b94dc2c5bd79ac2 100644 (file)
@@ -97,17 +97,17 @@ y@batch4
 userx@bdomain1
   router = r3, transport = smtp2
   host ipv4.ipv4.ipv4.ipv4 [ipv4.ipv4.ipv4.ipv4]
-  host 127.0.0.1 [127.0.0.1] 
+  host 127.0.0.1 [127.0.0.1]
   host the.local.host.name [ip4.ip4.ip4.ip4]
 userx@bdomain2
   router = r3, transport = smtp2
   host the.local.host.name [ip4.ip4.ip4.ip4]
-  host 127.0.0.1 [127.0.0.1] 
+  host 127.0.0.1 [127.0.0.1]
   host ipv4.ipv4.ipv4.ipv4 [ipv4.ipv4.ipv4.ipv4]
 userx@bdomain3
   router = r3, transport = smtp2
   host ipv4.ipv4.ipv4.ipv4 [ipv4.ipv4.ipv4.ipv4]
-  host 127.0.0.1 [127.0.0.1] 
+  host 127.0.0.1 [127.0.0.1]
   host the.local.host.name [ip4.ip4.ip4.ip4]
 
 ******** SERVER ********
index d951172b7882a1d3cbce61289e61cffbf2ce7981..120a1d4a85a75ab1946023585e07f20d3c44965a 100644 (file)
@@ -1,4 +1,4 @@
 xx@yy
   router = r1, transport = t1
-  host 1.2.3.4 [1.2.3.4] 
+  host 1.2.3.4 [1.2.3.4]
   host other2.test.ex [V4NET.12.3.2]
index 7914594b4eda8a5b79dc7cc47162312759f46506..1229b0ef83d816d4e485116ed20bc67759c4e245 100644 (file)
@@ -48,7 +48,7 @@ x@manual.route
   router = r1, transport = t1
   host ten-1.test.ex [V4NET.0.0.1] MX=5
   host eximtesthost.test.ex [ip4.ip4.ip4.ip4] MX=5
-  host ten-6.test.ex [V4NET.0.0.6] 
+  host ten-6.test.ex [V4NET.0.0.6]
 x@manual.route is undeliverable: lowest numbered MX record points to local host
 x@random.manual.route cannot be resolved at this time: lookup of host "localhost.test.ex" failed in r2 router
 x@random.manual.route
index 2657c47a9131b455a6622a6caaf777a53a867edc..1457c08d24d40f9dbaf469ed19fb6ecd9c52adce 100644 (file)
@@ -5,7 +5,7 @@ dnslookup@mx46.test.ex
 manualroute@test.ex
   router = r2, transport = smtp
   host 46.test.ex [V6NET:ffff:836f:a00:a:800:200a:c031]
-  host 46.test.ex [V4NET.0.0.4] 
+  host 46.test.ex [V4NET.0.0.4]
 dnslookup@v6.test.ex
   router = r1, transport = smtp
   host v6.test.ex [V6NET:ffff:836f:a00:a:800:200a:c032]
index 673bf0a2f11638f03387caee29795e8e84c4f622..7e86fa1825ce9441e14a6771e08fc96fc848f7ec 100644 (file)
@@ -1,7 +1,7 @@
 x@46.test.ex
   router = r1, transport = t1
   host 46.test.ex [V6NET:ffff:836f:a00:a:800:200a:c031]
-  host 46.test.ex [V4NET.0.0.4] 
+  host 46.test.ex [V4NET.0.0.4]
 x@mx46.test.ex
   router = r1, transport = t1
   host 46.test.ex [V6NET:ffff:836f:a00:a:800:200a:c031] MX=46
@@ -46,7 +46,7 @@ x@mx46.test.ex
   host 46.test.ex [V4NET.0.0.4] MX=46
 x@mx46.test.ex
   router = r0, transport = t1
-  host 46.test.ex [V4NET.0.0.4] 
+  host 46.test.ex [V4NET.0.0.4]
   host 46.test.ex [V6NET:ffff:836f:a00:a:800:200a:c031]
 x@mx46.test.ex
   router = r0, transport = t1
index 89a34ad4980b1c20413107c74aa04a7f6b688865..12c77344b1b6f5b4d48e25d964b4790610d646b3 100644 (file)
@@ -14,4 +14,4 @@ userx@other1.test.ex
 userx@other99.test.ex cannot be resolved at this time: host non-exist.test.ex not found when translating other99.test.ex [V4NET.99.0.1]
 userx@other99.test.ex
   router = lookuphost, transport = smtp
-  host non-exist.test.ex [unknown] 
+  host non-exist.test.ex [unknown]