)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"change_message_id":"1765d985e7b009e962cc20d08377b0bba793a81f","unresolved":false,"context_lines":[{"line_number":13,"context_line":"  to get those files first created."},{"line_number":14,"context_line":"- Fix error labels to now specify whether we are referring to"},{"line_number":15,"context_line":"  a sample or response when comparing response results"},{"line_number":16,"context_line":"- Eliminate the calling of _pretty_data twice in verify_something"},{"line_number":17,"context_line":"  which also cuts total execution time by .5 seconds"},{"line_number":18,"context_line":"- Create response template files which content is the same as"},{"line_number":19,"context_line":"  its response body instead of currently 0-length body."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAN3%2F%2F1s8%3D","line":16,"updated":"2013-03-09 23:56:36.000000000","message":"\"Eliminate the calling of _pretty_data twice in\" -\u003e \"Call _pretty_data just once in\"","commit_id":"5893cf5ac7fa84215ecc8d6159be171cdd186a74"},{"author":{"_account_id":4598,"name":"Giampaolo Lauria","email":"lauria@us.ibm.com","username":"lauria"},"change_message_id":"3ade05f9a71e627cab1ae5cd89cf9eed142ced75","unresolved":false,"context_lines":[{"line_number":13,"context_line":"  to get those files first created."},{"line_number":14,"context_line":"- Fix error labels to now specify whether we are referring to"},{"line_number":15,"context_line":"  a sample or response when comparing response results"},{"line_number":16,"context_line":"- Eliminate the calling of _pretty_data twice in verify_something"},{"line_number":17,"context_line":"  which also cuts total execution time by .5 seconds"},{"line_number":18,"context_line":"- Create response template files which content is the same as"},{"line_number":19,"context_line":"  its response body instead of currently 0-length body."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAN3%2F%2F1qo%3D","line":16,"in_reply_to":"AAAAN3%2F%2F1s8%3D","updated":"2013-03-10 01:25:18.000000000","message":"Done","commit_id":"5893cf5ac7fa84215ecc8d6159be171cdd186a74"}],"nova/tests/integrated/test_api_samples.py":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"3a05f162f268beaed139dc11885bcca444d2aef2","unresolved":false,"context_lines":[{"line_number":187,"context_line":"                raise NoMatch(_(\u0027Dictionary key mismatch:\\n\u0027"},{"line_number":188,"context_line":"                            \u0027Extra key(s) in template:\\n%(ex_delta)s\\n\u0027"},{"line_number":189,"context_line":"                            \u0027Extra key(s) in %(result_str)s:\\n%(res_delta)s\\n\u0027)"},{"line_number":190,"context_line":"                             % locals())"},{"line_number":191,"context_line":"            for key in ex_keys:"},{"line_number":192,"context_line":"                res \u003d self._compare_result(subs, expected[key], result[key],"},{"line_number":193,"context_line":"                                           result_str)"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAN3%2F%2F1bE%3D","line":190,"updated":"2013-03-10 14:50:06.000000000","message":"Alignment got messed up here.","commit_id":"ee1c083d5e0494a46e5443264659cf5b1d73058d"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"3a05f162f268beaed139dc11885bcca444d2aef2","unresolved":false,"context_lines":[{"line_number":203,"context_line":"                for i, ex_obj in enumerate(expected):"},{"line_number":204,"context_line":"                    try:"},{"line_number":205,"context_line":"                        matched_value \u003d self._compare_result(subs, ex_obj,"},{"line_number":206,"context_line":"                                                           res_obj, result_str)"},{"line_number":207,"context_line":"                        del expected[i]"},{"line_number":208,"context_line":"                        break"},{"line_number":209,"context_line":"                    except NoMatch:"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAN3%2F%2F1bU%3D","line":206,"updated":"2013-03-10 14:50:06.000000000","message":"Alignment got messed up here.","commit_id":"ee1c083d5e0494a46e5443264659cf5b1d73058d"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"3a05f162f268beaed139dc11885bcca444d2aef2","unresolved":false,"context_lines":[{"line_number":239,"context_line":"            if not match:"},{"line_number":240,"context_line":"                raise NoMatch(_(\u0027Values do not match:\\n\u0027"},{"line_number":241,"context_line":"                          \u0027Template: %(expected)s\\n%(result_str)s: %(result)s\u0027)"},{"line_number":242,"context_line":"                          % locals())"},{"line_number":243,"context_line":"            try:"},{"line_number":244,"context_line":"                matched_value \u003d match.group(\u0027id\u0027)"},{"line_number":245,"context_line":"            except IndexError:"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAN3%2F%2F1a4%3D","line":242,"updated":"2013-03-10 14:50:06.000000000","message":"Not sure if this alignment change was intentional or not, but it should probably be either visually-aligned, or eight spaces.","commit_id":"ee1c083d5e0494a46e5443264659cf5b1d73058d"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"3a05f162f268beaed139dc11885bcca444d2aef2","unresolved":false,"context_lines":[{"line_number":253,"context_line":"            if expected !\u003d result:"},{"line_number":254,"context_line":"                raise NoMatch(_(\u0027Values do not match:\\n\u0027"},{"line_number":255,"context_line":"                          \u0027Template: %(expected)s\\n%(result_str)s: %(result)s\u0027)"},{"line_number":256,"context_line":"                          % locals())"},{"line_number":257,"context_line":"        return matched_value"},{"line_number":258,"context_line":""},{"line_number":259,"context_line":"    def generalize_subs(self, subs, vanilla_regexes):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAN3%2F%2F1a0%3D","line":256,"updated":"2013-03-10 14:50:06.000000000","message":"Same here.","commit_id":"ee1c083d5e0494a46e5443264659cf5b1d73058d"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"3a05f162f268beaed139dc11885bcca444d2aef2","unresolved":false,"context_lines":[{"line_number":298,"context_line":"            subs[\u0027id\u0027] \u003d vanilla_regexes[\u0027id\u0027]"},{"line_number":299,"context_line":"            subs \u003d self.generalize_subs(subs, vanilla_regexes)"},{"line_number":300,"context_line":"            sample_data \u003d self._objectify(sample_data)"},{"line_number":301,"context_line":"            self._compare_result(subs, template_data, sample_data, \"Sample\")"},{"line_number":302,"context_line":"            return response_result"},{"line_number":303,"context_line":"        except NoMatch:"},{"line_number":304,"context_line":"            raise"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAN3%2F%2F1ak%3D","line":301,"updated":"2013-03-10 14:50:06.000000000","message":"Personally, I\u0027d rather see you pull the pretty_data() out to the top of verify_response and leave these two lines in the helper function instead of replicating them here twice.","commit_id":"ee1c083d5e0494a46e5443264659cf5b1d73058d"},{"author":{"_account_id":4598,"name":"Giampaolo Lauria","email":"lauria@us.ibm.com","username":"lauria"},"change_message_id":"a0bd7de09436ceb5659af6bf018fed6ddbd4a45e","unresolved":false,"context_lines":[{"line_number":298,"context_line":"            subs[\u0027id\u0027] \u003d vanilla_regexes[\u0027id\u0027]"},{"line_number":299,"context_line":"            subs \u003d self.generalize_subs(subs, vanilla_regexes)"},{"line_number":300,"context_line":"            sample_data \u003d self._objectify(sample_data)"},{"line_number":301,"context_line":"            self._compare_result(subs, template_data, sample_data, \"Sample\")"},{"line_number":302,"context_line":"            return response_result"},{"line_number":303,"context_line":"        except NoMatch:"},{"line_number":304,"context_line":"            raise"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAN3%2F%2F1Xw%3D","line":301,"in_reply_to":"AAAAN3%2F%2F1ak%3D","updated":"2013-03-10 15:52:38.000000000","message":"Hi Dan,\nMy biggest issue I am having w/ Python is not having a clue on how to align multi-line stmts. Would you provide me w/ some examples or a pointer to the rules I/we should follow? Thanks","commit_id":"ee1c083d5e0494a46e5443264659cf5b1d73058d"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"d97c34f418bfe7152805cdba8c2ccfb4b8693daa","unresolved":false,"context_lines":[{"line_number":241,"context_line":"            if not match:"},{"line_number":242,"context_line":"                raise NoMatch(_(\u0027Values do not match:\\n\u0027"},{"line_number":243,"context_line":"                        \u0027Template: %(expected)s\\n%(result_str)s: %(result)s\u0027)"},{"line_number":244,"context_line":"                        % locals())"},{"line_number":245,"context_line":"            try:"},{"line_number":246,"context_line":"                matched_value \u003d match.group(\u0027id\u0027)"},{"line_number":247,"context_line":"            except IndexError:"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAN3%2F%2F0dQ%3D","line":244,"updated":"2013-03-11 15:05:07.000000000","message":"These should be visually-aligned since the raise line doesn\u0027t end in (","commit_id":"833d9d81d74cc9cc850aec437e6bef8a0a2dbb50"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"d97c34f418bfe7152805cdba8c2ccfb4b8693daa","unresolved":false,"context_lines":[{"line_number":255,"context_line":"            if expected !\u003d result:"},{"line_number":256,"context_line":"                raise NoMatch(_(\u0027Values do not match:\\n\u0027"},{"line_number":257,"context_line":"                        \u0027Template: %(expected)s\\n%(result_str)s: %(result)s\u0027)"},{"line_number":258,"context_line":"                        % locals())"},{"line_number":259,"context_line":"        return matched_value"},{"line_number":260,"context_line":""},{"line_number":261,"context_line":"    def generalize_subs(self, subs, vanilla_regexes):"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAN3%2F%2F0dM%3D","line":258,"updated":"2013-03-11 15:05:07.000000000","message":"Ditto.","commit_id":"833d9d81d74cc9cc850aec437e6bef8a0a2dbb50"}]}