testdirectory: handle IPv6 addresses (#68)

testdirectory/directory.go

@@ -99,6 +99,10 @@ func Start(t TestingT, opt ...Option) *Directory {
 	if opts.withPort == 0 {
 		opts.withPort = FreePort(t)
 	}
+	// Handle IPv6 addresses
+	if strings.Contains(opts.withHost, ":") && !strings.ContainsAny(opts.withHost, "[]") {
+		opts.withHost = "[" + opts.withHost + "]"
+	}
 
 	d := &Directory{
 		t:                  t,

testdirectory/directory_test.go

@@ -146,6 +146,25 @@ func Test_Start(t *testing.T) {
 		// not sure there's anything that's assertable here...
 		_ = testdirectory.Start(t, testdirectory.WithDisablePanicRecovery(t, true))
 	})
+	t.Run("start-with-ipv6-localhost", func(t *testing.T) {
+		assert, require := assert.New(t), require.New(t)
+		port := testdirectory.FreePort(t)
+		td := testdirectory.Start(
+			t,
+			testdirectory.WithPort(t, port),
+			testdirectory.WithHost(t, "::"),
+			testdirectory.WithDefaults(t, &testdirectory.Defaults{Users: testEntries}),
+			testdirectory.WithLogger(t, testLogger),
+			testdirectory.WithNoTLS(t),
+		)
+		assert.Equal(port, td.Port())
+
+		c := td.Conn()
+		defer c.Close()
+
+		err := c.Bind(userDN, testPwd)
+		require.NoError(err)
+	})
 }
 
 type safeBuf struct {
@@ -724,3 +743,7 @@ func TestDirectory_DeleteResponse(t *testing.T) {
 		})
 	}
 }
+
+func Test_Directory_Start(t *testing.T) {
+	t.Parallel()
+}