)]}'
{"ironicclient/tests/functional/osc/v1/test_baremetal_port_basic.py":[{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"21256bc3f1c16fcc6d1f698b61b953473e0dea26","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"        Test steps:"},{"line_number":35,"context_line":"        1) Create baremetal node in setUp."},{"line_number":36,"context_line":"        2) Create one more baremetal node explicitly."},{"line_number":37,"context_line":"        3) Check that node successfully created."},{"line_number":38,"context_line":"        \"\"\""},{"line_number":39,"context_line":"        address \u003d data_utils.rand_mac_address()"}],"source_content_type":"text/x-python","patch_set":7,"id":"1aa78d24_3db744f6","line":36,"range":{"start_line":36,"start_character":11,"end_line":36,"end_character":53},"updated":"2016-07-06 19:24:18.000000000","message":"s/Crea..../create port","commit_id":"65930b1d2736b33e2e07c72a26920e9cb5bfecd3"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"af782ef1450a0c62d2f51fb89daaedd570b5dda9","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        3) Check that node successfully created."},{"line_number":38,"context_line":"        \"\"\""},{"line_number":39,"context_line":"        address \u003d data_utils.rand_mac_address()"},{"line_number":40,"context_line":"        raw_output \u003d self.openstack(\u0027baremetal port create -f json \u0027"},{"line_number":41,"context_line":"                                    \u0027--node {0} {1}\u0027"},{"line_number":42,"context_line":"                                    .format(self.node[\u0027uuid\u0027], address))"},{"line_number":43,"context_line":"        port \u003d json.loads(raw_output)"},{"line_number":44,"context_line":"        # TODO(): check when \u0027baremetal port delete\u0027 command will be available"},{"line_number":45,"context_line":"        self.delete_port(port[\u0027uuid\u0027], True)"},{"line_number":46,"context_line":"        # TODO(): check when \u0027baremetal port list\u0027 command will be available"}],"source_content_type":"text/x-python","patch_set":7,"id":"1aa78d24_9d4d9840","line":43,"range":{"start_line":40,"start_character":8,"end_line":43,"end_character":37},"updated":"2016-07-06 19:23:04.000000000","message":"would be better to create port_create method as for node","commit_id":"65930b1d2736b33e2e07c72a26920e9cb5bfecd3"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"af782ef1450a0c62d2f51fb89daaedd570b5dda9","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        self.delete_port(port[\u0027uuid\u0027], True)"},{"line_number":46,"context_line":"        # TODO(): check when \u0027baremetal port list\u0027 command will be available"},{"line_number":47,"context_line":"        port_list \u003d self.list_ports()"},{"line_number":48,"context_line":"        self.assertIn(port[\u0027uuid\u0027], [x[\u0027UUID\u0027] for x in port_list])"}],"source_content_type":"text/x-python","patch_set":7,"id":"1aa78d24_5d5690ce","line":48,"range":{"start_line":48,"start_character":8,"end_line":48,"end_character":67},"updated":"2016-07-06 19:23:04.000000000","message":"You are asserting deleted port?","commit_id":"65930b1d2736b33e2e07c72a26920e9cb5bfecd3"}],"ironicclient/tests/functional/osc/v1/test_baremetal_port_create.py":[{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"1a2022bfc4cd4a01ec9c370402a9f900b8b6140f","unresolved":false,"context_lines":[{"line_number":25,"context_line":"        super(BaremetalPortCreate, self).setUp()"},{"line_number":26,"context_line":"        self.node \u003d self.node_create()"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    @staticmethod"},{"line_number":29,"context_line":"    def generate_params(argument, params):"},{"line_number":30,"context_line":"        # wait for merge it here https://review.openstack.org/#/c/262055/"},{"line_number":31,"context_line":"        params_str \u003d \u0027\u0027"},{"line_number":32,"context_line":"        for key, value in params.iteritems():"},{"line_number":33,"context_line":"            params_str +\u003d \u0027 {0} {1}\u003d{2}\u0027.format(argument, key, value)"},{"line_number":34,"context_line":"        return params_str"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    def assertEqualDict(self, params, object_params):"},{"line_number":37,"context_line":"        # # wait for merge it here https://review.openstack.org/#/c/262055/"}],"source_content_type":"text/x-python","patch_set":19,"id":"ff0f0b1f_7cea2a0f","line":34,"range":{"start_line":28,"start_character":0,"end_line":34,"end_character":25},"updated":"2017-05-23 13:14:52.000000000","message":"you should rebase on dependent patch instead of copy/paste code","commit_id":"3db3cc588b7c1349dbe420d8038ff7a8e493c223"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"5b2af1988c01c6233d0303f026fea06cf55a83aa","unresolved":false,"context_lines":[{"line_number":25,"context_line":"        super(BaremetalPortCreate, self).setUp()"},{"line_number":26,"context_line":"        self.node \u003d self.node_create()"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    @staticmethod"},{"line_number":29,"context_line":"    def generate_params(argument, params):"},{"line_number":30,"context_line":"        # wait for merge it here https://review.openstack.org/#/c/262055/"},{"line_number":31,"context_line":"        params_str \u003d \u0027\u0027"},{"line_number":32,"context_line":"        for key, value in params.iteritems():"},{"line_number":33,"context_line":"            params_str +\u003d \u0027 {0} {1}\u003d{2}\u0027.format(argument, key, value)"},{"line_number":34,"context_line":"        return params_str"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    def assertEqualDict(self, params, object_params):"},{"line_number":37,"context_line":"        # # wait for merge it here https://review.openstack.org/#/c/262055/"}],"source_content_type":"text/x-python","patch_set":19,"id":"3f1d235d_12b2d249","line":34,"range":{"start_line":28,"start_character":0,"end_line":34,"end_character":25},"in_reply_to":"ff0f0b1f_7cea2a0f","updated":"2017-07-06 11:57:38.000000000","message":"Moved this to base.","commit_id":"3db3cc588b7c1349dbe420d8038ff7a8e493c223"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"1a2022bfc4cd4a01ec9c370402a9f900b8b6140f","unresolved":false,"context_lines":[{"line_number":33,"context_line":"            params_str +\u003d \u0027 {0} {1}\u003d{2}\u0027.format(argument, key, value)"},{"line_number":34,"context_line":"        return params_str"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    def assertEqualDict(self, params, object_params):"},{"line_number":37,"context_line":"        # # wait for merge it here https://review.openstack.org/#/c/262055/"},{"line_number":38,"context_line":"        object_params \u003d ast.literal_eval(str(object_params))"},{"line_number":39,"context_line":"        self.assertEqual(params, object_params)"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    def test_extras(self):"},{"line_number":42,"context_line":"        \"\"\"Check baremetal port create command with extra data."}],"source_content_type":"text/x-python","patch_set":19,"id":"ff0f0b1f_3cf032fd","line":39,"range":{"start_line":36,"start_character":0,"end_line":39,"end_character":47},"updated":"2017-05-23 13:14:52.000000000","message":"ditto","commit_id":"3db3cc588b7c1349dbe420d8038ff7a8e493c223"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"5b2af1988c01c6233d0303f026fea06cf55a83aa","unresolved":false,"context_lines":[{"line_number":33,"context_line":"            params_str +\u003d \u0027 {0} {1}\u003d{2}\u0027.format(argument, key, value)"},{"line_number":34,"context_line":"        return params_str"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    def assertEqualDict(self, params, object_params):"},{"line_number":37,"context_line":"        # # wait for merge it here https://review.openstack.org/#/c/262055/"},{"line_number":38,"context_line":"        object_params \u003d ast.literal_eval(str(object_params))"},{"line_number":39,"context_line":"        self.assertEqual(params, object_params)"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    def test_extras(self):"},{"line_number":42,"context_line":"        \"\"\"Check baremetal port create command with extra data."}],"source_content_type":"text/x-python","patch_set":19,"id":"3f1d235d_5295ea85","line":39,"range":{"start_line":36,"start_character":0,"end_line":39,"end_character":47},"in_reply_to":"ff0f0b1f_3cf032fd","updated":"2017-07-06 11:57:38.000000000","message":"Let\u0027s use another method.","commit_id":"3db3cc588b7c1349dbe420d8038ff7a8e493c223"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"1a2022bfc4cd4a01ec9c370402a9f900b8b6140f","unresolved":false,"context_lines":[{"line_number":53,"context_line":"            port \u003d self.port_create(self.node[\u0027uuid\u0027], params\u003dparams)"},{"line_number":54,"context_line":"            self.assertEqualDict(extra, port[\u0027extra\u0027])"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    def test_pxe(self):"},{"line_number":57,"context_line":"        \"\"\"Check baremetal port create command with PXE option."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"        Test steps:"}],"source_content_type":"text/x-python","patch_set":19,"id":"ff0f0b1f_3c96f23f","line":56,"range":{"start_line":56,"start_character":8,"end_line":56,"end_character":16},"updated":"2017-05-23 13:14:52.000000000","message":"nit: it would be nice to mark version specific tests, at least specify version in name","commit_id":"3db3cc588b7c1349dbe420d8038ff7a8e493c223"},{"author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"change_message_id":"5b2af1988c01c6233d0303f026fea06cf55a83aa","unresolved":false,"context_lines":[{"line_number":53,"context_line":"            port \u003d self.port_create(self.node[\u0027uuid\u0027], params\u003dparams)"},{"line_number":54,"context_line":"            self.assertEqualDict(extra, port[\u0027extra\u0027])"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    def test_pxe(self):"},{"line_number":57,"context_line":"        \"\"\"Check baremetal port create command with PXE option."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"        Test steps:"}],"source_content_type":"text/x-python","patch_set":19,"id":"3f1d235d_128bf267","line":56,"range":{"start_line":56,"start_character":8,"end_line":56,"end_character":16},"in_reply_to":"ff0f0b1f_3c96f23f","updated":"2017-07-06 11:57:38.000000000","message":"Something like test_pxe_1_19 ?","commit_id":"3db3cc588b7c1349dbe420d8038ff7a8e493c223"}]}