Stuart Padley

Add a "not yet implemented" error for all back-compat switches so user gets an actionable error message

1

If there are any switches left from the old ODBC sqlcmd that are not yet implemented in the new go-sqlcmd, please put in a stub implementation for go-sqlcmd now, which...

DRAFT: Not ready for code review yet. ---- This PR is a partial merge of PR #319, which rounds out the "use" scenario for containers, to allow the casual user...

In implementing `sqlcmd open ads` for Mac, we've hit an issue, in that the name of the credential that ADS is expecting (via SqlToolsService) is named using UTF16 characters (the...

Currently the Testcmd function just executes a command, but doesn't allow for further verification that the output generated was expected cmdparser.TestSetup(t) cmdparser.TestCmd[*AddEndpoint]() cmdparser.TestCmd[*AddContext]("--endpoint endpoint") cmdparser.TestCmd[*UseContext]("--name context") Provide a pattern that...

Ensure sqlcmd query doesn't return error after sqlcmd start

2

Right now `sqlcmd start` does not block waiting for SQL Server to be running and ready for client connections. So if `sqlcmd query` is immediately run, it will fail with:...

Enable user to override the default application name sqlcmd puts in the connection string

2

This is related to the other issue on application_name. Where we are looking to set application name to "sqlcmd", but this request go further and asks that the application name...

Pattern used for catching expected errors in tests should be able to verify for precise expected error

1

I've been using this pattern for negative tests, that are expecting a failure: defer func() { test.CatchExpectedError(recover(), t) }() This pattern ends up causing a test to pass for any...

In add-user.go line 83: we should prompt for the password on the console. we could also allow reading it from redirected stdin sqlcmd config add-user --username myuser < somefile.txt

Tracking issue from PR feedback

Tracking PR feedback: Where do we hook up the -U switch on the CLI as an override of the user name of the current context? Need to think about the...