Commit 70dd4775 authored by Kuniyuki Iwashima's avatar Kuniyuki Iwashima Committed by Jakub Kicinski
Browse files

selftest: packetdrill: Set ktap_set_plan properly for single protocol test. 



The cited commit forgot to update the ktap_set_plan call.

ktap_set_plan sets the number of tests (KSFT_NUM_TESTS), which must
match the number of executed tests (KTAP_CNT_PASS + KTAP_CNT_SKIP +
KTAP_CNT_XFAIL) in ktap_finished.

Otherwise, the selftest exit()s with 1.

Let's adjust KSFT_NUM_TESTS based on supported protocols.

While at it, misalignment is fixed up.

Fixes: a5c10aa3 ("selftests/net: packetdrill: Support single protocol test.")
Signed-off-by: default avatarKuniyuki Iwashima <kuniyu@google.com>
Reviewed-by: default avatarWillem de Bruijn <willemb@google.com>
Link: https://patch.msgid.link/20250927213022.1850048-2-kuniyu@google.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 4ed9db2d

tools/testing/selftests/net/packetdrill/ksft_runner.sh

+2 −2
Original line number Diff line number Diff line
@@ -48,7 +48,7 @@ elif [[ ! "$ip_versions" =~ ^ipv[46]$ ]]; then 
fi



ktap_print_header

ktap_set_plan 2

ktap_set_plan $(echo $ip_versions | wc -w)



for ip_version in $ip_versions; do

	unshare -n packetdrill ${ip_args[$ip_version]} ${optargs[@]} $script > /dev/null \