Replace 'assertTrue(a in b)' with 'assertIn(a, b)'
Supplement the patch:https://review.openstack.org/#/c/364652/ Change-Id: I6890d7f5aff751ddfad86fc954146cc5ac102ac8
This commit is contained in:
parent
5b88ddc8ea
commit
312731ab4c
|
@ -131,14 +131,14 @@ class OperationLogMiddlewareTest(test.TestCase):
|
||||||
self.assertEqual(302, resp.status_code)
|
self.assertEqual(302, resp.status_code)
|
||||||
log_args = mock_logger.info.call_args[0]
|
log_args = mock_logger.info.call_args[0]
|
||||||
logging_str = log_args[0] % log_args[1]
|
logging_str = log_args[0] % log_args[1]
|
||||||
self.assertTrue(request.user.username in logging_str)
|
self.assertIn(request.user.username, logging_str)
|
||||||
self.assertTrue(self.http_referer in logging_str)
|
self.assertIn(self.http_referer, logging_str)
|
||||||
self.assertTrue(settings.LOGIN_URL in logging_str)
|
self.assertIn(settings.LOGIN_URL, logging_str)
|
||||||
self.assertTrue('POST' in logging_str)
|
self.assertIn('POST', logging_str)
|
||||||
self.assertTrue('302' in logging_str)
|
self.assertIn('302', logging_str)
|
||||||
post_data = ['"username": "admin"', '"password": "********"']
|
post_data = ['"username": "admin"', '"password": "********"']
|
||||||
for data in post_data:
|
for data in post_data:
|
||||||
self.assertTrue(data in logging_str)
|
self.assertIn(data, logging_str)
|
||||||
|
|
||||||
@override_settings(OPERATION_LOG_ENABLED=True)
|
@override_settings(OPERATION_LOG_ENABLED=True)
|
||||||
@override_settings(OPERATION_LOG_OPTIONS={'target_methods': ['GET']})
|
@override_settings(OPERATION_LOG_OPTIONS={'target_methods': ['GET']})
|
||||||
|
@ -154,11 +154,11 @@ class OperationLogMiddlewareTest(test.TestCase):
|
||||||
self.assertEqual(302, resp.status_code)
|
self.assertEqual(302, resp.status_code)
|
||||||
log_args = mock_logger.info.call_args[0]
|
log_args = mock_logger.info.call_args[0]
|
||||||
logging_str = log_args[0] % log_args[1]
|
logging_str = log_args[0] % log_args[1]
|
||||||
self.assertTrue(request.user.username in logging_str)
|
self.assertIn(request.user.username, logging_str)
|
||||||
self.assertTrue(self.http_referer in logging_str)
|
self.assertIn(self.http_referer, logging_str)
|
||||||
self.assertTrue(request.path in logging_str)
|
self.assertIn(request.path, logging_str)
|
||||||
self.assertTrue('GET' in logging_str)
|
self.assertIn('GET', logging_str)
|
||||||
self.assertTrue('302' in logging_str)
|
self.assertIn('302', logging_str)
|
||||||
|
|
||||||
@override_settings(OPERATION_LOG_ENABLED=True)
|
@override_settings(OPERATION_LOG_ENABLED=True)
|
||||||
@patch(('horizon.middleware.operation_log.OperationLogMiddleware.'
|
@patch(('horizon.middleware.operation_log.OperationLogMiddleware.'
|
||||||
|
@ -186,10 +186,10 @@ class OperationLogMiddlewareTest(test.TestCase):
|
||||||
log_args = mock_logger.info.call_args[0]
|
log_args = mock_logger.info.call_args[0]
|
||||||
logging_str = log_args[0] % log_args[1]
|
logging_str = log_args[0] % log_args[1]
|
||||||
self.assertTrue(mock_logger.info.called)
|
self.assertTrue(mock_logger.info.called)
|
||||||
self.assertTrue(request.user.username in logging_str)
|
self.assertIn(request.user.username, logging_str)
|
||||||
self.assertTrue(self.http_referer in logging_str)
|
self.assertIn(self.http_referer, logging_str)
|
||||||
self.assertTrue(settings.LOGIN_URL in logging_str)
|
self.assertIn(settings.LOGIN_URL, logging_str)
|
||||||
self.assertTrue('Unexpected error occurred.' in logging_str)
|
self.assertIn('Unexpected error occurred.', logging_str)
|
||||||
post_data = ['"username": "admin"', '"password": "********"']
|
post_data = ['"username": "admin"', '"password": "********"']
|
||||||
for data in post_data:
|
for data in post_data:
|
||||||
self.assertTrue(data in logging_str)
|
self.assertTrue(data in logging_str)
|
||||||
|
|
|
@ -287,8 +287,8 @@ class SecurityGroupTabTests(test.TestCase):
|
||||||
|
|
||||||
create_action = self.getAndAssertTableAction(res, 'security_groups',
|
create_action = self.getAndAssertTableAction(res, 'security_groups',
|
||||||
'create')
|
'create')
|
||||||
self.assertTrue('disabled' in create_action.classes,
|
self.assertIn('disabled', create_action.classes,
|
||||||
'The create button should be disabled')
|
'The create button should be disabled')
|
||||||
|
|
||||||
def test_create_button_disabled_when_quota_exceeded_neutron_disabled(self):
|
def test_create_button_disabled_when_quota_exceeded_neutron_disabled(self):
|
||||||
self._test_create_button_disabled_when_quota_exceeded(False)
|
self._test_create_button_disabled_when_quota_exceeded(False)
|
||||||
|
|
|
@ -1244,8 +1244,8 @@ class NetworkViewTests(test.TestCase, NetworkStubMixin):
|
||||||
self.assertItemsEqual(networks, self.networks.list())
|
self.assertItemsEqual(networks, self.networks.list())
|
||||||
|
|
||||||
button = find_button_fn(res)
|
button = find_button_fn(res)
|
||||||
self.assertTrue('disabled' in button.classes,
|
self.assertIn('disabled', button.classes,
|
||||||
"The create button should be disabled")
|
"The create button should be disabled")
|
||||||
return button
|
return button
|
||||||
|
|
||||||
@test.create_stubs({api.neutron: ('network_list',),
|
@test.create_stubs({api.neutron: ('network_list',),
|
||||||
|
@ -1327,8 +1327,8 @@ class NetworkViewTests(test.TestCase, NetworkStubMixin):
|
||||||
self.assertItemsEqual(subnets, self.subnets.list())
|
self.assertItemsEqual(subnets, self.subnets.list())
|
||||||
|
|
||||||
create_action = self.getAndAssertTableAction(res, 'subnets', 'create')
|
create_action = self.getAndAssertTableAction(res, 'subnets', 'create')
|
||||||
self.assertTrue('disabled' in create_action.classes,
|
self.assertIn('disabled', create_action.classes,
|
||||||
'The create button should be disabled')
|
'The create button should be disabled')
|
||||||
|
|
||||||
@test.create_stubs({api.neutron: ('network_list',),
|
@test.create_stubs({api.neutron: ('network_list',),
|
||||||
quotas: ('tenant_quota_usages',)})
|
quotas: ('tenant_quota_usages',)})
|
||||||
|
|
|
@ -807,8 +807,8 @@ class RouterViewTests(RouterMixin, test.TestCase):
|
||||||
self.assertItemsEqual(routers, self.routers.list())
|
self.assertItemsEqual(routers, self.routers.list())
|
||||||
|
|
||||||
create_action = self.getAndAssertTableAction(res, 'routers', 'create')
|
create_action = self.getAndAssertTableAction(res, 'routers', 'create')
|
||||||
self.assertTrue('disabled' in create_action.classes,
|
self.assertIn('disabled', create_action.classes,
|
||||||
'Create button is not disabled')
|
'Create button is not disabled')
|
||||||
self.assertEqual('Create Router (Quota exceeded)',
|
self.assertEqual('Create Router (Quota exceeded)',
|
||||||
create_action.verbose_name)
|
create_action.verbose_name)
|
||||||
|
|
||||||
|
|
|
@ -1148,8 +1148,8 @@ class VolumeViewTests(test.TestCase):
|
||||||
|
|
||||||
snapshot_action = self._get_volume_row_action_from_ajax(
|
snapshot_action = self._get_volume_row_action_from_ajax(
|
||||||
res, 'snapshots', volume.id)
|
res, 'snapshots', volume.id)
|
||||||
self.assertTrue('disabled' in snapshot_action.classes,
|
self.assertIn('disabled', snapshot_action.classes,
|
||||||
'The create snapshot button should be disabled')
|
'The create snapshot button should be disabled')
|
||||||
|
|
||||||
@test.create_stubs({cinder: ('tenant_absolute_limits',
|
@test.create_stubs({cinder: ('tenant_absolute_limits',
|
||||||
'volume_list_paged',
|
'volume_list_paged',
|
||||||
|
@ -1224,8 +1224,8 @@ class VolumeViewTests(test.TestCase):
|
||||||
self.assertItemsEqual(volumes, self.cinder_volumes.list())
|
self.assertItemsEqual(volumes, self.cinder_volumes.list())
|
||||||
|
|
||||||
create_action = self.getAndAssertTableAction(res, 'volumes', 'create')
|
create_action = self.getAndAssertTableAction(res, 'volumes', 'create')
|
||||||
self.assertTrue('disabled' in create_action.classes,
|
self.assertIn('disabled', create_action.classes,
|
||||||
'The create button should be disabled')
|
'The create button should be disabled')
|
||||||
|
|
||||||
@test.create_stubs({cinder: ('tenant_absolute_limits',
|
@test.create_stubs({cinder: ('tenant_absolute_limits',
|
||||||
'volume_get',
|
'volume_get',
|
||||||
|
|
Loading…
Reference in New Issue