)]}'
{"glance/tests/unit/v1/test_api.py":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"27bfa8b803c0976d1e1ccae46b25d72629a65bc4","unresolved":false,"context_lines":[{"line_number":4220,"context_line":"        self.assertEqual(403, output.status_int)"},{"line_number":4221,"context_line":""},{"line_number":4222,"context_line":"    def test_create_protected_prop_check_case_insensitive(self):"},{"line_number":4223,"context_line":"        \"\"\""},{"line_number":4224,"context_line":"        Verify that role check is case-insensitive i.e. the property"},{"line_number":4225,"context_line":"        marked with role Member is creatable by the member role"},{"line_number":4226,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_542d7bfc","line":4223,"updated":"2015-03-18 10:17:38.000000000","message":"Doc string should not start with new line.\n\nhttp://docs.openstack.org/developer/hacking/#docstrings","commit_id":"03407f91179d6746fce90e817082943248ce1ea4"},{"author":{"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},"change_message_id":"beb2280acdda2d9ffd075586c7704d6ef217a38b","unresolved":false,"context_lines":[{"line_number":4220,"context_line":"        self.assertEqual(403, output.status_int)"},{"line_number":4221,"context_line":""},{"line_number":4222,"context_line":"    def test_create_protected_prop_check_case_insensitive(self):"},{"line_number":4223,"context_line":"        \"\"\""},{"line_number":4224,"context_line":"        Verify that role check is case-insensitive i.e. the property"},{"line_number":4225,"context_line":"        marked with role Member is creatable by the member role"},{"line_number":4226,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"3acd31a7_a2fb7ec0","line":4223,"in_reply_to":"3acd31a7_e9e4360e","updated":"2015-05-05 19:33:06.000000000","message":"Remaining consistent with the current formatting would be better here.","commit_id":"03407f91179d6746fce90e817082943248ce1ea4"},{"author":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"change_message_id":"e87e3531afe460c90c0c4395558cb5b341d4fbc1","unresolved":false,"context_lines":[{"line_number":4220,"context_line":"        self.assertEqual(403, output.status_int)"},{"line_number":4221,"context_line":""},{"line_number":4222,"context_line":"    def test_create_protected_prop_check_case_insensitive(self):"},{"line_number":4223,"context_line":"        \"\"\""},{"line_number":4224,"context_line":"        Verify that role check is case-insensitive i.e. the property"},{"line_number":4225,"context_line":"        marked with role Member is creatable by the member role"},{"line_number":4226,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"3acd31a7_e9e4360e","line":4223,"in_reply_to":"9a80dd14_542d7bfc","updated":"2015-04-30 13:57:21.000000000","message":"Perhaps, but it\u0027s consistent with every other test in this file :-)","commit_id":"03407f91179d6746fce90e817082943248ce1ea4"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"27bfa8b803c0976d1e1ccae46b25d72629a65bc4","unresolved":false,"context_lines":[{"line_number":4229,"context_line":"            path\u003d\u0027/images/%s\u0027 % image_id, method\u003d\u0027PUT\u0027)"},{"line_number":4230,"context_line":"        headers \u003d {\u0027x-auth-token\u0027: \u0027user:tenant:member\u0027,"},{"line_number":4231,"context_line":"                   \u0027x-image-meta-property-x_case_insensitive\u0027: \u00271\u0027}"},{"line_number":4232,"context_line":"        for k, v in six.iteritems(headers):"},{"line_number":4233,"context_line":"                another_request.headers[k] \u003d v"},{"line_number":4234,"context_line":"        output \u003d another_request.get_response(self.api)"},{"line_number":4235,"context_line":"        res_body \u003d jsonutils.loads(output.body)[\u0027image\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_b48797b5","line":4232,"updated":"2015-03-18 10:17:38.000000000","message":"to avoid code duplication, you can move this code to common method.\n\nExample:\n\ndef _update_request_headers(headers, another_request):\n    for k, v in six.iteritems(headers):\n        another_request.headers[k] \u003d v\n\nand call _update_request_headers(headers, another_request) here.","commit_id":"03407f91179d6746fce90e817082943248ce1ea4"},{"author":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"change_message_id":"e87e3531afe460c90c0c4395558cb5b341d4fbc1","unresolved":false,"context_lines":[{"line_number":4229,"context_line":"            path\u003d\u0027/images/%s\u0027 % image_id, method\u003d\u0027PUT\u0027)"},{"line_number":4230,"context_line":"        headers \u003d {\u0027x-auth-token\u0027: \u0027user:tenant:member\u0027,"},{"line_number":4231,"context_line":"                   \u0027x-image-meta-property-x_case_insensitive\u0027: \u00271\u0027}"},{"line_number":4232,"context_line":"        for k, v in six.iteritems(headers):"},{"line_number":4233,"context_line":"                another_request.headers[k] \u003d v"},{"line_number":4234,"context_line":"        output \u003d another_request.get_response(self.api)"},{"line_number":4235,"context_line":"        res_body \u003d jsonutils.loads(output.body)[\u0027image\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"3acd31a7_e992f656","line":4232,"in_reply_to":"9a80dd14_b48797b5","updated":"2015-04-30 13:57:21.000000000","message":"Again, this seems consistent with previously existing tests.","commit_id":"03407f91179d6746fce90e817082943248ce1ea4"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"27bfa8b803c0976d1e1ccae46b25d72629a65bc4","unresolved":false,"context_lines":[{"line_number":4236,"context_line":"        self.assertEqual(\u00271\u0027, res_body[\u0027properties\u0027][\u0027x_case_insensitive\u0027])"},{"line_number":4237,"context_line":""},{"line_number":4238,"context_line":"    def test_read_protected_prop_check_case_insensitive(self):"},{"line_number":4239,"context_line":"        \"\"\""},{"line_number":4240,"context_line":"        Verify that role check is case-insensitive i.e. the property"},{"line_number":4241,"context_line":"        marked with role Member is readable by the member role"},{"line_number":4242,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_9437f34c","line":4239,"updated":"2015-03-18 10:17:38.000000000","message":"ditto","commit_id":"03407f91179d6746fce90e817082943248ce1ea4"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"27bfa8b803c0976d1e1ccae46b25d72629a65bc4","unresolved":false,"context_lines":[{"line_number":4256,"context_line":"            \u00271\u0027, output.headers[\u0027x-image-meta-property-x_case_insensitive\u0027])"},{"line_number":4257,"context_line":""},{"line_number":4258,"context_line":"    def test_update_protected_props_check_case_insensitive(self):"},{"line_number":4259,"context_line":"        \"\"\""},{"line_number":4260,"context_line":"        Verify that role check is case-insensitive i.e. the property"},{"line_number":4261,"context_line":"        marked with role Member is updatable by the member role"},{"line_number":4262,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_b43dd72e","line":4259,"updated":"2015-03-18 10:17:38.000000000","message":"ditto","commit_id":"03407f91179d6746fce90e817082943248ce1ea4"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"27bfa8b803c0976d1e1ccae46b25d72629a65bc4","unresolved":false,"context_lines":[{"line_number":4273,"context_line":"        self.assertEqual(\u00272\u0027, res_body[\u0027properties\u0027][\u0027x_case_insensitive\u0027])"},{"line_number":4274,"context_line":""},{"line_number":4275,"context_line":"    def test_delete_protected_props_check_case_insensitive(self):"},{"line_number":4276,"context_line":"        \"\"\""},{"line_number":4277,"context_line":"        Verify that role check is case-insensitive i.e. the property"},{"line_number":4278,"context_line":"        marked with role Member is deletable by the member role"},{"line_number":4279,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a80dd14_d4388b3d","line":4276,"updated":"2015-03-18 10:17:38.000000000","message":"ditto","commit_id":"03407f91179d6746fce90e817082943248ce1ea4"}]}