Allow expectations to be reused about go-sqlmock

[l3pp4rd]

yes, might make sense for such an option like AllowRepeatedExpectationMatching. though, then it is ExpectedQuery, the Rows offset should be reset on each match. Would you care to implement it?

[mikemacd]

I don't see a way to be able to reset the rows offset in the driver Rows, so it would seem to me that this feature may not be achievable.

[l3pp4rd]

Since driver.Rows is an interface you can always cast it to type. And we are certain, if it is not nil that it will be rowSets

So the code on this block should be the following:

if expected.rows == nil {
	return nil, fmt.Errorf("Query '%s' with args %+v, must return a database/sql/driver.Rows, but it was not set for expectation %T as %+v", query, args, expected, expected)
}

// reset rows for next use if allowed
if rs, ok := expected.rows.(*rowSets); ok {
	rs.pos = 0
	for _, set := range rs.sets {
		set.pos = 0
	}
}

return expected, nil

Does it make sense?

[mikemacd]

I hadn't considered that approach. Thanks! I'll try using it.

[l3pp4rd]

see another better attempt #148 but after long thought, this may lead to inconsistencies in behavior. cannot accept that