diff --git a/masakari/context.py b/masakari/context.py index 9e7bd9a3..e2b94907 100644 --- a/masakari/context.py +++ b/masakari/context.py @@ -117,7 +117,12 @@ class RequestContext(context.RequestContext): LOG.debug('Arguments dropped when creating context: %s', str(kwargs)) - self.read_deleted = read_deleted + if read_deleted is None: + # If we did not get a value for read_deleted, ensure we default + # it to "no" as code expects it to be a string. + self.read_deleted = 'no' + else: + self.read_deleted = read_deleted self.remote_address = remote_address if not timestamp: timestamp = timeutils.utcnow() diff --git a/masakari/tests/unit/test_context.py b/masakari/tests/unit/test_context.py index 29bf48aa..6fb9b46b 100644 --- a/masakari/tests/unit/test_context.py +++ b/masakari/tests/unit/test_context.py @@ -57,6 +57,11 @@ class ContextTestCase(test.NoDBTestCase): ctxt.read_deleted = 'no' self.assertEqual('no', ctxt.read_deleted) + def test_request_context_read_deleted_none(self): + ctxt = context.RequestContext('111', + '222') + self.assertEqual('no', ctxt.read_deleted) + def test_request_context_read_deleted_invalid(self): self.assertRaises(ValueError, context.RequestContext,