diff options
| author | Joseph Sutton <josephsutton@catalyst.net.nz> | 2023-04-11 14:26:56 +1200 |
|---|---|---|
| committer | Andreas Schneider <asn@cryptomilk.org> | 2023-04-12 13:52:32 +0000 |
| commit | 23d2c9cb72d3aa4593edae40fc6b998c96242dd6 (patch) | |
| tree | 1d7bfe158048306ad8c74bdf044371d082f0b451 /testprogs/blackbox/common_test_fns.inc | |
| parent | dfb088aaed5799ada4e78b8862b5f4f7f7b1de06 (diff) | |
| download | samba-23d2c9cb72d3aa4593edae40fc6b998c96242dd6.tar.gz | |
testprogs: Make test_rpcclient_expect_failure_grep() return 0 on success
This is more consistent with the behaviour of the other expect_failure
functions.
Adjust all callers expecting the opposite behaviour to match.
Signed-off-by: Joseph Sutton <josephsutton@catalyst.net.nz>
Reviewed-by: Andreas Schneider <asn@samba.org>
Diffstat (limited to 'testprogs/blackbox/common_test_fns.inc')
| -rwxr-xr-x | testprogs/blackbox/common_test_fns.inc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/testprogs/blackbox/common_test_fns.inc b/testprogs/blackbox/common_test_fns.inc index ca7ed25b4a2..df8fee0e032 100755 --- a/testprogs/blackbox/common_test_fns.inc +++ b/testprogs/blackbox/common_test_fns.inc @@ -82,7 +82,7 @@ test_rpcclient_expect_failure_grep() status=$? if [ x$status = x0 ]; then printf '%s' "$output" | subunit_fail_test "$name" - return $status + return 1 fi printf '%s' "$output" | grep -q "$grep" gstatus=$? @@ -91,7 +91,7 @@ test_rpcclient_expect_failure_grep() else printf '%s' "$output" | subunit_fail_test "$name" fi - return $status + return $gstatus } kerberos_kinit() |