ctdb-tests: Force stub version of service in eventscript tests

Fedora 34 now has a shell function for the which command, which causes
these uses of which to return the enclosing function definition rather
than the executable file as expected.

The event script unit tests always expect the stub service command to
be used, so the conditional in these functions is unnecessary.
$CTDB_HELPER_BINDIR already conveniently points to the stub directory,
so use it here.

Signed-off-by: Martin Schwenke <martin@meltin.net>
Signed-off-by: Amitay Isaacs <amitay@gmail.com>

diff --git a/ctdb/tests/UNIT/eventscripts/etc-ctdb/rc.local b/ctdb/tests/UNIT/eventscripts/etc-ctdb/rc.local
index e9a7f99829af56f75e80a6a1a2c85457312449b5..2f506e907a20d7e769c4fb47ba194ad0f248f6f6 100755 (executable)
--- a/ctdb/tests/UNIT/eventscripts/etc-ctdb/rc.local
+++ b/ctdb/tests/UNIT/eventscripts/etc-ctdb/rc.local
@@ -1,24 +1,14 @@
 # Hey Emacs, this is a -*- shell-script -*- !!!  :-)
 
-# Use a "service" command in $PATH if one exists.
+# Always use stub version of service command
 service ()
 {
-    if _t=$(which "service" 2>/dev/null) ; then
-       "$_t" "$@"
-    else
-       _nice=""
-       _service "$@"
-    fi
+       "${CTDB_HELPER_BINDIR}/service" "$@"
 }
 
 nice_service ()
 {
-    if _t=$(which "service" 2>/dev/null) ; then
-       nice "$_t" "$@"
-    else
-       _nice="nice"
-       _service "$@"
-    fi
+       nice "${CTDB_HELPER_BINDIR}/service" "$@"
 }
 
 # Always succeeds