Skip to content
This repository was archived by the owner on Sep 7, 2021. It is now read-only.
This repository is currently being migrated. It's locked while the migration is in progress.

Commit 27bb758

Browse files
lunnylunny
committed
fix tests
1 parent ada24d3 commit 27bb758

15 files changed

+192
-254
lines changed

‎session_exist_test.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ func TestExistStruct(t *testing.T) {
5656
assert.NoError(t, err)
5757
assert.False(t, has)
5858

59-
tableName := mapper.Obj2Table("RecordExist")
59+
tableName := "`" + mapper.Obj2Table("RecordExist") + "`"
6060

6161
has, err = testEngine.SQL("select * from "+testEngine.TableName(tableName, true)+" where "+nameName+" = ?", "test1").Exist()
6262
assert.NoError(t, err)

‎session_find_test.go

+21-18
Original file line numberDiff line numberDiff line change
@@ -52,13 +52,13 @@ func TestJoinLimit(t *testing.T) {
5252
assert.NoError(t, err)
5353
assert.EqualValues(t, 1, cnt)
5454

55-
tableName := mapper.Obj2Table("CheckList")
56-
tableName2 := mapper.Obj2Table("Salary")
57-
tableName3 := mapper.Obj2Table("Empsetting")
55+
tableName := "`" + mapper.Obj2Table("CheckList") + "`"
56+
tableName2 := "`" + mapper.Obj2Table("Salary") + "`"
57+
tableName3 := "`" + mapper.Obj2Table("Empsetting") + "`"
5858

59-
idName := mapper.Obj2Table("Id")
60-
lIDName := mapper.Obj2Table("Lid")
61-
eIDName := mapper.Obj2Table("Eid")
59+
idName := "`" + mapper.Obj2Table("Id") + "`"
60+
lIDName := "`" + mapper.Obj2Table("Lid") + "`"
61+
eIDName := "`" + mapper.Obj2Table("Eid") + "`"
6262

6363
var salaries []Salary
6464
err = testEngine.Table(tableName2).
@@ -286,11 +286,11 @@ func TestGroupBy(t *testing.T) {
286286
assert.NoError(t, prepareEngine())
287287
assertSync(t, new(Userinfo))
288288

289-
idName := "id"
289+
idName := "`id`"
290290
userName := mapper.Obj2Table("Username")
291291

292292
users := make([]Userinfo, 0)
293-
err := testEngine.GroupBy(idName + ", " + userName).Find(&users)
293+
err := testEngine.GroupBy(idName + ", `" + userName + "`").Find(&users)
294294
assert.NoError(t, err)
295295
}
296296

@@ -609,17 +609,20 @@ func TestFindMapStringId(t *testing.T) {
609609

610610
deviceIDs := []string{"1"}
611611

612+
deviceIDName := "`" + mapper.Obj2Table("Deviceid") + "`"
613+
statusName := "`" + mapper.Obj2Table("Status") + "`"
614+
612615
deviceMaps := make(map[string]*FindMapDevice, len(deviceIDs))
613616
err = testEngine.
614-
Where("status = ?", 1).
615-
In("deviceid", deviceIDs).
617+
Where(statusName+" = ?", 1).
618+
In(deviceIDName, deviceIDs).
616619
Find(&deviceMaps)
617620
assert.NoError(t, err)
618621

619622
deviceMaps2 := make(map[string]FindMapDevice, len(deviceIDs))
620623
err = testEngine.
621-
Where("status = ?", 1).
622-
In("deviceid", deviceIDs).
624+
Where(statusName+" = ?", 1).
625+
In(deviceIDName, deviceIDs).
623626
Find(&deviceMaps2)
624627
assert.NoError(t, err)
625628

@@ -650,7 +653,7 @@ func TestFindMapStringId(t *testing.T) {
650653
assert.NoError(t, err)
651654
assert.EqualValues(t, 1, cnt)
652655

653-
sum, err := testEngine.SumInt(new(FindMapDevice), "status")
656+
sum, err := testEngine.SumInt(new(FindMapDevice), statusName)
654657
assert.NoError(t, err)
655658
assert.EqualValues(t, 2, sum)
656659

@@ -783,12 +786,12 @@ func TestFindJoin(t *testing.T) {
783786
assert.NoError(t, prepareEngine())
784787
assertSync(t, new(SceneItem), new(DeviceUserPrivrels))
785788

786-
tableName1 := mapper.Obj2Table("SceneItem")
787-
tableName2 := mapper.Obj2Table("DeviceUserPrivrels")
789+
tableName1 := "`" + mapper.Obj2Table("SceneItem") + "`"
790+
tableName2 := "`" + mapper.Obj2Table("DeviceUserPrivrels") + "`"
788791

789-
deviceIDName := mapper.Obj2Table("DeviceId")
790-
userIDName := mapper.Obj2Table("UserId")
791-
typeName := mapper.Obj2Table("Type")
792+
deviceIDName := "`" + mapper.Obj2Table("DeviceId") + "`"
793+
userIDName := "`" + mapper.Obj2Table("UserId") + "`"
794+
typeName := "`" + mapper.Obj2Table("Type") + "`"
792795

793796
var scenes []SceneItem
794797
err := testEngine.Join("LEFT OUTER", tableName2, tableName1+"."+deviceIDName+"="+tableName2+"."+deviceIDName).

‎session_get_test.go

+20-21
Original file line numberDiff line numberDiff line change
@@ -35,76 +35,75 @@ func TestGetVar(t *testing.T) {
3535
_, err := testEngine.InsertOne(&data)
3636
assert.NoError(t, err)
3737

38-
tableName := mapper.Obj2Table("GetVar")
38+
tableName := "`" + mapper.Obj2Table("GetVar") + "`"
39+
idName := mapper.Obj2Table("Id")
40+
msgName := mapper.Obj2Table("Msg")
41+
ageName := mapper.Obj2Table("Age")
42+
moneyName := mapper.Obj2Table("Money")
3943

4044
var msg string
41-
has, err := testEngine.Table(tableName).Cols("msg").Get(&msg)
45+
has, err := testEngine.Table(tableName).Cols("`" + msgName + "`").Get(&msg)
4246
assert.NoError(t, err)
4347
assert.Equal(t, true, has)
4448
assert.Equal(t, "hi", msg)
4549

4650
var age int
47-
has, err = testEngine.Table(tableName).Cols("age").Get(&age)
51+
has, err = testEngine.Table(tableName).Cols("`" + ageName + "`").Get(&age)
4852
assert.NoError(t, err)
4953
assert.Equal(t, true, has)
5054
assert.Equal(t, 28, age)
5155

5256
var age2 int64
53-
has, err = testEngine.Table(tableName).Cols("age").
54-
Where("age > ?", 20).
55-
And("age < ?", 30).
57+
has, err = testEngine.Table(tableName).Cols("`"+ageName+"`").
58+
Where("`"+ageName+"` > ?", 20).
59+
And("`"+ageName+"` < ?", 30).
5660
Get(&age2)
5761
assert.NoError(t, err)
5862
assert.Equal(t, true, has)
5963
assert.EqualValues(t, 28, age2)
6064

6165
var id sql.NullInt64
62-
has, err = testEngine.Table(tableName).Cols("id").Get(&id)
66+
has, err = testEngine.Table(tableName).Cols("`" + idName + "`").Get(&id)
6367
assert.NoError(t, err)
6468
assert.Equal(t, true, has)
6569
assert.Equal(t, true, id.Valid)
6670
assert.EqualValues(t, data.Id, id.Int64)
6771

6872
var msgNull sql.NullString
69-
has, err = testEngine.Table(tableName).Cols("msg").Get(&msgNull)
73+
has, err = testEngine.Table(tableName).Cols("`" + msgName + "`").Get(&msgNull)
7074
assert.NoError(t, err)
7175
assert.Equal(t, true, has)
7276
assert.Equal(t, true, msgNull.Valid)
7377
assert.EqualValues(t, data.Msg, msgNull.String)
7478

7579
var nullMoney sql.NullFloat64
76-
has, err = testEngine.Table(tableName).Cols("money").Get(&nullMoney)
80+
has, err = testEngine.Table(tableName).Cols("`" + moneyName + "`").Get(&nullMoney)
7781
assert.NoError(t, err)
7882
assert.Equal(t, true, has)
7983
assert.Equal(t, true, nullMoney.Valid)
8084
assert.EqualValues(t, data.Money, nullMoney.Float64)
8185

8286
var money float64
83-
has, err = testEngine.Table(tableName).Cols("money").Get(&money)
87+
has, err = testEngine.Table(tableName).Cols("`" + moneyName + "`").Get(&money)
8488
assert.NoError(t, err)
8589
assert.Equal(t, true, has)
8690
assert.Equal(t, "1.5", fmt.Sprintf("%.1f", money))
8791

8892
var money2 float64
8993
if testEngine.Dialect().DBType() == core.MSSQL {
90-
has, err = testEngine.SQL("SELECT TOP 1 money FROM " + testEngine.TableName(tableName, true)).Get(&money2)
94+
has, err = testEngine.SQL("SELECT TOP 1 `" + moneyName + "` FROM " + testEngine.TableName(tableName, true)).Get(&money2)
9195
} else {
92-
has, err = testEngine.SQL("SELECT money FROM " + testEngine.TableName(tableName, true) + " LIMIT 1").Get(&money2)
96+
has, err = testEngine.SQL("SELECT `" + moneyName + "` FROM " + testEngine.TableName(tableName, true) + " LIMIT 1").Get(&money2)
9397
}
9498
assert.NoError(t, err)
9599
assert.Equal(t, true, has)
96100
assert.Equal(t, "1.5", fmt.Sprintf("%.1f", money2))
97101

98102
var money3 float64
99-
has, err = testEngine.SQL("SELECT money FROM " + testEngine.TableName(tableName, true) + " WHERE money > 20").Get(&money3)
103+
has, err = testEngine.SQL("SELECT `" + moneyName + "` FROM " + testEngine.TableName(tableName, true) + " WHERE `" + moneyName + "` > 20").Get(&money3)
100104
assert.NoError(t, err)
101105
assert.Equal(t, false, has)
102106

103-
idName := mapper.Obj2Table("Id")
104-
ageName := mapper.Obj2Table("Age")
105-
msgName := mapper.Obj2Table("Msg")
106-
moneyName := mapper.Obj2Table("Money")
107-
108107
var valuesString = make(map[string]string)
109108
has, err = testEngine.Table(tableName).Get(&valuesString)
110109
assert.NoError(t, err)
@@ -118,7 +117,7 @@ func TestGetVar(t *testing.T) {
118117
// for mymysql driver, interface{} will be []byte, so ignore it currently
119118
if testEngine.Dialect().DriverName() != "mymysql" {
120119
var valuesInter = make(map[string]interface{})
121-
has, err = testEngine.Table(tableName).Where(idName+" = ?", 1).Select("*").Get(&valuesInter)
120+
has, err = testEngine.Table(tableName).Where("`"+idName+"` = ?", 1).Select("*").Get(&valuesInter)
122121
assert.NoError(t, err)
123122
assert.Equal(t, true, has)
124123
assert.Equal(t, 5, len(valuesInter))
@@ -336,10 +335,10 @@ func TestGetStructId(t *testing.T) {
336335
Id int64
337336
}
338337

339-
idName := mapper.Obj2Table("Id")
338+
idName := "`" + mapper.Obj2Table("Id") + "`"
340339
//var id int64
341340
var maxid maxidst
342-
sql := "select max(" + idName + ") as id from " + testEngine.TableName(&TestGetStruct{}, true)
341+
sql := "select max(" + idName + ") as id from " + testEngine.Quote(testEngine.TableName(&TestGetStruct{}, true))
343342
has, err := testEngine.SQL(sql).Get(&maxid)
344343
assert.NoError(t, err)
345344
assert.True(t, has)

‎session_insert_test.go

+16-12
Original file line numberDiff line numberDiff line change
@@ -768,10 +768,14 @@ func TestInsertMap(t *testing.T) {
768768
assert.NoError(t, prepareEngine())
769769
assertSync(t, new(InsertMap))
770770

771+
widthName := mapper.Obj2Table("Width")
772+
heightName := mapper.Obj2Table("Height")
773+
nameName := mapper.Obj2Table("Name")
774+
771775
cnt, err := testEngine.Table(new(InsertMap)).Insert(map[string]interface{}{
772-
"width": 20,
773-
"height": 10,
774-
"name": "lunny",
776+
widthName: 20,
777+
heightName: 10,
778+
nameName: "lunny",
775779
})
776780
assert.NoError(t, err)
777781
assert.EqualValues(t, 1, cnt)
@@ -787,9 +791,9 @@ func TestInsertMap(t *testing.T) {
787791
tableName := mapper.Obj2Table("InsertMap")
788792

789793
cnt, err = testEngine.Table(tableName).Insert(map[string]interface{}{
790-
"width": 30,
791-
"height": 10,
792-
"name": "lunny",
794+
widthName: 30,
795+
heightName: 10,
796+
nameName: "lunny",
793797
})
794798
assert.NoError(t, err)
795799
assert.EqualValues(t, 1, cnt)
@@ -807,14 +811,14 @@ func TestInsertMap(t *testing.T) {
807811

808812
cnt, err = testEngine.Table(tableName).Insert([]map[string]interface{}{
809813
{
810-
"width": 40,
811-
"height": 10,
812-
"name": "lunny",
814+
widthName: 40,
815+
heightName: 10,
816+
nameName: "lunny",
813817
},
814818
{
815-
"width": 50,
816-
"height": 10,
817-
"name": "lunny",
819+
widthName: 50,
820+
heightName: 10,
821+
nameName: "lunny",
818822
},
819823
})
820824
assert.NoError(t, err)

‎session_query_test.go

+15-15
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ func TestQueryString(t *testing.T) {
3939

4040
tableName := mapper.Obj2Table("GetVar2")
4141

42-
records, err := testEngine.QueryString("select * from " + testEngine.TableName(tableName, true))
42+
records, err := testEngine.QueryString("select * from `" + testEngine.TableName(tableName, true) + "`")
4343
assert.NoError(t, err)
4444
assert.Equal(t, 1, len(records))
4545
assert.Equal(t, 5, len(records[0]))
@@ -71,7 +71,7 @@ func TestQueryString2(t *testing.T) {
7171
_, err := testEngine.Insert(data)
7272
assert.NoError(t, err)
7373

74-
tableName := mapper.Obj2Table("GetVar3")
74+
tableName := "`" + mapper.Obj2Table("GetVar3") + "`"
7575

7676
idName := mapper.Obj2Table("Id")
7777
msgName := mapper.Obj2Table("Msg")
@@ -141,7 +141,7 @@ func TestQueryInterface(t *testing.T) {
141141
_, err := testEngine.InsertOne(data)
142142
assert.NoError(t, err)
143143

144-
tableName := mapper.Obj2Table("GetVarInterface")
144+
tableName := "`" + mapper.Obj2Table("GetVarInterface") + "`"
145145
idName := mapper.Obj2Table("Id")
146146
msgName := mapper.Obj2Table("Msg")
147147
ageName := mapper.Obj2Table("Age")
@@ -202,7 +202,7 @@ func TestQueryNoParams(t *testing.T) {
202202
assert.EqualValues(t, 3000, money)
203203
}
204204

205-
tableName := mapper.Obj2Table("QueryNoParams")
205+
tableName := "`" + mapper.Obj2Table("QueryNoParams") + "`"
206206
results, err := testEngine.Table(tableName).Limit(10).Query()
207207
assert.NoError(t, err)
208208
assertResult(t, results)
@@ -228,7 +228,7 @@ func TestQueryStringNoParam(t *testing.T) {
228228
_, err := testEngine.Insert(data)
229229
assert.NoError(t, err)
230230

231-
tableName := mapper.Obj2Table("GetVar4")
231+
tableName := "`" + mapper.Obj2Table("GetVar4") + "`"
232232

233233
idName := mapper.Obj2Table("Id")
234234
msgName := mapper.Obj2Table("Msg")
@@ -243,7 +243,7 @@ func TestQueryStringNoParam(t *testing.T) {
243243
assert.EqualValues(t, "0", records[0][msgName])
244244
}
245245

246-
records, err = testEngine.Table(tableName).Where(builder.Eq{idName: 1}).QueryString()
246+
records, err = testEngine.Table(tableName).Where(builder.Eq{"`" + idName + "`": 1}).QueryString()
247247
assert.NoError(t, err)
248248
assert.EqualValues(t, 1, len(records))
249249
assert.EqualValues(t, "1", records[0][idName])
@@ -270,7 +270,7 @@ func TestQuerySliceStringNoParam(t *testing.T) {
270270
_, err := testEngine.Insert(data)
271271
assert.NoError(t, err)
272272

273-
tableName := mapper.Obj2Table("GetVar6")
273+
tableName := "`" + mapper.Obj2Table("GetVar6") + "`"
274274

275275
records, err := testEngine.Table(tableName).Limit(1).QuerySliceString()
276276
assert.NoError(t, err)
@@ -283,7 +283,7 @@ func TestQuerySliceStringNoParam(t *testing.T) {
283283
}
284284

285285
idName := mapper.Obj2Table("Id")
286-
records, err = testEngine.Table(tableName).Where(builder.Eq{idName: 1}).QuerySliceString()
286+
records, err = testEngine.Table(tableName).Where(builder.Eq{"`" + idName + "`": 1}).QuerySliceString()
287287
assert.NoError(t, err)
288288
assert.EqualValues(t, 1, len(records))
289289
assert.EqualValues(t, "1", records[0][0])
@@ -310,7 +310,7 @@ func TestQueryInterfaceNoParam(t *testing.T) {
310310
_, err := testEngine.Insert(data)
311311
assert.NoError(t, err)
312312

313-
tableName := mapper.Obj2Table("GetVar5")
313+
tableName := "`" + mapper.Obj2Table("GetVar5") + "`"
314314
idName := mapper.Obj2Table("Id")
315315
msgName := mapper.Obj2Table("Msg")
316316

@@ -320,7 +320,7 @@ func TestQueryInterfaceNoParam(t *testing.T) {
320320
assert.EqualValues(t, 1, toInt64(records[0][idName]))
321321
assert.EqualValues(t, 0, toInt64(records[0][msgName]))
322322

323-
records, err = testEngine.Table(tableName).Where(builder.Eq{idName: 1}).QueryInterface()
323+
records, err = testEngine.Table(tableName).Where(builder.Eq{"`" + idName + "`": 1}).QueryInterface()
324324
assert.NoError(t, err)
325325
assert.EqualValues(t, 1, len(records))
326326
assert.EqualValues(t, 1, toInt64(records[0][idName]))
@@ -373,7 +373,7 @@ func TestQueryWithBuilder(t *testing.T) {
373373
}
374374

375375
tableName := mapper.Obj2Table("QueryWithBuilder")
376-
results, err := testEngine.Query(builder.Select("*").From(testEngine.TableName(tableName, true)))
376+
results, err := testEngine.Query(builder.Select("*").From("`" + testEngine.TableName(tableName, true) + "`"))
377377
assert.NoError(t, err)
378378
assertResult(t, results)
379379
}
@@ -414,11 +414,11 @@ func TestJoinWithSubQuery(t *testing.T) {
414414
assert.NoError(t, err)
415415
assert.EqualValues(t, 1, cnt)
416416

417-
tableName := mapper.Obj2Table("JoinWithSubQueryDepart")
418-
tableName1 := mapper.Obj2Table("JoinWithSubQuery1")
417+
tableName := "`" + mapper.Obj2Table("JoinWithSubQueryDepart") + "`"
418+
tableName1 := "`" + mapper.Obj2Table("JoinWithSubQuery1") + "`"
419419

420-
departID := mapper.Obj2Table("DepartId")
421-
idName := mapper.Obj2Table("Id")
420+
departID := "`" + mapper.Obj2Table("DepartId") + "`"
421+
idName := "`" + mapper.Obj2Table("Id") + "`"
422422

423423
var querys []JoinWithSubQuery1
424424
err = testEngine.Join("INNER", builder.Select(idName).From(testEngine.Quote(testEngine.TableName(tableName, true))),

‎session_raw_test.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -25,13 +25,13 @@ func TestExecAndQuery(t *testing.T) {
2525
uidName := mapper.Obj2Table("Uid")
2626
nameName := mapper.Obj2Table("Name")
2727

28-
res, err := testEngine.Exec("INSERT INTO "+testEngine.TableName(tableName, true)+" ("+uidName+", "+nameName+") VALUES (?, ?)", 1, "user")
28+
res, err := testEngine.Exec("INSERT INTO `"+testEngine.TableName(tableName, true)+"` (`"+uidName+"`, `"+nameName+"`) VALUES (?, ?)", 1, "user")
2929
assert.NoError(t, err)
3030
cnt, err := res.RowsAffected()
3131
assert.NoError(t, err)
3232
assert.EqualValues(t, 1, cnt)
3333

34-
results, err := testEngine.Query("select * from " + testEngine.TableName(tableName, true))
34+
results, err := testEngine.Query("select * from `" + testEngine.TableName(tableName, true) + "`")
3535
assert.NoError(t, err)
3636
assert.EqualValues(t, 1, len(results))
3737
id, err := strconv.Atoi(string(results[0][uidName]))

0 commit comments

Comments
 (0)