diff options
| author |   Cédric Ollivier <cedric.ollivier@orange.com> | 2019-05-25 09:38:40 +0200 |
|---|---|---|
| committer | Cédric Ollivier <cedric.ollivier@orange.com> | 2019-05-25 11:23:45 +0200 |
| commit | 5853dd1ef3c522a975f9685250b1b3e85588f738 (patch) | |
| tree | db375cd93716df8cd27cff410183af0c37c0833e /functest/tests/unit | |
| parent | 7d58f89103a8c44fd166c3e06b492ade63e5c5b3 (diff) | |
Set utf-8 in decode and encode calls
It eases backporting in iruya and hunter as a few logs can't be
decoded as ascii [1].
[1] https://build.opnfv.org/ci/job/functest-opnfv-functest-vnf-iruya-juju_epc-run/56/console
Change-Id: I52a6f9a25130cbf62e8ce3b9972064ab73177665
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
Diffstat (limited to 'functest/tests/unit')
| -rw-r--r-- | functest/tests/unit/openstack/rally/test_rally.py | 6 | ||||
| -rw-r--r-- | functest/tests/unit/utils/test_functest_utils.py | 8 |
2 files changed, 7 insertions, 7 deletions
diff --git a/functest/tests/unit/openstack/rally/test_rally.py b/functest/tests/unit/openstack/rally/test_rally.py index c51258060..83332c86f 100644 --- a/functest/tests/unit/openstack/rally/test_rally.py +++ b/functest/tests/unit/openstack/rally/test_rally.py @@ -78,13 +78,13 @@ class OSRallyTesting(unittest.TestCase): calls = [ mock.call(['rally', 'deployment', 'destroy', '--deployment', str(getattr(config.CONF, 'rally_deployment_name'))]), - mock.call().decode(), + mock.call().decode("utf-8"), mock.call(['rally', 'deployment', 'create', '--fromenv', '--name', str(getattr(config.CONF, 'rally_deployment_name'))], env=None), - mock.call().decode(), + mock.call().decode("utf-8"), mock.call(['rally', 'deployment', 'check']), - mock.call().decode()] + mock.call().decode("utf-8")] mock_exec.assert_has_calls(calls) @mock.patch('functest.opnfv_tests.openstack.rally.rally.os.path.exists') diff --git a/functest/tests/unit/utils/test_functest_utils.py b/functest/tests/unit/utils/test_functest_utils.py index ba022e377..4ec205894 100644 --- a/functest/tests/unit/utils/test_functest_utils.py +++ b/functest/tests/unit/utils/test_functest_utils.py @@ -99,7 +99,7 @@ class FunctestUtilsTesting(unittest.TestCase): mock.patch('six.moves.builtins.open', mock.mock_open()) as mopen: stream = six.BytesIO() - stream.write(self.cmd_readline().encode()) + stream.write(self.cmd_readline().encode("utf-8")) mock_obj2 = mock.Mock() attrs = {'stdout': stream, 'wait.return_value': 1} mock_obj2.configure_mock(**attrs) @@ -120,7 +120,7 @@ class FunctestUtilsTesting(unittest.TestCase): mock.patch('six.moves.builtins.open', mock.mock_open()) as mopen: stream = six.BytesIO() - stream.write(self.cmd_readline().encode()) + stream.write(self.cmd_readline().encode("utf-8")) mock_obj2 = mock.Mock() attrs = {'stdout': stream, 'wait.return_value': 0} mock_obj2.configure_mock(**attrs) @@ -139,7 +139,7 @@ class FunctestUtilsTesting(unittest.TestCase): with mock.patch('functest.utils.functest_utils.subprocess.Popen') \ as mock_subproc_open: stream = six.BytesIO() - stream.write(self.cmd_readline().encode()) + stream.write(self.cmd_readline().encode("utf-8")) mock_obj2 = mock.Mock() attrs = {'stdout': stream, 'wait.return_value': 0} mock_obj2.configure_mock(**attrs) @@ -155,7 +155,7 @@ class FunctestUtilsTesting(unittest.TestCase): with mock.patch('functest.utils.functest_utils.subprocess.Popen') \ as mock_subproc_open: stream = six.BytesIO() - stream.write(self.cmd_readline().encode()) + stream.write(self.cmd_readline().encode("utf-8")) mock_obj2 = mock.Mock() attrs = {'stdout': stream, 'wait.return_value': 1} mock_obj2.configure_mock(**attrs) |