From: Stefan Metzmacher <metze@samba.org>
Date: Sat, 27 Feb 2010 09:16:56 +0000 (+0100)
Subject: s4:RPC-ECHO: don't ignore errors in the Sleep test now that we support async rpc... 
X-Git-Tag: samba-3.6.0pre1~4862
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=522e4f3d2bcd260f7037fbafd4340a9a4513c89d;p=thirdparty%2Fsamba.git

s4:RPC-ECHO: don't ignore errors in the Sleep test now that we support async rpc over ncacn_np

metze
---

diff --git a/source4/torture/rpc/echo.c b/source4/torture/rpc/echo.c
index 2321500ca52..e0ee8a580e2 100644
--- a/source4/torture/rpc/echo.c
+++ b/source4/torture/rpc/echo.c
@@ -289,9 +289,9 @@ static bool test_sleep(struct torture_context *tctx,
 					torture_comment(tctx, "Slept for %u seconds (reply takes %u.%06u seconds - ok)\n", 
 							r[i].out.result, (unsigned int)diff[i].tv_sec, (unsigned int)diff[i].tv_usec);
 				} else {
-						torture_comment(tctx, "(Failed) - Not async - Slept for %u seconds (but reply takes %u.%06u seconds)", 
-							r[i].out.result, (unsigned int)diff[i].tv_sec, (unsigned int)diff[i].tv_usec);
-					/* TODO: let the test fail here, when we support async rpc on ncacn_np */
+					torture_fail(tctx, talloc_asprintf(tctx,
+						     "(Failed) - Not async - Slept for %u seconds (but reply takes %u.%06u seconds)\n",
+						     r[i].out.result, (unsigned int)diff[i].tv_sec, (unsigned int)diff[i].tv_usec));
 				}
 			}
 		}