Skip to content

Commit

Permalink
give perf test a properly separate name
Browse files Browse the repository at this point in the history
When adding this perf test I failed to realize that the first
parameter is the name of the test, which we can then use in metabase
to find the right benchmark.  Fix this to give the benchmark a unique
name, so we can query it properly in metabase.
  • Loading branch information
tgummerer committed Nov 14, 2023
1 parent 1042d8b commit 1a98352
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions misc/test/performance_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -125,9 +125,9 @@ func TestPolicyPacks(t *testing.T) {
err := npmInstallCmd.Run()
assert.NoError(t, err)

benchmark := bench("aws-py-s3-folder", "aws", "go", "go")
benchmark := bench("policy-test", "aws", "go", "go")
opts := integration.ProgramTestOptions{
Dir: path.Join(getCwd(t), "..", "..", benchmark.Name),
Dir: path.Join(getCwd(t), "..", "..", "aws-go-s3-folder"),
UpdateCommandlineFlags: []string{fmt.Sprintf("--policy-pack=%s", policyPack)},
ExtraRuntimeValidation: func(t *testing.T, stack integration.RuntimeValidationStackInfo) {
assertHTTPResult(t, "http://"+stack.Outputs["website_url"].(string), nil, func(body string) bool {
Expand Down

0 comments on commit 1a98352

Please sign in to comment.