diff options
| author | Joel Diaz <jdiaz@redhat.com> | 2017-02-28 16:22:08 +0000 |
|---|---|---|
| committer | Joel Diaz <jdiaz@redhat.com> | 2017-03-21 17:33:27 -0400 |
| commit | a11970d30c88d188392ec217c055b6b8169b3769 (patch) | |
| tree | 8c32341fde26780aa24332f9ff6aff09db6110dc /roles/lib_openshift/src/test/unit | |
| parent | 45fbfdad1b80c50276a9da3841d6e4089b109e35 (diff) | |
| download | openshift-a11970d30c88d188392ec217c055b6b8169b3769.tar.gz openshift-a11970d30c88d188392ec217c055b6b8169b3769.tar.bz2 openshift-a11970d30c88d188392ec217c055b6b8169b3769.tar.xz openshift-a11970d30c88d188392ec217c055b6b8169b3769.zip | |
clean up and clarify docs/comments
update unit tests
Diffstat (limited to 'roles/lib_openshift/src/test/unit')
| -rwxr-xr-x | roles/lib_openshift/src/test/unit/test_oc_user.py (renamed from roles/lib_openshift/src/test/unit/oc_user.py) | 16 |
1 files changed, 13 insertions, 3 deletions
diff --git a/roles/lib_openshift/src/test/unit/oc_user.py b/roles/lib_openshift/src/test/unit/test_oc_user.py index 920b06d0e..933e96ae2 100755 --- a/roles/lib_openshift/src/test/unit/oc_user.py +++ b/roles/lib_openshift/src/test/unit/test_oc_user.py @@ -24,7 +24,7 @@ import mock # place class in our python path module_path = os.path.join('/'.join(os.path.realpath(__file__).split('/')[:-4]), 'library') # noqa: E501 sys.path.insert(0, module_path) -from oc_user import OCUser # noqa: E402 +from oc_user import OCUser, locate_oc_binary # noqa: E402 class OCUserTest(unittest.TestCase): @@ -36,8 +36,9 @@ class OCUserTest(unittest.TestCase): ''' setup method will create a file and set to known configuration ''' pass + @mock.patch('oc_user.Utils.create_tmpfile_copy') @mock.patch('oc_user.OCUser._run') - def test_state_list(self, mock_cmd): + def test_state_list(self, mock_cmd, mock_tmpfile_copy): ''' Testing a user list ''' params = {'username': 'testuser@email.com', 'state': 'list', @@ -65,13 +66,18 @@ class OCUserTest(unittest.TestCase): (0, user, ''), ] + mock_tmpfile_copy.side_effect = [ + '/tmp/mocked_kubeconfig', + ] + results = OCUser.run_ansible(params, False) self.assertFalse(results['changed']) self.assertTrue(results['results'][0]['metadata']['name'] == "testuser@email.com") + @mock.patch('oc_user.Utils.create_tmpfile_copy') @mock.patch('oc_user.OCUser._run') - def test_state_present(self, mock_cmd): + def test_state_present(self, mock_cmd, mock_tmpfile_copy): ''' Testing a user list ''' params = {'username': 'testuser@email.com', 'state': 'present', @@ -102,6 +108,10 @@ class OCUserTest(unittest.TestCase): (0, created_user, ''), # get ] + mock_tmpfile_copy.side_effect = [ + '/tmp/mocked_kubeconfig', + ] + results = OCUser.run_ansible(params, False) self.assertTrue(results['changed']) |