Skip to content

Commit ecd34fd

Browse files
Zuulopenstack-gerrit
authored andcommitted
Merge "Trivial: debug_tracebacks_in_api=True for unit tests"
2 parents b77d736 + e905b0e commit ecd34fd

2 files changed

Lines changed: 4 additions & 0 deletions

File tree

ironic/tests/unit/api/base.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@ class BaseApiTest(db_base.DbTestCase):
4343
"""
4444

4545
SOURCE_DATA = {'test_source': {'somekey': '666'}}
46+
SKIP_TRACEBACK_OVERRIDE = False
4647

4748
root_controller = 'ironic.api.controllers.root.RootController'
4849

@@ -53,6 +54,8 @@ def setUp(self):
5354
cfg.CONF.set_override("admin_user", "admin",
5455
group='keystone_authtoken')
5556
cfg.CONF.set_override("auth_strategy", "noauth")
57+
if not self.SKIP_TRACEBACK_OVERRIDE:
58+
cfg.CONF.set_override("debug_tracebacks_in_api", True)
5659
self.app = self._make_app()
5760

5861
def reset_pecan():

ironic/tests/unit/api/test_hooks.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,7 @@ class TestNoExceptionTracebackHook(base.BaseApiTest):
9999
' getattr(proxyobj, method)(ctxt, **kwargs)']
100100
MSG_WITHOUT_TRACE = "Test exception message."
101101
MSG_WITH_TRACE = MSG_WITHOUT_TRACE + "\n" + "\n".join(TRACE)
102+
SKIP_TRACEBACK_OVERRIDE = True # do not mess with the defaults
102103

103104
def setUp(self):
104105
super(TestNoExceptionTracebackHook, self).setUp()

0 commit comments

Comments
 (0)