From 657190e759bf93b135ab42bef41698aade0545e0 Mon Sep 17 00:00:00 2001
From: "claude[bot]" <41898282+claude[bot]@users.noreply.github.com>
Date: Thu, 12 Feb 2026 15:02:18 +0000
Subject: [PATCH] fix(frontend): address latest CodeRabbit review suggestions

- Use valid sort value "runs" instead of undefined in MainSearchResultPage
  test defaultProps to match production default and satisfy type contract
- Remove redundant marketplacePage.goto() navigation in E2E test since
  the page is already at /marketplace after login

Co-authored-by: Ubbe <0ubbe@users.noreply.github.com>
---
 .../MainSearchResultPage/__tests__/auth-state.test.tsx          | 2 +-
 .../MainSearchResultPage/__tests__/error-handling.test.tsx      | 2 +-
 .../MainSearchResultPage/__tests__/no-results.test.tsx          | 2 +-
 .../MainSearchResultPage/__tests__/rendering.test.tsx           | 2 +-
 autogpt_platform/frontend/src/tests/marketplace-agent.spec.ts   | 1 -
 5 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/autogpt_platform/frontend/src/app/(platform)/marketplace/components/MainSearchResultPage/__tests__/auth-state.test.tsx b/autogpt_platform/frontend/src/app/(platform)/marketplace/components/MainSearchResultPage/__tests__/auth-state.test.tsx
index db3717a18e..28795c0a6e 100644
--- a/autogpt_platform/frontend/src/app/(platform)/marketplace/components/MainSearchResultPage/__tests__/auth-state.test.tsx
+++ b/autogpt_platform/frontend/src/app/(platform)/marketplace/components/MainSearchResultPage/__tests__/auth-state.test.tsx
@@ -9,7 +9,7 @@ import {
 
 const defaultProps = {
   searchTerm: "test-search",
-  sort: undefined,
+  sort: "runs" as const,
 };
 
 describe("MainSearchResultPage - Auth State", () => {
diff --git a/autogpt_platform/frontend/src/app/(platform)/marketplace/components/MainSearchResultPage/__tests__/error-handling.test.tsx b/autogpt_platform/frontend/src/app/(platform)/marketplace/components/MainSearchResultPage/__tests__/error-handling.test.tsx
index 50f4ebe3ce..08d37086ae 100644
--- a/autogpt_platform/frontend/src/app/(platform)/marketplace/components/MainSearchResultPage/__tests__/error-handling.test.tsx
+++ b/autogpt_platform/frontend/src/app/(platform)/marketplace/components/MainSearchResultPage/__tests__/error-handling.test.tsx
@@ -10,7 +10,7 @@ import { create500Handler } from "@/tests/integrations/helpers/create-500-handle
 
 const defaultProps = {
   searchTerm: "test-search",
-  sort: undefined,
+  sort: "runs" as const,
 };
 
 describe("MainSearchResultPage - Error Handling", () => {
diff --git a/autogpt_platform/frontend/src/app/(platform)/marketplace/components/MainSearchResultPage/__tests__/no-results.test.tsx b/autogpt_platform/frontend/src/app/(platform)/marketplace/components/MainSearchResultPage/__tests__/no-results.test.tsx
index 1d87d9a1cb..2a171d35a5 100644
--- a/autogpt_platform/frontend/src/app/(platform)/marketplace/components/MainSearchResultPage/__tests__/no-results.test.tsx
+++ b/autogpt_platform/frontend/src/app/(platform)/marketplace/components/MainSearchResultPage/__tests__/no-results.test.tsx
@@ -6,7 +6,7 @@ import { http, HttpResponse } from "msw";
 
 const defaultProps = {
   searchTerm: "nonexistent-search-term-xyz",
-  sort: undefined,
+  sort: "runs" as const,
 };
 
 describe("MainSearchResultPage - No Results", () => {
diff --git a/autogpt_platform/frontend/src/app/(platform)/marketplace/components/MainSearchResultPage/__tests__/rendering.test.tsx b/autogpt_platform/frontend/src/app/(platform)/marketplace/components/MainSearchResultPage/__tests__/rendering.test.tsx
index f450e54263..e7d67fb908 100644
--- a/autogpt_platform/frontend/src/app/(platform)/marketplace/components/MainSearchResultPage/__tests__/rendering.test.tsx
+++ b/autogpt_platform/frontend/src/app/(platform)/marketplace/components/MainSearchResultPage/__tests__/rendering.test.tsx
@@ -4,7 +4,7 @@ import { MainSearchResultPage } from "../MainSearchResultPage";
 
 const defaultProps = {
   searchTerm: "test-search",
-  sort: undefined,
+  sort: "runs" as const,
 };
 
 describe("MainSearchResultPage - Rendering", () => {
diff --git a/autogpt_platform/frontend/src/tests/marketplace-agent.spec.ts b/autogpt_platform/frontend/src/tests/marketplace-agent.spec.ts
index 92bb2b6d57..6d9fb3b71c 100644
--- a/autogpt_platform/frontend/src/tests/marketplace-agent.spec.ts
+++ b/autogpt_platform/frontend/src/tests/marketplace-agent.spec.ts
@@ -22,7 +22,6 @@ test.describe("Marketplace Agent Page - Cross-Page Flows", () => {
     const richUser = getTestUserWithLibraryAgents();
     await loginPage.login(richUser.email, richUser.password);
     await hasUrl(page, "/marketplace");
-    await marketplacePage.goto(page);
 
     const firstStoreCard = await marketplacePage.getFirstTopAgent();
     await firstStoreCard.click();