Avoid panic in TestRowsColumnTypes (#1426)

wayyoungboy · methane · web-flow · commit 0b40aee005da · 2023-05-06T18:07:41.000+09:00
* optimized the execution flow of the TestRowsColumnTypes unit test

* Update driver_test.go

---------

Co-authored-by: 渠磊 &lt;xuyan.wxy@oceanbase.com&gt;
Co-authored-by: Inada Naoki &lt;songofacandy@gmail.com&gt;
diff --git a/driver_test.go b/driver_test.go
@@ -2945,7 +2945,10 @@ func TestRowsColumnTypes(t *testing.T) {
 				continue
 			}
 		}
-
+		// Avoid panic caused by nil scantype.
+		if t.Failed() {
+			return
+		}
 		values := make([]interface{}, len(tt))
 		for i := range values {
 			values[i] = reflect.New(types[i]).Interface()

Original file line number	Diff line number	Diff line change
`@@ -2945,7 +2945,10 @@ func TestRowsColumnTypes(t *testing.T) {`
`2945`	`2945`	`continue`
`2946`	`2946`	`}`
`2947`	`2947`	`}`
`2948`		`-`
	`2948`	`+ // Avoid panic caused by nil scantype.`
	`2949`	`+ if t.Failed() {`
	`2950`	`+ return`
	`2951`	`+ }`
`2949`	`2952`	`values := make([]interface{}, len(tt))`
`2950`	`2953`	`for i := range values {`
`2951`	`2954`	`values[i] = reflect.New(types[i]).Interface()`