From a55d79aa68266c790f3767ba240c3bdee247d511 Mon Sep 17 00:00:00 2001 From: Phil Sutter Date: Mon, 19 Mar 2018 18:02:04 +0100 Subject: tests/shell: Fix flowtable test cases The major problem here was that existence of network interfaces 'eth0' and 'wlan0' was assumed. Overcome this by just using 'lo' instead, which exists even in newly created netns by default. Another minor issue was false naming of 0004delete_after_add0 - the expected return code is supposed to be separated by '_' from the remaining filename. Signed-off-by: Phil Sutter Signed-off-by: Pablo Neira Ayuso --- tests/shell/testcases/flowtable/0001flowtable_0 | 2 +- tests/shell/testcases/flowtable/0002create_flowtable_0 | 6 +++--- tests/shell/testcases/flowtable/0003add_after_flush_0 | 4 ++-- tests/shell/testcases/flowtable/0004delete_after_add0 | 6 ------ tests/shell/testcases/flowtable/0004delete_after_add_0 | 6 ++++++ tests/shell/testcases/flowtable/0005delete_in_use_1 | 2 +- tests/shell/testcases/flowtable/dumps/0001flowtable_0.nft | 2 +- 7 files changed, 14 insertions(+), 14 deletions(-) delete mode 100755 tests/shell/testcases/flowtable/0004delete_after_add0 create mode 100755 tests/shell/testcases/flowtable/0004delete_after_add_0 (limited to 'tests') diff --git a/tests/shell/testcases/flowtable/0001flowtable_0 b/tests/shell/testcases/flowtable/0001flowtable_0 index 95b193dc..90b23dc3 100755 --- a/tests/shell/testcases/flowtable/0001flowtable_0 +++ b/tests/shell/testcases/flowtable/0001flowtable_0 @@ -3,7 +3,7 @@ EXPECTED='table inet t { flowtable f { hook ingress priority 10 - devices = { eth0, wlan0 } + devices = { lo } } chain c { diff --git a/tests/shell/testcases/flowtable/0002create_flowtable_0 b/tests/shell/testcases/flowtable/0002create_flowtable_0 index b6941c58..4c85c3f8 100755 --- a/tests/shell/testcases/flowtable/0002create_flowtable_0 +++ b/tests/shell/testcases/flowtable/0002create_flowtable_0 @@ -2,11 +2,11 @@ set -e $NFT add table t -$NFT add flowtable t f { hook ingress priority 10 \; devices = { eth0, wlan0 }\; } -if $NFT create flowtable t f { hook ingress priority 10 \; devices = { eth0, wlan0 }\; } 2>/dev/null ; then +$NFT add flowtable t f { hook ingress priority 10 \; devices = { lo }\; } +if $NFT create flowtable t f { hook ingress priority 10 \; devices = { lo }\; } 2>/dev/null ; then echo "E: flowtable creation not failing on existing set" >&2 exit 1 fi -$NFT add flowtable t f { hook ingress priority 10 \; devices = { eth0, wlan0 }\; } +$NFT add flowtable t f { hook ingress priority 10 \; devices = { lo }\; } exit 0 diff --git a/tests/shell/testcases/flowtable/0003add_after_flush_0 b/tests/shell/testcases/flowtable/0003add_after_flush_0 index 1f3cb18a..481c7edd 100755 --- a/tests/shell/testcases/flowtable/0003add_after_flush_0 +++ b/tests/shell/testcases/flowtable/0003add_after_flush_0 @@ -2,7 +2,7 @@ set -e $NFT add table x -$NFT add flowtable x y { hook ingress priority 0\; devices = { eth0, wlan0 }\;} +$NFT add flowtable x y { hook ingress priority 0\; devices = { lo }\;} $NFT flush ruleset $NFT add table x -$NFT add flowtable x y { hook ingress priority 0\; devices = { eth0, wlan0 }\;} +$NFT add flowtable x y { hook ingress priority 0\; devices = { lo }\;} diff --git a/tests/shell/testcases/flowtable/0004delete_after_add0 b/tests/shell/testcases/flowtable/0004delete_after_add0 deleted file mode 100755 index b72977b3..00000000 --- a/tests/shell/testcases/flowtable/0004delete_after_add0 +++ /dev/null @@ -1,6 +0,0 @@ -#!/bin/bash - -set -e -$NFT add table x -$NFT add flowtable x y { hook ingress priority 0\; devices = { eth0, wlan0 }\;} -$NFT delete flowtable x y diff --git a/tests/shell/testcases/flowtable/0004delete_after_add_0 b/tests/shell/testcases/flowtable/0004delete_after_add_0 new file mode 100755 index 00000000..8d9a842d --- /dev/null +++ b/tests/shell/testcases/flowtable/0004delete_after_add_0 @@ -0,0 +1,6 @@ +#!/bin/bash + +set -e +$NFT add table x +$NFT add flowtable x y { hook ingress priority 0\; devices = { lo }\;} +$NFT delete flowtable x y diff --git a/tests/shell/testcases/flowtable/0005delete_in_use_1 b/tests/shell/testcases/flowtable/0005delete_in_use_1 index ce39e24e..1b239f41 100755 --- a/tests/shell/testcases/flowtable/0005delete_in_use_1 +++ b/tests/shell/testcases/flowtable/0005delete_in_use_1 @@ -3,7 +3,7 @@ set -e $NFT add table x $NFT add chain x x -$NFT add flowtable x y { hook ingress priority 0\; devices = { eth0, wlan0 }\;} +$NFT add flowtable x y { hook ingress priority 0\; devices = { lo }\;} $NFT add rule x x flow offload @y $NFT delete flowtable x y echo "E: delete flowtable in use" diff --git a/tests/shell/testcases/flowtable/dumps/0001flowtable_0.nft b/tests/shell/testcases/flowtable/dumps/0001flowtable_0.nft index 5188b207..32250699 100755 --- a/tests/shell/testcases/flowtable/dumps/0001flowtable_0.nft +++ b/tests/shell/testcases/flowtable/dumps/0001flowtable_0.nft @@ -1,7 +1,7 @@ table inet t { flowtable f { hook ingress priority 10 - devices = { eth0, wlan0 } + devices = { lo } } chain c { -- cgit v1.2.3