test: correct stale browser-flow comment · Entire
test: correct stale browser-flow comment
1d2b6ba→main·
toothbrush·1mo ago·1 file·+4 added/-3 removed
openBrowser reports failure under test (not a no-op), forcing the fallback URL path the test scrapes — update the doc comment to match. Addresses a PR review note.
Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com
Sessions
86eeeb1f9d81View transcript
[?
Switch Login Flow to Browser-Based DefaultClaude Code·1 step](/content/gh/entireio/cli/session/5d3033e2-31cd-46be-999c-6ae099f4757a#timeline-86eeeb1f9d81/index.html)
Changes
1
cmd/entire/cli/integration_test
Mlogin_test.go+4/-3
188 unmodified lines
189
190
191
192
193
194
192
193
194
195
196
197
198
188 unmodified lines
// TestLogin_BrowserFlow_SavesToken drives the loopback authorization-code
// flow end to end: ENTIRE_TEST_TTY=1 forces the interactive (browser)
// default, openBrowser is a no-op under test, and the test plays the role
// of the browser by GETting the loopback callback with a code + the state
// parsed out of the printed authorization URL.
// default, openBrowser reports failure under test (no usable browser on a
// headless host) so the flow prints the fallback URL, and the test plays
// the role of the browser by parsing that URL and GETting the loopback
// callback with a code + the state from it.
func TestLogin_BrowserFlow_SavesToken(t *testing.T) {
t.Parallel()
Mcmd/entire/cli/integration_test/login_test.go+4/-3