Handle missing image "src" properties

index.js

@@ -3,6 +3,10 @@ var loaderUtils = require('loader-utils');
 var steed = require('steed');
 
 function resolveImageSrc(loaderContext, image, callback) {
+  if (typeof image.src !== 'string') {
+    return callback(new Error('Missing image "src" property in Web App Manifest'));
+  }
+
   var dirname = path.dirname(loaderContext.resourcePath);
 
   // Resolve the image filename relative to the manifest file
@@ -34,7 +38,7 @@ function resolveImages(loaderContext, manifest, key, callback) {
     return callback(null);
   }
 
-  steed.each(manifest[key], resolveImageSrc.bind(null, loaderContext), function(err) {
+  steed.map(manifest[key], resolveImageSrc.bind(null, loaderContext), function(err) {
     if (err) {
       return callback(err);
     }

test/error-cases/missing-src/expected-error.js

@@ -0,0 +1 @@
+module.exports = 'Error: Missing image "src" property in Web App Manifest';

test/error-cases/missing-src/index.js

@@ -0,0 +1 @@
+require('./manifest.json');

test/error-cases/missing-src/manifest.json

@@ -0,0 +1,9 @@
+{
+  "name": "Missing Src",
+  "icons": [
+    {
+      "sizes": "192x192",
+      "type": "image/png"
+    }
+  ]
+}

test/error-cases/missing-src/webpack.config.js

@@ -0,0 +1,19 @@
+module.exports = {
+  entry: {
+    main: __dirname + '/index.js'
+  },
+
+  output: {
+    filename: 'index.js',
+    path: __dirname + '/actual-output'
+  },
+
+  module: {
+    loaders: [
+      {
+        test: /manifest.json$/,
+        loader: 'file?name=manifest.json!../../../index.js'
+      }
+    ]
+  }
+};