From 1b97d65873a61efe188167d3ed483e846e6c52df Mon Sep 17 00:00:00 2001
From: Freddy <freddygv@users.noreply.github.com>
Date: Wed, 24 Jul 2019 08:41:00 -0600
Subject: [PATCH] Make new config when retrying testServer creation (#6204)

---
 agent/consul/server_test.go | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/agent/consul/server_test.go b/agent/consul/server_test.go
index 1aeb1a590..018113deb 100644
--- a/agent/consul/server_test.go
+++ b/agent/consul/server_test.go
@@ -154,16 +154,18 @@ func testServerDCExpectNonVoter(t *testing.T, dc string, expect int) (string, *S
 }
 
 func testServerWithConfig(t *testing.T, cb func(*Config)) (string, *Server) {
-	dir, config := testServerConfig(t)
-	if cb != nil {
-		cb(config)
-	}
-
+	var dir string
+	var config *Config
 	var srv *Server
 	var err error
 
 	// Retry added to avoid cases where bind addr is already in use
 	retry.RunWith(retry.ThreeTimes(), t, func(r *retry.R) {
+		dir, config = testServerConfig(t)
+		if cb != nil {
+			cb(config)
+		}
+
 		srv, err = newServer(config)
 		if err != nil {
 			os.RemoveAll(dir)
@@ -183,10 +185,6 @@ func newServer(c *Config) (*Server, error) {
 			oldNotify()
 		}
 	}
-	// Restore old notify to guard against re-closing `up` on a retry
-	defer func() {
-		c.NotifyListen = oldNotify
-	}()
 
 	// start server
 	w := c.LogOutput