X-Git-Url: https://www.tinc-vpn.org/git/browse?a=blobdiff_plain;f=test%2Fns-ping.test;h=417ed32841c02bbe8c5b31e8948b497c76ea5cb8;hb=612ad2749ae504e6513748f881f0884ceeab2ed7;hp=5a5415cf0ff7462faad1d661d464911b960f0f6e;hpb=7b55200887d05c29abb8cea31aa914e03b806e9a;p=tinc

diff --git a/test/ns-ping.test b/test/ns-ping.test
index 5a5415cf..417ed328 100755
--- a/test/ns-ping.test
+++ b/test/ns-ping.test
@@ -1,72 +1,82 @@
 #!/bin/sh
 
-. "${0%/*}/testlib.sh"
+# shellcheck source=testlib.sh
+. ./testlib.sh
 
-# Skip this test if we aren't root or if "ip netns" does not exist
+echo "[STEP] Skip this test if we aren't root or if 'ip netns' does not exist"
 
-test "`id -u`" = "0" || exit 77
-ip netns list || exit 77
+test "$(id -u)" = "0" || exit $EXIT_SKIP_TEST
+test -e /dev/net/tun || exit $EXIT_SKIP_TEST
+ip netns list || exit $EXIT_SKIP_TEST
 
-# Initialize two nodes
+ip_foo=192.168.1.1
+ip_bar=192.168.1.2
+mask=24
 
-$tinc $c1 <<EOF
+echo [STEP] Create network namespaces
+
+ip netns add ping.test1
+ip netns add ping.test2
+
+cleanup_hook() {
+  ip netns del ping.test1
+  ip netns del ping.test2
+}
+
+echo [STEP] Initialize two nodes
+
+tinc foo <<EOF
 init foo
-set Subnet 192.168.1.1
+set Subnet $ip_foo
 set Interface ping.test1
-set Port 32577
+set Port 30090
 set Address localhost
 set AutoConnect no
 EOF
 
-cat >$d1/tinc-up <<EOF
-#!/bin/sh
-ip netns add ping.test1
-ip link set dev \$INTERFACE netns ping.test1
-ip netns exec ping.test1 ip addr add 192.168.1.1/24 dev \$INTERFACE
-ip netns exec ping.test1 ip link set \$INTERFACE up
-EOF
+# shellcheck disable=SC2016
+create_script foo tinc-up "
+  ip link set dev \$INTERFACE netns ping.test1
+  ip netns exec ping.test1 ip addr add $ip_foo/$mask dev \$INTERFACE
+  ip netns exec ping.test1 ip link set \$INTERFACE up
+"
 
-$tinc $c2 <<EOF
+tinc bar <<EOF
 init bar
-set Subnet 192.168.1.2
+set Subnet $ip_bar
 set Interface ping.test2
-set Port 32574
+set Port 30091
 set AutoConnect no
 EOF
 
-cat >$d2/tinc-up <<EOF
-#!/bin/sh
-ip netns add ping.test2
-ip link set dev \$INTERFACE netns ping.test2
-ip netns exec ping.test2 ip addr add 192.168.1.2/24 dev \$INTERFACE
-ip netns exec ping.test2 ip link set \$INTERFACE up
-EOF
-
-# Exchange configuration files
+# shellcheck disable=SC2016
+create_script bar tinc-up "
+  ip link set dev \$INTERFACE netns ping.test2
+  ip netns exec ping.test2 ip addr add $ip_bar/$mask dev \$INTERFACE
+  ip netns exec ping.test2 ip link set \$INTERFACE up
+"
 
-$tinc $c1 export | $tinc $c2 exchange | $tinc $c1 import
+echo [STEP] Exchange configuration files
 
-# Start tinc
+tinc foo export | tinc bar exchange | tinc foo import
 
-$tinc $c1 start $r1
-$tinc $c2 start $r2
+echo [STEP] Start tinc
 
-sleep 1
+start_tinc foo
+start_tinc bar
 
-# The nodes should not be able to ping each other if there is no connection
+wait_script foo tinc-up
+wait_script bar tinc-up
 
-ip netns exec ping.test1 ping -W1 -c3 192.168.1.2 && exit 1
+echo [STEP] The nodes should not be able to ping each other if there is no connection
 
-# After connecting they should be
+must_fail ip netns exec ping.test1 ping -W1 -c3 $ip_bar
 
-$tinc $c2 add ConnectTo foo
-sleep 1
+echo [STEP] After connecting they should be
 
-ip netns exec ping.test1 ping -W1 -c3 192.168.1.2
+create_script bar hosts/foo-up
 
-# Clean up
+tinc bar add ConnectTo foo
+wait_script bar hosts/foo-up
 
-$tinc $c2 stop
-$tinc $c1 stop
-ip netns del ping.test2
-ip netns del ping.test1
+ip netns exec ping.test1 ping -W1 -c3 $ip_bar