)]}'
{"ironic/tests/unit/api/test_acl.py":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"936068aa80827c144f4036a2852bd9e68a8d0acf","unresolved":false,"context_lines":[{"line_number":63,"context_line":"        else:"},{"line_number":64,"context_line":"            assert False, \u0027Unimplemented test method: %s\u0027 % method"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"        if assert_status:"},{"line_number":67,"context_line":"            self.assertEqual(assert_status, response.status_int)"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"        if assert_dict_contains:"}],"source_content_type":"text/x-python","patch_set":1,"id":"2571652d_b2cec380","line":66,"updated":"2021-01-11 17:19:54.000000000","message":"I feel like assert_status should be mandatory","commit_id":"5ee7185f91eca0f8699e694321ef97677bb5c53d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"936068aa80827c144f4036a2852bd9e68a8d0acf","unresolved":false,"context_lines":[{"line_number":66,"context_line":"        if assert_status:"},{"line_number":67,"context_line":"            self.assertEqual(assert_status, response.status_int)"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"        if assert_dict_contains:"},{"line_number":70,"context_line":"            for k, v in assert_dict_contains.items():"},{"line_number":71,"context_line":"                self.assertIn(k, response)"},{"line_number":72,"context_line":"                self.assertEqual(v.format(**self.format_data), response[k])"}],"source_content_type":"text/x-python","patch_set":1,"id":"80687311_fea5f365","line":69,"updated":"2021-01-11 17:19:54.000000000","message":"nit: I\u0027d call this somehow more specific, like assert_response_contains","commit_id":"5ee7185f91eca0f8699e694321ef97677bb5c53d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"936068aa80827c144f4036a2852bd9e68a8d0acf","unresolved":false,"context_lines":[{"line_number":77,"context_line":""},{"line_number":78,"context_line":"    def _create_test_data(self):"},{"line_number":79,"context_line":"        fake_db_node \u003d db_utils.create_test_node(chassis_id\u003dNone)"},{"line_number":80,"context_line":"        self.format_data[\u0027node_uuid\u0027] \u003d fake_db_node[\u0027uuid\u0027]"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    @ddt.file_data(\u0027test_acl_basic.yaml\u0027)"},{"line_number":83,"context_line":"    @ddt.unpack"}],"source_content_type":"text/x-python","patch_set":1,"id":"95c09dd4_601d0940","line":80,"updated":"2021-01-11 17:19:54.000000000","message":"I suspect this method will be common for most tests","commit_id":"5ee7185f91eca0f8699e694321ef97677bb5c53d"}],"ironic/tests/unit/api/test_acl_basic.yaml":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"936068aa80827c144f4036a2852bd9e68a8d0acf","unresolved":false,"context_lines":[{"line_number":2,"context_line":"  path: \u0026node_path \u0027/v1/nodes/{node_uuid}\u0027"},{"line_number":3,"context_line":"  assert_status: 401"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"authenticated:"},{"line_number":6,"context_line":"  path: *node_path"},{"line_number":7,"context_line":"  headers:"},{"line_number":8,"context_line":"    X-Auth-Token: \u0026admin_token \u00274562138218392831\u0027"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"6a511f38_bc30d062","line":5,"updated":"2021-01-11 17:19:54.000000000","message":"nit: assert_status: 200","commit_id":"5ee7185f91eca0f8699e694321ef97677bb5c53d"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"936068aa80827c144f4036a2852bd9e68a8d0acf","unresolved":false,"context_lines":[{"line_number":38,"context_line":"public_api_v1_json:"},{"line_number":39,"context_line":"  path: /v1.json"},{"line_number":40,"context_line":"  assert_status: 200"},{"line_number":41,"context_line":"  "},{"line_number":42,"context_line":"public_api_v1_xml:"},{"line_number":43,"context_line":"  path: /v1.xml"},{"line_number":44,"context_line":"  assert_status: 404"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"da6c78c7_84bf019f","line":41,"updated":"2021-01-11 17:19:54.000000000","message":"++","commit_id":"5ee7185f91eca0f8699e694321ef97677bb5c53d"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"16c7d6aa101d1c0c850764738788e04d8a52889a","unresolved":true,"context_lines":[{"line_number":38,"context_line":"public_api_v1_json:"},{"line_number":39,"context_line":"  path: /v1.json"},{"line_number":40,"context_line":"  assert_status: 200"},{"line_number":41,"context_line":"  "},{"line_number":42,"context_line":"public_api_v1_xml:"},{"line_number":43,"context_line":"  path: /v1.xml"},{"line_number":44,"context_line":"  assert_status: 404"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5396bec7_2147971f","line":41,"updated":"2021-01-08 22:23:40.000000000","message":"Extra whitespace?","commit_id":"5ee7185f91eca0f8699e694321ef97677bb5c53d"}]}