Merge branch 'master' into fix-xunit-result-xml-file

.github/workflows/ci.yml

@@ -0,0 +1,21 @@
+name: "CI"
+on:
+  push:
+  pull_request:
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - name: checkout
+        uses: actions/checkout@v4
+      - name: setup go
+        uses: actions/setup-go@v4
+        with:
+          go-version: 1.21
+      - name: vet
+        run: go vet ./...
+      - name: build
+        run: make build
+      - name: test
+        run: make test

Makefile

@@ -1,23 +1,25 @@
 .PHONY: all build test tidy clean
 
+GO := go
+
 default: build
 
 build:
-	go build -o mysql-tester ./src
+	$(GO) build -o mysql-tester ./src
 
 debug:
-	go build -gcflags="all=-N -l" -o mysql-tester ./src
+	$(GO) build -gcflags="all=-N -l" -o mysql-tester ./src
 
 test: build
-	go test -cover ./...
+	$(GO) test -cover ./...
 	#./mysql-tester -check-error
 
 tidy:
-	go mod tidy
+	$(GO) mod tidy
 
 clean:
-	go clean -i ./...
+	$(GO) clean -i ./...
 	rm -rf mysql-tester
 
 gen_perror: generate_perror/main.go
-	go build -o gen_perror ./generate_perror
+	$(GO) build -o gen_perror ./generate_perror

generate_perror/main.go

@@ -78,7 +78,7 @@ func scanErrCodeFile(fileName string, nameToNum map[string]int) {
 	r := regexp.MustCompile(`^\s+(\w+)*\s+=\s+(\d+)$`)
 	for s.Scan() {
 		m := r.FindStringSubmatch(s.Text())
-		if m != nil && len(m) == 3 && m[1] != "" && m[2] != "" {
+		if len(m) == 3 && m[1] != "" && m[2] != "" {
 			i, err := strconv.Atoi(m[2])
 			if err != nil {
 				log.Fatal(err)
@@ -134,7 +134,7 @@ func main() {
 		r := regexp.MustCompile(`^MySQL error code MY-0*(\d+) \((\w+)\)`)
 		for s.Scan() {
 			m := r.FindStringSubmatch(s.Text())
-			if m != nil && len(m) == 3 && m[1] != "" && m[2] != "" {
+			if len(m) == 3 && m[1] != "" && m[2] != "" {
 				c, err := strconv.Atoi(m[1])
 				if err != nil {
 					log.Fatal(err)
@@ -145,7 +145,10 @@ func main() {
 				checkNewErr(m[2], i, NameToNum)
 			}
 		}
-		cmd.Wait()
+		err = cmd.Wait()
+		if err != nil {
+			log.Fatal(err)
+		}
 	}
 	if maxError >= 1000 {
 		fmt.Printf("\r")
@@ -170,7 +173,7 @@ func main() {
 	sort.Slice(codes, func(i, j int) bool {
 		return codes[i].Code < codes[j].Code || codes[i].Code == codes[j].Code && codes[i].Name < codes[j].Name
 	})
-	for i, _ := range codes {
+	for i := range codes {
 		_, err = w.WriteString("\t\"" + codes[i].Name + `": ` + strconv.Itoa(codes[i].Code) + ",\n")
 		if err != nil {
 			log.Fatal(err)

src/main.go

@@ -291,7 +291,10 @@ func (t *tester) preProcess() {
 
 func (t *tester) postProcess() {
 	if !reserveSchema {
-		t.mdb.Exec(fmt.Sprintf("drop database `%s`", strings.ReplaceAll(t.name, "/", "__")))
+		_, err := t.mdb.Exec(fmt.Sprintf("drop database `%s`", strings.ReplaceAll(t.name, "/", "__")))
+		if err != nil {
+			log.Errorf("failed to drop database: %s", err.Error())
+		}
 	}
 	for _, v := range t.conn {
 		v.conn.Close()
@@ -918,13 +921,16 @@ func (t *tester) executeStmt(query string) error {
 	}
 
 	if t.enableInfo {
-		t.curr.conn.Raw(func(driverConn any) error {
+		err = t.curr.conn.Raw(func(driverConn any) error {
 			rowsAffected := driverConn.(*mysql.MysqlConn).RowsAffected()
 			lastMessage := driverConn.(*mysql.MysqlConn).LastMessage()
 			t.buf.WriteString(fmt.Sprintf("affected rows: %d\n", rowsAffected))
 			t.buf.WriteString(fmt.Sprintf("info: %s\n", lastMessage))
 			return nil
 		})
+		if err != nil {
+			log.Errorf("failed to get info: %s", err.Error())
+		}
 	}
 
 	if t.enableWarning {