mirror of
https://github.com/samba-team/samba.git
synced 2024-12-22 13:34:15 +03:00
tests/posixacl: use assertRaises to simplify code
BUG: https://bugzilla.samba.org/show_bug.cgi?id=13521 Signed-off-by: Joe Guo <joeg@catalyst.net.nz> Reviewed-by: Andrew Bartlett <abartlet@samba.org> Reviewed-by: Gary Lockyer <gary@catalyst.net.nz>
This commit is contained in:
parent
11e2c32b4c
commit
002987ab3d
@ -62,11 +62,8 @@ class PosixAclMappingTests(TestCaseInTempDir):
|
||||
smbd.set_simple_acl(self.tempf, 0o640)
|
||||
|
||||
# However, this only asks the xattr
|
||||
try:
|
||||
facl = getntacl(self.lp, self.tempf, direct_db_access=True)
|
||||
self.assertTrue(False)
|
||||
except TypeError:
|
||||
pass
|
||||
self.assertRaises(
|
||||
TypeError, getntacl, self.lp, self.tempf, direct_db_access=True)
|
||||
|
||||
def test_setntacl_invalidate_getntacl(self):
|
||||
acl = ACL
|
||||
@ -184,14 +181,9 @@ class PosixAclMappingTests(TestCaseInTempDir):
|
||||
self.assertEquals(posix_acl.acl[3].a_perm, 6)
|
||||
|
||||
def test_setposixacl_getntacl(self):
|
||||
acl = ""
|
||||
smbd.set_simple_acl(self.tempf, 0o750)
|
||||
try:
|
||||
facl = getntacl(self.lp, self.tempf)
|
||||
self.assertTrue(False)
|
||||
except TypeError:
|
||||
# We don't expect the xattr to be filled in in this case
|
||||
pass
|
||||
# We don't expect the xattr to be filled in in this case
|
||||
self.assertRaises(TypeError, getntacl, self.lp, self.tempf)
|
||||
|
||||
def test_setposixacl_getntacl_smbd(self):
|
||||
s4_passdb = passdb.PDB(self.lp.get("passdb backend"))
|
||||
|
Loading…
Reference in New Issue
Block a user