Skip to content

Commit

Permalink
Fix test
Browse files Browse the repository at this point in the history
Signed-off-by: Tim Vaillancourt <[email protected]>
  • Loading branch information
timvaillancourt committed Apr 17, 2024
1 parent e4834b5 commit e23ef0f
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions go/vt/vtgate/executor_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -1851,11 +1851,9 @@ func TestGetPlanPriority(t *testing.T) {
vCursor, err := newVCursorImpl(session, makeComments(""), r, nil, r.vm, r.VSchema(), r.resolver.resolver, nil, false, pv)
assert.NoError(t, err)

stmt, err := sqlparser.Parse(testCase.sql)
assert.NoError(t, err)
crticalityFromStatement, _ := sqlparser.GetPriorityFromStatement(stmt)

Check failure on line 1854 in go/vt/vtgate/executor_test.go

View workflow job for this annotation

GitHub Actions / Static Code Checks Etc

undefined: stmt

_, err = r.getPlan(context.Background(), vCursor, testCase.sql, stmt, makeComments("/* some comment */"), map[string]*querypb.BindVariable{}, nil, true, logStats)
_, err = r.getPlan(context.Background(), vCursor, testCase.sql, makeComments("/* some comment */"), map[string]*querypb.BindVariable{}, nil, true, logStats)

Check failure on line 1856 in go/vt/vtgate/executor_test.go

View workflow job for this annotation

GitHub Actions / Static Code Checks Etc

too many arguments in call to r.getPlan
if testCase.expectedError != nil {
assert.ErrorIs(t, err, testCase.expectedError)
} else {
Expand Down

0 comments on commit e23ef0f

Please sign in to comment.