Add tests on facets

Author: bidoubiwa

src/client/index.ts

@@ -21,11 +21,6 @@ export function instantMeiliSearch(
           indexName: indexUid,
         } = isSearchRequest
 
-        // console.log({
-        //   params: instantSearchParams,
-        //   indexName: indexUid,
-        // })
-
         const { paginationTotalHits, primaryKey, placeholderSearch } = options
         const { page, hitsPerPage } = instantSearchParams
         const client = this.MeiliSearchClient
@@ -37,9 +32,6 @@ export function instantMeiliSearch(
           hitsPerPage: hitsPerPage === undefined ? 20 : hitsPerPage, // 20 is the MeiliSearch's default limit value. `hitsPerPage` can be changed with `InsantSearch.configure`.
           page: page || 0, // default page is 0 if none is provided
         }
-
-        // console.log({instantSearchParams, context})
-
         // Transform IS params to MeiliSearch params
         const msSearchParams = transformToMeiliSearchParams(
           instantSearchParams,
@@ -58,8 +50,6 @@ export function instantMeiliSearch(
           instantSearchParams,
           context
         )
-        // console.log({ ISresponse })
-
         return ISresponse
       } catch (e) {
         console.error(e)

src/types/types.ts

@@ -1,8 +1,17 @@
 import * as MStypes from 'meilisearch'
 import * as IStypes from './instantsearch-types'
 
-export type ISSearchParams = IStypes.SearchRequestParameters &
-  MStypes.SearchParams<any>
+export type FacetFilter = Array<string | string[]>
+
+export type IMSearchParams = Omit<IStypes.SearchParameters, 'facetFilters'> & {
+  query?: string
+  facetFilters?: FacetFilter | FacetFilter[]
+}
+
+export type IMSearchRequest = {
+  indexName: string
+  params: IMSearchParams
+}
 
 export type InstantMeiliSearchOptions = {
   paginationTotalHits?: number
@@ -20,7 +29,7 @@ export type ISHits<T = Record<string, any>> = T & {
 }
 
 export type IMResponse = {
-  facets?: Record<string, object | undefined>
+  facets?: Record<string, Record<string, number> | undefined>
   exhaustiveFacetsCount?: boolean
   exhaustiveNbHits: boolean
   nbPages?: number
@@ -54,7 +63,7 @@ export type SnippetsParams = {
 
 export type CreateHighlighResult = (
   highLightParams: HighLightParams & FormattedHit
-) => { formattedHit: any } & ISSearchParams
+) => { formattedHit: any } & IMSearchParams
 
 export type ReplaceHighlightTags = (
   value: string,
@@ -64,7 +73,7 @@ export type ReplaceHighlightTags = (
 
 export type CreateSnippetResult = (
   snippetsParams: HighLightParams & SnippetsParams & FormattedHit
-) => { formattedHit: any } & ISSearchParams
+) => { formattedHit: any } & IMSearchParams
 
 export type SnippetValue = (
   value: string,
@@ -74,20 +83,20 @@ export type SnippetValue = (
 ) => string
 
 export type TransformToMeiliSearchParams = (
-  instantSearchParams: ISSearchParams,
+  instantSearchParams: IMSearchParams,
   instantMeiliSearchContext: InstantMeiliSearchContext
 ) => Record<string, any>
 
 export type TransformToISResponse = (
   indexUid: string,
   meiliSearchResponse: MStypes.SearchResponse<any, any>,
-  instantSearchParams: ISSearchParams,
+  instantSearchParams: IMSearchParams,
   instantMeiliSearchContext: InstantMeiliSearchContext
 ) => { results: SearchResponse[] }
 
 export type TransformToISHitsm = (
   meiliSearchHits: Array<Record<string, any>>,
-  instantSearchParams: ISSearchParams,
+  instantSearchParams: IMSearchParams,
   instantMeiliSearchContext: InstantMeiliSearchContext
 ) => ISHits[]
 
@@ -104,7 +113,7 @@ export type PaginateHits = (
 export type InstantMeiliSearchInstance = {
   MeiliSearchClient: MStypes.MeiliSearch
   search: (
-    requests: IStypes.SearchRequest[]
+    requests: IMSearchRequest[]
   ) => Promise<{ results: SearchResponse[] }>
 }
 

tests/facets.tests.ts

@@ -24,26 +24,13 @@ describe('Instant MeiliSearch Browser test', () => {
         indexName: 'movies',
         params: {
           query: '',
-          facetFilters: ['genres:action'],
-        },
-      },
-    ])
-    const hits = response.results[0].hits
-    expect(hits.length).toEqual(2)
-  })
-
-  test('Test multiple on facetsFilters', async () => {
-    const response = await searchClient.search([
-      {
-        indexName: 'movies',
-        params: {
-          query: '',
-          facetFilters: ['genres:action', 'genres:Thriller', 'genres:crime'],
+          facetFilters: ['genres:Adventure'],
         },
       },
     ])
     const hits = response.results[0].hits
     expect(hits.length).toEqual(1)
+    expect(hits[0].title).toEqual('Star Wars')
   })
 
   test('Test multiple on facetsFilters', async () => {
@@ -52,26 +39,25 @@ describe('Instant MeiliSearch Browser test', () => {
         indexName: 'movies',
         params: {
           query: '',
-          facetFilters: ['genres:action', 'genres:Thriller', 'genres:crime'],
+          facetFilters: ['genres:Comedy', 'genres:Crime'],
         },
       },
     ])
     const hits = response.results[0].hits
-    expect(hits.length).toEqual(1)
+    expect(hits.length).toEqual(2)
+    expect(hits[0].title).toEqual('Ariel')
   })
 
-  test('Test one facet on facetsDistribution', async () => {
-    const response = await searchClient.search([
-      {
-        indexName: 'movies',
-        params: {
-          query: '',
-          facets: ['genres'],
-        },
+  test('Test multiple nested on facetsFilters', async () => {
+    const params = {
+      indexName: 'movies',
+      params: {
+        query: '',
+        facetFilters: [['genres:action', 'genres:Thriller'], 'genres:crime'],
       },
-    ])
-    console.log(response.results[0])
-    // console.log(response.results[0].hits)
-    // expect(response.results[0].facets.genres).toEqual(1)
+    }
+    const response = await searchClient.search([params])
+    const hits = response.results[0].hits
+    expect(hits[0].title).toEqual('Judgment Night')
   })
 })

tests/facetsDistribution.tests.ts

@@ -0,0 +1,59 @@
+import { searchClient, dataset } from './assets/utils'
+
+describe('Instant MeiliSearch Browser test', () => {
+  beforeAll(async () => {
+    try {
+      await searchClient.MeiliSearchClient.deleteIndex('movies')
+    } catch (e) {
+      // movies does not exist
+    }
+    await searchClient.MeiliSearchClient.index(
+      'movies'
+    ).updateAttributesForFaceting(['genres'])
+    const moviesUpdate = await searchClient.MeiliSearchClient.index(
+      'movies'
+    ).addDocuments(dataset)
+    await searchClient.MeiliSearchClient.index('movies').waitForPendingUpdate(
+      moviesUpdate.updateId
+    )
+  })
+
+  test('Test empty array on facetsDistribution', async () => {
+    const response = await searchClient.search([
+      {
+        indexName: 'movies',
+        params: {
+          query: '',
+          facets: [],
+        },
+      },
+    ])
+    expect(response.results[0].facets?.genres).toEqual(undefined)
+  })
+
+  test('Test one facet on facetsDistribution', async () => {
+    const response = await searchClient.search([
+      {
+        indexName: 'movies',
+        params: {
+          query: '',
+          facets: ['genres'],
+        },
+      },
+    ])
+    expect(response.results[0].facets?.genres?.Action).toEqual(2)
+  })
+
+  test('Test non-existent facets on facetsDistribution', async () => {
+    const response = await searchClient.search([
+      {
+        indexName: 'movies',
+        params: {
+          query: '',
+          facets: ['genres', 'notKnown'],
+        },
+      },
+    ])
+    expect(response.results[0].facets?.genres?.Action).toEqual(2)
+  })
+})