From a763fed573e0c2df9be704be11fd3dd3ecfb031e Mon Sep 17 00:00:00 2001 From: kolaente Date: Fri, 5 Jun 2026 00:27:06 +0200 Subject: [PATCH] test(api/v2): assert both ETags non-empty in team permission test --- pkg/webtests/huma_team_test.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/pkg/webtests/huma_team_test.go b/pkg/webtests/huma_team_test.go index 2a677f827..a7e2b39e3 100644 --- a/pkg/webtests/huma_team_test.go +++ b/pkg/webtests/huma_team_test.go @@ -401,7 +401,10 @@ func TestHumaTeam_ETagReflectsPermission(t *testing.T) { member := humaRequest(t, e, http.MethodGet, "/api/v2/teams/1", "", humaTokenFor(t, &testuser2), "") require.Equal(t, http.StatusOK, member.Code, "body: %s", member.Body.String()) + // Both must be present; otherwise a missing member ETag would let the + // NotEqual below pass while silently dropping the conditional-request header. assert.NotEmpty(t, admin.Header().Get("ETag")) + assert.NotEmpty(t, member.Header().Get("ETag")) assert.NotEqual(t, admin.Header().Get("ETag"), member.Header().Get("ETag"), "same team, different caller permission must produce different ETags") }