test(e2e): assert requested audit model

Author: SantiagoDePolonia

internal/admin/handler_test.go

@@ -823,6 +823,54 @@ func TestAuditLog_Success(t *testing.T) {
 	}
 }
 
+func TestAuditLog_EmitsRequestedModel(t *testing.T) {
+	now := time.Now().UTC()
+	reader := &mockAuditReader{
+		logResult: &auditlog.LogListResult{
+			Entries: []auditlog.LogEntry{
+				{
+					ID:             "log-1",
+					Timestamp:      now,
+					RequestedModel: "does-not-exist-model",
+					StatusCode:     http.StatusBadRequest,
+					RequestID:      "req-1",
+					Method:         http.MethodPost,
+					Path:           "/v1/chat/completions",
+					ErrorType:      string(core.ErrorTypeInvalidRequest),
+				},
+			},
+			Total:  1,
+			Limit:  25,
+			Offset: 0,
+		},
+	}
+
+	h := NewHandler(nil, nil, WithAuditReader(reader))
+	c, rec := newHandlerContext("/admin/api/v1/audit/log?search=req-1")
+
+	if err := h.AuditLog(c); err != nil {
+		t.Fatalf("unexpected error: %v", err)
+	}
+	if rec.Code != http.StatusOK {
+		t.Fatalf("expected 200, got %d", rec.Code)
+	}
+
+	var result struct {
+		Entries []struct {
+			RequestedModel string `json:"requested_model"`
+		} `json:"entries"`
+	}
+	if err := json.Unmarshal(rec.Body.Bytes(), &result); err != nil {
+		t.Fatalf("failed to unmarshal: %v", err)
+	}
+	if len(result.Entries) != 1 {
+		t.Fatalf("expected 1 entry, got %d", len(result.Entries))
+	}
+	if result.Entries[0].RequestedModel != "does-not-exist-model" {
+		t.Fatalf("requested_model = %q, want does-not-exist-model", result.Entries[0].RequestedModel)
+	}
+}
+
 func TestAuditLog_EnrichesEntriesWithUsageSummary(t *testing.T) {
 	now := time.Now().UTC()
 	usageReader := &mockUsageReader{

tests/e2e/release-e2e-scenarios.md

@@ -819,12 +819,12 @@ jq -e '.error.type == "invalid_request_error"' "$BODY_FILE" >/dev/null
 sleep 6
 AUDIT_JSON_FILE="$QA_RUN_DIR/s61.audit.json"
 curl -fsS "$BASE_URL/admin/api/v1/audit/log?search=$REQUEST_ID&limit=5" > "$AUDIT_JSON_FILE"
-jq --arg request_id "$REQUEST_ID" '{total:(.entries|map(select(.request_id==$request_id))|length),entries:(.entries|map(select(.request_id==$request_id))|map({request_id,path,model,resolved_model,provider,status_code,error_type}))}' "$AUDIT_JSON_FILE"
+jq --arg request_id "$REQUEST_ID" '{total:(.entries|map(select(.request_id==$request_id))|length),entries:(.entries|map(select(.request_id==$request_id))|map({request_id,path,requested_model,resolved_model,provider,status_code,error_type}))}' "$AUDIT_JSON_FILE"
 jq -e --arg request_id "$REQUEST_ID" '
     any(.entries[]?;
       .request_id == $request_id
       and .path == "/v1/chat/completions"
-      and .model == "does-not-exist-model"
+      and .requested_model == "does-not-exist-model"
       and .status_code == 400
       and .error_type == "invalid_request_error"
     )
@@ -850,12 +850,12 @@ jq -e '.error.type == "invalid_request_error"' "$BODY_FILE" >/dev/null
 sleep 6
 AUDIT_JSON_FILE="$QA_RUN_DIR/s62.audit.json"
 curl -fsS "$BASE_URL/admin/api/v1/audit/log?search=$REQUEST_ID&limit=5" > "$AUDIT_JSON_FILE"
-jq --arg request_id "$REQUEST_ID" '{total:(.entries|map(select(.request_id==$request_id))|length),entries:(.entries|map(select(.request_id==$request_id))|map({request_id,path,model,provider,status_code,error_type}))}' "$AUDIT_JSON_FILE"
+jq --arg request_id "$REQUEST_ID" '{total:(.entries|map(select(.request_id==$request_id))|length),entries:(.entries|map(select(.request_id==$request_id))|map({request_id,path,requested_model,provider,status_code,error_type}))}' "$AUDIT_JSON_FILE"
 jq -e --arg request_id "$REQUEST_ID" '
     any(.entries[]?;
       .request_id == $request_id
       and .path == "/p/not-a-real-provider/responses"
-      and .model == "gpt-4.1-nano"
+      and .requested_model == "gpt-4.1-nano"
       and .provider == "not-a-real-provider"
       and .status_code == 400
       and .error_type == "invalid_request_error"