SqlCollector: enabled is true by default

xgfone · Aug 9, 2024 · f8246ca · f8246ca
1 parent 1447f8d
commit f8246ca
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/interceptor.go b/interceptor.go
@@ -58,9 +58,9 @@ type SqlCollector struct {
 	sqls map[string]struct{}
 }
 
-// NewSqlCollector returns a new SqlCollector.
+// NewSqlCollector returns a new SqlCollector with enabled==true.
 func NewSqlCollector() *SqlCollector {
-	return &SqlCollector{sqls: make(map[string]struct{}, 128)}
+	return (&SqlCollector{sqls: make(map[string]struct{}, 128)}).SetEnabled(true)
 }
 
 // Sqls returns the executed sqls.

diff --git a/interceptor_test.go b/interceptor_test.go
@@ -22,7 +22,7 @@ import (
 )
 
 func TestSqlCollector(t *testing.T) {
-	collector := NewSqlCollector()
+	collector := NewSqlCollector().SetEnabled(false)
 	_, _, _ = collector.Intercept("sql0", nil)
 
 	collector.SetEnabled(true)