piral-core 0.11.8-pre.1620 → 0.11.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "piral-core",
-  "version": "0.11.8-pre.1620",
+  "version": "0.11.8",
   "description": "The core library for creating a Piral instance.",
   "keywords": [
     "portal",
@@ -51,7 +51,7 @@
   },
   "dependencies": {
     "@dbeining/react-atom": "^4.0.0",
-    "piral-base": "^0.11.8-pre.1620"
+    "piral-base": "^0.11.8"
   },
-  "gitHead": "e027010f23a5a9fbda79a2c6443d08a378bad1d0"
+  "gitHead": "617df77a2d553e455ec66b04936967ff3ce21a47"
 }

package/src/components/Mediator.test.tsx

@@ -55,4 +55,4 @@ describe('Component Mediator', () => {
     );
     expect(deref<any>(context.state).app.layout).toEqual('tablet');
   });
-});
+});

package/src/helpers.test.tsx

@@ -3,13 +3,13 @@ import { globalDependencies } from './modules';
 import { PiletMetadata, Pilet, AvailableDependencies } from 'piral-base';
 import { Atom, swap } from '@dbeining/react-atom';
 
-function createMockApi(meta: PiletMetadata){
+function createMockApi(meta: PiletMetadata) {
   return {
     meta,
     emit: jest.fn(),
     off: jest.fn(),
     on: jest.fn(),
-  }as any;
+  } as any;
 }
 
 function createMockContainer() {
@@ -36,13 +36,11 @@ function createMockContainer() {
 }
 
 describe('Piral-Core helpers module', () => {
-
   it('setSharedDependencies set the shared dependecies', () => {
-
     // Arrange
     const dependencies: AvailableDependencies = {
-      "gg": {},
-      "ff": {}
+      gg: {},
+      ff: {},
     };
 
     // Act
@@ -51,14 +49,13 @@ describe('Piral-Core helpers module', () => {
 
     // Assert
     expect(result).not.toBeUndefined();
-  })
+  });
 
   it('extendSharedDependencies should extend the dependecies', () => {
-
     // Arrange
     const additionalDependencies: AvailableDependencies = {
-      "gg": {},
-      "ff": {}
+      gg: {},
+      ff: {},
     };
 
     // Act
@@ -69,14 +66,13 @@ describe('Piral-Core helpers module', () => {
   });
 
   it('createPiletOptions creates the options using the provided pilets', () => {
-
     const wasUndefined = process.env.DEBUG_PIRAL === undefined;
 
-    process.env.DEBUG_PIRAL = "true";
+    process.env.DEBUG_PIRAL = 'true';
 
     // Arrange
     const setupMock = jest.fn();
-    const globalContext = createMockContainer().context
+    const globalContext = createMockContainer().context;
     const providedPilets: Array<Pilet> = [
       {
         setup: setupMock,
@@ -99,8 +95,8 @@ describe('Piral-Core helpers module', () => {
       getDependencies: jest.fn(),
       loadPilet: jest.fn(),
       requestPilets: jest.fn(() => Promise.resolve(providedPilets)),
-      strategy: jest.fn()
-    }
+      strategy: jest.fn(),
+    };
 
     // Act
     const options = createPiletOptions(optionsConfig);
@@ -108,16 +104,15 @@ describe('Piral-Core helpers module', () => {
     // Assert
     expect(options.pilets.length).toEqual(providedPilets.length);
 
-    if(wasUndefined) {
+    if (wasUndefined) {
       process.env.DEBUG_PIRAL = undefined;
     }
   });
 
   it('createPiletOptions creates the options exposing the global dependencies', () => {
-
     // Arrange
     const setupMock = jest.fn();
-    const globalContext = createMockContainer().context
+    const globalContext = createMockContainer().context;
     const providedPilets: Array<Pilet> = [
       {
         setup: setupMock,
@@ -140,8 +135,8 @@ describe('Piral-Core helpers module', () => {
       getDependencies: () => globalDependencies,
       loadPilet: jest.fn(),
       requestPilets: jest.fn(() => Promise.resolve(providedPilets)),
-      strategy: jest.fn()
-    }
+      strategy: jest.fn(),
+    };
 
     // Act
     const options = createPiletOptions(optionsConfig);
@@ -151,14 +146,13 @@ describe('Piral-Core helpers module', () => {
   });
 
   it('createPiletOptions creates the options with provided requestPilets', () => {
-
     const wasUndefined = process.env.DEBUG_PIRAL === undefined;
 
     // Arrange
-    process.env.DEBUG_PIRAL = "true";
+    process.env.DEBUG_PIRAL = 'true';
     const setupMock = jest.fn();
     const requestPilets = jest.fn(() => Promise.resolve(providedPilets));
-    const globalContext = createMockContainer().context
+    const globalContext = createMockContainer().context;
     const providedPilets: Array<Pilet> = [
       {
         setup: setupMock,
@@ -181,8 +175,8 @@ describe('Piral-Core helpers module', () => {
       getDependencies: jest.fn(),
       loadPilet: jest.fn(),
       requestPilets: requestPilets,
-      strategy: jest.fn()
-    }
+      strategy: jest.fn(),
+    };
 
     // Act
     const options = createPiletOptions(optionsConfig);
@@ -191,7 +185,7 @@ describe('Piral-Core helpers module', () => {
     // Assert
     expect(requestPilets).toHaveBeenCalled();
 
-    if(wasUndefined) {
+    if (wasUndefined) {
       process.env.DEBUG_PIRAL = undefined;
     }
   });
@@ -200,10 +194,10 @@ describe('Piral-Core helpers module', () => {
     const wasUndefined = process.env.DEBUG_PILET === undefined;
 
     // Arrange
-    process.env.DEBUG_PILET = "localhost:1234";
+    process.env.DEBUG_PILET = 'localhost:1234';
     const setupMock = jest.fn();
     const requestPilets = jest.fn(() => Promise.resolve(providedPilets));
-    const globalContext = createMockContainer().context
+    const globalContext = createMockContainer().context;
     const providedPilets: Array<Pilet> = [
       {
         setup: setupMock,
@@ -226,8 +220,8 @@ describe('Piral-Core helpers module', () => {
       getDependencies: jest.fn(),
       loadPilet: jest.fn(),
       requestPilets: requestPilets,
-      strategy: jest.fn()
-    }
+      strategy: jest.fn(),
+    };
 
     let hasFailed = false;
 
@@ -235,15 +229,14 @@ describe('Piral-Core helpers module', () => {
     const options = createPiletOptions(optionsConfig);
     try {
       options.fetchPilets(); //This call should not work in node.js test environment
-    }
-    catch {
+    } catch {
       hasFailed = true;
     }
 
     // Assert
     expect(hasFailed).toBeTruthy();
 
-    if(wasUndefined) {
+    if (wasUndefined) {
       process.env.DEBUG_PILET = undefined;
     }
   });

package/src/state/withApi.test.tsx

@@ -2,6 +2,25 @@ import * as React from 'react';
 import * as hooks from '../hooks';
 import { mount } from 'enzyme';
 import { withApi } from './withApi';
+import { ComponentConverters } from '../types';
+import { StateContext } from '../state';
+import { Atom } from '@dbeining/react-atom';
+import { unmountComponentAtNode } from 'react-dom';
+
+function createMockContainer() {
+  const state = Atom.of({
+    portals: {},
+  });
+  return {
+    context: {
+      on: jest.fn(),
+      off: jest.fn(),
+      emit: jest.fn(),
+      state,
+      destroyPortal: id => {},
+    } as any,
+  };
+}
 
 jest.mock('../hooks');
 
@@ -12,6 +31,10 @@ jest.mock('../hooks');
     },
   });
 
+(hooks as any).useActions = () => ({
+  destroyPortal: jest.fn(),
+});
+
 const StubErrorInfo: React.FC = props => <div />;
 StubErrorInfo.displayName = 'StubErrorInfo';
 
@@ -60,4 +83,42 @@ describe('withApi Module', () => {
     mount(<Component shouldCrash />);
     expect(console.error).toHaveBeenCalled();
   });
+
+  it('Wraps component of type object', () => {
+    const api: any = {};
+    const converters: ComponentConverters<any> = {
+      html: component => {
+        return component.component;
+      },
+    };
+    const context = createMockContainer();
+    const Component = withApi(converters, { type: 'html', component: { mount: () => {} } }, api, 'unknown');
+
+    const node = mount(
+      <StateContext.Provider value={context.context}>
+        <Component />
+      </StateContext.Provider>,
+    );
+
+    expect(node.children.length).toBe(1);
+  });
+
+  it('Wraps component which is object == null.', () => {
+    const api: any = {};
+    const converters: ComponentConverters<any> = {
+      html: component => {
+        return component.component;
+      },
+    };
+    const context = createMockContainer();
+    const Component = withApi(converters, null, api, 'unknown');
+
+    const node = mount(
+      <StateContext.Provider value={context.context}>
+        <Component />
+      </StateContext.Provider>,
+    );
+
+    expect(Component.displayName).toBeUndefined();
+  });
 });

package/src/utils/foreign.test.ts

@@ -40,4 +40,4 @@ describe('Util Foreign.', () => {
     var result = renderInDom(context, element, DefaultLoadingIndicator, {});
     expect(result).toBeUndefined();
   });
-});
+});