fixed goimports and failing endtoend tests

Signed-off-by: Arvind Murty <10248018+arvind-murty@users.noreply.github.com>

go/test/endtoend/vtgate/queries/random/query_gen_test.go

@@ -28,9 +28,6 @@ import (
 
 // TestSeed makes sure that the seed is deterministic
 func TestSeed(t *testing.T) {
-	sel, err := sqlparser.Parse("select 1 < any (select empno from emp)")
-	fmt.Println(sel, err)
-
 	// specify the schema (that is defined in schema.sql)
 	schemaTables := []tableT{
 		{tableExpr: sqlparser.NewTableName("emp")},

go/test/endtoend/vtgate/queries/random/random_expr_test.go

@@ -17,28 +17,16 @@ limitations under the License.
 package random
 
 import (
-	"fmt"
 	"math/rand"
 	"testing"
 	"time"
 
-	"github.com/stretchr/testify/require"
-
-	"vitess.io/vitess/go/test/endtoend/utils"
-
 	"vitess.io/vitess/go/slices2"
 	"vitess.io/vitess/go/vt/sqlparser"
 )
 
 // This test tests that generating random expressions with a schema does not panic
 func TestRandomExprWithTables(t *testing.T) {
-	//t.Skip("Skip CI")
-	mcmp, closer := start(t)
-	defer closer()
-
-	require.NoError(t, utils.WaitForAuthoritative(t, keyspaceName, "emp", clusterInstance.VtgateProcess.ReadVSchema))
-	require.NoError(t, utils.WaitForAuthoritative(t, keyspaceName, "dept", clusterInstance.VtgateProcess.ReadVSchema))
-
 	// specify the schema (that is defined in schema.sql)
 	schemaTables := []tableT{
 		{tableExpr: sqlparser.NewTableName("emp")},
@@ -63,16 +51,9 @@ func TestRandomExprWithTables(t *testing.T) {
 	for i := 0; i < 100; i++ {
 
 		seed := time.Now().UnixNano()
-		fmt.Printf("seed: %d\n", seed)
-
 		r := rand.New(rand.NewSource(seed))
-		genConfig := sqlparser.NewExprGeneratorConfig(sqlparser.CannotAggregate, "", 0, false)
+		genConfig := sqlparser.NewExprGeneratorConfig(sqlparser.CanAggregate, "", 0, false)
 		g := sqlparser.NewGenerator(r, 3, slices2.Map(schemaTables, func(t tableT) sqlparser.ExprGenerator { return &t })...)
-		expr := g.Expression(genConfig)
-		fmt.Println(sqlparser.String(expr))
-
-		from := sqlparser.TableExprs{sqlparser.NewAliasedTableExpr(sqlparser.NewTableName("emp"), ""), sqlparser.NewAliasedTableExpr(sqlparser.NewTableName("dept"), "")}
-		query := sqlparser.NewSelect(nil, sqlparser.SelectExprs{sqlparser.NewAliasedExpr(expr, "")}, nil, nil, from, nil, nil, nil, nil)
-		mcmp.ExecAllowAndCompareError(sqlparser.String(query))
+		g.Expression(genConfig)
 	}
 }

go/test/vschemawrapper/vschema_wrapper.go

@@ -20,6 +20,7 @@ import (
 	"context"
 	"fmt"
 	"strings"
+
 	"vitess.io/vitess/go/mysql/collations"
 	"vitess.io/vitess/go/sqltypes"
 	"vitess.io/vitess/go/vt/key"
@@ -82,8 +83,7 @@ func (vw *VSchemaWrapper) PlanPrepareStatement(ctx context.Context, query string
 	return plan, stmt, nil
 }
 
-func (vw *VSchemaWrapper) ClearPrepareData(lowered string) {
-}
+func (vw *VSchemaWrapper) ClearPrepareData(string) {}
 
 func (vw *VSchemaWrapper) StorePrepareData(string, *vtgatepb.PrepareData) {}