)]}'
{"tacker/tests/functional/vnfpkgm/test_vnf_package.py":[{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"5a4c86b2db647d9d4e843451a7726df60a1d31ca","unresolved":false,"context_lines":[{"line_number":115,"context_line":"                                    \u0027../../etc/samples/\u0027 + file_name))"},{"line_number":116,"context_line":"        return file_path"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"    def test_upload_from_file_update_show_and_delete(self):"},{"line_number":119,"context_line":"        user_data \u003d jsonutils.dumps("},{"line_number":120,"context_line":"            {\"userDefinedData\": {\"key1\": \"val1\", \"key2\": \"val2\","},{"line_number":121,"context_line":"                                 \"key3\": \"val3\"}})"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f493fa4_5d786ef7","line":118,"updated":"2020-04-21 04:12:24.000000000","message":"I understand the need for including existing test_patch_in_onboarded_state(), but the name of this method is not clear for the behavior as the result. Could you add a comment for the method to explain the purpose and behavior simply?","commit_id":"2622c48e907848fd44da5645f6471a5087c24a39"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"1e560a752a3cbf9cb84577cf9d6fcf2c621e1902","unresolved":false,"context_lines":[{"line_number":115,"context_line":"                                    \u0027../../etc/samples/\u0027 + file_name))"},{"line_number":116,"context_line":"        return file_path"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"    def test_upload_from_file_update_show_and_delete(self):"},{"line_number":119,"context_line":"        user_data \u003d jsonutils.dumps("},{"line_number":120,"context_line":"            {\"userDefinedData\": {\"key1\": \"val1\", \"key2\": \"val2\","},{"line_number":121,"context_line":"                                 \"key3\": \"val3\"}})"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f493fa4_3303e0ba","line":118,"in_reply_to":"1f493fa4_5d786ef7","updated":"2020-04-23 04:51:45.000000000","message":"Done","commit_id":"2622c48e907848fd44da5645f6471a5087c24a39"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"f1f369425c804fe9c53e7d06dc49a976405f71d3","unresolved":false,"context_lines":[{"line_number":124,"context_line":"        file_path \u003d self._get_csar_file_path(\"sample_vnf_package_csar.zip\")"},{"line_number":125,"context_line":"        with open(file_path, \u0027rb\u0027) as file_object:"},{"line_number":126,"context_line":"            resp, resp_body \u003d self.http_client.do_request("},{"line_number":127,"context_line":"                \u0027{base_path}/{id}/package_content\u0027.format("},{"line_number":128,"context_line":"                    id\u003dvnf_package[\u0027id\u0027],"},{"line_number":129,"context_line":"                    base_path\u003dself.base_url),"},{"line_number":130,"context_line":"                \"PUT\", body\u003dfile_object, content_type\u003d\u0027application/zip\u0027)"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"        self.assertEqual(202, resp.status_code)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f493fa4_532fbff5","line":129,"range":{"start_line":127,"start_character":16,"end_line":129,"end_character":44},"updated":"2020-04-21 07:17:46.000000000","message":"I think it\u0027s better to define the url variable like show_url.","commit_id":"2622c48e907848fd44da5645f6471a5087c24a39"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"1e560a752a3cbf9cb84577cf9d6fcf2c621e1902","unresolved":false,"context_lines":[{"line_number":124,"context_line":"        file_path \u003d self._get_csar_file_path(\"sample_vnf_package_csar.zip\")"},{"line_number":125,"context_line":"        with open(file_path, \u0027rb\u0027) as file_object:"},{"line_number":126,"context_line":"            resp, resp_body \u003d self.http_client.do_request("},{"line_number":127,"context_line":"                \u0027{base_path}/{id}/package_content\u0027.format("},{"line_number":128,"context_line":"                    id\u003dvnf_package[\u0027id\u0027],"},{"line_number":129,"context_line":"                    base_path\u003dself.base_url),"},{"line_number":130,"context_line":"                \"PUT\", body\u003dfile_object, content_type\u003d\u0027application/zip\u0027)"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"        self.assertEqual(202, resp.status_code)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f493fa4_13fea4ad","line":129,"range":{"start_line":127,"start_character":16,"end_line":129,"end_character":44},"in_reply_to":"1f493fa4_532fbff5","updated":"2020-04-23 04:51:45.000000000","message":"Done","commit_id":"2622c48e907848fd44da5645f6471a5087c24a39"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"f1f369425c804fe9c53e7d06dc49a976405f71d3","unresolved":false,"context_lines":[{"line_number":149,"context_line":"            \"PATCH\", content_type\u003d\u0027application/json\u0027, body\u003dupdate_req_body)"},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"        self.assertEqual(200, resp.status_code)"},{"line_number":152,"context_line":"        self.assertEqual(expected_result, resp_body)"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"        # Show vnf package and verify whether the userDefinedData"},{"line_number":155,"context_line":"        # and operationalState parameters are updated correctly."}],"source_content_type":"text/x-python","patch_set":1,"id":"1f493fa4_73b75bda","line":152,"range":{"start_line":152,"start_character":8,"end_line":152,"end_character":52},"updated":"2020-04-21 07:17:46.000000000","message":"I think it\u0027s better to assert \u0027operationalState\u0027 and \u0027userDefinedData\u0027 separately like other codes.","commit_id":"2622c48e907848fd44da5645f6471a5087c24a39"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"1e560a752a3cbf9cb84577cf9d6fcf2c621e1902","unresolved":false,"context_lines":[{"line_number":149,"context_line":"            \"PATCH\", content_type\u003d\u0027application/json\u0027, body\u003dupdate_req_body)"},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"        self.assertEqual(200, resp.status_code)"},{"line_number":152,"context_line":"        self.assertEqual(expected_result, resp_body)"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"        # Show vnf package and verify whether the userDefinedData"},{"line_number":155,"context_line":"        # and operationalState parameters are updated correctly."}],"source_content_type":"text/x-python","patch_set":1,"id":"1f493fa4_1369848a","line":152,"range":{"start_line":152,"start_character":8,"end_line":152,"end_character":52},"in_reply_to":"1f493fa4_73b75bda","updated":"2020-04-23 04:51:45.000000000","message":"IMO, it\u0027s one and the same.","commit_id":"2622c48e907848fd44da5645f6471a5087c24a39"}]}