Skip to content

Commit a1816a9

Browse files
slievrlyCode-Fight
andauthored
optimize: fix go vet issue (#892)
fix go vet Co-authored-by: FengZhang <[email protected]>
1 parent f6da25c commit a1816a9

File tree

3 files changed

+10
-14
lines changed

3 files changed

+10
-14
lines changed

pkg/datasource/sql/datasource/base/meta_cache_test.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,6 @@ func (m *mockTrigger) LoadAll(ctx context.Context, dbName string, conn *sql.Conn
7373

7474
func TestBaseTableMetaCache_refresh(t *testing.T) {
7575
type fields struct {
76-
lock sync.RWMutex
7776
expireDuration time.Duration
7877
capity int32
7978
size int32
@@ -96,7 +95,6 @@ func TestBaseTableMetaCache_refresh(t *testing.T) {
9695
{
9796
name: "test1",
9897
fields: fields{
99-
lock: sync.RWMutex{},
10098
capity: capacity,
10199
size: 0,
102100
expireDuration: EexpireTime,
@@ -117,7 +115,6 @@ func TestBaseTableMetaCache_refresh(t *testing.T) {
117115
{
118116
name: "test2",
119117
fields: fields{
120-
lock: sync.RWMutex{},
121118
capity: capacity,
122119
size: 0,
123120
expireDuration: EexpireTime,
@@ -154,7 +151,6 @@ func TestBaseTableMetaCache_refresh(t *testing.T) {
154151
defer loadAllStub.Reset()
155152

156153
c := &BaseTableMetaCache{
157-
lock: tt.fields.lock,
158154
expireDuration: tt.fields.expireDuration,
159155
capity: tt.fields.capity,
160156
size: tt.fields.size,

pkg/datasource/sql/exec/at/base_executor_test.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -67,8 +67,8 @@ func TestBaseExecBuildLockKey(t *testing.T) {
6767
types.RecordImage{
6868
TableName: "test_name",
6969
Rows: []types.RowImage{
70-
{[]types.ColumnImage{getColumnImage("id", 1), getColumnImage("userId", "user1")}},
71-
{[]types.ColumnImage{getColumnImage("id", 2), getColumnImage("userId", "user2")}},
70+
{Columns: []types.ColumnImage{getColumnImage("id", 1), getColumnImage("userId", "user1")}},
71+
{Columns: []types.ColumnImage{getColumnImage("id", 2), getColumnImage("userId", "user2")}},
7272
},
7373
},
7474
"TEST_NAME:1_user1,2_user2",
@@ -84,9 +84,9 @@ func TestBaseExecBuildLockKey(t *testing.T) {
8484
types.RecordImage{
8585
TableName: "test2_name",
8686
Rows: []types.RowImage{
87-
{[]types.ColumnImage{getColumnImage("id", 1), getColumnImage("userId", "one"), getColumnImage("age", "11")}},
88-
{[]types.ColumnImage{getColumnImage("id", 2), getColumnImage("userId", "two"), getColumnImage("age", "22")}},
89-
{[]types.ColumnImage{getColumnImage("id", 3), getColumnImage("userId", "three"), getColumnImage("age", "33")}},
87+
{Columns: []types.ColumnImage{getColumnImage("id", 1), getColumnImage("userId", "one"), getColumnImage("age", "11")}},
88+
{Columns: []types.ColumnImage{getColumnImage("id", 2), getColumnImage("userId", "two"), getColumnImage("age", "22")}},
89+
{Columns: []types.ColumnImage{getColumnImage("id", 3), getColumnImage("userId", "three"), getColumnImage("age", "33")}},
9090
},
9191
},
9292
"TEST2_NAME:1_one_11,2_two_22,3_three_33",

pkg/datasource/sql/undo/builder/basic_undo_log_builder_test.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -93,8 +93,8 @@ func TestBuildLockKey(t *testing.T) {
9393
types.RecordImage{
9494
TableName: "test_name",
9595
Rows: []types.RowImage{
96-
{[]types.ColumnImage{getColumnImage("id", 1), getColumnImage("userId", "one")}},
97-
{[]types.ColumnImage{getColumnImage("id", 2), getColumnImage("userId", "two")}},
96+
{Columns: []types.ColumnImage{getColumnImage("id", 1), getColumnImage("userId", "one")}},
97+
{Columns: []types.ColumnImage{getColumnImage("id", 2), getColumnImage("userId", "two")}},
9898
},
9999
},
100100
"TEST_NAME:1_one,2_two",
@@ -110,9 +110,9 @@ func TestBuildLockKey(t *testing.T) {
110110
types.RecordImage{
111111
TableName: "test2_name",
112112
Rows: []types.RowImage{
113-
{[]types.ColumnImage{getColumnImage("id", 1), getColumnImage("userId", "one"), getColumnImage("age", "11")}},
114-
{[]types.ColumnImage{getColumnImage("id", 2), getColumnImage("userId", "two"), getColumnImage("age", "22")}},
115-
{[]types.ColumnImage{getColumnImage("id", 3), getColumnImage("userId", "three"), getColumnImage("age", "33")}},
113+
{Columns: []types.ColumnImage{getColumnImage("id", 1), getColumnImage("userId", "one"), getColumnImage("age", "11")}},
114+
{Columns: []types.ColumnImage{getColumnImage("id", 2), getColumnImage("userId", "two"), getColumnImage("age", "22")}},
115+
{Columns: []types.ColumnImage{getColumnImage("id", 3), getColumnImage("userId", "three"), getColumnImage("age", "33")}},
116116
},
117117
},
118118
"TEST2_NAME:1_one_11,2_two_22,3_three_33",

0 commit comments

Comments
 (0)