Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: typo in variable set ui and comments #3037

Merged
merged 1 commit into from
Aug 8, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion testing/cli-e2etest/testscenarios/test/run_test_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ func TestRunTestWithHttpTriggerAndVariableSetFile(t *testing.T) {
// And I have my server recently created
// And the datasource is already set

// When I create an variable set
// When I create a variable set
// Then it should be created correctly
environmentFile := env.GetTestResourcePath(t, "variableSet-file")

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ func TestDeleteVariableSet(t *testing.T) {
// Given I am a Tracetest CLI user
// And I have my server recently created

// When I try to delete an variable set that don't exist
// When I try to delete a variable set that don't exist
// Then it should return an error and say that this resource does not exist
result := tracetestcli.Exec(t, "delete variableset --id .env", tracetestcli.WithCLIConfig(cliConfig))
helpers.RequireExitCodeEqual(t, result, 1)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -51,9 +51,9 @@ func TestGetVariableSet(t *testing.T) {
t.Run("get with YAML format", func(t *testing.T) {
// Given I am a Tracetest CLI user
// And I have my server recently created
// And I have an variable set already set
// And I have a variable set already set

// When I try to get an variable set on yaml mode
// When I try to get a variable set on yaml mode
// Then it should print a YAML
result := tracetestcli.Exec(t, "get variableset --id .env --output yaml", tracetestcli.WithCLIConfig(cliConfig))
helpers.RequireExitCodeEqual(t, result, 0)
Expand All @@ -73,9 +73,9 @@ func TestGetVariableSet(t *testing.T) {
t.Run("get with JSON format", func(t *testing.T) {
// Given I am a Tracetest CLI user
// And I have my server recently created
// And I have an variable set already set
// And I have a variable set already set

// When I try to get an variable set on json mode
// When I try to get a variable set on json mode
// Then it should print a json
result := tracetestcli.Exec(t, "get variableset --id .env --output json", tracetestcli.WithCLIConfig(cliConfig))
helpers.RequireExitCodeEqual(t, result, 0)
Expand All @@ -95,9 +95,9 @@ func TestGetVariableSet(t *testing.T) {
t.Run("get with pretty format", func(t *testing.T) {
// Given I am a Tracetest CLI user
// And I have my server recently created
// And I have an variable set already set
// And I have a variable set already set

// When I try to get an variable set on pretty mode
// When I try to get a variable set on pretty mode
// Then it should print a table with 4 lines printed: header, separator, variable set item and empty line
result := tracetestcli.Exec(t, "get variableset --id .env --output pretty", tracetestcli.WithCLIConfig(cliConfig))
helpers.RequireExitCodeEqual(t, result, 0)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ function getOptionsMetadata({
[CliCommandOption.UseHostname]: {label: 'Specify Tracetest server hostname'},
[CliCommandOption.UseCurrentVariableSet]: {
label: 'Use selected variable set',
help: !isVariableSetSelected ? 'This option is only available when an variable set is selected' : undefined,
help: !isVariableSetSelected ? 'This option is only available when a variable set is selected' : undefined,
disabled: !isVariableSetSelected,
},
[CliCommandOption.GeneratesJUnit]: {label: 'Generate JUnit report'},
Expand Down