From 2d2dbafd2b761d6ae554484a22283338da224f02 Mon Sep 17 00:00:00 2001 From: Andrew Bartlett Date: Thu, 31 Oct 2013 16:57:47 +1300 Subject: [PATCH] torture-samr: Make failures easier to trace with torture_assert Change-Id: I729ba2f0a0501575357977754401a0cb40d95b34 Signed-off-by: Andrew Bartlett Reviewed-by: Stefan Metzmacher --- source4/torture/rpc/samr.c | 59 +++++++++++++--------------------------------- 1 file changed, 17 insertions(+), 42 deletions(-) diff --git a/source4/torture/rpc/samr.c b/source4/torture/rpc/samr.c index 27086ee3cd9..dbd13149289 100644 --- a/source4/torture/rpc/samr.c +++ b/source4/torture/rpc/samr.c @@ -4890,57 +4890,32 @@ static bool test_user_ops(struct dcerpc_pipe *p, case TORTURE_SAMR_PASSWORDS_PWDLASTSET: /* test last password change timestamp behaviour */ - if (!test_SetPassword_pwdlastset(p, tctx, base_acct_flags, - base_acct_name, - user_handle, &password, - machine_credentials)) { - ret = false; - } - - if (ret == true) { - torture_comment(tctx, "pwdLastSet test succeeded\n"); - } else { - torture_result(tctx, TORTURE_FAIL, "pwdLastSet test failed\n"); - } - + torture_assert(tctx, test_SetPassword_pwdlastset(p, tctx, base_acct_flags, + base_acct_name, + user_handle, &password, + machine_credentials), + "pwdLastSet test failed\n"); break; case TORTURE_SAMR_PASSWORDS_BADPWDCOUNT: /* test bad pwd count change behaviour */ - if (!test_Password_badpwdcount_wrap(p, tctx, base_acct_flags, - base_acct_name, - domain_handle, - user_handle, &password, - machine_credentials)) { - ret = false; - } - - if (ret == true) { - torture_comment(tctx, "badPwdCount test succeeded\n"); - } else { - torture_result(tctx, TORTURE_FAIL, "badPwdCount test failed\n"); - } - + torture_assert(tctx, test_Password_badpwdcount_wrap(p, tctx, base_acct_flags, + base_acct_name, + domain_handle, + user_handle, &password, + machine_credentials), + "badPwdCount test failed\n"); break; case TORTURE_SAMR_PASSWORDS_LOCKOUT: - if (!test_Password_lockout_wrap(p, tctx, base_acct_flags, - base_acct_name, - domain_handle, - user_handle, &password, - machine_credentials)) - { - ret = false; - } - - if (ret == true) { - torture_comment(tctx, "lockout test succeeded\n"); - } else { - torture_result(tctx, TORTURE_FAIL, "lockout test failed\n"); - } - + torture_assert(tctx, test_Password_lockout_wrap(p, tctx, base_acct_flags, + base_acct_name, + domain_handle, + user_handle, &password, + machine_credentials), + "Lockout test failed"); break; -- 2.11.4.GIT