[Devel] [PATCH vz10] selftests/net: source net_helper.sh in fcnal-test.sh

[Eva Kurchatova]

Commit 15c068cb214d ("selftests: net: replace sleeps in fcnal-test
with waits") replaced the 'sleep 1' calls that followed server
spawns with wait_local_port_listen(). Upstream did this together with
commit d9d836bfa5e6 ("selftests: net: move wait_local_port_listen to
lib.sh") so the helper became reachable from lib.sh, which
fcnal-test.sh already sources.

Our tree backports the first commit (as 810beccc278e) but not the
second, leaving wait_local_port_listen undefined when fcnal-test.sh
runs and producing dozens of subtest failures of the form:

    ./fcnal-test.sh: line 1198: wait_local_port_listen: command not found
    TEST: Global server - ns-A loopback IP     [FAIL]
        expected rc 0; actual rc 1

Pull in the function explicitly by sourcing net_helper.sh in addition
to lib.sh.  This keeps the change minimal compared to backporting the
full d9d836bfa5e6 (which deletes net_helper.sh and rewrites 14 other
callers).

Fixes: 810beccc278e ("ms/selftests: net: replace sleeps in fcnal-test with waits")
Signed-off-by: Eva Kurchatova <eva.kurchatova at virtuozzo.com>

https://virtuozzo.atlassian.net/browse/VSTOR-134204
Feature: fix selftests
---
 tools/testing/selftests/net/fcnal-test.sh | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tools/testing/selftests/net/fcnal-test.sh b/tools/testing/selftests/net/fcnal-test.sh
index bb2b789541ff..0b8b77bd8e9f 100755
--- a/tools/testing/selftests/net/fcnal-test.sh
+++ b/tools/testing/selftests/net/fcnal-test.sh
@@ -38,6 +38,7 @@
 # server / client nomenclature relative to ns-A
 
 source lib.sh
+source net_helper.sh
 
 PATH=$PWD:$PWD/tools/testing/selftests/net:$PATH
 
-- 
2.54.0