diff options
author | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2012-10-22 08:23:49 -0400 |
---|---|---|
committer | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2012-10-22 12:23:19 -0400 |
commit | 20cf8f21bc7043de0070644cfb434ce77e24d28c (patch) | |
tree | e51718c51b6cecc4fba4850e775c945ff4eb37f2 /testsuite/Testsrc/Testlib/TestClient | |
parent | b492fd38950599f4414d0cbd8b1b03b162503aa6 (diff) | |
download | bcfg2-20cf8f21bc7043de0070644cfb434ce77e24d28c.tar.gz bcfg2-20cf8f21bc7043de0070644cfb434ce77e24d28c.tar.bz2 bcfg2-20cf8f21bc7043de0070644cfb434ce77e24d28c.zip |
fixed some POSIX unit tests
Diffstat (limited to 'testsuite/Testsrc/Testlib/TestClient')
-rw-r--r-- | testsuite/Testsrc/Testlib/TestClient/TestTools/TestPOSIX/Testbase.py | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/testsuite/Testsrc/Testlib/TestClient/TestTools/TestPOSIX/Testbase.py b/testsuite/Testsrc/Testlib/TestClient/TestTools/TestPOSIX/Testbase.py index 332722074..0e0234719 100644 --- a/testsuite/Testsrc/Testlib/TestClient/TestTools/TestPOSIX/Testbase.py +++ b/testsuite/Testsrc/Testlib/TestClient/TestTools/TestPOSIX/Testbase.py @@ -587,11 +587,15 @@ class TestPOSIXTool(Bcfg2TestCase): def inner(mock_getfilecon, mock_stat): mock_getfilecon.return_value = [len(context) + 1, context] mock_stat.return_value = MagicMock() + mock_stat.return_value.__getitem__.return_value = MagicMock() # disable acls for this call and test them separately - state = Bcfg2.Client.Tools.POSIX.base.HAS_ACLS + state = (Bcfg2.Client.Tools.POSIX.base.HAS_ACLS, + Bcfg2.Client.Tools.POSIX.base.HAS_SELINUX) Bcfg2.Client.Tools.POSIX.base.HAS_ACLS = False + Bcfg2.Client.Tools.POSIX.base.HAS_SELINUX = True self.assertEqual(self.ptool._gather_data(path)[4], 'root_t') - Bcfg2.Client.Tools.POSIX.base.HAS_ACLS = state + Bcfg2.Client.Tools.POSIX.base.HAS_ACLS, \ + Bcfg2.Client.Tools.POSIX.base.HAS_SELINUX = state mock_getfilecon.assert_called_with(path) inner() @@ -606,11 +610,15 @@ class TestPOSIXTool(Bcfg2TestCase): mock_list_file_acls.return_value = acls path = '/test' mock_stat.return_value = MagicMock() + mock_stat.return_value.__getitem__.return_value = MagicMock() # disable selinux for this call and test it separately - state = Bcfg2.Client.Tools.POSIX.base.HAS_SELINUX + state = (Bcfg2.Client.Tools.POSIX.base.HAS_ACLS, + Bcfg2.Client.Tools.POSIX.base.HAS_SELINUX) + Bcfg2.Client.Tools.POSIX.base.HAS_ACLS = True Bcfg2.Client.Tools.POSIX.base.HAS_SELINUX = False self.assertItemsEqual(self.ptool._gather_data(path)[5], acls) - Bcfg2.Client.Tools.POSIX.base.HAS_SELINUX = state + Bcfg2.Client.Tools.POSIX.base.HAS_ACLS, \ + Bcfg2.Client.Tools.POSIX.base.HAS_SELINUX = state mock_list_file_acls.assert_called_with(path) @patchIf(HAS_SELINUX, "selinux.matchpathcon") |