diff --git a/cinder/tests/unit/targets/test_cxt_driver.py b/cinder/tests/unit/targets/test_cxt_driver.py index 49c79edd0..9ae932192 100644 --- a/cinder/tests/unit/targets/test_cxt_driver.py +++ b/cinder/tests/unit/targets/test_cxt_driver.py @@ -70,7 +70,7 @@ class TestCxtAdmDriver(tf.TargetDriverFixture): tmp_file.seek(0) expected = ('otzL', '234Z') - with mock.patch('__builtin__.open') as mock_open: + with mock.patch('six.moves.builtins.open') as mock_open: ctx = context.get_admin_context() mock_open.return_value = contextlib.closing(tmp_file) self.assertEqual(expected, @@ -79,7 +79,7 @@ class TestCxtAdmDriver(tf.TargetDriverFixture): self.assertTrue(mock_open.called) def test_get_target_chap_auth_negative(self): - with mock.patch('__builtin__.open') as mock_open: + with mock.patch('six.moves.builtins.open') as mock_open: e = IOError() e.errno = 123 mock_open.side_effect = e @@ -87,8 +87,8 @@ class TestCxtAdmDriver(tf.TargetDriverFixture): self.assertRaises(IOError, self.target._get_target_chap_auth, ctxt, self.test_vol) - mock_open.side_effect = StandardError() - self.assertRaises(StandardError, + mock_open.side_effect = ZeroDivisionError() + self.assertRaises(ZeroDivisionError, self.target._get_target_chap_auth, ctxt, self.test_vol) diff --git a/cinder/tests/unit/targets/test_tgt_driver.py b/cinder/tests/unit/targets/test_tgt_driver.py index efcbe089d..4ef1a9e39 100644 --- a/cinder/tests/unit/targets/test_tgt_driver.py +++ b/cinder/tests/unit/targets/test_tgt_driver.py @@ -153,7 +153,7 @@ class TestTgtAdmDriver(tf.TargetDriverFixture): 'bspath': self.testvol_path} with open(os.path.join(self.fake_volumes_dir, self.test_vol.split(':')[1]), - 'wb') as tmp_file: + 'w') as tmp_file: tmp_file.write(persist_file) ctxt = context.get_admin_context() expected = ('otzL', '234Z') @@ -162,7 +162,7 @@ class TestTgtAdmDriver(tf.TargetDriverFixture): self.test_vol)) def test_get_target_chap_auth_negative(self): - with mock.patch('__builtin__.open') as mock_open: + with mock.patch('six.moves.builtins.open') as mock_open: e = IOError() e.errno = 123 mock_open.side_effect = e @@ -170,8 +170,8 @@ class TestTgtAdmDriver(tf.TargetDriverFixture): self.assertRaises(IOError, self.target._get_target_chap_auth, ctxt, self.test_vol) - mock_open.side_effect = StandardError() - self.assertRaises(StandardError, + mock_open.side_effect = ZeroDivisionError() + self.assertRaises(ZeroDivisionError, self.target._get_target_chap_auth, ctxt, self.test_vol) diff --git a/tox.ini b/tox.ini index 7c36705e4..1ff355b83 100644 --- a/tox.ini +++ b/tox.ini @@ -29,6 +29,12 @@ downloadcache = ~/cache/pip [testenv:py34] commands = python -m testtools.run \ + cinder.tests.unit.targets.test_base_iscsi_driver \ + cinder.tests.unit.targets.test_cxt_driver \ + cinder.tests.unit.targets.test_iser_driver \ + cinder.tests.unit.targets.test_lio_driver \ + cinder.tests.unit.targets.test_scst_driver \ + cinder.tests.unit.targets.test_tgt_driver \ cinder.tests.unit.test_api_urlmap \ cinder.tests.unit.test_backup \ cinder.tests.unit.test_backup_ceph \