Skip to content

Commit

Permalink
roachtest: fix parameters passed to require.NoError
Browse files Browse the repository at this point in the history
When context is passed to an assertion, the parameters *must* be a
string format, followed by arguments (as you would in a call to
`fmt.Sprintf`). The previous code would panic trying to cast int to
string.

Informs #95416

Release note: None
  • Loading branch information
renatolabs committed Jan 25, 2023
1 parent 1f0aeb4 commit 43f1777
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/cmd/roachtest/tests/multitenant_distsql.go
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@ func runMultiTenantDistSQL(
default:
// procede to report error
}
require.NoError(t, err, li, iter)
require.NoError(t, err, "instance idx = %d, iter = %d", li, iter)
iter++
}
})
Expand Down

0 comments on commit 43f1777

Please sign in to comment.