)]}'
{"pkg/remote/redfish/redfish_test.go":[{"author":{"_account_id":30797,"name":"kanwar saad bin liaqat","email":"kanwar.sbl@gmail.com","username":"ekanwli"},"change_message_id":"76e70b0318fe6aece58a2206df51b8c6d02e1171","unresolved":false,"context_lines":[{"line_number":58,"context_line":"\t\tdefaultURL,"},{"line_number":59,"context_line":"\t\tcomputerSystemID,"},{"line_number":60,"context_line":"\t\t\"/tmp/test.iso\","},{"line_number":61,"context_line":"\t\t\"On\","},{"line_number":62,"context_line":"\t)"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"\tassert.NoError(t, err)"}],"source_content_type":"text/x-go","patch_set":7,"id":"3fa7e38b_4f7b897a","line":61,"range":{"start_line":61,"start_character":2,"end_line":61,"end_character":6},"updated":"2019-11-11 22:47:55.000000000","message":"you are setting resettype to \"On\" which means current reset code will not be executed and your code will be executed. which will fail the current tests because current tests are expecting to call the current reset function. Here you have to set this to empty string \"\" because that\u0027s how you decide in the code which code portion to execute. then create your own additional tests to test your new functionality.","commit_id":"a4944d3141766a61050c7c4acb294c302d83bd21"}],"pkg/remote/redfish/utils.go":[{"author":{"_account_id":30797,"name":"kanwar saad bin liaqat","email":"kanwar.sbl@gmail.com","username":"ekanwli"},"change_message_id":"76e70b0318fe6aece58a2206df51b8c6d02e1171","unresolved":false,"context_lines":[{"line_number":129,"context_line":"func IsValidResetActionType(actionType string) bool {"},{"line_number":130,"context_line":"\tswitch actionType {"},{"line_number":131,"context_line":"\tcase"},{"line_number":132,"context_line":"\t\t\"On\","},{"line_number":133,"context_line":"\t\t\"ForceOff\","},{"line_number":134,"context_line":"\t\t\"GracefulShutdown\","},{"line_number":135,"context_line":"\t\t\"GracefulRestart\","},{"line_number":136,"context_line":"\t\t\"ForceRestart\","},{"line_number":137,"context_line":"\t\t\"Nmi\","},{"line_number":138,"context_line":"\t\t\"ForceOn\":"},{"line_number":139,"context_line":"\t\treturn true"},{"line_number":140,"context_line":"\t}"},{"line_number":141,"context_line":"\treturn false"}],"source_content_type":"text/x-go","patch_set":7,"id":"3fa7e38b_af76bda2","line":138,"range":{"start_line":132,"start_character":1,"end_line":138,"end_character":12},"updated":"2019-11-11 22:47:55.000000000","message":"you dont\u0027 need to define these string constants again. they are already defined in redfish library.\n\ncheck redfishClient.RESETTYPE_*","commit_id":"a4944d3141766a61050c7c4acb294c302d83bd21"}]}