Use our client mock in our test suite

This was causing some test failures as I forgot to update the client mock

Author: rylnd

x-pack/plugins/siem/server/lib/detection_engine/routes/__mocks__/request_context.ts

@@ -13,14 +13,15 @@ import {
 import { alertsClientMock } from '../../../../../../alerting/server/mocks';
 import { actionsClientMock } from '../../../../../../actions/server/mocks';
 import { licensingMock } from '../../../../../../licensing/server/mocks';
+import { siemMock } from '../../../../mocks';
 
 const createMockClients = () => ({
   actionsClient: actionsClientMock.create(),
   alertsClient: alertsClientMock.create(),
   clusterClient: elasticsearchServiceMock.createScopedClusterClient(),
   licensing: { license: licensingMock.createLicenseMock() },
   savedObjectsClient: savedObjectsClientMock.create(),
-  siemClient: { signalsIndex: 'mockSignalsIndex' },
+  siemClient: siemMock.createClient(),
 });
 
 const createRequestContextMock = (

x-pack/plugins/siem/server/lib/detection_engine/routes/rules/import_rules_route.test.ts

@@ -126,6 +126,7 @@ describe('import_rules_route', () => {
     });
 
     test('returns an error if the index does not exist', async () => {
+      clients.siemClient.getSignalsIndex.mockReturnValue('mockSignalsIndex');
       clients.clusterClient.callAsCurrentUser.mockResolvedValue(getEmptyIndex());
       const response = await server.inject(request, context);
       expect(response.status).toEqual(200);