)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9370935fa3e7138bd7591c4138882e9fd5b4aa97","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Manik Bindlish \u003cmanikbindlish19@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-08-09 05:54:11 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Removing duplicate test from \"test_verify_tempest_config.py\""},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There is a duplicate test: \"test_verify_glance_version_no_v2_with_v1_1\""},{"line_number":10,"context_line":"It is checking same thing as"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3f79a3b5_e3f4c0d1","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":60},"updated":"2018-08-09 06:05:24.000000000","message":"this is not true now, you are fixing the tests not removing.","commit_id":"01fd15d384f0b3aed03a348df97369976b75abfe"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9370935fa3e7138bd7591c4138882e9fd5b4aa97","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2018-08-09 05:54:11 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Removing duplicate test from \"test_verify_tempest_config.py\""},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There is a duplicate test: \"test_verify_glance_version_no_v2_with_v1_1\""},{"line_number":10,"context_line":"It is checking same thing as"},{"line_number":11,"context_line":"in \"test_verify_glance_version_no_v2_with_v1_0\""},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"So this patch set will"},{"line_number":14,"context_line":"correct the \"test_verify_glance_version_no_v2_with_v1_1\" which was"},{"line_number":15,"context_line":"mistakenly became duplicate."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: Ic7a9454ed61216311de731b43058357e0dbae337"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3f79a3b5_83038cb8","line":15,"range":{"start_line":8,"start_character":0,"end_line":15,"end_character":28},"updated":"2018-08-09 06:05:24.000000000","message":"ditto","commit_id":"01fd15d384f0b3aed03a348df97369976b75abfe"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9370935fa3e7138bd7591c4138882e9fd5b4aa97","unresolved":false,"context_lines":[{"line_number":13,"context_line":"So this patch set will"},{"line_number":14,"context_line":"correct the \"test_verify_glance_version_no_v2_with_v1_1\" which was"},{"line_number":15,"context_line":"mistakenly became duplicate."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: Ic7a9454ed61216311de731b43058357e0dbae337"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3f79a3b5_6332f0a9","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":0},"updated":"2018-08-09 06:05:24.000000000","message":"add below patch link also for history and easy to review.\n\nBelow patch made this test testing 1.0 instead of 1.1\n- https://review.openstack.org/#/c/486165/7\\\n\nalso add BP link for CLI testing, even that is new tests or fixing old tests - Partially-Implements: blueprint tempest-cli-unit-test-coverage","commit_id":"01fd15d384f0b3aed03a348df97369976b75abfe"}],"tempest/tests/cmd/test_verify_tempest_config.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"2a280d5b8296f1bdad8ce50b6604f467af1d4398","unresolved":false,"context_lines":[{"line_number":225,"context_line":"        # This test verifies that wrong config api_v2 \u003d True is detected"},{"line_number":226,"context_line":"        class FakeClient(object):"},{"line_number":227,"context_line":"            def get_versions(self):"},{"line_number":228,"context_line":"                return (None, [\u0027v1.0\u0027])"},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"        fake_os \u003d mock.MagicMock()"},{"line_number":231,"context_line":"        fake_module \u003d mock.MagicMock()"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_0e256ac2","side":"PARENT","line":228,"range":{"start_line":228,"start_character":16,"end_line":228,"end_character":39},"updated":"2018-08-08 22:25:17.000000000","message":"It is better to change here to\n\n return (None, [\u0027v1.1\u0027])\n\ninstead of removing.\nThe commit I65e2a685f683c0a3f141cb5349abc6065143bcdc made these tests same accidentally. So it is good to change the test for the original purpose (v1.1).","commit_id":"0a13ede274b1152983d8b094de922e73abbb9079"},{"author":{"_account_id":27078,"name":"Manik Bindlish","email":"manikbindlish19@gmail.com","username":"manik"},"change_message_id":"63b2c99d787396e0440e534e9821107d3fcaf601","unresolved":false,"context_lines":[{"line_number":225,"context_line":"        # This test verifies that wrong config api_v2 \u003d True is detected"},{"line_number":226,"context_line":"        class FakeClient(object):"},{"line_number":227,"context_line":"            def get_versions(self):"},{"line_number":228,"context_line":"                return (None, [\u0027v1.0\u0027])"},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"        fake_os \u003d mock.MagicMock()"},{"line_number":231,"context_line":"        fake_module \u003d mock.MagicMock()"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_e046f621","side":"PARENT","line":228,"range":{"start_line":228,"start_character":16,"end_line":228,"end_character":39},"in_reply_to":"3f79a3b5_0e256ac2","updated":"2018-08-09 04:47:56.000000000","message":"I have fixed for now and added \u0027v1.1\u0027 instead of \u0027v1.0\u0027\nBut cann\u0027t we make both test cases a single test case by using:\nreturn (None, [\u0027v1.x\u0027])","commit_id":"0a13ede274b1152983d8b094de922e73abbb9079"}]}