)]}'
{"ironicclient/tests/functional/test_ironicclient.py":[{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"857bbc3ac4abd202f7b28a9e54ebb37a6e8c0299","unresolved":false,"context_lines":[{"line_number":23,"context_line":"                                 \u0027Power State\u0027, \u0027Provisioning State\u0027,"},{"line_number":24,"context_line":"                                 \u0027Maintenance\u0027], list)"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    def test_node_create_get_delete(self):"},{"line_number":27,"context_line":"        node \u003d self.create_node()"},{"line_number":28,"context_line":"        got \u003d self.ironic(\u0027node-show\u0027, params\u003dnode[\u0027uuid\u0027])"},{"line_number":29,"context_line":"        expected_node \u003d self.get_dict_from_output(got)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a29b11f_28f033c9","side":"PARENT","line":26,"updated":"2015-10-23 15:15:25.000000000","message":"Please don\u0027t delete existing test, you can just create another test scope and move it.","commit_id":"0fdfc9d6198ecf442f8d9c5194042cdafe789ada"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"857bbc3ac4abd202f7b28a9e54ebb37a6e8c0299","unresolved":false,"context_lines":[{"line_number":29,"context_line":"            2) check table structure"},{"line_number":30,"context_line":"        \"\"\""},{"line_number":31,"context_line":"        shassis_list \u003d self.ironic(\u0027chassis-list\u0027)"},{"line_number":32,"context_line":"        self.assertTableHeaders([\u0027Description\u0027], shassis_list)"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    def test_node_list(self):"},{"line_number":35,"context_line":"        \"\"\"Test scenario:"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a29b11f_a8ae837f","line":32,"updated":"2015-10-23 15:15:25.000000000","message":"and \u0027UUID\u0027 ?","commit_id":"861bb48b5f3a806161a61c33d6bc5c8f3f13c03e"},{"author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"change_message_id":"786c01d8966d714fa4fce31949590cc90eb8155c","unresolved":false,"context_lines":[{"line_number":19,"context_line":"class SimpleReadonlyIronicClientTest(base.ClientTestBase):"},{"line_number":20,"context_line":"    \"\"\"Basic, read-only tests for Ironic CLI client."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    Basic smoke test for the Ironic CLI commands which do not require"},{"line_number":23,"context_line":"    creating or modifying ironic objects."},{"line_number":24,"context_line":"    \"\"\""},{"line_number":25,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3a29b11f_53ebb7b0","line":22,"updated":"2015-10-26 11:29:27.000000000","message":"Do you mean that this is a read-only test for ironic CLI commands, or that this is a test for read-only ironic CLI commands?","commit_id":"216b80bdf3eea2ee0e2d3895dfea31cfaa368d19"},{"author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"change_message_id":"786c01d8966d714fa4fce31949590cc90eb8155c","unresolved":false,"context_lines":[{"line_number":23,"context_line":"    creating or modifying ironic objects."},{"line_number":24,"context_line":"    \"\"\""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    def test_shassis_list(self):"},{"line_number":27,"context_line":"        \"\"\"Test scenario:"},{"line_number":28,"context_line":"            1) get shassis-list"},{"line_number":29,"context_line":"            2) check table structure"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a29b11f_b39ecb61","line":26,"updated":"2015-10-26 11:29:27.000000000","message":"Typo: s/shassis/chassis/g. Also, the test names aren\u0027t very explicit: \"test_chassis_list_table_structure\" (and so on) might be better.","commit_id":"216b80bdf3eea2ee0e2d3895dfea31cfaa368d19"},{"author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"change_message_id":"9036788ec6f1794e62b499fd8f4c122a2bb86211","unresolved":false,"context_lines":[{"line_number":23,"context_line":"    which do not require creating or modifying ironic objects."},{"line_number":24,"context_line":"    \"\"\""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    def test_shassis_list_table_structure(self):"},{"line_number":27,"context_line":"        \"\"\"Test scenario:"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"            1) get shassis-list"}],"source_content_type":"text/x-python","patch_set":6,"id":"3a29b11f_edd77f21","line":26,"updated":"2015-10-26 14:23:51.000000000","message":"Please replace \"shassis\" with \"chassis\" throughout. Sorry about English\u0027s habit of stealing words from other languages and keeping the original spelling :-(","commit_id":"38e78011efd0ce297626d00e6eaefd5d3abf87df"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"d4cf86db177ce923f384db9d25e3e4fa9a44d0d6","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    def test_port_list_table_structure(self):"},{"line_number":47,"context_line":"        \"\"\"Test scenario:"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"            1) get port-slist"},{"line_number":50,"context_line":"            2) check table structure"},{"line_number":51,"context_line":"        \"\"\""},{"line_number":52,"context_line":"        port_list \u003d self.ironic(\u0027port-list\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"1a26ad4f_12a0d49e","line":49,"updated":"2015-10-28 16:01:12.000000000","message":"nit: s/port-slist/port-list","commit_id":"76de605bd2d31bd19e18b566fb454b490b80916e"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"d4cf86db177ce923f384db9d25e3e4fa9a44d0d6","unresolved":false,"context_lines":[{"line_number":55,"context_line":"    def test_driver_list_table_structure(self):"},{"line_number":56,"context_line":"        \"\"\"Test scenario:"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"            1) get driver-slist"},{"line_number":59,"context_line":"            2) check table structure"},{"line_number":60,"context_line":"        \"\"\""},{"line_number":61,"context_line":"        driver_list \u003d self.ironic(\u0027driver-list\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"1a26ad4f_9282a4ff","line":58,"updated":"2015-10-28 16:01:12.000000000","message":"s/driver-slist/driver-list","commit_id":"76de605bd2d31bd19e18b566fb454b490b80916e"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"e87f9bda10b2e80d4799c8f490e01eddc51ce1b8","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright 2015 Hewlett-Packard Development Company, L.P."},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-python","patch_set":10,"id":"fa80f949_305ecf12","side":"PARENT","line":1,"updated":"2015-11-06 13:23:19.000000000","message":"Not sure that deleting it is good.","commit_id":"1ced89b9e99ddefbc1e77495f1a430f4e7de6e84"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"e87f9bda10b2e80d4799c8f490e01eddc51ce1b8","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"class TestIronicClient(base.FunctionalTestBase):"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"    def test_node_list(self):"},{"line_number":19,"context_line":"        list \u003d self.ironic(\u0027node-list\u0027)"},{"line_number":20,"context_line":"        self.assertTableHeaders([\u0027UUID\u0027, \u0027Name\u0027, \u0027Instance UUID\u0027,"},{"line_number":21,"context_line":"                                 \u0027Power State\u0027, \u0027Provisioning State\u0027,"}],"source_content_type":"text/x-python","patch_set":10,"id":"fa80f949_906c5bd1","line":18,"updated":"2015-11-06 13:23:19.000000000","message":"This is not needed anymore :)","commit_id":"16c042d61d3df7cb0842e3621de9676a0f2192c2"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"5b21cf3430780370255bc1b8e1d2e22015c331b2","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright 2015 Hewlett-Packard Development Company, L.P."},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":5,"context_line":"# not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":11,"id":"da85f559_e9a2fd4a","side":"PARENT","line":2,"updated":"2015-11-09 16:56:47.000000000","message":"I\u0027m not sure if removing copyrights is a good thing to do..","commit_id":"1ced89b9e99ddefbc1e77495f1a430f4e7de6e84"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"3ea7ca41cce87b2b239e030f2df1fdbf5db246e9","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright 2015 Hewlett-Packard Development Company, L.P."},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":5,"context_line":"# not use this file except in compliance with the License. You may obtain"},{"line_number":6,"context_line":"# a copy of the License at"}],"source_content_type":"text/x-python","patch_set":11,"id":"da85f559_896ed12d","side":"PARENT","line":3,"updated":"2015-11-09 17:14:11.000000000","message":"unrelated changes here, plus as you don\u0027t work for HP it feels a bit invasive to do it.","commit_id":"1ced89b9e99ddefbc1e77495f1a430f4e7de6e84"}],"ironicclient/tests/functional/test_table_structure.py":[{"author":{"_account_id":8592,"name":"Anastasia Kuznetsova","email":"akuznetsova@mirantis.com","username":"nastya-kuz"},"change_message_id":"a3db96ff8039ee8c43550ce93ff9736a4cb40c44","unresolved":false,"context_lines":[{"line_number":16,"context_line":"class TableStructureIronicCLITests(base.FunctionalTestBase):"},{"line_number":17,"context_line":"    \"\"\"Basic, read-only table structure tests for Ironic CLI commands."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"    Basic smoke test for the Ironic CLI commands to check table structure"},{"line_number":20,"context_line":"    which do not require creating or modifying ironic objects."},{"line_number":21,"context_line":"    \"\"\""},{"line_number":22,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"fa80f949_612567bf","line":19,"updated":"2015-11-06 14:37:13.000000000","message":"Soft: test - \u003e tests","commit_id":"1ad2790df7677d4ec1e8ed4583b6ddec39f2daad"},{"author":{"_account_id":17270,"name":"Sergii Turivnyi","email":"sturivnyi@mirantis.com","username":"sturivnyi"},"change_message_id":"36e5a46946ecd96d975f246f1447c9bc58608c48","unresolved":false,"context_lines":[{"line_number":16,"context_line":"class TableStructureIronicCLITests(base.FunctionalTestBase):"},{"line_number":17,"context_line":"    \"\"\"Basic, read-only table structure tests for Ironic CLI commands."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"    Basic smoke test for the Ironic CLI commands to check table structure"},{"line_number":20,"context_line":"    which do not require creating or modifying ironic objects."},{"line_number":21,"context_line":"    \"\"\""},{"line_number":22,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"fa80f949_bc337813","line":19,"in_reply_to":"fa80f949_612567bf","updated":"2015-11-06 14:46:18.000000000","message":"Done","commit_id":"1ad2790df7677d4ec1e8ed4583b6ddec39f2daad"},{"author":{"_account_id":8592,"name":"Anastasia Kuznetsova","email":"akuznetsova@mirantis.com","username":"nastya-kuz"},"change_message_id":"a3db96ff8039ee8c43550ce93ff9736a4cb40c44","unresolved":false,"context_lines":[{"line_number":17,"context_line":"    \"\"\"Basic, read-only table structure tests for Ironic CLI commands."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"    Basic smoke test for the Ironic CLI commands to check table structure"},{"line_number":20,"context_line":"    which do not require creating or modifying ironic objects."},{"line_number":21,"context_line":"    \"\"\""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    def test_chassis_list_table_structure(self):"}],"source_content_type":"text/x-python","patch_set":11,"id":"fa80f949_c11a137d","line":20,"updated":"2015-11-06 14:37:13.000000000","message":"Soft: ironic - \u003e Ironic","commit_id":"1ad2790df7677d4ec1e8ed4583b6ddec39f2daad"},{"author":{"_account_id":17270,"name":"Sergii Turivnyi","email":"sturivnyi@mirantis.com","username":"sturivnyi"},"change_message_id":"36e5a46946ecd96d975f246f1447c9bc58608c48","unresolved":false,"context_lines":[{"line_number":17,"context_line":"    \"\"\"Basic, read-only table structure tests for Ironic CLI commands."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"    Basic smoke test for the Ironic CLI commands to check table structure"},{"line_number":20,"context_line":"    which do not require creating or modifying ironic objects."},{"line_number":21,"context_line":"    \"\"\""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    def test_chassis_list_table_structure(self):"}],"source_content_type":"text/x-python","patch_set":11,"id":"fa80f949_7c3990f1","line":20,"in_reply_to":"fa80f949_c11a137d","updated":"2015-11-06 14:46:18.000000000","message":"Done","commit_id":"1ad2790df7677d4ec1e8ed4583b6ddec39f2daad"}],"ironicclient/tests/functional/test_table_structure_ironicclient.py":[{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"e87f9bda10b2e80d4799c8f490e01eddc51ce1b8","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":2,"context_line":"# not use this file except in compliance with the License. You may obtain"},{"line_number":3,"context_line":"# a copy of the License at"},{"line_number":4,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":10,"id":"fa80f949_907a1b76","line":1,"updated":"2015-11-06 13:23:19.000000000","message":"I suppose ironicclient is redundant in file name, why not just test_table_structure?","commit_id":"16c042d61d3df7cb0842e3621de9676a0f2192c2"}]}