openstack/aodh
Code Issues Proposed changes

Merge "Avoid swallowing AssertionError in test skipping logic"

Browse Source
This commit is contained in:
Jenkins 2014-03-26 08:29:50 +00:00 committed by Gerrit Code Review
commit 2a86c41b72
2 changed files with 8 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
ceilometer/tests/api/v2/test_statistics_scenarios.py
View File

@ -1470,12 +1470,13 @@ class TestSelectableAggregates(FunctionalTest,
def test_bad_selectable_parameterized_aggregate(self):
agg_args = {'aggregate.func': 'cardinality',
'aggregate.param': 'injection_attack'}
resp = self.get_json(self.PATH, expect_errors=True,
resp = self.get_json(self.PATH, status=[400],
groupby=['project_id'], **agg_args)
self.assertTrue(400, resp.status_code)
self.assertTrue('error_message' in resp.json)
self.assertEqual('Bad aggregate: cardinality.injection_attack',
resp.json['error_message'].get('faultstring'))
self.assertTrue('error_message' in resp)
self.assertEqual(resp['error_message'].get('faultcode'),
'Client')
self.assertEqual(resp['error_message'].get('faultstring'),
'Bad aggregate: cardinality.injection_attack')
class TestUnparameterizedAggregates(FunctionalTest,

2
ceilometer/tests/base.py
View File

@ -59,6 +59,8 @@ def _skip_decorator(func):
def skip_if_not_implemented(*args, **kwargs):
try:
return func(*args, **kwargs)
except AssertionError:
raise
except NotImplementedError as e:
raise testcase.TestSkipped(six.text_type(e))
except Exception as e: