From 89e2944e1f8578f221e87908184b5daf0eea7b9b Mon Sep 17 00:00:00 2001
From: blam <ben@blam.sh>
Date: Thu, 3 Jun 2021 11:03:20 +0200
Subject: [PATCH] chore: fixing tests and making the thing nice

Signed-off-by: blam <ben@blam.sh>
---
 .../src/components/RadarComponent.test.tsx    | 33 ++++++++++++++++---
 1 file changed, 28 insertions(+), 5 deletions(-)

diff --git a/plugins/tech-radar/src/components/RadarComponent.test.tsx b/plugins/tech-radar/src/components/RadarComponent.test.tsx
index 59cababbc9..4b740597b5 100644
--- a/plugins/tech-radar/src/components/RadarComponent.test.tsx
+++ b/plugins/tech-radar/src/components/RadarComponent.test.tsx
@@ -24,6 +24,7 @@ import { withLogCollector } from '@backstage/test-utils';
 
 import GetBBoxPolyfill from '../utils/polyfills/getBBox';
 import RadarComponent from './RadarComponent';
+import { TechRadarLoaderResponse, techRadarApiRef, TechRadarApi } from '../api';
 
 describe('RadarComponent', () => {
   beforeAll(() => {
@@ -34,13 +35,30 @@ describe('RadarComponent', () => {
     GetBBoxPolyfill.remove();
   });
 
+  class MockClient implements TechRadarApi {
+    async load(): Promise<TechRadarLoaderResponse> {
+      return {
+        entries: [],
+        quadrants: [],
+        rings: [],
+      };
+    }
+  }
+
+  const mockClient = new MockClient();
+
   it('should render a progress bar', async () => {
     jest.useFakeTimers();
 
     const errorApi = { post: () => {} };
     const { getByTestId, queryByTestId } = render(
       <ThemeProvider theme={lightTheme}>
-        <ApiProvider apis={ApiRegistry.from([[errorApiRef, errorApi]])}>
+        <ApiProvider
+          apis={ApiRegistry.from([
+            [errorApiRef, errorApi],
+            [techRadarApiRef, mockClient],
+          ])}
+        >
           <RadarComponent
             width={1200}
             height={800}
@@ -61,16 +79,21 @@ describe('RadarComponent', () => {
 
   it('should call the errorApi if load fails', async () => {
     const errorApi = { post: jest.fn() };
-    const techRadarLoadFail = () =>
-      Promise.reject(new Error('404 Page Not Found'));
+    jest
+      .spyOn(mockClient, 'load')
+      .mockRejectedValue(new Error('404 Page Not Found'));
 
     const { queryByTestId } = render(
       <ThemeProvider theme={lightTheme}>
-        <ApiProvider apis={ApiRegistry.from([[errorApiRef, errorApi]])}>
+        <ApiProvider
+          apis={ApiRegistry.from([
+            [errorApiRef, errorApi],
+            [techRadarApiRef, mockClient],
+          ])}
+        >
           <RadarComponent
             width={1200}
             height={800}
-            getData={techRadarLoadFail}
             svgProps={{ 'data-testid': 'tech-radar-svg' }}
           />
         </ApiProvider>