fix: APPS-2668 SearchHome.vue query param should be q (#507)

fix: query param in searchhome Co-authored-by: Jess Divers <[email protected]>
UCLALibrary · May 9, 2024 · 3ab4e57 · 3ab4e57 · github-actions · May 9, 2024
1 parent 7d8e2dd
commit 3ab4e57
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/lib-components/SearchHome.vue b/src/lib-components/SearchHome.vue
@@ -86,7 +86,7 @@ function doSearch() {
   if (isSiteSearch.value) {
     router.push({
       path: actionUrl.value,
-      query: { [queryParam as any]: searchWords.value },
+      query: { [queryParam.value as string]: searchWords.value },
     })
   }
   else {

diff --git a/src/stories/SearchHome.spec.js b/src/stories/SearchHome.spec.js
@@ -7,4 +7,11 @@ describe('Search / Search Home', () => {
 
     cy.percySnapshot('Search / Home: Default')
   })
+  it('uses q for query parameter', () => {
+    cy.visit('/iframe.html?id=search-search-home--default&args=&viewMode=story')
+    // click the library search tab, then submit empty search
+    cy.contains('button', 'Site Search').click()
+    cy.get('button[class="button-submit"]').click()
+    cy.url().should('include', '?q=')
+  })
 })