Extend hosts API client with search param

CMK-35352

Change-Id: I428861ec1d0f018ec70c32d59dea5563a22d7dde

Author: makanakoeln

packages/cmk-frontend-vue/src/monitoring/all-hosts/api/hosts.ts

@@ -12,11 +12,15 @@ import type { HostsRequest, HostsResponse } from '../../shared/api/types'
 export interface HostQueryParams {
   limit?: number
   sort?: SortingState
+  searchQuery?: string
 }
 
 export class HostApi {
   public async fetchHosts(params: HostQueryParams = {}): Promise<HostsResponse> {
     const sort = (params.sort ?? []).map((s) => `${s.id}:${s.desc ? 'desc' : 'asc'}`)
+    // Drop an empty or whitespace-only search query so an empty search matches a request without
+    // the `q` param.
+    const searchQuery = params.searchQuery?.trim()
     // The schema types limit as string and sort as string (not string[]); openapi-fetch's
     // defaultQuerySerializer handles string[] as repeated params at runtime, so the cast
     // is only needed to bridge the incorrectly-generated schema types.
@@ -25,7 +29,8 @@ export class HostApi {
         params: {
           query: {
             ...(params.limit !== undefined && { limit: String(params.limit) }),
-            ...(sort.length > 0 && { sort })
+            ...(sort.length > 0 && { sort }),
+            ...(searchQuery !== undefined && searchQuery !== '' && { q: searchQuery })
           } as unknown as NonNullable<HostsRequest>
         }
       })

packages/cmk-frontend-vue/tests/monitoring/all-hosts/api/hosts.test.ts

@@ -130,6 +130,42 @@ describe('HostApi.fetchHosts', () => {
     await expect(new HostApi().fetchHosts()).rejects.toThrow()
   })
 
+  it('forwards a non-empty search query as the q param', async () => {
+    mockSuccess(makeHostsResponse([]))
+
+    await new HostApi().fetchHosts({ searchQuery: 'web-server' })
+
+    expect(getSpy).toHaveBeenCalledWith('/monitor/hosts', {
+      params: { query: { q: 'web-server' } }
+    })
+  })
+
+  it('omits the q param when the search query is empty', async () => {
+    mockSuccess(makeHostsResponse([]))
+
+    await new HostApi().fetchHosts({ searchQuery: '' })
+
+    expect(getSpy).toHaveBeenCalledWith('/monitor/hosts', { params: { query: {} } })
+  })
+
+  it('omits the q param when the search query is only whitespace', async () => {
+    mockSuccess(makeHostsResponse([]))
+
+    await new HostApi().fetchHosts({ searchQuery: '   ' })
+
+    expect(getSpy).toHaveBeenCalledWith('/monitor/hosts', { params: { query: {} } })
+  })
+
+  it('keeps other params when omitting an empty q', async () => {
+    mockSuccess(makeHostsResponse([]))
+
+    await new HostApi().fetchHosts({ limit: 50, searchQuery: '' })
+
+    expect(getSpy).toHaveBeenCalledWith('/monitor/hosts', {
+      params: { query: { limit: '50' } }
+    })
+  })
+
   it('returns the response data from the API', async () => {
     const hosts = [makeHost({ name: 'db-1', state: 'DOWN' }), makeHost({ name: 'web-1' })]
     const response = makeHostsResponse(hosts)