chore: remove support for deprecate sourceMap option

mjeanroy · mjeanroy · commit afed2dba9add · 2020-04-06T22:13:00.000+02:00
diff --git a/src/rollup-plugin-prettier.js b/src/rollup-plugin-prettier.js
@@ -36,7 +36,6 @@ import prettier from 'prettier';
  */
 const OPTIONS = new Set([
   'sourcemap',
-  'sourceMap',
   'cwd',
 ]);
 
@@ -78,9 +77,6 @@ export class RollupPluginPrettier {
     // Check if sourcemap is enabled by default.
     if (hasIn(options, 'sourcemap')) {
       this._sourcemap = options.sourcemap;
-    } else if (hasIn(options, 'sourceMap')) {
-      console.warn(`[${this.name}] The sourceMap option is deprecated, please use sourcemap instead.`);
-      this._sourcemap = options.sourceMap;
     } else {
       this._sourcemap = null;
     }
diff --git a/test/index.spec.js b/test/index.spec.js
@@ -29,7 +29,7 @@ import {verifyWarnLogsBecauseOfSourcemap} from './utils/verify-warn-logs-because
 import {verifyWarnLogsNotTriggered} from './utils/verify-warn-logs-not-triggered.js';
 import {installWarnSpy} from './utils/install-warn-spy.js';
 
-describe('rollup-plugin-prettier [stable]', () => {
+describe('rollup-plugin-prettier', () => {
   beforeEach(() => {
     installWarnSpy();
   });
@@ -94,25 +94,6 @@ describe('rollup-plugin-prettier [stable]', () => {
     );
   });
 
-  it('should run prettier with sourcemap (camelcase) in plugin options', () => {
-    const instance = rollupPluginPrettier({
-      sourceMap: true,
-      parser: 'babel',
-    });
-
-    const code = 'var foo=0;var test="hello world";';
-    const chunk = {isEntry: false, imports: []};
-    const outputOptions = {};
-    const result = instance.renderChunk(code, chunk, outputOptions);
-
-    verifyWarnLogsBecauseOfSourcemap();
-    expect(result.map).toBeDefined();
-    expect(result.code).toBe(
-        'var foo = 0;\n' +
-        'var test = "hello world";\n'
-    );
-  });
-
   it('should run prettier with sourcemap disabled in output options', () => {
     const instance = rollupPluginPrettier({
       parser: 'babel',
diff --git a/test/it/it.spec.js b/test/it/it.spec.js
@@ -240,42 +240,6 @@ describe('rollup-plugin-prettier', () => {
         });
   });
 
-  it('should enable sourcemap (camelcase) on plugin', (done) => {
-    const output = path.join(tmpDir.name, 'bundle.js');
-    const config = {
-      input: getBundlePath(),
-
-      output: {
-        file: output,
-        format: 'es',
-      },
-
-      plugins: [
-        prettier({
-          parser: 'babel',
-          sourceMap: true,
-        }),
-      ],
-    };
-
-    rollup.rollup(config)
-        .then((bundle) => bundle.write(config.output))
-        .then(() => {
-          fs.readFile(output, 'utf8', (err, data) => {
-            if (err) {
-              done.fail(err);
-              return;
-            }
-
-            verifyWarnLogsBecauseOfSourcemap();
-            done();
-          });
-        })
-        .catch((err) => {
-          done.fail(err);
-        });
-  });
-
   /**
    * Get the output bundle absolute path.
    *
diff --git a/test/rollup-plugin-prettier.spec.js b/test/rollup-plugin-prettier.spec.js
@@ -78,25 +78,6 @@ describe('RollupPluginPrettier', () => {
     );
   });
 
-  it('should run prettier with sourceMap (camelcase)', () => {
-    const plugin = new RollupPluginPrettier({
-      parser: 'babel',
-      sourceMap: true,
-    });
-
-    const code = 'var foo=0;var test="hello world";';
-    const result = plugin.reformat(code);
-
-    expect(plugin.getSourcemap()).toBe(true);
-
-    verifyWarnLogsBecauseOfSourcemap();
-    expect(result.map).toBeDefined();
-    expect(result.code).toBe(
-        'var foo = 0;\n' +
-        'var test = "hello world";\n'
-    );
-  });
-
   it('should run prettier with sourcemap if it has been enabled', () => {
     const plugin = new RollupPluginPrettier({
       parser: 'babel',
