DATA-DOG / go-sqlmock

Sql mock driver for golang to test database interactions
Other
6.02k stars 406 forks source link

Fix ExpectedExec Stringer implementation #249

Closed maguro closed 3 years ago

maguro commented 3 years ago

Sometimes the result is incorrectly set, so the cast that takes place in the String() method returns nil.

codecov-io commented 3 years ago

Codecov Report

Merging #249 (ad48e46) into master (c35a79d) will not change coverage. The diff coverage is 66.66%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #249   +/-   ##
=======================================
  Coverage   93.98%   93.98%           
=======================================
  Files          19       19           
  Lines         814      814           
=======================================
  Hits          765      765           
  Misses         34       34           
  Partials       15       15           
Impacted Files Coverage Δ
expectations.go 84.25% <66.66%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update c35a79d...ad48e46. Read the comment docs.

l3pp4rd commented 3 years ago

Thanks!