diff options
author | Günther Deschner <gd@samba.org> | 2010-02-19 14:27:31 +0100 |
---|---|---|
committer | Günther Deschner <gd@samba.org> | 2010-02-19 14:27:31 +0100 |
commit | 0e95a6a09cda6579be7d01c9e4c78f9f7f7cc54e (patch) | |
tree | e929ea84de1e953615f10b5a32fb31fdf01ad059 | |
parent | f2ecec3dcdafd63cd72d13019a998bf99539a9ba (diff) | |
download | samba-0e95a6a09cda6579be7d01c9e4c78f9f7f7cc54e.tar.gz samba-0e95a6a09cda6579be7d01c9e4c78f9f7f7cc54e.tar.bz2 samba-0e95a6a09cda6579be7d01c9e4c78f9f7f7cc54e.zip |
s4-smbtorture: explain failure conditions in printer device mode tests a little more.
Guenther
-rw-r--r-- | source4/torture/rpc/spoolss.c | 31 |
1 files changed, 20 insertions, 11 deletions
diff --git a/source4/torture/rpc/spoolss.c b/source4/torture/rpc/spoolss.c index e22032d8bc..981eb93f2a 100644 --- a/source4/torture/rpc/spoolss.c +++ b/source4/torture/rpc/spoolss.c @@ -1664,7 +1664,8 @@ static bool test_PrinterInfo_SDs(struct torture_context *tctx, sd2 = info.info3.secdesc; - torture_assert(tctx, test_security_descriptor_equal(tctx, sd1, sd2), ""); + torture_assert(tctx, test_security_descriptor_equal(tctx, sd1, sd2), + "SD level 2 != SD level 3"); /* query level 2, set level 2, query level 2 */ @@ -1683,7 +1684,8 @@ static bool test_PrinterInfo_SDs(struct torture_context *tctx, sd1->type &= ~SEC_DESC_DACL_DEFAULTED; } - torture_assert(tctx, test_security_descriptor_equal(tctx, sd1, sd2), ""); + torture_assert(tctx, test_security_descriptor_equal(tctx, sd1, sd2), + "SD level 2 != SD level 2 after SD has been set via level 2"); /* query level 2, set level 3, query level 2 */ @@ -1698,8 +1700,8 @@ static bool test_PrinterInfo_SDs(struct torture_context *tctx, sd2 = info.info2.secdesc; - torture_assert(tctx, test_security_descriptor_equal(tctx, sd1, sd2), ""); - + torture_assert(tctx, test_security_descriptor_equal(tctx, sd1, sd2), + "SD level 2 != SD level 2 after SD has been set via level 3"); /* set modified sd level 3, query level 2 */ @@ -1724,7 +1726,9 @@ static bool test_PrinterInfo_SDs(struct torture_context *tctx, sd1->type &= ~SEC_DESC_DACL_DEFAULTED; } - torture_assert(tctx, test_security_descriptor_equal(tctx, sd1, sd2), ""); + torture_assert(tctx, test_security_descriptor_equal(tctx, sd1, sd2), + "modified SD level 2 != SD level 2 after SD has been set via level 3"); + return true; } @@ -1916,7 +1920,8 @@ static bool test_PrinterInfo_DevModes(struct torture_context *tctx, devmode2 = info.info2.devmode; - torture_assert(tctx, test_devicemode_equal(tctx, devmode, devmode2), ""); + torture_assert(tctx, test_devicemode_equal(tctx, devmode, devmode2), + "DM level 8 != DM level 2"); /* change formname upon open and see if it persists in getprinter calls */ @@ -1964,13 +1969,15 @@ static bool test_PrinterInfo_DevModes(struct torture_context *tctx, devmode2 = info.info8.devmode; - torture_assert(tctx, test_devicemode_equal(tctx, devmode, devmode2), ""); + torture_assert(tctx, test_devicemode_equal(tctx, devmode, devmode2), + "modified DM level 8 != DM level 8 after DM has been set via level 8"); torture_assert(tctx, test_GetPrinter_level(tctx, p, handle, 2, &info), ""); devmode2 = info.info2.devmode; - torture_assert(tctx, test_devicemode_equal(tctx, devmode, devmode2), ""); + torture_assert(tctx, test_devicemode_equal(tctx, devmode, devmode2), + "modified DM level 8 != DM level 2"); /* set devicemode level 2 and see if it persists */ @@ -1978,19 +1985,21 @@ static bool test_PrinterInfo_DevModes(struct torture_context *tctx, devmode->copies = 39; devmode->formname = talloc_strdup(tctx, "Letter"); - torture_assert(tctx, test_devmode_set_level(tctx, p, handle, 8, devmode), ""); + torture_assert(tctx, test_devmode_set_level(tctx, p, handle, 2, devmode), ""); torture_assert(tctx, test_GetPrinter_level(tctx, p, handle, 8, &info), ""); devmode2 = info.info8.devmode; - torture_assert(tctx, test_devicemode_equal(tctx, devmode, devmode2), ""); + torture_assert(tctx, test_devicemode_equal(tctx, devmode, devmode2), + "modified DM level 8 != DM level 8 after DM has been set via level 2"); torture_assert(tctx, test_GetPrinter_level(tctx, p, handle, 2, &info), ""); devmode2 = info.info2.devmode; - torture_assert(tctx, test_devicemode_equal(tctx, devmode, devmode2), ""); + torture_assert(tctx, test_devicemode_equal(tctx, devmode, devmode2), + "modified DM level 8 != DM level 2"); return true; |