Skip to content

Commit 40c917f

Browse files
authored
tests: add db:close when query failed.
1 parent 15a4b08 commit 40c917f

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

README.markdown

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,7 @@ Synopsis
9999

100100
if not ok then
101101
ngx.say("failed to connect: ", err, ": ", errcode, " ", sqlstate)
102+
db:close()
102103
return
103104
end
104105

@@ -108,6 +109,7 @@ Synopsis
108109
db:query("drop table if exists cats")
109110
if not res then
110111
ngx.say("bad result: ", err, ": ", errcode, ": ", sqlstate, ".")
112+
db:close()
111113
return
112114
end
113115

@@ -117,6 +119,7 @@ Synopsis
117119
.. "name varchar(5))")
118120
if not res then
119121
ngx.say("bad result: ", err, ": ", errcode, ": ", sqlstate, ".")
122+
db:close()
120123
return
121124
end
122125

@@ -127,6 +130,7 @@ Synopsis
127130
.. "values (\'Bob\'),(\'\'),(null)")
128131
if not res then
129132
ngx.say("bad result: ", err, ": ", errcode, ": ", sqlstate, ".")
133+
db:close()
130134
return
131135
end
132136

@@ -139,6 +143,7 @@ Synopsis
139143
db:query("select * from cats order by id asc", 10)
140144
if not res then
141145
ngx.say("bad result: ", err, ": ", errcode, ": ", sqlstate, ".")
146+
db:close()
142147
return
143148
end
144149

@@ -150,6 +155,7 @@ Synopsis
150155
local ok, err = db:set_keepalive(10000, 100)
151156
if not ok then
152157
ngx.say("failed to set keepalive: ", err)
158+
db:close()
153159
return
154160
end
155161

@@ -416,6 +422,7 @@ Below is a trivial example for this:
416422
res, err, errcode, sqlstate = db:query("select 1; select 2; select 3;")
417423
if not res then
418424
ngx.log(ngx.ERR, "bad result #1: ", err, ": ", errcode, ": ", sqlstate, ".")
425+
db:close()
419426
return ngx.exit(500)
420427
end
421428

@@ -426,6 +433,7 @@ Below is a trivial example for this:
426433
res, err, errcode, sqlstate = db:read_result()
427434
if not res then
428435
ngx.log(ngx.ERR, "bad result #", i, ": ", err, ": ", errcode, ": ", sqlstate, ".")
436+
db:close()
429437
return ngx.exit(500)
430438
end
431439

@@ -436,6 +444,7 @@ Below is a trivial example for this:
436444
local ok, err = db:set_keepalive(10000, 50)
437445
if not ok then
438446
ngx.log(ngx.ERR, "failed to set keepalive: ", err)
447+
db:close()
439448
ngx.exit(500)
440449
end
441450
```

0 commit comments

Comments
 (0)