feat(engine): 🏄‍♂️Engine tests are passing.

lowdefy · May 28, 2022 · d737e1c · d737e1c
1 parent 92182a2
commit d737e1c
Show file tree

Hide file tree

Showing 31 changed files with 433 additions and 435 deletions.
diff --git a/package.json b/package.json
@@ -52,7 +52,7 @@
     "start:server:next-dev:docs": "yarn start:cli:build:docs && yarn && yarn workspace @lowdefy/server dev",
     "start:server-dev:next-dev:app": "yarn workspace lowdefy start build --config-directory ../../app --server-directory ../server-dev --output-directory ../ && yarn workspace @lowdefy/server-dev next dev",
     "test": "lerna run test",
-    "test:ci": "yarn test --ignore='@lowdefy/engine' --ignore='@lowdefy/format' --ignore='@lowdefy/blocks-*' --ignore='@lowdefy/plugin-aws'"
+    "test:ci": "yarn test --ignore='@lowdefy/format' --ignore='@lowdefy/blocks-*' --ignore='@lowdefy/plugin-aws'"
   },
   "devDependencies": {
     "@yarnpkg/pnpify": "3.1.1-rc.13",

diff --git a/packages/engine/src/actions/createCallMethod.test.js b/packages/engine/src/actions/createCallMethod.test.js
@@ -17,20 +17,7 @@ import { jest } from '@jest/globals';
 
 import testContext from '../../test/testContext.js';
 
-const lowdefy = {
-  _internal: {
-    actions: {
-      CallMethod: ({ methods: { callMethod }, params }) => {
-        return callMethod(params);
-      },
-    },
-    blockComponents: {
-      Button: { meta: { category: 'display' } },
-      List: { meta: { category: 'list' } },
-      TextInput: { meta: { category: 'input' } },
-    },
-  },
-};
+const lowdefy = {};
 
 const RealDate = Date;
 const mockDate = jest.fn(() => ({ date: 0 }));
@@ -91,7 +78,7 @@ test('CallMethod with no args, synchronous method', async () => {
   textInput.registerMethod('blockMethod', blockMethod);
   const res = await button.triggerEvent({ name: 'onClick' });
   expect(res).toEqual({
-    id: 'button',
+    blockId: 'button',
     bounced: false,
     event: undefined,
     eventName: 'onClick',
@@ -166,7 +153,7 @@ test('CallMethod method return a promise', async () => {
   textInput.registerMethod('blockMethod', blockMethod);
   const res = await button.triggerEvent({ name: 'onClick' });
   expect(res).toEqual({
-    id: 'button',
+    blockId: 'button',
     bounced: false,
     event: undefined,
     eventName: 'onClick',
@@ -229,7 +216,7 @@ test('CallMethod with args not an array', async () => {
   textInput.registerMethod('blockMethod', blockMethod);
   const res = await button.triggerEvent({ name: 'onClick' });
   expect(res).toEqual({
-    id: 'button',
+    blockId: 'button',
     bounced: false,
     event: undefined,
     eventName: 'onClick',
@@ -238,7 +225,7 @@ test('CallMethod with args not an array', async () => {
         id: 'a',
         params: {
           args: 'arg',
-          blockId: 'block:root:textInput:0',
+          blockId: 'textInput',
           method: 'blockMethod',
         },
         type: 'CallMethod',
@@ -505,7 +492,7 @@ test('CallMethod with method does not exist', async () => {
       action: {
         id: 'a',
         params: {
-          blockId: 'block:root:textInput:0',
+          blockId: 'textInput',
           method: 'no-method',
         },
         type: 'CallMethod',

diff --git a/packages/engine/src/actions/createDisplayMessage.test.js b/packages/engine/src/actions/createDisplayMessage.test.js
@@ -30,9 +30,6 @@ const lowdefy = {
         });
       },
     },
-    blockComponents: {
-      Button: { meta: { category: 'display' } },
-    },
     displayMessage: mockMessage,
   },
 };

diff --git a/packages/engine/src/actions/createGetActions.test.js b/packages/engine/src/actions/createGetActions.test.js
@@ -25,9 +25,6 @@ const lowdefy = {
       },
       Custom: () => 'response',
     },
-    blockComponents: {
-      Button: { meta: { category: 'display' } },
-    },
   },
 };
 

diff --git a/packages/engine/src/actions/createGetBlockId.test.js b/packages/engine/src/actions/createGetBlockId.test.js
@@ -24,9 +24,6 @@ const lowdefy = {
         return getBlockId(params);
       },
     },
-    blockComponents: {
-      Button: { meta: { category: 'display' } },
-    },
   },
 };
 

diff --git a/packages/engine/src/actions/createGetEvent.test.js b/packages/engine/src/actions/createGetEvent.test.js
@@ -24,9 +24,6 @@ const lowdefy = {
         return getEvent(params);
       },
     },
-    blockComponents: {
-      Button: { meta: { category: 'display' } },
-    },
   },
 };
 

diff --git a/packages/engine/src/actions/createGetGlobal.test.js b/packages/engine/src/actions/createGetGlobal.test.js
@@ -24,9 +24,6 @@ const lowdefy = {
         return getGlobal(params);
       },
     },
-    blockComponents: {
-      Button: { meta: { category: 'display' } },
-    },
   },
   lowdefyGlobal: {
     some: 'data',
@@ -52,7 +49,7 @@ afterAll(() => {
 test('getGlobal params is true', async () => {
   const pageConfig = {
     id: 'root',
-    tyoe: 'Box',
+    type: 'Box',
     blocks: [
       {
         id: 'button',

diff --git a/packages/engine/src/actions/createGetInput.test.js b/packages/engine/src/actions/createGetInput.test.js
@@ -24,12 +24,9 @@ const lowdefy = {
         return getInput(params);
       },
     },
-    blockComponents: {
-      Button: { meta: { category: 'display' } },
-    },
   },
   inputs: {
-    test: { some: 'data' },
+    'page:root': { some: 'data' },
   },
 };
 

diff --git a/packages/engine/src/actions/createGetPageId.test.js b/packages/engine/src/actions/createGetPageId.test.js
@@ -24,9 +24,6 @@ const lowdefy = {
         return getPageId(params);
       },
     },
-    blockComponents: {
-      Button: { meta: { category: 'display' } },
-    },
   },
 };
 

diff --git a/packages/engine/src/actions/createGetRequestDetails.test.js b/packages/engine/src/actions/createGetRequestDetails.test.js
@@ -43,12 +43,6 @@ const lowdefy = {
       Action: ({ methods: { getRequestDetails }, params }) => {
         return getRequestDetails(params);
       },
-      Request: ({ methods: { request }, params }) => {
-        return request(params);
-      },
-    },
-    blockComponents: {
-      Button: { meta: { category: 'display' } },
     },
     callRequest: mockCallRequest,
   },
@@ -81,7 +75,8 @@ test('getRequestDetails params is true', async () => {
     type: 'Box',
     requests: [
       {
-        requestId: 'req_one',
+        id: 'req_one',
+        type: 'Fetch',
       },
     ],
     blocks: [
@@ -142,7 +137,8 @@ test('getRequestDetails params is req_one', async () => {
     type: 'Box',
     requests: [
       {
-        requestId: 'req_one',
+        id: 'req_one',
+        type: 'Fetch',
       },
     ],
     blocks: [
@@ -201,7 +197,8 @@ test('getRequestDetails params is none', async () => {
     type: 'Box',
     requests: [
       {
-        requestId: 'req_one',
+        id: 'req_one',
+        type: 'Fetch',
       },
     ],
     blocks: [
@@ -267,7 +264,8 @@ test('getRequestDetails params.key is null', async () => {
     type: 'Box',
     requests: [
       {
-        requestId: 'req_one',
+        id: 'req_one',
+        type: 'Fetch',
       },
     ],
     blocks: [
@@ -325,7 +323,8 @@ test('getRequestDetails params.all is true', async () => {
     type: 'Box',
     requests: [
       {
-        requestId: 'req_one',
+        id: 'req_one',
+        type: 'Fetch',
       },
     ],
     blocks: [
@@ -388,7 +387,8 @@ test('getRequestDetails params.key is not string or int', async () => {
     type: 'Box',
     requests: [
       {
-        requestId: 'req_one',
+        id: 'req_one',
+        type: 'Fetch',
       },
     ],
     blocks: [
@@ -463,7 +463,8 @@ test('getRequestDetails params.key is req_one', async () => {
     type: 'Box',
     requests: [
       {
-        requestId: 'req_one',
+        id: 'req_one',
+        type: 'Fetch',
       },
     ],
     blocks: [

diff --git a/packages/engine/src/actions/createGetState.test.js b/packages/engine/src/actions/createGetState.test.js
@@ -24,9 +24,6 @@ const lowdefy = {
         return getState(params);
       },
     },
-    blockComponents: {
-      Button: { meta: { category: 'display' } },
-    },
   },
 };
 
@@ -55,7 +52,7 @@ test('getState params is true', async () => {
         {
           id: 'initState',
           type: 'SetState',
-          params: { textInput: 'init' },
+          params: { some: 'data' },
         },
       ],
     },

diff --git a/packages/engine/src/actions/createGetUrlQuery.test.js b/packages/engine/src/actions/createGetUrlQuery.test.js
@@ -24,9 +24,6 @@ const lowdefy = {
         return getUrlQuery(params);
       },
     },
-    blockComponents: {
-      Button: { meta: { category: 'display' } },
-    },
   },
   urlQuery: {
     some: 'data',

diff --git a/packages/engine/src/actions/createGetUser.test.js b/packages/engine/src/actions/createGetUser.test.js
@@ -24,9 +24,6 @@ const lowdefy = {
         return getUser(params);
       },
     },
-    blockComponents: {
-      Button: { meta: { category: 'display' } },
-    },
   },
   user: {
     some: 'data',

diff --git a/packages/engine/src/actions/createLink.test.js b/packages/engine/src/actions/createLink.test.js
@@ -34,11 +34,6 @@ const lowdefy = {
         }
       },
     },
-    blockComponents: {
-      Button: { meta: { category: 'display' } },
-      List: { meta: { category: 'list' } },
-      TextInput: { meta: { category: 'input' } },
-    },
     link: jest.fn(),
   },
 };

diff --git a/packages/engine/src/actions/createLogin.test.js b/packages/engine/src/actions/createLogin.test.js
@@ -24,9 +24,6 @@ const lowdefy = {
         return login(params);
       },
     },
-    blockComponents: {
-      Button: { meta: { category: 'display' } },
-    },
     auth: {
       login: jest.fn(),
     },

diff --git a/packages/engine/src/actions/createLogout.test.js b/packages/engine/src/actions/createLogout.test.js
@@ -24,9 +24,6 @@ const lowdefy = {
         return logout();
       },
     },
-    blockComponents: {
-      Button: { meta: { category: 'display' } },
-    },
     auth: {
       logout: jest.fn(),
     },

diff --git a/packages/engine/src/actions/createRequest.test.js b/packages/engine/src/actions/createRequest.test.js
@@ -50,9 +50,6 @@ const lowdefy = {
         return request(params);
       },
     },
-    blockComponents: {
-      Button: { meta: { category: 'display' } },
-    },
     callRequest: mockCallRequest,
   },
 };
@@ -84,7 +81,8 @@ test('Request call one request', async () => {
     type: 'Box',
     requests: [
       {
-        requestId: 'req_one',
+        id: 'req_one',
+        type: 'Fetch',
       },
     ],
     blocks: [
@@ -133,10 +131,12 @@ test('Request call all requests', async () => {
     type: 'Box',
     requests: [
       {
-        requestId: 'req_one',
+        id: 'req_one',
+        type: 'Fetch',
       },
       {
-        requestId: 'req_two',
+        id: 'req_two',
+        type: 'Fetch',
       },
     ],
     blocks: [
@@ -204,10 +204,12 @@ test('Request call array of requests', async () => {
     type: 'Box',
     requests: [
       {
-        requestId: 'req_one',
+        id: 'req_one',
+        type: 'Fetch',
       },
       {
-        requestId: 'req_two',
+        id: 'req_two',
+        type: 'Fetch',
       },
     ],
     blocks: [
@@ -275,10 +277,12 @@ test('Request pass if params are none', async () => {
     type: 'Box',
     requests: [
       {
-        requestId: 'req_one',
+        id: 'req_one',
+        type: 'Fetch',
       },
       {
-        requestId: 'req_two',
+        id: 'req_two',
+        type: 'Fetch',
       },
     ],
     blocks: [
@@ -306,7 +310,8 @@ test('Request call request error', async () => {
     type: 'Box',
     requests: [
       {
-        requestId: 'req_error',
+        id: 'req_error',
+        type: 'Fetch',
       },
     ],
     blocks: [

diff --git a/packages/engine/src/actions/createReset.js b/packages/engine/src/actions/createReset.js
@@ -22,7 +22,6 @@ function createReset({ context }) {
     context._internal.RootBlocks.reset(
       serializer.deserializeFromString(context._internal.State.frozenState)
     );
-    context._internal.update();
   };
 }