Skip to content

Commit ea62fd6

Browse files
committed
PR comments review
1 parent a458146 commit ea62fd6

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

commands/printcase.go

+5-5
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ func PrintTestCaseOutput(testCase *TestCaseIO) {
7373

7474
func PrintCaseAction(context *cli.Context) error {
7575
if context.Bool("input-only") && context.Bool("output-only") {
76-
color.Red("--input-only and --output-only cannot be exiting both.")
76+
color.Red("only --input-only or --output-only should be set.")
7777
return errors.New("Invalid commands arguments")
7878
}
7979

@@ -85,7 +85,7 @@ func PrintCaseAction(context *cli.Context) error {
8585
id, err := strconv.Atoi(context.Args().Get(0))
8686

8787
if err != nil {
88-
color.Red("Cannot parse test id, a number required!")
88+
color.Red(fmt.Sprintf("Cannot parse test id = '%s', a number required!", context.Args().Get(0)))
8989
return errors.New(fmt.Sprintf("Failed to parse test id = %s", context.Args().Get(0)))
9090
}
9191

@@ -110,7 +110,7 @@ func PrintCaseAction(context *cli.Context) error {
110110

111111
testCase := GetTestCase(metaData, id)
112112
if testCase == nil {
113-
color.Red("Count not find test case with given id")
113+
color.Red(fmt.Sprintf("Could not find test case with id = %d", id))
114114
return errors.New(fmt.Sprintf("Unknown test case with id %d", id))
115115
}
116116

@@ -126,9 +126,9 @@ func PrintCaseAction(context *cli.Context) error {
126126
}
127127

128128
// Command to print a test case. this command can be used to print inputs and/or outputs
129-
// to the consol. The user can choose the print the input only or the output only. The
129+
// to the consol. The user can choose to print the input only or the output only. The
130130
// user should provide a valid test id.
131-
// Running this command will fetch egor meta data, get the Test case with the given id,
131+
// Running this command will fetch egor meta data, get the test case with the given id,
132132
// and then print the content of the input and/or of the output files.
133133
var PrintCaseCommand = cli.Command{
134134
Name: "printcase",

0 commit comments

Comments
 (0)