Allow this.getState references (not calls) in lifecycle methods.

[benmosher]

I think this should not be considered an error/warning by this rule: (my actual code follows)

componentDidMount() {
    this.handleStoreEvent = _.compose(this.setState, getStoreState).bind(this);
    dependencies.add(this.handleStoreEvent);
},

While I don't know that this is great code per se, it is essentially binding setState to a list of Flux stores via a helper (dependencies). This seems to me like a valid use of setState within componentDidMount.

I refactored both rules pertaining to setState to respond to CallExpression nodes instead of MemberExpression, and it returns if the callee is not a MemberExpression. The original logic follows thereafter.

I can imagine also potentially making this reduction in strength an option, but I believe the behavior here captures the spirit of the rule, and thus an option should not be necessary.

Submitted for your consideration. 👍

[coveralls]

Coverage decreased (-1.57%) to 98.43% when pulling 9e670c7 on benmosher:master into d79eadc on yannickcr:master.

[coveralls]

Coverage remained the same at 100.0% when pulling e42ba31 on benmosher:master into d79eadc on yannickcr:master.

[yannickcr]

Looks good to me

And I think this is the way this rule should work, so an option is not necessary.

Thanks for your contribution :)