diff options
author | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2014-04-25 09:10:05 -0400 |
---|---|---|
committer | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2014-04-25 09:10:05 -0400 |
commit | c3edef5bd2b55266fee96327f17a093a0fb9cce4 (patch) | |
tree | 1e5c15e305b4513ccb6cb4cd8e87149e5e46f735 /testsuite/Testsrc/Testlib/TestServer/TestPlugins/TestProperties.py | |
parent | 148239de876a0c8d2fb53fd644ffaabc70540370 (diff) | |
download | bcfg2-c3edef5bd2b55266fee96327f17a093a0fb9cce4.tar.gz bcfg2-c3edef5bd2b55266fee96327f17a093a0fb9cce4.tar.bz2 bcfg2-c3edef5bd2b55266fee96327f17a093a0fb9cce4.zip |
testsuite: more unit test fixes
Diffstat (limited to 'testsuite/Testsrc/Testlib/TestServer/TestPlugins/TestProperties.py')
-rw-r--r-- | testsuite/Testsrc/Testlib/TestServer/TestPlugins/TestProperties.py | 30 |
1 files changed, 1 insertions, 29 deletions
diff --git a/testsuite/Testsrc/Testlib/TestServer/TestPlugins/TestProperties.py b/testsuite/Testsrc/Testlib/TestServer/TestPlugins/TestProperties.py index 896f5861e..7bdf344b5 100644 --- a/testsuite/Testsrc/Testlib/TestServer/TestPlugins/TestProperties.py +++ b/testsuite/Testsrc/Testlib/TestServer/TestPlugins/TestProperties.py @@ -292,21 +292,18 @@ class TestXMLPropertyFile(TestPropertyFile, TestStructFile): @patchIf(HAS_CRYPTO, "Bcfg2.Encryption.ssl_decrypt") @patchIf(HAS_CRYPTO, "Bcfg2.Encryption.get_algorithm") @patchIf(HAS_CRYPTO, "Bcfg2.Encryption.get_passphrases") - @patchIf(HAS_CRYPTO, "Bcfg2.Encryption.bruteforce_decrypt") - def test_decrypt(self, mock_bruteforce, mock_get_passphrases, + def test_decrypt(self, mock_get_passphrases, mock_get_algorithm, mock_ssl): pf = self.get_obj() Bcfg2.Server.Plugins.Properties.SETUP = MagicMock() def reset(): - mock_bruteforce.reset_mock() mock_get_algorithm.reset_mock() mock_get_passphrases.reset_mock() mock_ssl.reset_mock() # test element without text contents self.assertIsNone(pf._decrypt(lxml.etree.Element("Test"))) - self.assertFalse(mock_bruteforce.called) self.assertFalse(mock_get_passphrases.called) self.assertFalse(mock_ssl.called) @@ -325,7 +322,6 @@ class TestXMLPropertyFile(TestPropertyFile, TestStructFile): Bcfg2.Server.Plugins.Properties.SETUP) mock_ssl.assert_called_with(el.text, "foopass", algorithm="bf_cbc") - self.assertFalse(mock_bruteforce.called) # test failure to decrypt element with a passphrase in the config reset() @@ -337,35 +333,11 @@ class TestXMLPropertyFile(TestPropertyFile, TestStructFile): Bcfg2.Server.Plugins.Properties.SETUP) mock_ssl.assert_called_with(el.text, "foopass", algorithm="bf_cbc") - self.assertFalse(mock_bruteforce.called) # test element without valid passphrase reset() el.set("encrypted", "true") - mock_bruteforce.return_value = "decrypted with bruteforce" - self.assertEqual(pf._decrypt(el), mock_bruteforce.return_value) - mock_get_passphrases.assert_called_with( - Bcfg2.Server.Plugins.Properties.SETUP) - mock_get_algorithm.assert_called_with( - Bcfg2.Server.Plugins.Properties.SETUP) - mock_bruteforce.assert_called_with(el.text, - passphrases=["foopass", - "barpass"], - algorithm="bf_cbc") - self.assertFalse(mock_ssl.called) - - # test failure to decrypt element without valid passphrase - reset() - mock_bruteforce.side_effect = EVPError self.assertRaises(EVPError, pf._decrypt, el) - mock_get_passphrases.assert_called_with( - Bcfg2.Server.Plugins.Properties.SETUP) - mock_get_algorithm.assert_called_with( - Bcfg2.Server.Plugins.Properties.SETUP) - mock_bruteforce.assert_called_with(el.text, - passphrases=["foopass", - "barpass"], - algorithm="bf_cbc") self.assertFalse(mock_ssl.called) @patch("copy.copy") |