)]}'
{"keystone/tests/unit/test_v3_resource.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"2630697ddd2a0710af412ce62acf1509a8741616","unresolved":false,"context_lines":[{"line_number":323,"context_line":""},{"line_number":324,"context_line":"        self.delete(\u0027/domains/%(domain_id)s\u0027 % {"},{"line_number":325,"context_line":"            \u0027domain_id\u0027: CONF.identity.default_domain_id},"},{"line_number":326,"context_line":"            expected_status\u003dexception.ForbiddenAction.code)"},{"line_number":327,"context_line":""},{"line_number":328,"context_line":"    def test_delete_new_default_domain_fails(self):"},{"line_number":329,"context_line":"        # If change the default domain ID, deleting the new default domain"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_69bdc801","side":"PARENT","line":326,"updated":"2016-01-07 21:00:57.000000000","message":"This test should be updated to expect a 204 No Content, which I think would be:\n\n  expected_status\u003dhttp_client.NO_CONTENT\n\nAnd furthermore, all of these expected_status\u003dexception.* are reaching into our implementation to validatate our implementation... and should use http_client.* (a third party package) instead.","commit_id":"1a67c2a7efcd31213eadc0e292e74b651440a656"}]}