usql icon indicating copy to clipboard operation
usql copied to clipboard

Fix substitution of quoted variables

Open murfffi opened this issue 1 year ago • 1 comments

testcli.go failed before the fix and passes after it. Also includes a fix to testcli.go itself.

Closes #505 .

murfffi avatar Dec 07 '24 11:12 murfffi

Please advise if a new unit test is needed here. stmt_test.go contains tests for statement parsing (TestNextResetState) and substitution (TestVarSubstitute) but I didn't see a test for both at the same time. (I might have missed it - there are a lot of tests.)

Maybe this is where testcli.go comes in, though it took me some time to get from its failure to the root cause.

murfffi avatar Dec 17 '24 07:12 murfffi