feat: add end-to-end tests for sqlcmd binary (fixes microsoft#641)

Add e2e tests that build the sqlcmd binary and exercise real-world scenarios:

- TestE2E_Help: verifies --help flag works
- TestE2E_Version: verifies --version flag works
- TestE2E_PipedInput_NoPanic: regression test for microsoft#607 (piped input panic)
- TestE2E_PipedInput_EmptyInput: empty piped input doesn't panic
- TestE2E_InvalidFlag: invalid flags produce helpful errors
- TestE2E_QueryFlag_NoServer: -Q flag without server doesn't panic
- TestE2E_InputFile_NotFound: missing input file errors gracefully
- TestE2E_PipedInput_WithStdinReader: GO batches in piped input work

The tests build the binary once and reuse it for all tests.
Non-connection tests can run anywhere without a SQL Server.

Addresses feedback from @shueybubbles in PR microsoft#640.

Author: dlevy-msft-sql

cmd/modern/e2e_test.go

@@ -0,0 +1,208 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT license.
+
+package main
+
+import (
+	"bytes"
+	"os"
+	"os/exec"
+	"path/filepath"
+	"runtime"
+	"strings"
+	"sync"
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+)
+
+var (
+	binaryPath string
+	buildOnce  sync.Once
+	buildErr   error
+)
+
+// buildBinary compiles the sqlcmd binary once for all e2e tests.
+// The binary is placed in a temporary directory and cleaned up after tests complete.
+func buildBinary(t *testing.T) string {
+	t.Helper()
+	buildOnce.Do(func() {
+		tmpDir, err := os.MkdirTemp("", "sqlcmd-e2e-*")
+		if err != nil {
+			buildErr = err
+			return
+		}
+
+		binaryName := "sqlcmd"
+		if runtime.GOOS == "windows" {
+			binaryName = "sqlcmd.exe"
+		}
+		binaryPath = filepath.Join(tmpDir, binaryName)
+
+		cmd := exec.Command("go", "build", "-o", binaryPath, ".")
+		cmd.Dir = filepath.Dir(binaryPath)
+		// Build from the cmd/modern directory
+		wd, _ := os.Getwd()
+		cmd.Dir = wd
+		output, err := cmd.CombinedOutput()
+		if err != nil {
+			buildErr = &buildError{err: err, output: string(output)}
+			return
+		}
+	})
+	if buildErr != nil {
+		t.Fatalf("Failed to build sqlcmd binary: %v", buildErr)
+	}
+	return binaryPath
+}
+
+type buildError struct {
+	err    error
+	output string
+}
+
+func (e *buildError) Error() string {
+	return e.err.Error() + ": " + e.output
+}
+
+// TestE2E_Help verifies that --help flag works and produces expected output.
+func TestE2E_Help(t *testing.T) {
+	binary := buildBinary(t)
+
+	cmd := exec.Command(binary, "--help")
+	output, err := cmd.CombinedOutput()
+
+	require.NoError(t, err, "sqlcmd --help should not error")
+	assert.Contains(t, string(output), "sqlcmd", "help output should mention sqlcmd")
+	assert.Contains(t, string(output), "Usage:", "help output should contain Usage section")
+}
+
+// TestE2E_Version verifies that --version flag works.
+func TestE2E_Version(t *testing.T) {
+	binary := buildBinary(t)
+
+	cmd := exec.Command(binary, "--version")
+	output, err := cmd.CombinedOutput()
+
+	require.NoError(t, err, "sqlcmd --version should not error")
+	// Version output should contain version info
+	outputStr := string(output)
+	assert.True(t, strings.Contains(outputStr, "Version") || strings.Contains(outputStr, "version") || strings.Contains(outputStr, "v"),
+		"version output should contain version info: %s", outputStr)
+}
+
+// TestE2E_PipedInput_NoPanic verifies that piping input to sqlcmd with -G flag
+// does not cause a nil pointer panic. This is a regression test for issue #607.
+// The command will fail to connect (no server specified properly), but it should
+// NOT panic - that's the key behavior we're testing.
+func TestE2E_PipedInput_NoPanic(t *testing.T) {
+	binary := buildBinary(t)
+
+	// Create a command that pipes input
+	cmd := exec.Command(binary, "-G", "-S", "nonexistent.database.windows.net", "-d", "testdb")
+	cmd.Stdin = strings.NewReader("SELECT 1")
+
+	// Run the command - we expect it to fail (can't connect), but NOT panic
+	output, err := cmd.CombinedOutput()
+	outputStr := string(output)
+
+	// The command should fail with a connection error, not a panic
+	if err != nil {
+		// This is expected - we can't connect to a non-existent server
+		// But we should NOT see a panic in the output
+		assert.NotContains(t, outputStr, "panic:", "sqlcmd should not panic when piping input")
+		assert.NotContains(t, outputStr, "nil pointer", "sqlcmd should not have nil pointer error")
+		assert.NotContains(t, outputStr, "runtime error", "sqlcmd should not have runtime error")
+	}
+	// If it somehow succeeded (unlikely), that's fine too
+}
+
+// TestE2E_PipedInput_EmptyInput verifies that piping empty input doesn't panic.
+func TestE2E_PipedInput_EmptyInput(t *testing.T) {
+	binary := buildBinary(t)
+
+	cmd := exec.Command(binary, "-S", "nonexistent.server")
+	cmd.Stdin = strings.NewReader("")
+
+	output, err := cmd.CombinedOutput()
+	outputStr := string(output)
+
+	// Should fail with connection error, not panic
+	if err != nil {
+		assert.NotContains(t, outputStr, "panic:", "sqlcmd should not panic with empty piped input")
+		assert.NotContains(t, outputStr, "nil pointer", "sqlcmd should not have nil pointer error")
+	}
+}
+
+// TestE2E_InvalidFlag verifies that invalid flags produce a helpful error message.
+func TestE2E_InvalidFlag(t *testing.T) {
+	binary := buildBinary(t)
+
+	cmd := exec.Command(binary, "--this-flag-does-not-exist")
+	output, err := cmd.CombinedOutput()
+
+	assert.Error(t, err, "invalid flag should cause an error")
+	outputStr := string(output)
+	// Should have some kind of error message about unknown flag
+	assert.True(t, strings.Contains(outputStr, "unknown") || strings.Contains(outputStr, "invalid") || strings.Contains(outputStr, "flag"),
+		"error message should indicate unknown/invalid flag: %s", outputStr)
+}
+
+// TestE2E_QueryFlag_NoServer verifies -Q flag behavior without a server.
+func TestE2E_QueryFlag_NoServer(t *testing.T) {
+	binary := buildBinary(t)
+
+	cmd := exec.Command(binary, "-Q", "SELECT 1")
+	output, err := cmd.CombinedOutput()
+	outputStr := string(output)
+
+	// Should fail because no server is specified, but not panic
+	if err != nil {
+		assert.NotContains(t, outputStr, "panic:", "sqlcmd should not panic")
+	}
+}
+
+// TestE2E_InputFile_NotFound verifies proper error when input file doesn't exist.
+func TestE2E_InputFile_NotFound(t *testing.T) {
+	binary := buildBinary(t)
+
+	cmd := exec.Command(binary, "-i", "/nonexistent/path/to/file.sql", "-S", "localhost")
+	output, err := cmd.CombinedOutput()
+
+	assert.Error(t, err, "non-existent input file should cause an error")
+	outputStr := string(output)
+	assert.NotContains(t, outputStr, "panic:", "should not panic on missing input file")
+}
+
+// TestE2E_PipedInput_WithStdinReader verifies piping works with bytes.Buffer.
+func TestE2E_PipedInput_WithStdinReader(t *testing.T) {
+	binary := buildBinary(t)
+
+	input := bytes.NewBufferString("SELECT @@VERSION\nGO\n")
+	cmd := exec.Command(binary, "-S", "nonexistent.server", "-C")
+	cmd.Stdin = input
+
+	output, err := cmd.CombinedOutput()
+	outputStr := string(output)
+
+	// Should fail to connect, but not panic
+	if err != nil {
+		assert.NotContains(t, outputStr, "panic:", "should not panic when piping SQL with GO")
+		assert.NotContains(t, outputStr, "nil pointer", "should not have nil pointer error")
+	}
+}
+
+// cleanupBinary can be called to remove the test binary.
+// In practice, os.TempDir cleanup handles this, but this is here for explicit cleanup if needed.
+func cleanupBinary() {
+	if binaryPath != "" {
+		os.RemoveAll(filepath.Dir(binaryPath))
+	}
+}
+
+func TestMain(m *testing.M) {
+	code := m.Run()
+	cleanupBinary()
+	os.Exit(code)
+}