test: the admin requests list test names the requester by their seeded username (#6265)
release / govulncheck (push) Successful in 17s
release / web (push) Successful in 1m36s
release / go (push) Successful in 1m54s
release / integration (push) Successful in 5m37s
release / android (push) Successful in 5m48s
release / Build signed APK (releases and dev) (push) Successful in 5m48s
release / Attach APK to the Release (tag releases only) (push) Skipped
release / Build + push container image (push) Successful in 1m9s
release / Verify release artifacts (tag releases only) (push) Skipped
release / govulncheck (push) Successful in 17s
release / web (push) Successful in 1m36s
release / go (push) Successful in 1m54s
release / integration (push) Successful in 5m37s
release / android (push) Successful in 5m48s
release / Build signed APK (releases and dev) (push) Successful in 5m48s
release / Attach APK to the Release (tag releases only) (push) Skipped
release / Build + push container image (push) Successful in 1m9s
release / Verify release artifacts (tag releases only) (push) Skipped
seedUser stores users as test-<name>, so the assertion compares against alice.Username rather than the literal "alice". Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
@@ -90,8 +90,8 @@ func TestHandleListAdminRequests_DefaultStatus_Pending(t *testing.T) {
|
||||
if rv.Status != "pending" {
|
||||
t.Errorf("result status = %q, want pending", rv.Status)
|
||||
}
|
||||
if rv.Username != "alice" {
|
||||
t.Errorf("result username = %q, want alice (the requester)", rv.Username)
|
||||
if rv.Username != alice.Username {
|
||||
t.Errorf("result username = %q, want %q (the requester)", rv.Username, alice.Username)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user