Ensure options are always an object

babel · Mar 6, 2017 · e8aa302 · e8aa302
1 parent b9209e7
commit e8aa302
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 1 deletion.
diff --git a/src/index.js b/src/index.js
@@ -93,7 +93,7 @@ module.exports = function(source, inputSourceMap) {
   const filename = webpackRemainingChain[webpackRemainingChain.length - 1];
 
   // Handle options
-  const loaderOptions = loaderUtils.getOptions(this);
+  const loaderOptions = loaderUtils.getOptions(this) || {};
   const defaultOptions = {
     metadataSubscribers: [],
     inputSourceMap: inputSourceMap,

diff --git a/test/loader.test.js b/test/loader.test.js
@@ -116,3 +116,29 @@ test.cb("should use correct env", (t) => {
     t.end();
   });
 });
+
+test.cb("should not throw without config", (t) => {
+  const config = {
+    entry: path.join(__dirname, "fixtures/basic.js"),
+    output: {
+      path: t.context.directory,
+    },
+    module: {
+      loaders: [
+        {
+          test: /\.jsx?/,
+          loader: babelLoader,
+          exclude: /node_modules/,
+        },
+      ],
+    },
+  };
+
+  webpack(config, (err, stats) => {
+    t.is(err, null);
+
+    t.true(stats.compilation.errors.length === 0);
+
+    t.end();
+  });
+});