Re: [PATCH 3/4 V6 nft] test: shell: Add tests for deleting rule by description

2016-08-22 Thread Pablo Neira Ayuso
Hi Carlos, One of this test fails... so please send me a follow up to fix it. W: [FAILED] ./testcases/rule_management/0010delete-by-desc_0 This chunk also looks a bit strange to me. set +e; # Next commands can return 0 REMAINS_RULE2DEL=$($NFT list -a ruleset | grep -c "$RULE2DEL")

[PATCH 3/4 V6 nft] test: shell: Add tests for deleting rule by description

2016-08-21 Thread Carlos Falgueras García
They checks if commands like "nft delete rule " works as is expected. First one checks if command deletes only one of the matched rules. Second one checks if command fails when rule did not found. Signed-off-by: Carlos Falgueras García ---