fix: remove duplicate -n

kubeshop · Feb 19, 2024 · dcee053 · dcee053
1 parent 0ca234e
commit dcee053
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/contrib/executor/jmeterd/pkg/runner/runner.go b/contrib/executor/jmeterd/pkg/runner/runner.go
@@ -300,6 +300,7 @@ func removeDuplicatedArgs(args []string) []string {
 func mergeDuplicatedArgs(args []string) []string {
  allowed := map[string]int{
  "-e": 0,
+ "-n": 0,
  }
 
  for i := len(args) - 1; i >= 0; i-- {

diff --git a/contrib/executor/jmeterd/pkg/runner/runner_test.go b/contrib/executor/jmeterd/pkg/runner/runner_test.go
@@ -210,13 +210,13 @@ func TestMergeDuplicatedArgs(t *testing.T) {
  },
  {
  name: "Multiple duplicated args",
- args: []string{"<envVars>", "-e", "-e", "-l"},
- expectedArgs: []string{"<envVars>", "-e", "-l"},
+ args: []string{"<envVars>", "-e", "-e", "-n", "-n", "-l"},
+ expectedArgs: []string{"<envVars>", "-e", "-n", "-l"},
  },
  {
  name: "Non duplicated args",
- args: []string{"-e", "<envVars>", "-l"},
- expectedArgs: []string{"-e", "<envVars>", "-l"},
+ args: []string{"-e", "-n", "<envVars>", "-l"},
+ expectedArgs: []string{"-e", "-n", "<envVars>", "-l"},
  },
  }