Fix login integration tests for new output wording. · Entire
Fix login integration tests for new output wording.
7fdfa6b→main·
toothbrush·1mo ago·1 file·+5 added/-5 removed
Match the renamed "Login URL:" prefix and lowercase "login complete." strings from the cosmetic login output tweak.
Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com
Sessions
bb20852cb016View transcript
Fix Login Integration Test Output
Changes
1
cmd/entire/cli/integration_test
- Mlogin_test.go+5/-5
95 unmodified lines
96
97
98
99
99
100
101
102
39 unmodified lines
142
143
144
145
145
146
147
148
32 unmodified lines
181
182
183
184
184
185
186
187
67 unmodified lines
255
256
257
258
259
258
259
260
261
262
95 unmodified lines
t.Fatalf("output missing wait message:\n%s", output)
}
if !strings.Contains(output, "Login complete.") {
if !strings.Contains(output, "login complete.") {
t.Fatalf("output missing login complete message (token save likely failed):\n%s", output)
}
39 unmodified lines
t.Fatalf("expected expired message, got:\n%s", output)
}
if strings.Contains(output, "Login complete.") {
if strings.Contains(output, "login complete.") {
t.Fatal("output should NOT contain login complete for expired flow")
}
}
32 unmodified lines
t.Fatalf("expected denied message, got:\n%s", output)
}
if strings.Contains(output, "Login complete.") {
if strings.Contains(output, "login complete.") {
t.Fatal("output should NOT contain login complete for denied flow")
}
}
67 unmodified lines
switch {
case strings.HasPrefix(line, "Device code: "):
deviceCode = strings.TrimPrefix(line, "Device code: "):
case strings.HasPrefix(line, "Approval URL: "):
approvalURL = strings.TrimPrefix(line, "Approval URL: "):
case strings.HasPrefix(line, "Login URL:"):
approvalURL = strings.TrimSpace(strings.TrimPrefix(line, "Login URL:"));
}
if approvalURL != "" && deviceCode != "" {