Merge pull request #21 from labd/feature/product-projection-search-without-filter

fix productProjection search without filters

Author: mikedebock

src/repositories/product-projection.ts

@@ -50,7 +50,8 @@ export class ProductProjectionRepository extends AbstractResourceRepository {
   }
 
   search(projectKey: string, query: ParsedQs) {
-    const wherePredicate = parseFilterExpression(query.filter as any)
+    const filter = (query['filter.query'] ?? query.filter) as any
+    const wherePredicate = filter ? parseFilterExpression(filter) : undefined
 
     const results = this._storage.query(projectKey, this.getTypeId(), {
       where: wherePredicate,

src/services/product-projection.test.ts

@@ -66,8 +66,10 @@ describe('Product Projection', () => {
 
     expect(response.status).toBe(404)
   })
+})
 
-  test('Search product projection', async () => {
+describe('Product Projection Search', () => {
+  beforeAll(() => {
     ctMock.project('dummy').add('product-projection', {
       id: '',
       version: 1,
@@ -83,7 +85,9 @@ describe('Product Projection', () => {
       lastModifiedAt: '',
       categories: [],
     })
+  })
 
+  test('Search product projection', async () => {
     const response = await supertest(ctMock.app).get(
       '/dummy/product-projections/search?' +
         qs.stringify({
@@ -113,4 +117,31 @@ describe('Product Projection', () => {
       ],
     })
   })
+
+  test('Search product projection with query.filter', async () => {
+    const response = await supertest(ctMock.app).get(
+      '/dummy/product-projections/search?' +
+        qs.stringify({
+          'query.filter': ['masterVariant.sku:"1337"'],
+        })
+    )
+
+    const projection: ProductProjection = response.body
+    expect(projection).toMatchObject({
+      count: 1,
+      results: [{ masterVariant: { id: 1, sku: '1337' } }],
+    })
+  })
+
+  test('Search product projection without filter', async () => {
+    const response = await supertest(ctMock.app).get(
+      '/dummy/product-projections/search'
+    )
+
+    const projection: ProductProjection = response.body
+    expect(projection).toMatchObject({
+      count: 1,
+      results: [{ masterVariant: { id: 1, sku: '1337' } }],
+    })
+  })
 })