dal-go / dalgo / 25914147254

100%

push

github

fix(dal): GroupCondition.String() spaces around the operator

Bug: `strings.Join(s, string(v.operator))` produced concatenations
like `'a'AND'b'` for AND-grouped conditions — invalid SQL in every
dialect. Fix: surround the operator with spaces, yielding
`'a' AND 'b'`. The empty-group case still collapses to "()".

Unlike the LIMIT/TOP emission (which is dialect-specific and was
fixed downstream in dalgo2sql), space-padding around AND/OR/= is
universal SQL — every text-passthrough driver needs it. Belongs in
dalgo core, not the driver layer.

Triggered by datatug-cli's db-copy filtering Feature
(TestCopy_WhereAndComposition: USA AND SupportRepId=3). Existing
GroupCondition test updated to reflect the spaced form; AND/OR/1-item
cases added to lock in coverage.

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

3653 of 3653 relevant lines covered (100.0%)

1.1 hits per line