mirror of
https://github.com/saltstack/salt.git
synced 2025-04-17 10:10:20 +00:00
Add unit tests to cover recursive cases of linux_acl states
This commit is contained in:
parent
f739b6f995
commit
4a018490b8
1 changed files with 62 additions and 1 deletions
|
@ -51,6 +51,8 @@ class LinuxAclTestCase(TestCase, LoaderModuleMockMixin):
|
|||
{name: {acl_type: [{}]}},
|
||||
{name: {acl_type: [{}]}},
|
||||
{name: {acl_type: [{}]}},
|
||||
{name: {acl_type: [{acl_name: {'octal': 7}}]}, name+"/foo": {acl_type: [{acl_name: {'octal': 'A'}}]}},
|
||||
{name: {acl_type: [{acl_name: {'octal': 7}}]}, name+"/foo": {acl_type: [{acl_name: {'octal': 7}}]}},
|
||||
{name: {acl_type: ''}}])
|
||||
mock_modfacl = MagicMock(return_value=True)
|
||||
|
||||
|
@ -146,6 +148,41 @@ class LinuxAclTestCase(TestCase, LoaderModuleMockMixin):
|
|||
self.assertDictEqual(linux_acl.present(name, acl_type,
|
||||
acl_name, perms),
|
||||
ret)
|
||||
|
||||
# New - recurse true
|
||||
with patch.dict(linux_acl.__salt__, {'acl.getfacl': mock}):
|
||||
# Update - test=True
|
||||
with patch.dict(linux_acl.__opts__, {'test': True}):
|
||||
comt = ('Updated permissions will be applied for {0}: 7 -> {1}'
|
||||
''.format(acl_name, perms))
|
||||
ret = {'name': name,
|
||||
'comment': comt,
|
||||
'changes': {},
|
||||
'pchanges': {'new': {'acl_name': acl_name,
|
||||
'acl_type': acl_type,
|
||||
'perms': perms},
|
||||
'old': {'acl_name': acl_name,
|
||||
'acl_type': acl_type,
|
||||
'perms': '7'}},
|
||||
'result': None}
|
||||
|
||||
self.assertDictEqual(linux_acl.present(name, acl_type, acl_name,
|
||||
perms, recurse=False), ret)
|
||||
|
||||
# New - recurse true - nothing to do
|
||||
with patch.dict(linux_acl.__salt__, {'acl.getfacl': mock}):
|
||||
# Update - test=True
|
||||
with patch.dict(linux_acl.__opts__, {'test': True}):
|
||||
comt = ('Permissions are in the desired state')
|
||||
ret = {'name': name,
|
||||
'comment': comt,
|
||||
'changes': {},
|
||||
'pchanges': {},
|
||||
'result': True}
|
||||
|
||||
self.assertDictEqual(linux_acl.present(name, acl_type, acl_name,
|
||||
perms, recurse=True), ret)
|
||||
|
||||
# No acl type
|
||||
comt = ('ACL Type does not exist')
|
||||
ret = {'name': name, 'comment': comt, 'result': False,
|
||||
|
@ -153,7 +190,7 @@ class LinuxAclTestCase(TestCase, LoaderModuleMockMixin):
|
|||
self.assertDictEqual(linux_acl.present(name, acl_type, acl_name,
|
||||
perms), ret)
|
||||
|
||||
# 'absent' function tests: 1
|
||||
# 'absent' function tests: 2
|
||||
|
||||
def test_absent(self):
|
||||
'''
|
||||
|
@ -180,3 +217,27 @@ class LinuxAclTestCase(TestCase, LoaderModuleMockMixin):
|
|||
comt = ('ACL Type does not exist')
|
||||
ret.update({'comment': comt, 'result': False})
|
||||
self.assertDictEqual(linux_acl.absent(name, acl_type, acl_name, perms), ret)
|
||||
|
||||
|
||||
def test_absent_recursive(self):
|
||||
'''
|
||||
Test to ensure a Linux ACL does not exist
|
||||
'''
|
||||
name = '/root'
|
||||
acl_type = 'users'
|
||||
acl_name = 'damian'
|
||||
perms = 'rwx'
|
||||
|
||||
ret = {'name': name,
|
||||
'result': None,
|
||||
'comment': '',
|
||||
'changes': {}}
|
||||
|
||||
mock = MagicMock(side_effect=[
|
||||
{name: {acl_type: [{acl_name: {'octal': 'rwx'}}]}, name+"/foo": {acl_type: [{acl_name: {'octal': 'A'}}]}}
|
||||
])
|
||||
with patch.dict(linux_acl.__salt__, {'acl.getfacl': mock}):
|
||||
with patch.dict(linux_acl.__opts__, {'test': True}):
|
||||
comt = ('Removing permissions')
|
||||
ret.update({'comment': comt})
|
||||
self.assertDictEqual(linux_acl.absent(name, acl_type, acl_name, perms, recurse=True), ret)
|
||||
|
|
Loading…
Add table
Reference in a new issue