Fix ProductBadge query value (#305)

* Reset queryParams on ProductBadge click * Add todo test
hashicorp-forge · Aug 28, 2023 · 497f46f · 497f46f
1 parent 282626a
commit 497f46f
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/web/app/components/product-badge-link.ts b/web/app/components/product-badge-link.ts
@@ -28,6 +28,11 @@ export default class ProductBadgeLinkComponent extends Component<ProductBadgeLin
     if (this.args.productArea) {
       return {
         product: [this.args.productArea],
+        docType: [],
+        owners: [],
+        page: 1,
+        sortBy: "dateDesc",
+        status: [],
       };
     } else {
       return {};

diff --git a/web/tests/acceptance/authenticated/all-test.ts b/web/tests/acceptance/authenticated/all-test.ts
@@ -63,4 +63,16 @@ module("Acceptance | authenticated/all", function (hooks) {
       .dom(PRODUCT_BADGE_LINK_SELECTOR)
       .hasAttribute("href", "/all?product=%5B%22Labs%22%5D");
   });
+
+  /**
+   * We want to test that clicking the product badge replaces filters
+   * rather than compound them, but we don't yet have the Mirage
+   * factories to support this.
+   */
+  todo(
+    "product badges have the correct hrefs when other filters are active",
+    async function (this: AuthenticatedAllRouteTestContext, assert) {
+      assert.true(false);
+    }
+  );
 });