Skip to content

Commit 55a0289

Browse files
committed
Merge pull request #38 from jamiecuthill/master
Fixes issue with false positive on met expectations
2 parents 3769fed + 721ae9b commit 55a0289

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

sqlmock.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,6 @@ func (c *sqlmock) Exec(query string, args []driver.Value) (res driver.Result, er
231231
}
232232

233233
defer expected.Unlock()
234-
expected.triggered = true
235234

236235
if !expected.queryMatches(query) {
237236
return nil, fmt.Errorf("exec query '%s', does not match regex '%s'", query, expected.sqlRegex.String())
@@ -248,6 +247,7 @@ func (c *sqlmock) Exec(query string, args []driver.Value) (res driver.Result, er
248247
if expected.result == nil {
249248
return nil, fmt.Errorf("exec query '%s' with args %+v, must return a database/sql/driver.result, but it was not set for expectation %T as %+v", query, args, expected, expected)
250249
}
250+
expected.triggered = true
251251
return expected.result, err
252252
}
253253

0 commit comments

Comments
 (0)