)]}'
{"ironicclient/tests/unit/v1/test_node_shell.py":[{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"e5bfa92f3fd5ff988c341a935a1f27df7346b290","unresolved":false,"context_lines":[{"line_number":88,"context_line":"        client_mock \u003d mock.MagicMock()"},{"line_number":89,"context_line":"        args \u003d mock.MagicMock()"},{"line_number":90,"context_line":"        args.node \u003d \u0027node_uuid\u0027"},{"line_number":91,"context_line":"        args.op \u003d \u0027foo\u0027"},{"line_number":92,"context_line":"        args.attributes \u003d [[\u0027arg1\u003dval1\u0027, \u0027arg2\u003dval2\u0027]]"},{"line_number":93,"context_line":"        self.assertRaises(exceptions.CommandError,"},{"line_number":94,"context_line":"                          n_shell.do_node_update,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa1b9901_82a0f7a0","line":91,"updated":"2015-08-18 16:38:27.000000000","message":"If this test is added, maybe also add tests for commands like set-power-state, set-provision-state with wrong choices? Or maybe this should not be added as there is choices\u003d in decorator.","commit_id":"a6537372e4e83be0589678be6e1fdaa12b4a52c2"}],"ironicclient/tests/unit/v1/test_port_shell.py":[{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"e5bfa92f3fd5ff988c341a935a1f27df7346b290","unresolved":false,"context_lines":[{"line_number":243,"context_line":"            address\u003d\u0027address\u0027, node_uuid\u003d\u0027uuid\u0027, extra\u003d{\u0027key1\u0027: \u0027val1\u0027,"},{"line_number":244,"context_line":"                                                        \u0027key2\u0027: \u0027val2\u0027})"},{"line_number":245,"context_line":""},{"line_number":246,"context_line":"    def test_do_port_create_invalid_extr_fields_values(self):"},{"line_number":247,"context_line":"        client_mock \u003d mock.MagicMock()"},{"line_number":248,"context_line":"        args \u003d mock.MagicMock()"},{"line_number":249,"context_line":"        args.address \u003d \u0027address\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa1b9901_620923dd","line":246,"updated":"2015-08-18 16:38:27.000000000","message":"Should there be such test for node creation attributes?","commit_id":"a6537372e4e83be0589678be6e1fdaa12b4a52c2"}],"ironicclient/v1/node_shell.py":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"7f52edce1e5170f0ac22635ad36a16732096aadc","unresolved":false,"context_lines":[{"line_number":326,"context_line":"    \"\"\"List the ports associated with a node.\"\"\""},{"line_number":327,"context_line":"    if args.detail:"},{"line_number":328,"context_line":"        fields \u003d res_fields.PORT_DETAILED_RESOURCE.fields"},{"line_number":329,"context_line":"        field_labels \u003d res_fields.PORT_DETAILED_RESOURCE.labels"},{"line_number":330,"context_line":"    elif args.fields:"},{"line_number":331,"context_line":"        utils.check_for_invalid_fields("},{"line_number":332,"context_line":"            args.fields[0], res_fields.PORT_DETAILED_RESOURCE.fields)"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba15a1d1_fb58ae8c","line":329,"updated":"2015-09-21 09:09:55.000000000","message":"would be nice to mention this change in the commit message, as it fixes a bug","commit_id":"b2e64c133aa577ad45883ea6f27914e35e1a2935"}]}