fixed tests, removed verbose logging

lib/src/commands/OptionsProcessor.js

@@ -11,7 +11,7 @@ class OptionsProcessor {
       if (_.isEqual(key, 'icon') || _.endsWith(key, 'Icon') || _.endsWith(key, 'Image')) {
         navigationOptions[key] = resolveAssetSource(navigationOptions[key]);
       }
-      if (_.isPlainObject(value) || _.isArray(value)) {
+      if (_.isObject(value) || _.isArray(value)) {
         OptionsProcessor.processOptions(value);
       }
     });

lib/src/commands/OptionsProcessor.test.js

@@ -1,4 +1,5 @@
 const OptionsProcessor = require('./OptionsProcessor');
+const NavigationOptions = require('./../params/NavigationOptions');
 
 describe('navigation options', () => {
   let navigationOptions;
@@ -69,16 +70,16 @@ describe('navigation options', () => {
   });
 
   it('any nested recursive keys ending with Color', () => {
-    navigationOptions.innerObj = { theKeyColor: 'red' };
-    navigationOptions.innerObj.innerMostObj = { anotherColor: 'yellow' };
+    navigationOptions.topBar = { textColor: 'red' };
+    navigationOptions.topBar.innerMostObj = { anotherColor: 'yellow' };
     OptionsProcessor.processOptions(navigationOptions);
-    expect(navigationOptions.innerObj.theKeyColor).toEqual(0xffff0000);
-    expect(navigationOptions.innerObj.innerMostObj.anotherColor).toEqual(0xffffff00);
+    expect(navigationOptions.topBar.textColor).toEqual(0xffff0000);
+    expect(navigationOptions.topBar.innerMostObj.anotherColor).toEqual(0xffffff00);
   });
 
   it('resolve image sources with name/ending with icon', () => {
     navigationOptions.icon = 'require("https://wix.github.io/react-native-navigation/_images/logo.png");';
-    navigationOptions.innerObj = {
+    navigationOptions.topBar = {
       myIcon: 'require("https://wix.github.io/react-native-navigation/_images/logo.png");',
       myOtherValue: 'value'
     };
@@ -88,7 +89,18 @@ describe('navigation options', () => {
     // I assign the icons to strings (what the require would generally look like)
     // and expect the value to be resovled, in this case it doesn't find anything and returns null
     expect(navigationOptions.icon).toEqual(null);
-    expect(navigationOptions.innerObj.myIcon).toEqual(null);
-    expect(navigationOptions.innerObj.myOtherValue).toEqual('value');
+    expect(navigationOptions.topBar.myIcon).toEqual(null);
+    expect(navigationOptions.topBar.myOtherValue).toEqual('value');
+  });
+
+  it('processes NavigationOptions jsDoc object', () => {
+    navigationOptions.someKeyColor = 'rgb(255, 0, 255)';
+    navigationOptions.topBar = { textColor: 'red' };
+    navigationOptions.topBar.innerMostObj = { anotherColor: 'yellow' };
+    const navigationOptionsObj = new NavigationOptions(navigationOptions);
+
+    OptionsProcessor.processOptions(navigationOptionsObj);
+
+    expect(navigationOptionsObj.topBar.textColor).toEqual(0xffff0000);
   });
 });

scripts/test.e2e.ios.js

@@ -7,6 +7,6 @@ run();
 
 function run() {
   const conf = release ? `release` : `debug`;
-  exec.execSync(`detox build --configuration ios.sim.${conf} && detox test --configuration ios.sim.${conf} ${process.env.CI ? '--cleanup' : ''} --loglevel verbose`);
+  exec.execSync(`detox build --configuration ios.sim.${conf} && detox test --configuration ios.sim.${conf} ${process.env.CI ? '--cleanup' : ''}`);
 }