huandu / go-sqlbuilder / 32086299391

96%

master: 96%

Pull #244

github

Interpolate: keep ClickHouse whole-number floats and non-finite values typed correctly

In encodeValue the Float32/Float64 cases rendered floats with
strconv.AppendFloat(..., 'g', -1, N) and ignored flavor, unlike the adjacent
flavor-aware String/Slice cases. For ClickHouse this had two symptoms: a
whole-number float such as float64(1) rendered as bare '1' (ClickHouse infers an
integer type and silently narrows a Float column), and NaN/+Inf/-Inf rendered in
Go's mixed case (ClickHouse only accepts lowercase nan/inf/-inf).

Route both cases through a new encodeFloat helper that, for ClickHouse only,
emits lowercase non-finite spellings and appends '.0' when strconv produced a
bare integer (detected via bytes.ContainsAny(rendered, ".eE") so 1e+21 is left
untouched). Every other flavor keeps its exact existing rendering.

Fixes #243

21 of 21 new or added lines in 1 file covered. (100.0%)

3998 of 4156 relevant lines covered (96.2%)

1.08 hits per line