detached 285d237

test(auth): add symmetric empty-username test for isInternalAuth

Add test case for symmetric auth validation when username is empty to improve test coverage of internal authentication logic.
commit
285d237e0bd677e006b3d8ecc8bcbd37c04b2e3b
author
brice · · 2026-02-06 05:56 UTC
parent
cee9def
signature
GPG VerifiedCD88F9E564E0102646FB6077A1A07C7C7C3E349D · arch-gpg Verified for the committer email
1 file · +11 -1
M internal/adapters/in/http/auth/handler_test.go +11 -1
@@ -463,7 +463,7 @@ func TestHandler_isInternalAuth(t *testing.T) {
463463 wantResult: false,
464464 },
465465 {
466- name: "partially empty internal auth",
466+ name: "partially empty internal auth - empty password",
467467 username: "gordon-internal",
468468 password: "internal-secret",
469469 internalAuth: InternalAuth{
@@ -472,6 +472,16 @@ func TestHandler_isInternalAuth(t *testing.T) {
472472 },
473473 wantResult: false,
474474 },
475+ {
476+ name: "partially empty internal auth - empty username",
477+ username: "gordon-internal",
478+ password: "internal-secret",
479+ internalAuth: InternalAuth{
480+ Username: "",
481+ Password: "internal-secret",
482+ },
483+ wantResult: false,
484+ },
475485 }
476486
477487 for _, tt := range tests {
@@ -463,7 +463,7 @@ func TestHandler_isInternalAuth(t *testing.T) {
463 wantResult: false,463 wantResult: false,
464 },464 },
465 {465 {
466- name: "partially empty internal auth",466+ name: "partially empty internal auth - empty password",
467 username: "gordon-internal",467 username: "gordon-internal",
468 password: "internal-secret",468 password: "internal-secret",
469 internalAuth: InternalAuth{469 internalAuth: InternalAuth{
@@ -472,6 +472,16 @@ func TestHandler_isInternalAuth(t *testing.T) {
472 },472 },
473 wantResult: false,473 wantResult: false,
474 },474 },
475+ {
476+ name: "partially empty internal auth - empty username",
477+ username: "gordon-internal",
478+ password: "internal-secret",
479+ internalAuth: InternalAuth{
480+ Username: "",
481+ Password: "internal-secret",
482+ },
483+ wantResult: false,
484+ },
475 }485 }
476 486
477 for _, tt := range tests {487 for _, tt := range tests {

Keyboard shortcuts

Single-key shortcuts are on. ctrl k and esc work whatever this says.

ctrl k :
go to a section, branch, tag, file or repository
?
this list
j k
move the row cursor down / up
enter
open the row under the cursor; on Files, a directory opens or closes in place
↑ ↓ ← → home end
in the file tree (focus on a row): previous / next row, close / open a directory or go to its parent / first entry, first / last row. They never open a file
l h
on Files: open a directory or a file / close a directory or go up to its parent, then to the parent page. Elsewhere: into the side panel (on a merge request, its files) / back to the content; there j k move between its controls
g g G
first / last row
/
filter the list; on Files, find a file anywhere in the repository (esc clears, enter opens the best match)
b
switch branch or tag
g s g f g c g b g m g i g r g a
tabs: summary, files, commits, branches & tags, merge requests, issues, releases, actions (when available)
g h
home
w
wide mode: use the whole width of the screen, or not (remembered)
y
copy a permalink pinned to the commit
d u x
diff: split / unified / collapse or expand all
e E
diff: the whole file in view / every file, with the lines the hunks hide
n
merge request: next note in the diff
esc
close a dialog, leave a field

Branches 7

Tags 106

↑↓ moveenter openesc close