refactoring

src/commands/Commands.js

@@ -4,8 +4,7 @@ const {NativeNavigation} = NativeModules;
 import * as LayoutBuilder from './LayoutBuilder';
 
 export function startApp(params) {
-  const layout = LayoutBuilder.parse(params);
-  NativeNavigation.startApp(layout);
+  NativeNavigation.startApp(LayoutBuilder.parse(params));
 }
 //
 //function parseParams(params) {

src/commands/Commands.test.js

@@ -36,37 +36,5 @@ describe('Commands', () => {
       });
     });
 
-    it('adds uniqueId to passed sideMenu', () => {
-      uut.startApp({
-        container: {
-          key: 'com.example.MyScreen'
-        },
-        sideMenu: {
-          left: {
-            key: 'com.example.SideMenu1'
-          },
-          right: {
-            key: 'com.example.SideMenu2'
-          }
-        }
-      });
-
-      expect(mockNativeNavigation.startApp).toHaveBeenCalledWith({
-        container: {
-          key: 'com.example.MyScreen',
-          id: 'container123'
-        },
-        sideMenu: {
-          left: {
-            key: 'com.example.SideMenu1',
-            id: 'container123'
-          },
-          right: {
-            key: 'com.example.SideMenu2',
-            id: 'container123'
-          }
-        }
-      });
-    });
   });
 });

src/commands/LayoutBuilder.js

@@ -1,8 +1,11 @@
+import _ from 'lodash';
 import {uniqueId} from '../providers/UniqueIdProvider';
 
 export function parse(params) {
-  const layout = params;
-  layout.container.id = uniqueId(`container`);
+  const layout = _.assign({}, params);
+  if (layout.container) {
+    layout.container.id = uniqueId(`container`);
+  }
   if (layout.sideMenu) {
     if (layout.sideMenu.left) {
       layout.sideMenu.left.id = uniqueId(`container`);

src/commands/LayoutBuilder.test.js

@@ -0,0 +1,46 @@
+describe('LayoutBuilder', () => {
+  let uut;
+
+  beforeEach(() => {
+    jest.mock('../providers/UniqueIdProvider');
+    uut = require('./LayoutBuilder');
+  });
+
+  describe('parse', () => {
+    it('returns a new object', () => {
+      const input = {};
+      expect(uut.parse(input)).not.toBe(input);
+    });
+
+    it('adds uniqueId to passed sideMenu', () => {
+      expect(uut.parse({
+        container: {
+          key: 'com.example.MyScreen'
+        },
+        sideMenu: {
+          left: {
+            key: 'com.example.SideMenu1'
+          },
+          right: {
+            key: 'com.example.SideMenu2'
+          }
+        }
+      })).toEqual({
+        container: {
+          key: 'com.example.MyScreen',
+          id: 'container123'
+        },
+        sideMenu: {
+          left: {
+            key: 'com.example.SideMenu1',
+            id: 'container123'
+          },
+          right: {
+            key: 'com.example.SideMenu2',
+            id: 'container123'
+          }
+        }
+      });
+    });
+  });
+});