[API] Deprecate the property option (#10)

Author: oliviertassinari

README.md

@@ -53,7 +53,7 @@ class Foo extends React.Component {
 
 **.babelrc**
 
-without options
+without options:
 ```json
 {
   "env": {
@@ -64,18 +64,7 @@ without options
 }
 ```
 
-with options
-```json
-{
-  "env": {
-    "production": {
-      "plugins": ["react-remove-properties", {"property": "data-test"}]
-    }
-  }
-}
-```
-
-the options also accepts an array of property names
+with options. We accepts an array of property names:
 
 ```json
 {

src/index.js

@@ -6,6 +6,12 @@ export default function() {
         const properties = state.opts.properties || [];
 
         if (state.opts.property) {
+          /* eslint-disable no-console */
+          console.warn(`
+            babel-plugin-react-remove-properties:
+            The property option is deprecated, instead use the properties one.
+          `);
+          /* eslint-enable no-console */
           properties.push(state.opts.property);
         }
 

test/index.js

@@ -18,10 +18,10 @@ function getPluginOptionsForDirectory(directory, filename = 'options.json') {
   }
 }
 
-describe('remove react propTypes', () => {
+describe('remove react properties', () => {
   const fixturesDir = path.join(__dirname, 'fixtures');
   fs.readdirSync(fixturesDir).map((caseName) => {
-    it(`should ${caseName.split('-').join(' ')}`, () => {
+    it(`should work with ${caseName.split('-').join(' ')}`, () => {
       const fixtureDir = path.join(fixturesDir, caseName);
       const pluginOptions = getPluginOptionsForDirectory(fixtureDir);
       const actual = transformFileSync(path.join(fixtureDir, 'actual.js'), {
@@ -31,7 +31,7 @@ describe('remove react propTypes', () => {
       }).code;
       const expected = fs.readFileSync(path.join(fixtureDir, 'expected.js')).toString();
 
-      assert.equal(trim(actual), trim(expected));
+      assert.strictEqual(trim(actual), trim(expected));
     });
   });
 });