ksuid icon indicating copy to clipboard operation
ksuid copied to clipboard

tweaked scan method to work with most sql drivers // Added Must

Open juliankoehn opened this issue 4 years ago • 1 comments

Stripped the Scan method a 'lil bit. As i had problems to Scan with sqlx library. All tests pass.

=== RUN   TestScan
--- PASS: TestScan (0.00s)
=== RUN   TestValue
--- PASS: TestValue (0.00s)

UnmarshalText(), UnmarshalBinary()are no longer required for Scan.

TestSqlScanner() from ksuid_test.go is still implemented and PASS

--- PASS: TestSqlScanner (0.00s)
    --- PASS: TestSqlScanner/<nil> (0.00s)
    --- PASS: TestSqlScanner/string (0.00s)
    --- PASS: TestSqlScanner/[]uint8 (0.00s)

Same for TestSqlValuer() --- PASS: TestSqlValuer (0.00s)

juliankoehn avatar Apr 19 '20 13:04 juliankoehn

@achille-roussel going to leave this one to you :) LMK if you want anything from me on it.

rbranson avatar Apr 20 '20 19:04 rbranson