)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"3593db7f7f461ddf7e6d7acb148db4f914a8eba1","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Negative test for the Ironic CLI commands which checks  actions with"},{"line_number":10,"context_line":"node command like use commands without arguments or with"},{"line_number":11,"context_line":"incorrect arguments and check that correct error message raised."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Closes Bug: #1616972"},{"line_number":14,"context_line":"Change-Id: Ie8f252aa6b9719566cc8d08b9a3bbe22e6667dfc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa7ab95a_0c47d41d","line":12,"updated":"2016-09-01 12:10:11.000000000","message":"Same grammar nits as in docstring.","commit_id":"56d81d836dad67d1b8624726a51f85d764657cf4"},{"author":{"_account_id":15784,"name":"Igor Pukha","email":"ipukha@mirantis.com","username":"ingwarr"},"change_message_id":"05c76b78daf1890374b3fc560db25311de774e93","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Negative test for the Ironic CLI commands which checks  actions with"},{"line_number":10,"context_line":"node command like use commands without arguments or with"},{"line_number":11,"context_line":"incorrect arguments and check that correct error message raised."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Closes Bug: #1616972"},{"line_number":14,"context_line":"Change-Id: Ie8f252aa6b9719566cc8d08b9a3bbe22e6667dfc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa7ab95a_189b8b73","line":12,"in_reply_to":"fa7ab95a_0c47d41d","updated":"2016-09-01 13:38:59.000000000","message":"Yep, better to change database-like style to human-readable","commit_id":"56d81d836dad67d1b8624726a51f85d764657cf4"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"a0b3d7f5f00df4f21cffc97429547635ddc2b187","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Negative tests for testing actions with node-port-list."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Negative test for the Ironic CLI commands which checks  actions with"},{"line_number":10,"context_line":"node command like use commands without arguments or with"},{"line_number":11,"context_line":"incorrect arguments and check that correct error message raised."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Closes Bug: #1616972"},{"line_number":14,"context_line":"Change-Id: Ie8f252aa6b9719566cc8d08b9a3bbe22e6667dfc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"5a3905b3_b1b86f2b","line":11,"range":{"start_line":9,"start_character":0,"end_line":11,"end_character":64},"updated":"2017-01-24 12:28:20.000000000","message":"Let\u0027s rewrite it for easier readability :)","commit_id":"f7d8f8eadc55ea3e9b304936ba4bd78650167dec"}],"ironicclient/tests/functional/test_node_negative.py":[{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"86bbae7cdca44be9bf221858ef1c9c98410c2587","unresolved":false,"context_lines":[{"line_number":12,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":13,"context_line":"# under the License."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"import ddt"},{"line_number":16,"context_line":"import six"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from tempest.lib import exceptions"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from ironicclient.tests.functional import base"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa7ab95a_dbbcaf08","line":18,"range":{"start_line":15,"start_character":0,"end_line":18,"end_character":34},"updated":"2016-08-29 11:09:41.000000000","message":"These should be the same group of imports, please remove the blank line.","commit_id":"8c6f0364b0eec64801c36c96348bfe57846c489e"},{"author":{"_account_id":17270,"name":"Sergii Turivnyi","email":"sturivnyi@mirantis.com","username":"sturivnyi"},"change_message_id":"30ffd052b76a3e62408c085698b717454e41da48","unresolved":false,"context_lines":[{"line_number":12,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":13,"context_line":"# under the License."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"import ddt"},{"line_number":16,"context_line":"import six"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from tempest.lib import exceptions"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from ironicclient.tests.functional import base"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa7ab95a_b2359ef8","line":18,"range":{"start_line":15,"start_character":0,"end_line":18,"end_character":34},"in_reply_to":"fa7ab95a_dbbcaf08","updated":"2016-08-31 15:30:11.000000000","message":"Done","commit_id":"8c6f0364b0eec64801c36c96348bfe57846c489e"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"86bbae7cdca44be9bf221858ef1c9c98410c2587","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"@ddt.ddt"},{"line_number":24,"context_line":"class NodePortListNegativeTests(base.FunctionalTestBase):"},{"line_number":25,"context_line":"    \"\"\"Negative tests for testing actions with node-port-list."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    Negative test for the Ironic CLI commands which checks  actions with"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa7ab95a_3b383bef","line":24,"updated":"2016-08-29 11:09:41.000000000","message":"I\u0027m not sure I get what\u0027s the intent of this patch. All the tests here doing the following:\n\n ironic node-port-list \u003cno node identifier\u003e --\u003csome flag\u003e \u003csome value\u003e\n\nIf you do not pass the node identifier the requests will obviously fail. Why do we need to check all the possible flags for that? I think this will explode the amount of tests we have with no obvious value. I\u0027d rather test passing incorrect values to flags with using some existing node uuid.","commit_id":"8c6f0364b0eec64801c36c96348bfe57846c489e"},{"author":{"_account_id":17270,"name":"Sergii Turivnyi","email":"sturivnyi@mirantis.com","username":"sturivnyi"},"change_message_id":"30ffd052b76a3e62408c085698b717454e41da48","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"@ddt.ddt"},{"line_number":24,"context_line":"class NodePortListNegativeTests(base.FunctionalTestBase):"},{"line_number":25,"context_line":"    \"\"\"Negative tests for testing actions with node-port-list."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    Negative test for the Ironic CLI commands which checks  actions with"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa7ab95a_32226eba","line":24,"in_reply_to":"fa7ab95a_3b383bef","updated":"2016-08-31 15:30:11.000000000","message":"Done","commit_id":"8c6f0364b0eec64801c36c96348bfe57846c489e"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"86bbae7cdca44be9bf221858ef1c9c98410c2587","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    def test_node_port_list_detail(self, value, ex_text):"},{"line_number":49,"context_line":"        six.assertRaisesRegex("},{"line_number":50,"context_line":"            self, exceptions.CommandFailed, ex_text, self.ironic,"},{"line_number":51,"context_line":"            \u0027node-port-list\u0027, params\u003d\u0027--detail {0}\u0027.format(value))"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    @ddt.data("},{"line_number":54,"context_line":"        (\u0027\u0027, (r\u0027expected one argument\u0027)),"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa7ab95a_db910ff3","line":51,"range":{"start_line":51,"start_character":47,"end_line":51,"end_character":50},"updated":"2016-08-29 11:09:41.000000000","message":"detail flag does not accept any value, so {0} here is considered a node identifier.","commit_id":"8c6f0364b0eec64801c36c96348bfe57846c489e"},{"author":{"_account_id":17270,"name":"Sergii Turivnyi","email":"sturivnyi@mirantis.com","username":"sturivnyi"},"change_message_id":"30ffd052b76a3e62408c085698b717454e41da48","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    def test_node_port_list_detail(self, value, ex_text):"},{"line_number":49,"context_line":"        six.assertRaisesRegex("},{"line_number":50,"context_line":"            self, exceptions.CommandFailed, ex_text, self.ironic,"},{"line_number":51,"context_line":"            \u0027node-port-list\u0027, params\u003d\u0027--detail {0}\u0027.format(value))"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    @ddt.data("},{"line_number":54,"context_line":"        (\u0027\u0027, (r\u0027expected one argument\u0027)),"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa7ab95a_9217ba53","line":51,"range":{"start_line":51,"start_character":47,"end_line":51,"end_character":50},"in_reply_to":"fa7ab95a_db910ff3","updated":"2016-08-31 15:30:11.000000000","message":"Done","commit_id":"8c6f0364b0eec64801c36c96348bfe57846c489e"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"3593db7f7f461ddf7e6d7acb148db4f914a8eba1","unresolved":false,"context_lines":[{"line_number":24,"context_line":"class NodePortListNegativeTests(base.FunctionalTestBase):"},{"line_number":25,"context_line":"    \"\"\"Negative tests for testing actions with node-port-list."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    Negative test for the Ironic CLI commands which checks  actions with"},{"line_number":28,"context_line":"    node command like use commands without arguments or with"},{"line_number":29,"context_line":"    incorrect arguments and check that correct error message raised."},{"line_number":30,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_aca50805","line":27,"range":{"start_line":27,"start_character":52,"end_line":27,"end_character":58},"updated":"2016-09-01 12:10:11.000000000","message":"check","commit_id":"56d81d836dad67d1b8624726a51f85d764657cf4"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"3593db7f7f461ddf7e6d7acb148db4f914a8eba1","unresolved":false,"context_lines":[{"line_number":24,"context_line":"class NodePortListNegativeTests(base.FunctionalTestBase):"},{"line_number":25,"context_line":"    \"\"\"Negative tests for testing actions with node-port-list."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    Negative test for the Ironic CLI commands which checks  actions with"},{"line_number":28,"context_line":"    node command like use commands without arguments or with"},{"line_number":29,"context_line":"    incorrect arguments and check that correct error message raised."},{"line_number":30,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_0c6434e3","line":27,"range":{"start_line":27,"start_character":13,"end_line":27,"end_character":17},"updated":"2016-09-01 12:10:11.000000000","message":"tests","commit_id":"56d81d836dad67d1b8624726a51f85d764657cf4"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"3593db7f7f461ddf7e6d7acb148db4f914a8eba1","unresolved":false,"context_lines":[{"line_number":24,"context_line":"class NodePortListNegativeTests(base.FunctionalTestBase):"},{"line_number":25,"context_line":"    \"\"\"Negative tests for testing actions with node-port-list."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    Negative test for the Ironic CLI commands which checks  actions with"},{"line_number":28,"context_line":"    node command like use commands without arguments or with"},{"line_number":29,"context_line":"    incorrect arguments and check that correct error message raised."},{"line_number":30,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_2c91f899","line":27,"range":{"start_line":27,"start_character":59,"end_line":27,"end_character":60},"updated":"2016-09-01 12:10:11.000000000","message":"too many spaces :)","commit_id":"56d81d836dad67d1b8624726a51f85d764657cf4"}]}