Simplify error message and remove redundant test code

Co-authored-by: dlevy-msft-sql <[email protected]>

Author: Copilot

pkg/sqlcmd/sqlcmd.go

@@ -420,7 +420,7 @@ func safeColumnTypes(rows *sql.Rows) (cols []*sql.ColumnType, err error) {
 	defer func() {
 		if r := recover(); r != nil {
 			// Convert panic to error
-			err = fmt.Errorf("failed to get column types: %v (this may be due to unsupported data types like GEOGRAPHY or GEOMETRY)", r)
+			err = fmt.Errorf("failed to get column types: %v", r)
 			cols = nil
 		}
 	}()

pkg/sqlcmd/sqlcmd_test.go

@@ -719,13 +719,6 @@ func TestSqlcmdPrefersSharedMemoryProtocol(t *testing.T) {
 // wrapper ensures this panic is caught regardless of where in the ColumnTypes()
 // call stack it originates.
 func TestSafeColumnTypesHandlesPanic(t *testing.T) {
-	// Verify that the defer/recover mechanism works by triggering any panic
-	defer func() {
-		if r := recover(); r != nil {
-			t.Errorf("safeColumnTypes should not panic, but got: %v", r)
-		}
-	}()
-
 	// This will trigger a panic due to nil pointer, but safeColumnTypes should catch it
 	var rows *sql.Rows
 	cols, err := safeColumnTypes(rows)