more places to check

Author: lee-leonardo

lib/projectBuilder.js

@@ -115,6 +115,10 @@ function createApps(w3cManifestInfo, rootDir, platforms, options, href, callback
     }
   }
 
+  if (!w3cManifestInfo || !w3cManifestInfo.content) {
+    return Q.reject(new Error("missing manifest information")).nodeify(callback);
+  }
+
   if (!href) {
     href = '/';
   }

lib/validations.js

@@ -1,4 +1,4 @@
-'use strict';
+'use strict';
 
 var platformTools = require('./platformTools'),
   manifestTools = require('./manifestTools');
@@ -36,7 +36,11 @@ function manifestFormatValid(format) {
 }
 
 function isExpectedValidationError(errorResult) {
-  const checkForPurposeList = errorResult.member.includes("icons") && errorResult.member.includes("purpose")
+  if (!errorResult.member) {
+    return false;
+  }
+
+  const checkForPurposeList = errorResult.member.includes("icons") && errorResult.member.includes("purpose");
 
   return checkForPurposeList
 }
@@ -46,7 +50,7 @@ function isExpectedCase(errorResult, w3cManifestInfo) {
     return true;
   }
 
-  const errorParams = errorResult.member.split('/').slice(1) //example: /icons/0/purpose -> ['icons', '0', 'purpose']
+  const errorParams = errorResult.member.split('/').slice(1); //example: /icons/0/purpose -> ['icons', '0', 'purpose']
   const isIconsPurpose = errorResult.member.includes("icons") && errorResult.member.includes("purpose");
   const isShortcuts = errorResult.member.includes("shortcuts");
   const isScreenshots = errorResult.member.includes("screenshots"); // in the case we need to extend it to screenshots as well...
@@ -56,7 +60,7 @@ function isExpectedCase(errorResult, w3cManifestInfo) {
     return true;
 
   } else  if (isIconsPurpose) {
-    const [icons, index, purpose] = errorParams
+    const [icons, index, purpose] = errorParams;
 
     return w3cManifestInfo.content.icons
       && w3cManifestInfo.content.icons[index]