Add comment describing why --force-tty is neede dfor test

bryphe-coder · bryphe-coder · commit 1faa215f2128 · 2022-02-12T04:50:01.000Z
diff --git a/cli/login_test.go b/cli/login_test.go
@@ -26,6 +26,9 @@ func TestLogin(t *testing.T) {
 		console, err := expect.NewConsole(expect.WithStdout(clitest.StdoutLogs(t)))
 		require.NoError(t, err)
 		client := coderdtest.New(t)
+		// The --force-tty is required on Windows, because the `isatty` library does not
+		// accurately detect Windows ptys when they are not attached to a process:
+		// https://github.com/mattn/go-isatty/issues/59
 		root, _ := clitest.New(t, "login", client.URL.String(), "--force-tty")
 		root.SetIn(console.InTty())
 		root.SetOut(console.OutTty())
