diff options
author | Pierre-Louis Bonicoli <pierre-louis.bonicoli@libregerbil.fr> | 2017-04-18 17:08:21 +0200 |
---|---|---|
committer | Pierre-Louis Bonicoli <pierre-louis.bonicoli@libregerbil.fr> | 2017-04-18 17:56:10 +0200 |
commit | 39607fb86f992253b600b8563d6c6c971fd42fb7 (patch) | |
tree | 3d647f85440af5145507552d75d4fa61aa225845 /roles/lib_openshift/src/test/unit/test_oc_adm_router.py | |
parent | 233cb72777a5bdea68e5a7703bad53bb012c0bd0 (diff) | |
download | openshift-39607fb86f992253b600b8563d6c6c971fd42fb7.tar.gz openshift-39607fb86f992253b600b8563d6c6c971fd42fb7.tar.bz2 openshift-39607fb86f992253b600b8563d6c6c971fd42fb7.tar.xz openshift-39607fb86f992253b600b8563d6c6c971fd42fb7.zip |
Add missing mock for locate_oc_binary method
When locate_oc_binary has not been mocked, the test suite fails when oc
executable is available.
Diffstat (limited to 'roles/lib_openshift/src/test/unit/test_oc_adm_router.py')
-rwxr-xr-x | roles/lib_openshift/src/test/unit/test_oc_adm_router.py | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/roles/lib_openshift/src/test/unit/test_oc_adm_router.py b/roles/lib_openshift/src/test/unit/test_oc_adm_router.py index 51393dbaf..5481ac623 100755 --- a/roles/lib_openshift/src/test/unit/test_oc_adm_router.py +++ b/roles/lib_openshift/src/test/unit/test_oc_adm_router.py @@ -286,10 +286,11 @@ class RouterTest(unittest.TestCase): ] }''' + @mock.patch('oc_adm_router.locate_oc_binary') @mock.patch('oc_adm_router.Utils._write') @mock.patch('oc_adm_router.Utils.create_tmpfile_copy') @mock.patch('oc_adm_router.Router._run') - def test_state_present(self, mock_cmd, mock_tmpfile_copy, mock_write): + def test_state_present(self, mock_cmd, mock_tmpfile_copy, mock_write, mock_oc_binary): ''' Testing a create ''' params = {'state': 'present', 'debug': False, @@ -345,6 +346,10 @@ class RouterTest(unittest.TestCase): '/tmp/mocked_kubeconfig', ] + mock_oc_binary.side_effect = [ + 'oc', + ] + results = Router.run_ansible(params, False) self.assertTrue(results['changed']) |