)]}'
{"tests/test_backend.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"8097194e107816d503a627ecdc62aa9e24b15b6e","unresolved":false,"context_lines":[{"line_number":2028,"context_line":"        token_id4 \u003d self.create_token_sample_data(tenant_id\u003dtenant2)"},{"line_number":2029,"context_line":"        # test for existing but empty tenant (LP:1078497)"},{"line_number":2030,"context_line":"        token_id5 \u003d self.create_token_sample_data("},{"line_number":2031,"context_line":"            tenant_id\u003d\u0027existing_but_empty\u0027)"},{"line_number":2032,"context_line":"        tokens \u003d self.token_api.list_tokens(\u0027testuserid\u0027)"},{"line_number":2033,"context_line":"        self.assertEquals(len(tokens), 3)"},{"line_number":2034,"context_line":"        self.assertNotIn(token_id1, tokens)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAOn%2F%2F1RY%3D","line":2031,"updated":"2013-04-19 16:26:40.000000000","message":"passing this string as the tenant_id here is a bit misleading, without knowing the implementation of create_token_sample_data\n\ni would simply prefer that \u0027existing_but_empty\u0027 be replaced with a constant defined at the module level, e.g. NULL \u003d object(), such that you can use:\n\n  self.create_token_sample_data(tenant_id\u003dNULL)\n\nAnd then revise the create_token_sample_data to simply test for:\n\n  if tenant_id is NULL: data[\u0027tenant\u0027] \u003d None","commit_id":"bdfbcef1cd3e702610fa76ccc651510e19690a1b"},{"author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"change_message_id":"29a99c2997d4fdc1813dad4aaa18795c9b73b376","unresolved":false,"context_lines":[{"line_number":28,"context_line":"CONF \u003d config.CONF"},{"line_number":29,"context_line":"DEFAULT_DOMAIN_ID \u003d CONF.identity.default_domain_id"},{"line_number":30,"context_line":"TIME_FORMAT \u003d \u0027%Y-%m-%dT%H:%M:%S.%fZ\u0027"},{"line_number":31,"context_line":"NULL \u003d object()"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"class IdentityTests(object):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAOn%2F%2FzZ4%3D","line":31,"updated":"2013-04-22 20:36:37.000000000","message":"Can this be changed from NULL to NULL_OBJECT so that it is more clear what you are doing here and not confused with the Python Null Object Design Pattern?","commit_id":"558adcf9a73d7bda93b9c3b1b9667fe382cfcdc7"}]}