From 891fcd5482f57e93da64b58654a74989c219b7cf Mon Sep 17 00:00:00 2001
From: shmaram <shaharshmaram@gmail.com>
Date: Fri, 22 Dec 2023 14:02:26 +0200
Subject: [PATCH] Fix tests

Signed-off-by: shmaram <shaharshmaram@gmail.com>
---
 plugins/home/src/api/VisitsStorageApi.test.ts |  2 +-
 .../VisitedByType/Content.test.tsx            | 41 +++++++++++++++++++
 2 files changed, 42 insertions(+), 1 deletion(-)

diff --git a/plugins/home/src/api/VisitsStorageApi.test.ts b/plugins/home/src/api/VisitsStorageApi.test.ts
index 6faeb62350..e361c624b9 100644
--- a/plugins/home/src/api/VisitsStorageApi.test.ts
+++ b/plugins/home/src/api/VisitsStorageApi.test.ts
@@ -250,7 +250,7 @@ describe('VisitsStorageApi.create', () => {
       ]);
     });
 
-    it('filters by timestamp with >', async () => {
+    it('filters by timestamp with value >', async () => {
       const visits = await api.list({
         filterBy: [{ field: 'timestamp', operator: '>', value: baseDate }],
       });
diff --git a/plugins/home/src/homePageComponents/VisitedByType/Content.test.tsx b/plugins/home/src/homePageComponents/VisitedByType/Content.test.tsx
index c6c0e6d81a..1234291f1a 100644
--- a/plugins/home/src/homePageComponents/VisitedByType/Content.test.tsx
+++ b/plugins/home/src/homePageComponents/VisitedByType/Content.test.tsx
@@ -197,6 +197,47 @@ describe('<Content kind="recent"/>', () => {
       });
     });
   });
+
+  it('allows recent items to have no filter if the filter config is not valid', async () => {
+    const configApiMock = new MockConfigApi({
+      home: {
+        recentVisits: {
+          filterBy: [
+            {
+              operator: '==',
+              value: '/tech-radar',
+            },
+          ],
+        },
+      },
+    });
+
+    const listSpy = jest.spyOn(mockVisitsApi, 'list');
+
+    await renderInTestApp(
+      <TestApiProvider
+        apis={[
+          [configApiRef, configApiMock],
+          [visitsApiRef, mockVisitsApi],
+        ]}
+      >
+        <ContextProvider>
+          <Content kind="recent" />
+        </ContextProvider>
+      </TestApiProvider>,
+    );
+    await waitFor(() => {
+      expect(listSpy).toHaveBeenCalledWith({
+        limit: 8,
+        orderBy: [
+          {
+            direction: 'desc',
+            field: 'timestamp',
+          },
+        ],
+      });
+    });
+  });
 });
 
 describe('<Content kind="top"/>', () => {