added pure-component option

Author: Keyan Zhang

transforms/__testfixtures__/class-pure-mixin3.input.js

@@ -0,0 +1,20 @@
+// for this file we disable the `pure-component` option
+// so the output should be just nothing
+var React = require('React');
+var ReactComponentWithPureRenderMixin = require('ReactComponentWithPureRenderMixin');
+
+var ComponentWithOnlyPureRenderMixin = React.createClass({
+  mixins: [ReactComponentWithPureRenderMixin],
+
+  getInitialState: function() {
+    return {
+      counter: this.props.initialNumber + 1,
+    };
+  },
+
+  render: function() {
+    return (
+      <div>{this.state.counter}</div>
+    );
+  },
+});

transforms/__testfixtures__/class-pure-mixin3.output.js

@@ -14,19 +14,23 @@ const defineTest = require('jscodeshift/dist/testUtils').defineTest;
 
 const pureMixinAlternativeOption = {
   'mixin-module-name': 'ReactComponentWithPureRenderMixin',
+  'pure-component': true,
 };
 
+const enableFlowOption = {flow: true};
+
 defineTest(__dirname, 'class');
-defineTest(__dirname, 'class', {flow: true}, 'class-anonymous');
+defineTest(__dirname, 'class', enableFlowOption, 'class-anonymous');
 defineTest(__dirname, 'class', pureMixinAlternativeOption, 'class-test2');
-defineTest(__dirname, 'class', {flow: true}, 'export-default-class');
+defineTest(__dirname, 'class', enableFlowOption, 'export-default-class');
 defineTest(__dirname, 'class', pureMixinAlternativeOption, 'class-pure-mixin1');
-defineTest(__dirname, 'class', {flow: true}, 'class-pure-mixin2');
-defineTest(__dirname, 'class', {flow: true}, 'class-initial-state');
-defineTest(__dirname, 'class', {flow: true}, 'class-property-field');
-defineTest(__dirname, 'class', {flow: true}, 'class-flow1');
-defineTest(__dirname, 'class', {flow: true}, 'class-flow2');
-defineTest(__dirname, 'class', {flow: true}, 'class-flow3');
-defineTest(__dirname, 'class', {flow: true}, 'class-flow4');
-defineTest(__dirname, 'class', {flow: true}, 'class-flow5');
-defineTest(__dirname, 'class', {flow: true}, 'class-flow6');
+defineTest(__dirname, 'class', {...enableFlowOption, 'pure-component': true}, 'class-pure-mixin2');
+defineTest(__dirname, 'class', null, 'class-pure-mixin3');
+defineTest(__dirname, 'class', enableFlowOption, 'class-initial-state');
+defineTest(__dirname, 'class', enableFlowOption, 'class-property-field');
+defineTest(__dirname, 'class', enableFlowOption, 'class-flow1');
+defineTest(__dirname, 'class', enableFlowOption, 'class-flow2');
+defineTest(__dirname, 'class', enableFlowOption, 'class-flow3');
+defineTest(__dirname, 'class', enableFlowOption, 'class-flow4');
+defineTest(__dirname, 'class', enableFlowOption, 'class-flow5');
+defineTest(__dirname, 'class', enableFlowOption, 'class-flow6');

transforms/__tests__/class-test.js

@@ -998,7 +998,7 @@ module.exports = (file, api, options) => {
     const mixinsFilter = (classPath) => {
       if (!ReactUtils.hasMixins(classPath)) {
         return true;
-      } else if (pureRenderMixinPathAndBinding) {
+      } else if (options['pure-component'] && pureRenderMixinPathAndBinding) {
         const {binding} = pureRenderMixinPathAndBinding;
         if (areMixinsConvertible([binding], classPath)) {
           return true;