The mssql-cli client uses old and unnecessary logic to right-strip semicolons. This was introduced in a refactor a few years ago, making it unclear why it was added in the first place. All of our tests passed after removing the semicolon-strip logic, however we needed to make minor updates to a few tests to expect semicolons in the input text.
This PR also introduces new tests to ensure the script in the #422 bug report work in multiline and non-interactive modes.
Fixes #422
The mssql-cli client uses old and unnecessary logic to right-strip semicolons. This was introduced in a refactor a few years ago, making it unclear why it was added in the first place. All of our tests passed after removing the semicolon-strip logic, however we needed to make minor updates to a few tests to expect semicolons in the input text.
This PR also introduces new tests to ensure the script in the #422 bug report work in multiline and non-interactive modes.