diff options
author | Joseph Sutton <josephsutton@catalyst.net.nz> | 2023-04-11 14:04:59 +1200 |
---|---|---|
committer | Andreas Schneider <asn@cryptomilk.org> | 2023-04-12 13:52:32 +0000 |
commit | 833cefe8eeb80e1e5475fc277e09cefdc89b6dce (patch) | |
tree | 4c7a991320f544004927f1c262c3244738f75c70 /nsswitch | |
parent | b1df85e7d30c59eeb37c243d88efcab8e3d1f9a6 (diff) | |
download | samba-833cefe8eeb80e1e5475fc277e09cefdc89b6dce.tar.gz |
testprogs: Make testit_expect_failure() return 0 on success
This is the behaviour that most existing callers expect, but the
function actually returns a non-zero status code in that case.
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 'nsswitch')
-rwxr-xr-x | nsswitch/tests/test_wbinfo.sh | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/nsswitch/tests/test_wbinfo.sh b/nsswitch/tests/test_wbinfo.sh index 8b48abc151b..9b2b6823052 100755 --- a/nsswitch/tests/test_wbinfo.sh +++ b/nsswitch/tests/test_wbinfo.sh @@ -312,9 +312,9 @@ if test "$TARGET" = "ad_member"; then testit "wbinfo --dc-info=$DOMAIN" $wbinfo --dc-info=$DOMAIN || failed=$(expr $failed + 1) fi -testit_expect_failure "wbinfo -a against $TARGET with invalid password" $wbinfo -a "$DOMAIN/$USERNAME%InvalidPassword" && failed=$(expr $failed + 1) +testit_expect_failure "wbinfo -a against $TARGET with invalid password" $wbinfo -a "$DOMAIN/$USERNAME%InvalidPassword" || failed=$(expr $failed + 1) -testit_expect_failure "wbinfo -K against $TARGET with invalid password" $wbinfo -K "$DOMAIN/$USERNAME%InvalidPassword" && failed=$(expr $failed + 1) +testit_expect_failure "wbinfo -K against $TARGET with invalid password" $wbinfo -K "$DOMAIN/$USERNAME%InvalidPassword" || failed=$(expr $failed + 1) rm -f $KRB5CCNAME_PATH |