On Mon, Jun 12, 2017 at 04:16:16PM +0530, Shyam Saini wrote:
> On Mon, Jun 12, 2017 at 2:52 PM, Pablo Neira Ayuso <pablo@xxxxxxxxxxxxx> wrote:
> > On Fri, Jun 09, 2017 at 09:31:00PM +0530, Shyam Saini wrote:
> >> diff --git a/tests/shell/testcases/sets/0023unknown_value_to_use_0 b/tests/shell/testcases/sets/0023unknown_value_to_use_0
> >> new file mode 100755
> >> index 0000000..ee9be68
> >> --- /dev/null
> >> +++ b/tests/shell/testcases/sets/0023unknown_value_to_use_0
> >> @@ -0,0 +1,33 @@
> >> +#!/bin/bash
> >> +
> >> + # This test checks bug identified and fixed in the commit Id "986dea8".
> >> + # i.e, If in a statement there are multiple src data then it would be totally ambiguous to decide which value to set.
> >> +
> >> + # Before this commit 986dea8, nft returns 134 which indicates the bug but after this commit it returns 1.
> >> + # We don't add this test in python testsuite, because there we can't detect 134 != 1 (returns code stating failure)
> >> +
> >> +declare -a rules=(
> >> +"tcp dport set {1, 2, 3}" "udp dport set {1, 2, 3}"
> >> +"meta pkttype set {unicast, multicast, broadcast}"
> >> +"meta mark set {0xffff, 0xcc}"
> >> +"ct mark set {0x11333, 0x11}" "ct zone set {123, 127}"
> >> +"ct label set {123, 127}"
> >> +"ct event set {new, related, destroy, label}"
> >> +"ether daddr set {01:00:5e:00:01:01, 01:00:5e:00:02:02}"
> >> +"ip saddr set {192.19.1.2, 191.1.22.1}"
> >> +)
> >> +
> >> +$NFT add table t
> >> +$NFT add chain t c
> >> +
> >> +for (( i = 0 ; i < ${#rules[@]} ; i++ ))
> >> +do
> >> + $NFT add rule t c ${rules[$i]}
> >> + ret=$?
> >> + if [ $ret -ne 1 ];
> >> + then
> >> + echo "E: failed test: returned $ret instead of 1" >&2
> >> + exit 1
> >> + fi
> >> +done
> >
> > Better use tests/py/ for this?
>
> We only have "ok" and "fail" as return codes in python test suites and
> we can't can't detect 134 != 1 there.
Not sure what you mean "we can't can't detect 134 != 1 there."
You probably refer to the fact that we cannot distinguish between
assertion failure and command failure?
> So it was added in the tests/shell/.
>
> further we would need "*.payload" file for that respective test.
>
> $ nft --debug=netlink add rule t c udp dport set {1, 3}
>
> But in our case it is not generated at all.
We have no *.payload for tests that fail anyway, right?
> Is it right ?
Probably I missing anything, I'm asking because I just wondering if py
tests would be a better fit for this.
--
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
