add test for show non default port

Author: jennifersp

testing/go/framework.go

@@ -164,7 +164,6 @@ func RunScript(t *testing.T, script ScriptTest, normalizeRows bool) {
 			controller.Stop()
 			err := controller.WaitForStop()
 			require.NoError(t, err)
-			currentPort = 5432
 		}()
 	}
 
@@ -339,9 +338,15 @@ func init() {
 // when the connection is closed (or loses its connection to the server). The accompanying WaitGroup may be used to wait
 // until the server has closed.
 func CreateServer(t *testing.T, database string) (context.Context, *Connection, *svcs.Controller) {
-	require.NotEmpty(t, database)
 	port := GetUnusedPort(t)
-	currentPort = port
+	return CreateServerWithPort(t, database, port)
+}
+
+// CreateServerWithPort creates a server with the given database and port, returning a connection to the server. The server will close
+// when the connection is closed (or loses its connection to the server). The accompanying WaitGroup may be used to wait
+// until the server has closed.
+func CreateServerWithPort(t *testing.T, database string, port int) (context.Context, *Connection, *svcs.Controller) {
+	require.NotEmpty(t, database)
 	controller, err := dserver.RunInMemory(&servercfg.DoltgresConfig{
 		ListenerConfig: &servercfg.DoltgresListenerConfig{
 			PortNumber: &port,

testing/go/server_config_variable_test.go

@@ -0,0 +1,78 @@
+// Copyright 2025 Dolthub, Inc.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+package _go
+
+import (
+	"context"
+	"fmt"
+	"github.com/dolthub/dolt/go/libraries/utils/svcs"
+	"github.com/stretchr/testify/require"
+	"testing"
+
+	"github.com/jackc/pgx/v5"
+
+	"github.com/dolthub/go-mysql-server/sql"
+)
+
+func TestServerConfigVariableStatement(t *testing.T) {
+	scriptDatabase := "postgres"
+	var ctx context.Context
+	var conn *Connection
+	port := GetUnusedPort(t)
+
+	if runOnPostgres {
+		ctx = context.Background()
+		pgxConn, err := pgx.Connect(ctx, fmt.Sprintf("postgres://postgres:[email protected]:%d/%s?sslmode=disable", 5432, scriptDatabase))
+		require.NoError(t, err)
+		conn = &Connection{
+			Default: pgxConn,
+			Current: pgxConn,
+		}
+		require.NoError(t, pgxConn.Ping(ctx))
+		defer func() {
+			conn.Close(ctx)
+		}()
+	} else {
+		var controller *svcs.Controller
+		ctx, conn, controller = CreateServerWithPort(t, scriptDatabase, port)
+		defer func() {
+			conn.Close(ctx)
+			controller.Stop()
+			err := controller.WaitForStop()
+			require.NoError(t, err)
+		}()
+	}
+
+	t.Run("show 'port' configuration variable", func(t *testing.T) {
+		runScript(t, ctx, ScriptTest{
+			Name:        "set 'port' configuration variable",
+			SetUpScript: []string{},
+			Assertions: []ScriptTestAssertion{
+				{
+					Query:    "SHOW port",
+					Expected: []sql.Row{{port}},
+				},
+				{
+					Query:       "SET port TO '5432'",
+					ExpectedErr: "is a read only variable",
+				},
+				{
+					Query:    "SELECT current_setting('port')",
+					Expected: []sql.Row{{fmt.Sprintf("%v", port)}},
+				},
+			},
+		}, conn, true)
+	})
+}

testing/go/set_test.go

@@ -5836,24 +5836,6 @@ var setStmts = []ScriptTest{
 			},
 		},
 	},
-	{
-		Name:        "set 'port' configuration variable",
-		SetUpScript: []string{},
-		Assertions: []ScriptTestAssertion{
-			{
-				Query:    "SHOW port",
-				Expected: []sql.Row{{currentPort}},
-			},
-			{
-				Query:       "SET port TO '5432'",
-				ExpectedErr: "is a read only variable",
-			},
-			{
-				Query:    "SELECT current_setting('port')",
-				Expected: []sql.Row{{fmt.Sprintf("%v", currentPort)}},
-			},
-		},
-	},
 	{
 		Name:        "set 'post_auth_delay' configuration variable",
 		SetUpScript: []string{},