31
83%
master: 83%

Ran 04 Aug 2026 01:57AM UTC

Files 19

Run time 1s

Badge

Embed ▾

Committed 04 Aug 2026 01:02AM UTC coverage: 82.017%. First build

Job # 30870040810.31

Build Type

Pull #1

github

Committed by

morgo

Commit Message

conn: reject named params and fix scan types on the unified path

Two divergences from the normal query path, both found in review.

QueryResultContext accepted driver.NamedValue with Name set and bound
it positionally. QueryContext and ExecContext route args through
namedValueToValue, which rejects named parameters outright; the unified
path skipped that and went straight to CheckNamedValue. Under an
InterpolateParams DSN a caller passing sql.Named would have silently
got a positional bind. Reuse namedValueToValue for the rejection, then
apply CheckNamedValue to the extracted values -- Raw still bypasses
database/sql's conversion, so that part is still needed here.

ColumnTypeScanType reported the parsed Go type (int32, sql.NullTime,
...) while raw rows deliver every non-NULL cell as []byte, so a caller
asserting driver.RowsColumnTypeScanType was told the wrong thing about
what Next yields. Report scanTypeBytes when raw is set. The other
ColumnType methods describe the column rather than the delivered value
and stay as they are.

Both are covered in TestQueryResultContext and fail without the fix.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Pull Request Pull Request #1: conn: add unified QueryResultContext for arbitrary SQL

Coverage Stats

3416 of 4165 relevant lines covered (82.02%)

79171.94 hits per line

block / mysql / 30870040810 / 31
83%
master: 83%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job Linux-Go-1.26-DB-mariadb-11.8 - 30870040810.31

block / mysql / 30870040810 / 31 83% master: 83%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job Linux-Go-1.26-DB-mariadb-11.8 - 30870040810.31

block / mysql / 30870040810 / 31
83%
master: 83%

README BADGES
x