go/go-sqlmock
1
0
Fork 0
mirror of https://github.com/DATA-DOG/go-sqlmock.git synced 2025-05-15 22:06:43 +02:00
Code Issues Releases Activity

Merge pull request #38 from jamiecuthill/master

Browse Source 
Fixes issue with false positive on met expectations
This commit is contained in:
Gediminas Morkevicius 2016-02-26 12:27:59 +00:00
commit 55a0289da5
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
sqlmock.go
View File

@ -231,7 +231,6 @@ func (c *sqlmock) Exec(query string, args []driver.Value) (res driver.Result, er
} }
defer expected.Unlock() defer expected.Unlock()
expected.triggered = true
if !expected.queryMatches(query) { if !expected.queryMatches(query) {
return nil, fmt.Errorf("exec query '%s', does not match regex '%s'", query, expected.sqlRegex.String()) 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
if expected.result == nil { if expected.result == nil {
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) 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)
} }
expected.triggered = true
return expected.result, err return expected.result, err
} }