)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9152,"name":"Castulo J. Martinez","email":"castulo.martinez@intel.com","username":"cas"},"change_message_id":"05f16b049de447da6866d7db5bcb7fa4d7b01e7a","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Alexander Petrov \u003capetrov@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-01-27 12:19:00 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Identity V3 API: this patch adds the test needed for test coverage of method \u0027Default configuration settings\u0027 (GET /domains/config/default)"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I1d95b51e52f1c1ad283cf518edd63c8aa22aa0bd"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5a5ae5dd_e776dfa2","line":7,"range":{"start_line":7,"start_character":70,"end_line":7,"end_character":76},"updated":"2016-02-08 21:42:24.000000000","message":"The commit description is longer than the 72 characters mentioned here: https://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"6ecf4679217f380a775b350b55c3b2df81bbae83"}],"tempest/api/identity/admin/v3/test_domains.py":[{"author":{"_account_id":9152,"name":"Castulo J. Martinez","email":"castulo.martinez@intel.com","username":"cas"},"change_message_id":"694ca5866d28ca9eafe57d4f8aa1fc7912cc1ebe","unresolved":false,"context_lines":[{"line_number":152,"context_line":"        identity_diff \u003d (set(expected_options[\u0027identity\u0027]) -"},{"line_number":153,"context_line":"                         set(identity_options))"},{"line_number":154,"context_line":"        ldap_diff \u003d set(expected_options[\u0027ldap\u0027]) - set(ldap_options)"},{"line_number":155,"context_line":"        self.assertFalse(0, identity_diff)"},{"line_number":156,"context_line":"        self.assertFalse(0, ldap_diff)"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"class DefaultDomainTestJSON(base.BaseIdentityV3AdminTest):"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a5ae5dd_d12faaf4","line":156,"range":{"start_line":155,"start_character":8,"end_line":156,"end_character":38},"updated":"2016-02-09 17:25:36.000000000","message":"Why are you doing these assertions with a hardcoded 0? They will always pass this way.","commit_id":"a681644b3b5d72fd3619c06ea987873d362b7a42"},{"author":{"_account_id":18796,"name":"Alex Petrov","email":"apetrov@mirantis.com","username":"apetrov"},"change_message_id":"71a236095b031cf7fb8fac869d41815dead8c92b","unresolved":false,"context_lines":[{"line_number":152,"context_line":"        identity_diff \u003d (set(expected_options[\u0027identity\u0027]) -"},{"line_number":153,"context_line":"                         set(identity_options))"},{"line_number":154,"context_line":"        ldap_diff \u003d set(expected_options[\u0027ldap\u0027]) - set(ldap_options)"},{"line_number":155,"context_line":"        self.assertFalse(0, identity_diff)"},{"line_number":156,"context_line":"        self.assertFalse(0, ldap_diff)"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"class DefaultDomainTestJSON(base.BaseIdentityV3AdminTest):"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a57f1b5_21adb988","line":156,"range":{"start_line":155,"start_character":8,"end_line":156,"end_character":38},"in_reply_to":"5a5ae5dd_d12faaf4","updated":"2016-02-09 20:44:01.000000000","message":"Castulo, thanks for your review. I agree. \nI am going to do it this way: self.assertEmpty(identity_diff)\nself.assertEmpty(ldap_diff)\n\nThere is another problem with this test. it won\u0027t pass jenkins jobs because it is valid only for keystone version v3.5.","commit_id":"a681644b3b5d72fd3619c06ea987873d362b7a42"}]}