From 6bd6ddb34001330bc921e452c9207a623be9b386 Mon Sep 17 00:00:00 2001
From: Ben Irvin <ben@innerdvations.com>
Date: Thu, 17 Nov 2022 12:15:42 +0100
Subject: [PATCH] test invalid strategy

---
 .../lib/engine/__tests__/engine.test.ts       | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

diff --git a/packages/core/data-transfer/lib/engine/__tests__/engine.test.ts b/packages/core/data-transfer/lib/engine/__tests__/engine.test.ts
index 068960d62e..3c9d3b187c 100644
--- a/packages/core/data-transfer/lib/engine/__tests__/engine.test.ts
+++ b/packages/core/data-transfer/lib/engine/__tests__/engine.test.ts
@@ -188,6 +188,7 @@ describe('Transfer engine', () => {
       await mergeEngine.transfer();
       expect(mergeEngine).toBeValidTransferEngine();
 
+      // undefined strategy
       await expect(
         (async () => {
           const invalidEngine = createTransferEngine(
@@ -198,15 +199,21 @@ describe('Transfer engine', () => {
           await invalidEngine.transfer();
         })()
       ).rejects.toThrow();
+
+      // invalid strategy
+      await expect(
+        (async () => {
+          const invalidEngine = createTransferEngine(mockedSource, mockedDestination, {
+            ...engineOptions,
+            strategy: 'foo',
+          } as unknown as ITransferEngineOptions);
+          await invalidEngine.transfer();
+        })()
+      ).rejects.toThrow();
     });
 
     test('calls all provider stages', async () => {
-      const engineOptions = {
-        versionMatching: 'exact',
-        exclude: [],
-        strategy: 'restore',
-      } as unknown as ITransferEngineOptions;
-      const engine = createTransferEngine(mockedSource, mockedDestination, engineOptions);
+      const engine = createTransferEngine(mockedSource, mockedDestination, defaultOptions);
 
       await engine.transfer();