Fix source map disabling

TrySound · Aug 30, 2018 · 24faf66 · 24faf66
1 parent 55c4d6f
commit 24faf66
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -7,7 +7,8 @@ function uglify(userOptions = {}) {
   }
 
   const options = Object.assign({}, userOptions, {
-    sourceMap: userOptions.sourcemap !== false
+    sourceMap: userOptions.sourcemap !== false,
+    sourcemap: undefined
   });
 
   return {

diff --git a/test/test.js b/test/test.js
@@ -37,12 +37,21 @@ test("minify with sourcemaps", async () => {
   expect(result.map).toBeTruthy();
 });
 
+test("allow to disable source maps", async () => {
+  const bundle = await rollup({
+    input: "test/fixtures/sourcemap.js",
+    plugins: [uglify({ sourcemap: false })]
+  });
+  await bundle.generate({ format: "cjs" });
+});
+
 test("does not allow to pass sourceMap", async () => {
   try {
     const bundle = await rollup({
       input: "test/fixtures/sourcemap.js",
       plugins: [uglify({ sourceMap: false })]
     });
+    await bundle.generate({ format: "cjs" });
     expect(true).toBeFalsy();
   } catch (error) {
     expect(error.toString()).toMatch(/sourceMap option is removed/);