On Mon, Oct 02, 2017 at 04:31:54PM +0530, Varsha Rao wrote: > This patch adds test for chain rename use case. > > Signed-off-by: Varsha Rao <rvarsha016@xxxxxxxxx> > --- > tests/shell/testcases/chains/0013rename_1 | 9 +++++++++ > 1 file changed, 9 insertions(+) > create mode 100755 tests/shell/testcases/chains/0013rename_1 > > diff --git a/tests/shell/testcases/chains/0013rename_1 b/tests/shell/testcases/chains/0013rename_1 > new file mode 100755 > index 0000000..a69d855 > --- /dev/null > +++ b/tests/shell/testcases/chains/0013rename_1 > @@ -0,0 +1,9 @@ > +#!/bin/bash > + > +set -e > + > +$NFT add table t > +$NFT add chain t c1 > +# kernel should return EEXIST Wait, I guess you mean here: "kernel should not return EEXIST as per 0d18779be13766b33c69cbc26df38383598da373", right? I can mangle this line here before applying. > +$NFT rename chain t c1 c2 2>/dev/null > +echo "E: Renamed chain with non existing chain" >&2 BTW, could you also add a test to make sure we don't allow a rename from 'c1' to 'c2' if chain 'c2' already exists. -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html