{children}<\/div>;\n+ void

{children}<\/div>;\n expect(console.error.calls.length).toBe(1);\n expect(console.error.calls[0].args[0]).toContain(\n 'Any use of a keyed object'\n@@ -37,7 +37,7 @@ describe('ReactFragment', function() {\n x: ,\n y: \n };\n-

{sameChildren}<\/div>;\n+ void

{sameChildren}<\/div>;\n expect(console.error.calls.length).toBe(1);\n });\n \n@@ -65,7 +65,7 @@ describe('ReactFragment', function() {\n y: \n };\n var frag = ReactFragment.create(children);\n- frag.x;\n+ void frag.x;\n frag.y = 10;\n expect(console.error.calls.length).toBe(1);\n expect(console.error.calls[0].args[0]).toContain(\ndiff --git a\/src\/addons\/__tests__\/renderSubtreeIntoContainer.js b\/src\/addons\/__tests__\/renderSubtreeIntoContainer.js\nindex 80de6145cf..ba50112f1c 100644\n--- a\/src\/addons\/__tests__\/renderSubtreeIntoContainer.js\n+++ b\/src\/addons\/__tests__\/renderSubtreeIntoContainer.js\n@@ -33,7 +33,7 @@ describe('renderSubtreeIntoContainer', function() {\n \n var Parent = React.createClass({\n childContextTypes: {\n- foo: React.PropTypes.string.isRequired,\n+ foo: React.PropTypes.string.isRequired\n },\n \n getChildContext: function() {\n@@ -72,7 +72,7 @@ describe('renderSubtreeIntoContainer', function() {\n \n var Parent = React.createClass({\n childContextTypes: {\n- foo: React.PropTypes.string.isRequired,\n+ foo: React.PropTypes.string.isRequired\n },\n \n getChildContext: function() {\ndiff --git a\/src\/addons\/__tests__\/update-test.js b\/src\/addons\/__tests__\/update-test.js\nindex 67ef8efb40..e6c78094eb 100644\n--- a\/src\/addons\/__tests__\/update-test.js\n+++ b\/src\/addons\/__tests__\/update-test.js\n@@ -72,7 +72,7 @@ describe('update', function() {\n });\n \n it('should support apply', function() {\n- expect(update(2, {$apply: function(x) { return x * 2; }})).toEqual(4);\n+ expect(update(2, {$apply: (x) => x * 2})).toEqual(4);\n expect(update.bind(null, 2, {$apply: 123})).toThrow(\n 'Invariant Violation: update(): expected spec of $apply to be a ' +\n 'function; got 123.'\ndiff --git a\/src\/addons\/link\/__tests__\/LinkedStateMixin-test.js b\/src\/addons\/link\/__tests__\/LinkedStateMixin-test.js\nindex 56b0ab50fe..8e0ffc1d24 100644\n--- a\/src\/addons\/link\/__tests__\/LinkedStateMixin-test.js\n+++ b\/src\/addons\/link\/__tests__\/LinkedStateMixin-test.js\n@@ -16,13 +16,11 @@\n describe('LinkedStateMixin', function() {\n var LinkedStateMixin;\n var React;\n- var ReactLink;\n var ReactTestUtils;\n \n beforeEach(function() {\n LinkedStateMixin = require('LinkedStateMixin');\n React = require('React');\n- ReactLink = require('ReactLink');\n ReactTestUtils = require('ReactTestUtils');\n });\n \ndiff --git a\/src\/addons\/transitions\/__tests__\/ReactCSSTransitionGroup-test.js b\/src\/addons\/transitions\/__tests__\/ReactCSSTransitionGroup-test.js\nindex 9ddcd78ace..733044fd5c 100644\n--- a\/src\/addons\/transitions\/__tests__\/ReactCSSTransitionGroup-test.js\n+++ b\/src\/addons\/transitions\/__tests__\/ReactCSSTransitionGroup-test.js\n@@ -13,7 +13,6 @@\n \n var React;\n var ReactCSSTransitionGroup;\n-var mocks;\n \n \/\/ Most of the real functionality is covered in other unit tests, this just\n \/\/ makes sure we're wired up correctly.\n@@ -23,7 +22,6 @@ describe('ReactCSSTransitionGroup', function() {\n beforeEach(function() {\n React = require('React');\n ReactCSSTransitionGroup = require('ReactCSSTransitionGroup');\n- mocks = require('mocks');\n \n container = document.createElement('div');\n spyOn(console, 'error');\n@@ -52,7 +50,7 @@ describe('ReactCSSTransitionGroup', function() {\n \n \/\/ For some reason jst is adding extra setTimeout()s and grunt test isn't,\n \/\/ so we need to do this disgusting hack.\n- for (var i = 0 ; i < setTimeout.mock.calls.length; i++) {\n+ for (var i = 0; i < setTimeout.mock.calls.length; i++) {\n if (setTimeout.mock.calls[i][1] === 5000) {\n setTimeout.mock.calls[i][0]();\n break;\ndiff --git a\/src\/addons\/transitions\/__tests__\/ReactTransitionGroup-test.js b\/src\/addons\/transitions\/__tests__\/ReactTransitionGroup-test.js\nindex f14d546bb6..27daf3836a 100644\n--- a\/src\/addons\/transitions\/__tests__\/ReactTransitionGroup-test.js\n+++ b\/src\/addons\/transitions\/__tests__\/ReactTransitionGroup-test.js\n@@ -13,7 +13,6 @@\n \n var React;\n var ReactTransitionGroup;\n-var mocks;\n \n \/\/ Most of the real functionality is covered in other unit tests, this just\n \/\/ makes sure we're wired up correctly.\n@@ -23,7 +22,6 @@ describe('ReactTransitionGroup', function() {\n beforeEach(function() {\n React = require('React');\n ReactTransitionGroup = require('ReactTransitionGroup');\n- mocks = require('mocks');\n \n container = document.createElement('div');\n });\n@@ -94,15 +92,15 @@ describe('ReactTransitionGroup', function() {\n \n it('should handle enter\/leave\/enter\/leave correctly', function() {\n var log = [];\n- var cb;\n+ var willEnterCb;\n \n var Child = React.createClass({\n componentDidMount: function() {\n log.push('didMount');\n },\n- componentWillEnter: function(_cb) {\n+ componentWillEnter: function(cb) {\n log.push('willEnter');\n- cb = _cb;\n+ willEnterCb = cb;\n },\n componentDidEnter: function() {\n log.push('didEnter');\n@@ -139,12 +137,13 @@ describe('ReactTransitionGroup', function() {\n expect(log).toEqual(['didMount']);\n instance.setState({count: 2});\n expect(log).toEqual(['didMount', 'didMount', 'willEnter']);\n- for (var i = 0; i < 5; i++) {\n+ for (var k = 0; k < 5; k++) {\n instance.setState({count: 2});\n expect(log).toEqual(['didMount', 'didMount', 'willEnter']);\n instance.setState({count: 1});\n }\n- cb();\n+ \/\/ other animations are blocked until willEnterCb is called\n+ willEnterCb();\n expect(log).toEqual([\n 'didMount', 'didMount', 'willEnter',\n 'didEnter', 'willLeave', 'didLeave', 'willUnmount'\n@@ -153,15 +152,15 @@ describe('ReactTransitionGroup', function() {\n \n it('should handle enter\/leave\/enter correctly', function() {\n var log = [];\n- var cb;\n+ var willEnterCb;\n \n var Child = React.createClass({\n componentDidMount: function() {\n log.push('didMount');\n },\n- componentWillEnter: function(_cb) {\n+ componentWillEnter: function(cb) {\n log.push('willEnter');\n- cb = _cb;\n+ willEnterCb = cb;\n },\n componentDidEnter: function() {\n log.push('didEnter');\n@@ -198,12 +197,12 @@ describe('ReactTransitionGroup', function() {\n expect(log).toEqual(['didMount']);\n instance.setState({count: 2});\n expect(log).toEqual(['didMount', 'didMount', 'willEnter']);\n- for (var i = 0; i < 5; i++) {\n+ for (var k = 0; k < 5; k++) {\n instance.setState({count: 1});\n expect(log).toEqual(['didMount', 'didMount', 'willEnter']);\n instance.setState({count: 2});\n }\n- cb();\n+ willEnterCb();\n expect(log).toEqual([\n 'didMount', 'didMount', 'willEnter', 'didEnter'\n ]);\n@@ -211,7 +210,6 @@ describe('ReactTransitionGroup', function() {\n \n it('should handle entering\/leaving several elements at once', function() {\n var log = [];\n- var cb;\n \n var Child = React.createClass({\n componentDidMount: function() {\ndiff --git a\/src\/isomorphic\/children\/__tests__\/ReactChildren-test.js b\/src\/isomorphic\/children\/__tests__\/ReactChildren-test.js\nindex 62c943cfcb..111d04d710 100644\n--- a\/src\/isomorphic\/children\/__tests__\/ReactChildren-test.js\n+++ b\/src\/isomorphic\/children\/__tests__\/ReactChildren-test.js\n@@ -22,10 +22,6 @@ describe('ReactChildren', function() {\n ReactFragment = require('ReactFragment');\n });\n \n- function frag(obj) {\n- return ReactFragment.create(obj);\n- }\n-\n function nthChild(mappedChildren, n) {\n var result = null;\n ReactChildren.forEach(mappedChildren, function(child, index) {\n@@ -227,7 +223,7 @@ describe('ReactChildren', function() {\n \n var instance = (\n

{\n- [frag({\n+ [ReactFragment.create({\n firstHalfKey: [zero, one, two],\n secondHalfKey: [three, four],\n keyFive: five\n@@ -345,7 +341,9 @@ describe('ReactChildren', function() {\n var zero =

;\n var one = ;\n \n- var mapFn = function(component) { return component; };\n+ var mapFn = function(component) {\n+ return component;\n+ };\n var instance = (\n

{zero}{one}<\/div>\n );\n@@ -410,7 +408,7 @@ describe('ReactChildren', function() {\n \n var instance = (\n

{\n- [frag({\n+ [ReactFragment.create({\n firstHalfKey: [zero, one, two],\n secondHalfKey: [three, four],\n keyFive: five\n@@ -438,7 +436,7 @@ describe('ReactChildren', function() {\n return c;\n });\n for (var key in frag) {\n- frag[key];\n+ void frag[key];\n break;\n }\n expect(console.error.calls.length).toBe(1);\n@@ -447,8 +445,8 @@ describe('ReactChildren', function() {\n var frag2 = ReactChildren.map([child, child], function(c) {\n return c;\n });\n- for (var key in frag2) {\n- frag2[key] = 123;\n+ for (var key2 in frag2) {\n+ frag2[key2] = 123;\n break;\n }\n expect(console.error.calls.length).toBe(2);\ndiff --git a\/src\/isomorphic\/classic\/__tests__\/ReactContextValidator-test.js b\/src\/isomorphic\/classic\/__tests__\/ReactContextValidator-test.js\nindex 7f79c1fb10..d7252a3f9c 100644\n--- a\/src\/isomorphic\/classic\/__tests__\/ReactContextValidator-test.js\n+++ b\/src\/isomorphic\/classic\/__tests__\/ReactContextValidator-test.js\n@@ -21,7 +21,6 @@ var React;\n var ReactTestUtils;\n \n var reactComponentExpect;\n-var mocks;\n \n describe('ReactContextValidator', function() {\n beforeEach(function() {\n@@ -30,7 +29,6 @@ describe('ReactContextValidator', function() {\n React = require('React');\n ReactTestUtils = require('ReactTestUtils');\n reactComponentExpect = require('reactComponentExpect');\n- mocks = require('mocks');\n \n spyOn(console, 'error');\n });\n@@ -86,7 +84,7 @@ describe('ReactContextValidator', function() {\n getChildContext: function() {\n return {\n foo: this.props.foo,\n- bar: \"bar\"\n+ bar: 'bar'\n };\n },\n \n@@ -125,7 +123,7 @@ describe('ReactContextValidator', function() {\n \n var instance = ;\n instance = ReactTestUtils.renderIntoDocument(instance);\n- instance.replaceProps({foo: \"def\"});\n+ instance.replaceProps({foo: 'def'});\n expect(actualComponentWillReceiveProps).toEqual({foo: 'def'});\n expect(actualShouldComponentUpdate).toEqual({foo: 'def'});\n expect(actualComponentWillUpdate).toEqual({foo: 'def'});\ndiff --git a\/src\/isomorphic\/classic\/class\/__tests__\/ReactBind-test.js b\/src\/isomorphic\/classic\/class\/__tests__\/ReactBind-test.js\nindex 78caa70c64..1bbdd33c72 100644\n--- a\/src\/isomorphic\/classic\/class\/__tests__\/ReactBind-test.js\n+++ b\/src\/isomorphic\/classic\/class\/__tests__\/ReactBind-test.js\n@@ -36,7 +36,7 @@ describe('autobinding', function() {\n \/\/ auto binding only occurs on top level functions in class defs.\n badIdeas: {\n badBind: function() {\n- this.state.something;\n+ void this.state.something;\n }\n },\n \n@@ -131,7 +131,7 @@ describe('autobinding', function() {\n }\n });\n \n- ReactTestUtils.renderIntoDocument()\n+ ReactTestUtils.renderIntoDocument();\n \n expect(console.error.argsForCall.length).toBe(1);\n expect(console.error.argsForCall[0][0]).toBe(\n@@ -159,7 +159,7 @@ describe('autobinding', function() {\n }\n });\n \n- ReactTestUtils.renderIntoDocument()\n+ ReactTestUtils.renderIntoDocument();\n \n expect(console.error.argsForCall.length).toBe(0);\n });\ndiff --git a\/src\/isomorphic\/classic\/class\/__tests__\/ReactClass-test.js b\/src\/isomorphic\/classic\/class\/__tests__\/ReactClass-test.js\nindex 41fb38cf34..912ae102e6 100644\n--- a\/src\/isomorphic\/classic\/class\/__tests__\/ReactClass-test.js\n+++ b\/src\/isomorphic\/classic\/class\/__tests__\/ReactClass-test.js\n@@ -175,7 +175,8 @@ describe('ReactClass-spec', function() {\n 'because the function is expected to return a value.'\n );\n \n- var NamedComponent = React.createClass({\n+ React.createClass({\n+ displayName: 'NamedComponent',\n componentShouldUpdate: function() {\n return false;\n },\n@@ -189,8 +190,6 @@ describe('ReactClass-spec', function() {\n 'mean shouldComponentUpdate()? The name is phrased as a question ' +\n 'because the function is expected to return a value.'\n );\n-\n- ; \/\/ Shut up lint\n });\n \n it('should throw if a reserved property is in statics', function() {\ndiff --git a\/src\/isomorphic\/classic\/class\/__tests__\/ReactClassMixin-test.js b\/src\/isomorphic\/classic\/class\/__tests__\/ReactClassMixin-test.js\nindex d56bb7052f..53f01e4726 100644\n--- a\/src\/isomorphic\/classic\/class\/__tests__\/ReactClassMixin-test.js\n+++ b\/src\/isomorphic\/classic\/class\/__tests__\/ReactClassMixin-test.js\n@@ -15,7 +15,6 @@ var mocks = require('mocks');\n \n var React;\n var ReactTestUtils;\n-var reactComponentExpect;\n \n var TestComponent;\n var TestComponentWithPropTypes;\n@@ -28,7 +27,6 @@ describe('ReactClass-mixin', function() {\n beforeEach(function() {\n React = require('React');\n ReactTestUtils = require('ReactTestUtils');\n- reactComponentExpect = require('reactComponentExpect');\n mixinPropValidator = mocks.getMockFunction();\n componentPropValidator = mocks.getMockFunction();\n \n@@ -288,18 +286,22 @@ describe('ReactClass-mixin', function() {\n );\n });\n \n- it(\"should throw if mixins override functions in statics\", function() {\n+ it('should throw if mixins override functions in statics', function() {\n expect(function() {\n var Mixin = {\n statics: {\n- abc: function() { console.log('foo'); }\n+ abc: function() {\n+ console.log('foo');\n+ }\n }\n };\n React.createClass({\n mixins: [Mixin],\n \n statics: {\n- abc: function() { console.log('bar'); }\n+ abc: function() {\n+ console.log('bar');\n+ }\n },\n \n render: function() {\n@@ -313,7 +315,7 @@ describe('ReactClass-mixin', function() {\n );\n });\n \n- it(\"should throw if the mixin is a React component\", function() {\n+ it('should throw if the mixin is a React component', function() {\n expect(function() {\n React.createClass({\n mixins: [

],\n@@ -328,7 +330,7 @@ describe('ReactClass-mixin', function() {\n );\n });\n \n- it(\"should throw if the mixin is a React component class\", function() {\n+ it('should throw if the mixin is a React component class', function() {\n expect(function() {\n var Component = React.createClass({\n render: function() {\n@@ -351,7 +353,9 @@ describe('ReactClass-mixin', function() {\n \n it('should have bound the mixin methods to the component', function() {\n var mixin = {\n- mixinFunc: function() {return this;}\n+ mixinFunc: function() {\n+ return this;\n+ }\n };\n \n var Component = React.createClass({\n@@ -368,24 +372,24 @@ describe('ReactClass-mixin', function() {\n });\n \n it('should include the mixin keys in even if their values are falsy',\n- function() {\n- var mixin = {\n- keyWithNullValue: null,\n- randomCounter: 0\n- };\n+ function() {\n+ var mixin = {\n+ keyWithNullValue: null,\n+ randomCounter: 0\n+ };\n \n- var Component = React.createClass({\n- mixins: [mixin],\n- componentDidMount: function() {\n- expect(this.randomCounter).toBe(0);\n- expect(this.keyWithNullValue).toBeNull();\n- },\n- render: function() {\n- return ;\n- }\n- });\n- var instance = ;\n- instance = ReactTestUtils.renderIntoDocument(instance);\n+ var Component = React.createClass({\n+ mixins: [mixin],\n+ componentDidMount: function() {\n+ expect(this.randomCounter).toBe(0);\n+ expect(this.keyWithNullValue).toBeNull();\n+ },\n+ render: function() {\n+ return ;\n+ }\n+ });\n+ var instance = ;\n+ instance = ReactTestUtils.renderIntoDocument(instance);\n });\n \n it('should work with a null getInitialState return value and a mixin', () => {\ndiff --git a\/src\/isomorphic\/classic\/element\/__tests__\/ReactElement-test.js b\/src\/isomorphic\/classic\/element\/__tests__\/ReactElement-test.js\nindex 946158c26e..539f149bfc 100644\n--- a\/src\/isomorphic\/classic\/element\/__tests__\/ReactElement-test.js\n+++ b\/src\/isomorphic\/classic\/element\/__tests__\/ReactElement-test.js\n@@ -14,8 +14,6 @@\n \/\/ NOTE: We're explicitly not using JSX in this file. This is intended to test\n \/\/ classic JS without JSX.\n \n-var mocks;\n-\n var React;\n var ReactTestUtils;\n \n@@ -25,12 +23,12 @@ describe('ReactElement', function() {\n beforeEach(function() {\n require('mock-modules').dumpCache();\n \n- mocks = require('mocks');\n-\n React = require('React');\n ReactTestUtils = require('ReactTestUtils');\n ComponentClass = React.createClass({\n- render: function() { return React.createElement('div'); }\n+ render: function() {\n+ return React.createElement('div');\n+ }\n });\n });\n \n@@ -143,7 +141,7 @@ describe('ReactElement', function() {\n it('allows static methods to be called using the type property', function() {\n spyOn(console, 'error');\n \n- var ComponentClass = React.createClass({\n+ var StaticMethodComponentClass = React.createClass({\n statics: {\n someStaticMethod: function() {\n return 'someReturnValue';\n@@ -157,7 +155,7 @@ describe('ReactElement', function() {\n }\n });\n \n- var element = React.createElement(ComponentClass);\n+ var element = React.createElement(StaticMethodComponentClass);\n expect(element.type.someStaticMethod()).toBe('someReturnValue');\n expect(console.error.argsForCall.length).toBe(0);\n });\n@@ -177,7 +175,7 @@ describe('ReactElement', function() {\n expect(React.isValidElement(null)).toEqual(false);\n expect(React.isValidElement(true)).toEqual(false);\n expect(React.isValidElement({})).toEqual(false);\n- expect(React.isValidElement(\"string\")).toEqual(false);\n+ expect(React.isValidElement('string')).toEqual(false);\n expect(React.isValidElement(React.DOM.div)).toEqual(false);\n expect(React.isValidElement(Component)).toEqual(false);\n });\n@@ -193,8 +191,8 @@ describe('ReactElement', function() {\n }\n });\n \n- expect(typeof Component.specialType).toBe(\"function\");\n- expect(typeof Component.specialType.isRequired).toBe(\"function\");\n+ expect(typeof Component.specialType).toBe('function');\n+ expect(typeof Component.specialType.isRequired).toBe('function');\n });\n \n it('is indistinguishable from a plain object', function() {\n@@ -285,7 +283,7 @@ describe('ReactElement', function() {\n \n it('warns when adding a prop after element creation', function() {\n spyOn(console, 'error');\n- var el = document.createElement('div');\n+ var container = document.createElement('div');\n var Outer = React.createClass({\n getDefaultProps: () => ({sound: 'meow'}),\n render: function() {\n@@ -299,7 +297,7 @@ describe('ReactElement', function() {\n return el;\n }\n });\n- var outer = React.render(, el);\n+ var outer = React.render(, container);\n expect(React.findDOMNode(outer).textContent).toBe('meow');\n expect(React.findDOMNode(outer).className).toBe('quack');\n \n@@ -315,7 +313,7 @@ describe('ReactElement', function() {\n \n var newOuterEl = ;\n newOuterEl.props.sound = 'oink';\n- outer = React.render(newOuterEl, el);\n+ outer = React.render(newOuterEl, container);\n expect(React.findDOMNode(outer).textContent).toBe('oink');\n expect(React.findDOMNode(outer).className).toBe('quack');\n \ndiff --git a\/src\/isomorphic\/classic\/element\/__tests__\/ReactElementClone-test.js b\/src\/isomorphic\/classic\/element\/__tests__\/ReactElementClone-test.js\nindex e7e0134625..769e5ca9ba 100644\n--- a\/src\/isomorphic\/classic\/element\/__tests__\/ReactElementClone-test.js\n+++ b\/src\/isomorphic\/classic\/element\/__tests__\/ReactElementClone-test.js\n@@ -13,8 +13,6 @@\n \n require('mock-modules');\n \n-var mocks = require('mocks');\n-\n var React;\n var ReactTestUtils;\n \ndiff --git a\/src\/isomorphic\/classic\/element\/__tests__\/ReactElementValidator-test.js b\/src\/isomorphic\/classic\/element\/__tests__\/ReactElementValidator-test.js\nindex 2f8d265956..6237fe075f 100644\n--- a\/src\/isomorphic\/classic\/element\/__tests__\/ReactElementValidator-test.js\n+++ b\/src\/isomorphic\/classic\/element\/__tests__\/ReactElementValidator-test.js\n@@ -28,7 +28,9 @@ describe('ReactElementValidator', function() {\n ReactFragment = require('ReactFragment');\n ReactTestUtils = require('ReactTestUtils');\n ComponentClass = React.createClass({\n- render: function() { return React.createElement('div'); }\n+ render: function() {\n+ return React.createElement('div');\n+ }\n });\n });\n \n@@ -213,7 +215,8 @@ describe('ReactElementValidator', function() {\n );\n });\n \n- it('does not warn for numeric keys in entry iterables in rest args', function() {\n+ it('does not warn for numeric keys in entry iterables in rest args',\n+ function() {\n spyOn(console, 'error');\n var Component = React.createFactory(ComponentClass);\n \n@@ -283,7 +286,7 @@ describe('ReactElementValidator', function() {\n });\n \n it('gives a helpful error when passing null, undefined, boolean, or number',\n- () => {\n+ function() {\n spyOn(console, 'error');\n React.createElement(undefined);\n React.createElement(null);\n@@ -315,7 +318,7 @@ describe('ReactElementValidator', function() {\n });\n \n it('includes the owner name when passing null, undefined, boolean, or number',\n- () => {\n+ function() {\n spyOn(console, 'error');\n var ParentComp = React.createClass({\n render: function() {\ndiff --git a\/src\/isomorphic\/classic\/types\/__tests__\/ReactPropTypes-test.js b\/src\/isomorphic\/classic\/types\/__tests__\/ReactPropTypes-test.js\nindex 0992f4f6eb..1bf284aa87 100644\n--- a\/src\/isomorphic\/classic\/types\/__tests__\/ReactPropTypes-test.js\n+++ b\/src\/isomorphic\/classic\/types\/__tests__\/ReactPropTypes-test.js\n@@ -55,7 +55,7 @@ describe('ReactPropTypes', function() {\n });\n \n describe('Primitive Types', function() {\n- it(\"should warn for invalid strings\", function() {\n+ it('should warn for invalid strings', function() {\n typeCheckFail(\n PropTypes.string,\n [],\n@@ -97,7 +97,7 @@ describe('ReactPropTypes', function() {\n );\n });\n \n- it(\"should not warn for valid values\", function() {\n+ it('should not warn for valid values', function() {\n typeCheckPass(PropTypes.array, []);\n typeCheckPass(PropTypes.bool, false);\n typeCheckPass(PropTypes.func, function() {});\n@@ -108,12 +108,12 @@ describe('ReactPropTypes', function() {\n typeCheckPass(PropTypes.object, \/please\/);\n });\n \n- it(\"should be implicitly optional and not warn without values\", function() {\n+ it('should be implicitly optional and not warn without values', function() {\n typeCheckPass(PropTypes.string, null);\n typeCheckPass(PropTypes.string, undefined);\n });\n \n- it(\"should warn for missing required values\", function() {\n+ it('should warn for missing required values', function() {\n typeCheckFail(PropTypes.string.isRequired, null, requiredMessage);\n typeCheckFail(PropTypes.string.isRequired, undefined, requiredMessage);\n });\n@@ -126,12 +126,12 @@ describe('ReactPropTypes', function() {\n typeCheckPass(PropTypes.any, []);\n });\n \n- it(\"should be implicitly optional and not warn without values\", function() {\n+ it('should be implicitly optional and not warn without values', function() {\n typeCheckPass(PropTypes.any, null);\n typeCheckPass(PropTypes.any, undefined);\n });\n \n- it(\"should warn for missing required values\", function() {\n+ it('should warn for missing required values', function() {\n typeCheckFail(PropTypes.any.isRequired, null, requiredMessage);\n typeCheckFail(PropTypes.any.isRequired, undefined, requiredMessage);\n });\n@@ -203,12 +203,12 @@ describe('ReactPropTypes', function() {\n typeCheckPass(PropTypes.arrayOf(PropTypes.number), []);\n });\n \n- it(\"should be implicitly optional and not warn without values\", function() {\n+ it('should be implicitly optional and not warn without values', function() {\n typeCheckPass(PropTypes.arrayOf(PropTypes.number), null);\n typeCheckPass(PropTypes.arrayOf(PropTypes.number), undefined);\n });\n \n- it(\"should warn for missing required values\", function() {\n+ it('should warn for missing required values', function() {\n typeCheckFail(\n PropTypes.arrayOf(PropTypes.number).isRequired,\n null,\n@@ -263,19 +263,19 @@ describe('ReactPropTypes', function() {\n expect(console.error.argsForCall.length).toBe(1);\n });\n \n- it(\"should be implicitly optional and not warn without values\", function() {\n+ it('should be implicitly optional and not warn without values', function() {\n typeCheckPass(PropTypes.element, null);\n typeCheckPass(PropTypes.element, undefined);\n });\n \n- it(\"should warn for missing required values\", function() {\n+ it('should warn for missing required values', function() {\n typeCheckFail(PropTypes.element.isRequired, null, requiredMessage);\n typeCheckFail(PropTypes.element.isRequired, undefined, requiredMessage);\n });\n });\n \n describe('Instance Types', function() {\n- it(\"should warn for invalid instances\", function() {\n+ it('should warn for invalid instances', function() {\n function Person() {}\n var personName = Person.name || '<>';\n var dateName = Date.name || '<>';\n@@ -313,7 +313,7 @@ describe('ReactPropTypes', function() {\n );\n });\n \n- it(\"should not warn for valid values\", function() {\n+ it('should not warn for valid values', function() {\n function Person() {}\n function Engineer() {}\n Engineer.prototype = new Person();\n@@ -325,12 +325,12 @@ describe('ReactPropTypes', function() {\n typeCheckPass(PropTypes.instanceOf(RegExp), \/please\/);\n });\n \n- it(\"should be implicitly optional and not warn without values\", function() {\n+ it('should be implicitly optional and not warn without values', function() {\n typeCheckPass(PropTypes.instanceOf(String), null);\n typeCheckPass(PropTypes.instanceOf(String), undefined);\n });\n \n- it(\"should warn for missing required values\", function() {\n+ it('should warn for missing required values', function() {\n typeCheckFail(\n PropTypes.instanceOf(String).isRequired, null, requiredMessage\n );\n@@ -500,12 +500,12 @@ describe('ReactPropTypes', function() {\n typeCheckPass(PropTypes.objectOf(PropTypes.number), {});\n });\n \n- it(\"should be implicitly optional and not warn without values\", function() {\n+ it('should be implicitly optional and not warn without values', function() {\n typeCheckPass(PropTypes.objectOf(PropTypes.number), null);\n typeCheckPass(PropTypes.objectOf(PropTypes.number), undefined);\n });\n \n- it(\"should warn for missing required values\", function() {\n+ it('should warn for missing required values', function() {\n typeCheckFail(\n PropTypes.objectOf(PropTypes.number).isRequired,\n null,\n@@ -520,7 +520,7 @@ describe('ReactPropTypes', function() {\n });\n \n describe('OneOf Types', function() {\n- it(\"should fail for invalid argument\", function() {\n+ it('should fail for invalid argument', function() {\n typeCheckFail(\n PropTypes.oneOf('red', 'blue'),\n 'red',\n@@ -528,7 +528,7 @@ describe('ReactPropTypes', function() {\n );\n });\n \n- it(\"should warn for invalid strings\", function() {\n+ it('should warn for invalid strings', function() {\n typeCheckFail(\n PropTypes.oneOf(['red', 'blue']),\n true,\n@@ -555,17 +555,17 @@ describe('ReactPropTypes', function() {\n );\n });\n \n- it(\"should not warn for valid values\", function() {\n+ it('should not warn for valid values', function() {\n typeCheckPass(PropTypes.oneOf(['red', 'blue']), 'red');\n typeCheckPass(PropTypes.oneOf(['red', 'blue']), 'blue');\n });\n \n- it(\"should be implicitly optional and not warn without values\", function() {\n+ it('should be implicitly optional and not warn without values', function() {\n typeCheckPass(PropTypes.oneOf(['red', 'blue']), null);\n typeCheckPass(PropTypes.oneOf(['red', 'blue']), undefined);\n });\n \n- it(\"should warn for missing required values\", function() {\n+ it('should warn for missing required values', function() {\n typeCheckFail(\n PropTypes.oneOf(['red', 'blue']).isRequired,\n null,\n@@ -580,7 +580,7 @@ describe('ReactPropTypes', function() {\n });\n \n describe('Union Types', function() {\n- it(\"should fail for invalid argument\", function() {\n+ it('should fail for invalid argument', function() {\n typeCheckFail(\n PropTypes.oneOfType(PropTypes.string, PropTypes.number),\n 'red',\n@@ -623,7 +623,7 @@ describe('ReactPropTypes', function() {\n typeCheckPass(checker, {b: 1});\n });\n \n- it(\"should be implicitly optional and not warn without values\", function() {\n+ it('should be implicitly optional and not warn without values', function() {\n typeCheckPass(\n PropTypes.oneOfType([PropTypes.string, PropTypes.number]), null\n );\n@@ -632,7 +632,7 @@ describe('ReactPropTypes', function() {\n );\n });\n \n- it(\"should warn for missing required values\", function() {\n+ it('should warn for missing required values', function() {\n typeCheckFail(\n PropTypes.oneOfType([PropTypes.string, PropTypes.number]).isRequired,\n null,\n@@ -647,7 +647,7 @@ describe('ReactPropTypes', function() {\n });\n \n describe('Shape Types', function() {\n- it(\"should warn for non objects\", function() {\n+ it('should warn for non objects', function() {\n typeCheckFail(\n PropTypes.shape({}),\n 'some string',\n@@ -662,25 +662,25 @@ describe('ReactPropTypes', function() {\n );\n });\n \n- it(\"should not warn for empty values\", function() {\n+ it('should not warn for empty values', function() {\n typeCheckPass(PropTypes.shape({}), undefined);\n typeCheckPass(PropTypes.shape({}), null);\n typeCheckPass(PropTypes.shape({}), {});\n });\n \n- it(\"should not warn for an empty object\", function() {\n+ it('should not warn for an empty object', function() {\n typeCheckPass(PropTypes.shape({}).isRequired, {});\n });\n \n- it(\"should not warn for non specified types\", function() {\n+ it('should not warn for non specified types', function() {\n typeCheckPass(PropTypes.shape({}), {key: 1});\n });\n \n- it(\"should not warn for valid types\", function() {\n+ it('should not warn for valid types', function() {\n typeCheckPass(PropTypes.shape({key: PropTypes.number}), {key: 1});\n });\n \n- it(\"should warn for required valid types\", function() {\n+ it('should warn for required valid types', function() {\n typeCheckFail(\n PropTypes.shape({key: PropTypes.number.isRequired}),\n {},\n@@ -688,7 +688,7 @@ describe('ReactPropTypes', function() {\n );\n });\n \n- it(\"should warn for the first required type\", function() {\n+ it('should warn for the first required type', function() {\n typeCheckFail(\n PropTypes.shape({\n key: PropTypes.number.isRequired,\n@@ -699,7 +699,7 @@ describe('ReactPropTypes', function() {\n );\n });\n \n- it(\"should warn for invalid key types\", function() {\n+ it('should warn for invalid key types', function() {\n typeCheckFail(PropTypes.shape({key: PropTypes.number}),\n {key: 'abc'},\n 'Invalid prop `testProp.key` of type `string` supplied to `testComponent`, ' +\n@@ -707,7 +707,7 @@ describe('ReactPropTypes', function() {\n );\n });\n \n- it(\"should be implicitly optional and not warn without values\", function() {\n+ it('should be implicitly optional and not warn without values', function() {\n typeCheckPass(\n PropTypes.shape(PropTypes.shape({key: PropTypes.number})), null\n );\n@@ -716,7 +716,7 @@ describe('ReactPropTypes', function() {\n );\n });\n \n- it(\"should warn for missing required values\", function() {\n+ it('should warn for missing required values', function() {\n typeCheckFail(\n PropTypes.shape({key: PropTypes.number}).isRequired,\n null,\n@@ -738,7 +738,7 @@ describe('ReactPropTypes', function() {\n \n it('should have been called with the right params', function() {\n var spy = jasmine.createSpy();\n- var Component = React.createClass({\n+ Component = React.createClass({\n propTypes: {num: spy},\n \n render: function() {\n@@ -756,7 +756,7 @@ describe('ReactPropTypes', function() {\n \n it('should have been called even if the prop is not present', function() {\n var spy = jasmine.createSpy();\n- var Component = React.createClass({\n+ Component = React.createClass({\n propTypes: {num: spy},\n \n render: function() {\n@@ -778,7 +778,7 @@ describe('ReactPropTypes', function() {\n }\n }\n );\n- var Component = React.createClass({\n+ Component = React.createClass({\n propTypes: {num: spy},\n \n render: function() {\n@@ -790,7 +790,8 @@ describe('ReactPropTypes', function() {\n instance = ReactTestUtils.renderIntoDocument(instance);\n expect(console.error.argsForCall.length).toBe(1);\n expect(console.error.argsForCall[0][0]).toBe(\n- 'Warning: Failed propType: num must be 5!');\n+ 'Warning: Failed propType: num must be 5!'\n+ );\n });\n \n it('should not warn if the validator returned null',\n@@ -800,7 +801,7 @@ describe('ReactPropTypes', function() {\n return null;\n }\n );\n- var Component = React.createClass({\n+ Component = React.createClass({\n propTypes: {num: spy},\n \n render: function() {\ndiff --git a\/src\/isomorphic\/deprecated\/__tests__\/cloneWithProps-test.js b\/src\/isomorphic\/deprecated\/__tests__\/cloneWithProps-test.js\nindex 9ef5d7fa12..6869fbcaaa 100644\n--- a\/src\/isomorphic\/deprecated\/__tests__\/cloneWithProps-test.js\n+++ b\/src\/isomorphic\/deprecated\/__tests__\/cloneWithProps-test.js\n@@ -15,8 +15,6 @@ require('mock-modules')\n .dontMock('cloneWithProps')\n .dontMock('emptyObject');\n \n-var mocks = require('mocks');\n-\n var React;\n var ReactTestUtils;\n \ndiff --git a\/src\/isomorphic\/modern\/class\/__tests__\/ReactClassEquivalence-test.js b\/src\/isomorphic\/modern\/class\/__tests__\/ReactClassEquivalence-test.js\nindex cd342ad487..f38d2718c1 100644\n--- a\/src\/isomorphic\/modern\/class\/__tests__\/ReactClassEquivalence-test.js\n+++ b\/src\/isomorphic\/modern\/class\/__tests__\/ReactClassEquivalence-test.js\n@@ -9,6 +9,8 @@\n * @emails react-core\n *\/\n \n+'use strict';\n+\n var MetaMatchers = require('MetaMatchers');\n \n describe('ReactClassEquivalence', function() {\ndiff --git a\/src\/isomorphic\/modern\/element\/__tests__\/ReactJSXElement-test.js b\/src\/isomorphic\/modern\/element\/__tests__\/ReactJSXElement-test.js\nindex ddc319003c..25a0b26680 100644\n--- a\/src\/isomorphic\/modern\/element\/__tests__\/ReactJSXElement-test.js\n+++ b\/src\/isomorphic\/modern\/element\/__tests__\/ReactJSXElement-test.js\n@@ -11,8 +11,6 @@\n \n 'use strict';\n \n-var mocks;\n-\n var React;\n var ReactTestUtils;\n \n@@ -22,12 +20,12 @@ describe('ReactJSXElement', function() {\n beforeEach(function() {\n require('mock-modules').dumpCache();\n \n- mocks = require('mocks');\n-\n React = require('React');\n ReactTestUtils = require('ReactTestUtils');\n Component = class {\n- render() { return

; }\n+ render() {\n+ return

;\n+ }\n };\n });\n \n@@ -145,7 +143,7 @@ describe('ReactJSXElement', function() {\n expect(React.isValidElement(null)).toEqual(false);\n expect(React.isValidElement(true)).toEqual(false);\n expect(React.isValidElement({})).toEqual(false);\n- expect(React.isValidElement(\"string\")).toEqual(false);\n+ expect(React.isValidElement('string')).toEqual(false);\n expect(React.isValidElement(Component)).toEqual(false);\n });\n \ndiff --git a\/src\/isomorphic\/modern\/element\/__tests__\/ReactJSXElementValidator-test.js b\/src\/isomorphic\/modern\/element\/__tests__\/ReactJSXElementValidator-test.js\nindex 1e15d2d3b9..47dde23764 100644\n--- a\/src\/isomorphic\/modern\/element\/__tests__\/ReactJSXElementValidator-test.js\n+++ b\/src\/isomorphic\/modern\/element\/__tests__\/ReactJSXElementValidator-test.js\n@@ -29,7 +29,9 @@ describe('ReactJSXElementValidator', function() {\n ReactTestUtils = require('ReactTestUtils');\n \n Component = class {\n- render() { return

; }\n+ render() {\n+ return

;\n+ }\n };\n });\n \n@@ -40,7 +42,7 @@ describe('ReactJSXElementValidator', function() {\n it('warns for keys for arrays of elements in children position', function() {\n spyOn(console, 'error');\n \n- {[, ]}<\/Component>;\n+ void {[, ]}<\/Component>;\n \n expect(console.error.argsForCall.length).toBe(1);\n expect(console.error.argsForCall[0][0]).toContain(\n@@ -55,7 +57,7 @@ describe('ReactJSXElementValidator', function() {\n render() {\n return {this.props.childSet}<\/Component>;\n }\n- };\n+ }\n \n class ComponentWrapper {\n render() {\n@@ -65,7 +67,7 @@ describe('ReactJSXElementValidator', function() {\n \/>\n );\n }\n- };\n+ }\n \n ReactTestUtils.renderIntoDocument();\n \n@@ -92,7 +94,7 @@ describe('ReactJSXElementValidator', function() {\n }\n };\n \n- {iterable}<\/Component>;\n+ void {iterable}<\/Component>;\n \n expect(console.error.argsForCall.length).toBe(1);\n expect(console.error.argsForCall[0][0]).toContain(\n@@ -103,7 +105,7 @@ describe('ReactJSXElementValidator', function() {\n it('does not warns for arrays of elements with keys', function() {\n spyOn(console, 'error');\n \n- {[, ]}<\/Component>;\n+ void {[, ]}<\/Component>;\n \n expect(console.error.argsForCall.length).toBe(0);\n });\n@@ -126,7 +128,7 @@ describe('ReactJSXElementValidator', function() {\n }\n };\n \n- {iterable}<\/Component>\n+ void {iterable}<\/Component>;\n \n expect(console.error.argsForCall.length).toBe(0);\n });\n@@ -134,7 +136,7 @@ describe('ReactJSXElementValidator', function() {\n it('warns for numeric keys on objects as children', function() {\n spyOn(console, 'error');\n \n- {frag({1: , 2: })}<\/Component>;\n+ void {frag({1: , 2: })}<\/Component>;\n \n expect(console.error.argsForCall.length).toBe(1);\n expect(console.error.argsForCall[0][0]).toContain(\n@@ -158,7 +160,7 @@ describe('ReactJSXElementValidator', function() {\n };\n iterable.entries = iterable['@@iterator'];\n \n- {iterable}<\/Component>;\n+ void {iterable}<\/Component>;\n \n expect(console.error.argsForCall.length).toBe(0);\n });\n@@ -166,7 +168,7 @@ describe('ReactJSXElementValidator', function() {\n it('does not warn when the element is directly as children', function() {\n spyOn(console, 'error');\n \n- <\/Component>;\n+ void <\/Component>;\n \n expect(console.error.argsForCall.length).toBe(0);\n });\n@@ -174,7 +176,7 @@ describe('ReactJSXElementValidator', function() {\n it('does not warn when the child array contains non-elements', function() {\n spyOn(console, 'error');\n \n- {[ {}, {} ]}<\/Component>;\n+ void {[ {}, {} ]}<\/Component>;\n \n expect(console.error.argsForCall.length).toBe(0);\n });\n@@ -215,10 +217,10 @@ describe('ReactJSXElementValidator', function() {\n var Num = 123;\n var Div = 'div';\n spyOn(console, 'error');\n- ;\n- ;\n- ;\n- ;\n+ void ;\n+ void ;\n+ void ;\n+ void ;\n expect(console.error.calls.length).toBe(4);\n expect(console.error.calls[0].args[0]).toContain(\n 'type should not be null, undefined, boolean, or number. It should be ' +\n@@ -236,7 +238,7 @@ describe('ReactJSXElementValidator', function() {\n 'type should not be null, undefined, boolean, or number. It should be ' +\n 'a string (for DOM elements) or a ReactClass (for composite components).'\n );\n-

;\n+ void

;\n expect(console.error.calls.length).toBe(4);\n });\n \ndiff --git a\/src\/renderers\/dom\/client\/__tests__\/ReactDOM-test.js b\/src\/renderers\/dom\/client\/__tests__\/ReactDOM-test.js\nindex b52cfcca13..71760b3203 100644\n--- a\/src\/renderers\/dom\/client\/__tests__\/ReactDOM-test.js\n+++ b\/src\/renderers\/dom\/client\/__tests__\/ReactDOM-test.js\n@@ -45,13 +45,13 @@ describe('ReactDOM', function() {\n });\n *\/\n \n- it(\"allows a DOM element to be used with a string\", function() {\n+ it('allows a DOM element to be used with a string', function() {\n var element = React.createElement('div', {className: 'foo'});\n var instance = ReactTestUtils.renderIntoDocument(element);\n expect(React.findDOMNode(instance).tagName).toBe('DIV');\n });\n \n- it(\"should allow children to be passed as an argument\", function() {\n+ it('should allow children to be passed as an argument', function() {\n var argDiv = ReactTestUtils.renderIntoDocument(\n div(null, 'child')\n );\n@@ -59,7 +59,7 @@ describe('ReactDOM', function() {\n expect(argNode.innerHTML).toBe('child');\n });\n \n- it(\"should overwrite props.children with children argument\", function() {\n+ it('should overwrite props.children with children argument', function() {\n var conflictDiv = ReactTestUtils.renderIntoDocument(\n div({children: 'fakechild'}, 'child')\n );\n@@ -71,7 +71,7 @@ describe('ReactDOM', function() {\n * We need to make sure that updates occur to the actual node that's in the\n * DOM, instead of a stale cache.\n *\/\n- it(\"should purge the DOM cache when removing nodes\", function() {\n+ it('should purge the DOM cache when removing nodes', function() {\n var myDiv = ReactTestUtils.renderIntoDocument(\n

,\ndiff --git a\/src\/renderers\/dom\/client\/__tests__\/ReactDOMIDOperations-test.js b\/src\/renderers\/dom\/client\/__tests__\/ReactDOMIDOperations-test.js\nindex 83ebbe0395..be744c7a5f 100644\n--- a\/src\/renderers\/dom\/client\/__tests__\/ReactDOMIDOperations-test.js\n+++ b\/src\/renderers\/dom\/client\/__tests__\/ReactDOMIDOperations-test.js\n@@ -18,8 +18,8 @@ describe('ReactDOMIDOperations', function() {\n var keyOf = require('keyOf');\n \n it('should disallow updating special properties', function() {\n- spyOn(ReactMount, \"getNode\");\n- spyOn(DOMPropertyOperations, \"setValueForProperty\");\n+ spyOn(ReactMount, 'getNode');\n+ spyOn(DOMPropertyOperations, 'setValueForProperty');\n \n expect(function() {\n ReactDOMIDOperations.updatePropertyByID(\n@@ -40,7 +40,7 @@ describe('ReactDOMIDOperations', function() {\n \n it('should update innerHTML and preserve whitespace', function() {\n var stubNode = document.createElement('div');\n- spyOn(ReactMount, \"getNode\").andReturn(stubNode);\n+ spyOn(ReactMount, 'getNode').andReturn(stubNode);\n \n var html = '\\n \\t \\n testContent \\t <\/span> \\n \\t';\n \ndiff --git a\/src\/renderers\/dom\/client\/__tests__\/ReactMount-test.js b\/src\/renderers\/dom\/client\/__tests__\/ReactMount-test.js\nindex bba0ee9b4f..e8c4e25d28 100644\n--- a\/src\/renderers\/dom\/client\/__tests__\/ReactMount-test.js\n+++ b\/src\/renderers\/dom\/client\/__tests__\/ReactMount-test.js\n@@ -169,25 +169,27 @@ describe('ReactMount', function() {\n );\n });\n \n- (WebComponents === undefined ? xit : it)\n- ('should allow mounting\/unmounting to document fragment container', function() {\n- var shadowRoot;\n- var proto = Object.create(HTMLElement.prototype, {\n- createdCallback: {\n- value: function() {\n+ if (WebComponents !== undefined) {\n+ it('should allow mounting\/unmounting to document fragment container',\n+ function() {\n+ var shadowRoot;\n+ var proto = Object.create(HTMLElement.prototype, {\n+ createdCallback: {\n+ value: function() {\n shadowRoot = this.createShadowRoot();\n React.render(

Hi, from within a WC!<\/div>, shadowRoot);\n expect(shadowRoot.firstChild.tagName).toBe('DIV');\n React.render(Hi, from within a WC!<\/span>, shadowRoot);\n expect(shadowRoot.firstChild.tagName).toBe('SPAN');\n+ }\n }\n- }\n+ });\n+ proto.unmount = function() {\n+ React.unmountComponentAtNode(shadowRoot);\n+ };\n+ document.registerElement('x-foo', {prototype: proto});\n+ var element = document.createElement('x-foo');\n+ element.unmount();\n });\n- proto.unmount = function() {\n- React.unmountComponentAtNode(shadowRoot);\n- };\n- document.registerElement('x-foo', {prototype: proto});\n- var element = document.createElement('x-foo');\n- element.unmount();\n- });\n+ }\n });\ndiff --git a\/src\/renderers\/dom\/client\/__tests__\/ReactMountDestruction-test.js b\/src\/renderers\/dom\/client\/__tests__\/ReactMountDestruction-test.js\nindex 9b510de5d3..d13071f547 100644\n--- a\/src\/renderers\/dom\/client\/__tests__\/ReactMountDestruction-test.js\n+++ b\/src\/renderers\/dom\/client\/__tests__\/ReactMountDestruction-test.js\n@@ -14,7 +14,7 @@\n var React = require('React');\n \n describe('ReactMount', function() {\n- it(\"should destroy a react root upon request\", function() {\n+ it('should destroy a react root upon request', function() {\n var mainContainerDiv = document.createElement('div');\n document.body.appendChild(mainContainerDiv);\n \ndiff --git a\/src\/renderers\/dom\/client\/__tests__\/findDOMNode-test.js b\/src\/renderers\/dom\/client\/__tests__\/findDOMNode-test.js\nindex 75e797d883..e0ae69e0fe 100644\n--- a\/src\/renderers\/dom\/client\/__tests__\/findDOMNode-test.js\n+++ b\/src\/renderers\/dom\/client\/__tests__\/findDOMNode-test.js\n@@ -34,14 +34,18 @@ describe('findDOMNode', function() {\n });\n \n it('findDOMNode should reject random objects', function() {\n- expect(function() {React.findDOMNode({foo: 'bar'});})\n+ expect(function() {\n+ React.findDOMNode({foo: 'bar'});\n+ })\n .toThrow('Invariant Violation: Element appears to be neither ' +\n 'ReactComponent nor DOMNode (keys: foo)'\n );\n });\n \n it('findDOMNode should reject unmounted objects with render func', function() {\n- expect(function() {React.findDOMNode({render: function() {}});})\n+ expect(function() {\n+ React.findDOMNode({render: function() {}});\n+ })\n .toThrow('Invariant Violation: Component (with keys: render) ' +\n 'contains `render` method but is not mounted in the DOM'\n );\ndiff --git a\/src\/renderers\/dom\/client\/eventPlugins\/__tests__\/AnalyticsEventPlugin-test.js b\/src\/renderers\/dom\/client\/eventPlugins\/__tests__\/AnalyticsEventPlugin-test.js\nindex 64c04d1e93..51c8eb1a57 100644\n--- a\/src\/renderers\/dom\/client\/eventPlugins\/__tests__\/AnalyticsEventPlugin-test.js\n+++ b\/src\/renderers\/dom\/client\/eventPlugins\/__tests__\/AnalyticsEventPlugin-test.js\n@@ -18,7 +18,6 @@ describe('AnalyticsEventPlugin', function() {\n var EventPluginHub;\n var EventPluginRegistry;\n var React;\n- var ReactBrowserEventEmitter;\n var ReactTestUtils;\n \n var DefaultEventPluginOrder;\n@@ -32,7 +31,6 @@ describe('AnalyticsEventPlugin', function() {\n EventPluginHub = require('EventPluginHub');\n EventPluginRegistry = require('EventPluginRegistry');\n React = require('React');\n- ReactBrowserEventEmitter = require('ReactBrowserEventEmitter');\n ReactTestUtils = require('ReactTestUtils');\n \n EventPluginRegistry._resetEventPlugins();\n@@ -65,7 +63,7 @@ describe('AnalyticsEventPlugin', function() {\n return (\n

\n+ data-analytics-events=\"click,doubleClick\">\n Test\n <\/div>\n );\n@@ -118,8 +116,8 @@ describe('AnalyticsEventPlugin', function() {\n render: function() {\n return (\n

\n+ data-analytics-id=\"test_invalid_events\"\n+ data-analytics-events=\"click,123\">\n Test\n <\/div>\n );\ndiff --git a\/src\/renderers\/dom\/client\/wrappers\/__tests__\/ReactDOMInput-test.js b\/src\/renderers\/dom\/client\/wrappers\/__tests__\/ReactDOMInput-test.js\nindex f67d43d1b8..b4759a07aa 100644\n--- a\/src\/renderers\/dom\/client\/wrappers\/__tests__\/ReactDOMInput-test.js\n+++ b\/src\/renderers\/dom\/client\/wrappers\/__tests__\/ReactDOMInput-test.js\n@@ -56,7 +56,7 @@ describe('ReactDOMInput', function() {\n it('should display \"foobar\" for `defaultValue` of `objToString`', function() {\n var objToString = {\n toString: function() {\n- return \"foobar\";\n+ return 'foobar';\n }\n };\n \n@@ -86,7 +86,7 @@ describe('ReactDOMInput', function() {\n expect(node.value).toEqual('true');\n });\n \n- it(\"should allow setting `value` to `false`\", function() {\n+ it('should allow setting `value` to `false`', function() {\n var stub = ;\n stub = ReactTestUtils.renderIntoDocument(stub);\n var node = React.findDOMNode(stub);\n@@ -106,7 +106,7 @@ describe('ReactDOMInput', function() {\n \n var objToString = {\n toString: function() {\n- return \"foobar\";\n+ return 'foobar';\n }\n };\n \ndiff --git a\/src\/renderers\/dom\/client\/wrappers\/__tests__\/ReactDOMSelect-test.js b\/src\/renderers\/dom\/client\/wrappers\/__tests__\/ReactDOMSelect-test.js\nindex 3166d913a5..72543b3bce 100644\n--- a\/src\/renderers\/dom\/client\/wrappers\/__tests__\/ReactDOMSelect-test.js\n+++ b\/src\/renderers\/dom\/client\/wrappers\/__tests__\/ReactDOMSelect-test.js\n@@ -154,7 +154,7 @@ describe('ReactDOMSelect', function() {\n });\n \n it('should reset child options selected when they are changed and `value` is set', function() {\n- var stub = ;\n stub = ReactTestUtils.renderIntoDocument(stub);\n \n stub.setProps({\n@@ -174,7 +174,7 @@ describe('ReactDOMSelect', function() {\n \n it('should allow setting `value` with `objectToString`', function() {\n var objectToString = {\n- animal: \"giraffe\",\n+ animal: 'giraffe',\n toString: function() {\n return this.animal;\n }\n@@ -194,7 +194,7 @@ describe('ReactDOMSelect', function() {\n expect(node.options[2].selected).toBe(false); \/\/ gorilla\n \n \/\/ Changing the `value` prop should change the selected options.\n- objectToString.animal = \"monkey\";\n+ objectToString.animal = 'monkey';\n stub.forceUpdate();\n \n expect(node.options[0].selected).toBe(true); \/\/ monkey\ndiff --git a\/src\/renderers\/dom\/client\/wrappers\/__tests__\/ReactDOMTextarea-test.js b\/src\/renderers\/dom\/client\/wrappers\/__tests__\/ReactDOMTextarea-test.js\nindex 902edd5e1b..9cd4659237 100644\n--- a\/src\/renderers\/dom\/client\/wrappers\/__tests__\/ReactDOMTextarea-test.js\n+++ b\/src\/renderers\/dom\/client\/wrappers\/__tests__\/ReactDOMTextarea-test.js\n@@ -67,7 +67,7 @@ describe('ReactDOMTextarea', function() {\n it('should display \"foobar\" for `defaultValue` of `objToString`', function() {\n var objToString = {\n toString: function() {\n- return \"foobar\";\n+ return 'foobar';\n }\n };\n \n@@ -136,7 +136,7 @@ describe('ReactDOMTextarea', function() {\n \n var objToString = {\n toString: function() {\n- return \"foo\";\n+ return 'foo';\n }\n };\n stub.replaceProps({value: objToString, onChange: emptyFunction});\n@@ -186,7 +186,7 @@ describe('ReactDOMTextarea', function() {\n spyOn(console, 'error');\n var obj = {\n toString: function() {\n- return \"sharkswithlasers\";\n+ return 'sharkswithlasers';\n }\n };\n var node = React.findDOMNode(renderTextarea(

{obj}<\/textarea>));\ndiff --git a\/src\/renderers\/dom\/server\/__tests__\/ReactServerRendering-test.js b\/src\/renderers\/dom\/server\/__tests__\/ReactServerRendering-test.js\nindex 1a9986dac9..05db7d7882 100644\n--- a\/src\/renderers\/dom\/server\/__tests__\/ReactServerRendering-test.js\n+++ b\/src\/renderers\/dom\/server\/__tests__\/ReactServerRendering-test.js\n@@ -14,7 +14,6 @@\n require('mock-modules')\n   .dontMock('ExecutionEnvironment')\n   .dontMock('React')\n-  .dontMock('ReactMount')\n   .dontMock('ReactServerRendering')\n   .dontMock('ReactTestUtils')\n   .dontMock('ReactMarkupChecksum');\n@@ -24,7 +23,6 @@ var mocks = require('mocks');\n var ExecutionEnvironment;\n var React;\n var ReactMarkupChecksum;\n-var ReactMount;\n var ReactReconcileTransaction;\n var ReactTestUtils;\n var ReactServerRendering;\n@@ -36,7 +34,6 @@ describe('ReactServerRendering', function() {\n     require('mock-modules').dumpCache();\n     React = require('React');\n     ReactMarkupChecksum = require('ReactMarkupChecksum');\n-    ReactMount = require('ReactMount');\n     ReactTestUtils = require('ReactTestUtils');\n     ReactReconcileTransaction = require('ReactReconcileTransaction');\n \ndiff --git a\/src\/renderers\/dom\/shared\/__tests__\/DOMPropertyOperations-test.js b\/src\/renderers\/dom\/shared\/__tests__\/DOMPropertyOperations-test.js\nindex 27b57aed6d..c9711ee77e 100644\n--- a\/src\/renderers\/dom\/shared\/__tests__\/DOMPropertyOperations-test.js\n+++ b\/src\/renderers\/dom\/shared\/__tests__\/DOMPropertyOperations-test.js\n@@ -212,7 +212,11 @@ describe('DOMPropertyOperations', function() {\n     it('should convert attribute values to string first', function() {\n       \/\/ Browsers default to this behavior, but some test environments do not.\n       \/\/ This ensures that we have consistent behavior.\n-      var obj = {toString: function() { return '<html>'; }};\n+      var obj = {\n+        toString: function() {\n+          return '<html>';\n+        }\n+      };\n       DOMPropertyOperations.setValueForProperty(stubNode, 'role', obj);\n       expect(stubNode.getAttribute('role')).toBe('<html>');\n     });\ndiff --git a\/src\/renderers\/dom\/shared\/__tests__\/Danger-test.js b\/src\/renderers\/dom\/shared\/__tests__\/Danger-test.js\nindex 7d7ae4e2c5..811ab51c8e 100644\n--- a\/src\/renderers\/dom\/shared\/__tests__\/Danger-test.js\n+++ b\/src\/renderers\/dom\/shared\/__tests__\/Danger-test.js\n@@ -99,7 +99,7 @@ describe('Danger', function() {\n         'Invariant Violation: dangerouslyRenderMarkup(...): Missing markup.'\n       );\n \n-      spyOn(console, \"error\");\n+      spyOn(console, 'error');\n \n       var renderedMarkup = Danger.dangerouslyRenderMarkup(['<p><\/p><p><\/p>']);\n       var args = console.error.argsForCall[0];\ndiff --git a\/src\/renderers\/dom\/shared\/__tests__\/ReactDOMComponent-test.js b\/src\/renderers\/dom\/shared\/__tests__\/ReactDOMComponent-test.js\nindex e45ad0b08b..dda078a441 100644\n--- a\/src\/renderers\/dom\/shared\/__tests__\/ReactDOMComponent-test.js\n+++ b\/src\/renderers\/dom\/shared\/__tests__\/ReactDOMComponent-test.js\n@@ -25,7 +25,7 @@ describe('ReactDOMComponent', function() {\n       ReactTestUtils = require('ReactTestUtils');\n     });\n \n-    it(\"should handle className\", function() {\n+    it('should handle className', function() {\n       var container = document.createElement('div');\n       React.render(<div style={{}} \/>, container);\n \n@@ -37,7 +37,7 @@ describe('ReactDOMComponent', function() {\n       expect(container.firstChild.className).toEqual('');\n     });\n \n-    it(\"should gracefully handle various style value types\", function() {\n+    it('should gracefully handle various style value types', function() {\n       var container = document.createElement('div');\n       React.render(<div style={{}} \/>, container);\n       var stubStyle = container.firstChild.style;\n@@ -59,7 +59,7 @@ describe('ReactDOMComponent', function() {\n     });\n \n     \/\/ TODO: (poshannessy) deprecate this pattern.\n-    it(\"should update styles when mutating style object\", function() {\n+    it('should update styles when mutating style object', function() {\n       \/\/ not actually used. Just to suppress the style mutation warning\n       spyOn(console, 'error');\n \n@@ -138,7 +138,7 @@ describe('ReactDOMComponent', function() {\n       expect(console.error.argsForCall.length).toBe(2);\n     });\n \n-    it(\"should update styles if initially null\", function() {\n+    it('should update styles if initially null', function() {\n       var styles = null;\n       var container = document.createElement('div');\n       React.render(<div style={styles} \/>, container);\n@@ -151,7 +151,7 @@ describe('ReactDOMComponent', function() {\n       expect(stubStyle.display).toEqual('block');\n     });\n \n-    it(\"should update styles if updated to null multiple times\", function() {\n+    it('should update styles if updated to null multiple times', function() {\n       var styles = null;\n       var container = document.createElement('div');\n       React.render(<div style={styles} \/>, container);\n@@ -172,25 +172,25 @@ describe('ReactDOMComponent', function() {\n       expect(stubStyle.display).toEqual('');\n     });\n \n-    it(\"should remove attributes\", function() {\n+    it('should remove attributes', function() {\n       var container = document.createElement('div');\n-      React.render(<img height='17' \/>, container);\n+      React.render(<img height=\"17\" \/>, container);\n \n       expect(container.firstChild.hasAttribute('height')).toBe(true);\n       React.render(<img \/>, container);\n       expect(container.firstChild.hasAttribute('height')).toBe(false);\n     });\n \n-    it(\"should remove properties\", function() {\n+    it('should remove properties', function() {\n       var container = document.createElement('div');\n-      React.render(<div className='monkey' \/>, container);\n+      React.render(<div className=\"monkey\" \/>, container);\n \n       expect(container.firstChild.className).toEqual('monkey');\n       React.render(<div \/>, container);\n       expect(container.firstChild.className).toEqual('');\n     });\n \n-    it(\"should clear a single style prop when changing 'style'\", function() {\n+    it('should clear a single style prop when changing `style`', function() {\n       var styles = {display: 'none', color: 'red'};\n       var container = document.createElement('div');\n       React.render(<div style={styles} \/>, container);\n@@ -203,7 +203,7 @@ describe('ReactDOMComponent', function() {\n       expect(stubStyle.color).toEqual('green');\n     });\n \n-    it(\"should clear all the styles when removing 'style'\", function() {\n+    it('should clear all the styles when removing `style`', function() {\n       var styles = {display: 'none', color: 'red'};\n       var container = document.createElement('div');\n       React.render(<div style={styles} \/>, container);\n@@ -215,7 +215,7 @@ describe('ReactDOMComponent', function() {\n       expect(stubStyle.color).toEqual('');\n     });\n \n-    it(\"should update styles when 'style' changes from null to object\", function() {\n+    it('should update styles when `style` changes from null to object', function() {\n       var container = document.createElement('div');\n       var styles = {color: 'red'};\n       React.render(<div style={styles} \/>, container);\n@@ -226,7 +226,7 @@ describe('ReactDOMComponent', function() {\n       expect(stubStyle.color).toEqual('red');\n     });\n \n-    it(\"should empty element when removing innerHTML\", function() {\n+    it('should empty element when removing innerHTML', function() {\n       var container = document.createElement('div');\n       React.render(<div dangerouslySetInnerHTML={{__html: ':)'}} \/>, container);\n \n@@ -235,7 +235,7 @@ describe('ReactDOMComponent', function() {\n       expect(container.firstChild.innerHTML).toEqual('');\n     });\n \n-    it(\"should transition from string content to innerHTML\", function() {\n+    it('should transition from string content to innerHTML', function() {\n       var container = document.createElement('div');\n       React.render(<div>hello<\/div>, container);\n \n@@ -247,7 +247,7 @@ describe('ReactDOMComponent', function() {\n       expect(container.firstChild.innerHTML).toEqual('goodbye');\n     });\n \n-    it(\"should transition from innerHTML to string content\", function() {\n+    it('should transition from innerHTML to string content', function() {\n       var container = document.createElement('div');\n       React.render(\n         <div dangerouslySetInnerHTML={{__html: 'bonjour'}} \/>,\n@@ -259,7 +259,7 @@ describe('ReactDOMComponent', function() {\n       expect(container.firstChild.innerHTML).toEqual('adieu');\n     });\n \n-    it(\"should not incur unnecessary DOM mutations\", function() {\n+    it('should not incur unnecessary DOM mutations', function() {\n       var container = document.createElement('div');\n       React.render(<div value=\"\" \/>, container);\n \n@@ -287,7 +287,7 @@ describe('ReactDOMComponent', function() {\n     var genMarkup;\n \n     function quoteRegexp(str) {\n-      return (str + '').replace(\/([.?*+\\^$\\[\\]\\\\(){}|-])\/g, \"\\\\$1\");\n+      return (str + '').replace(\/([.?*+\\^$\\[\\]\\\\(){}|-])\/g, '\\\\$1');\n     }\n \n     beforeEach(function() {\n@@ -315,7 +315,7 @@ describe('ReactDOMComponent', function() {\n       this.addMatchers({\n         toHaveAttribute: function(attr, value) {\n           var expected = '(?:^|\\\\s)' + attr + '=[\\\\\\'\"]';\n-          if (typeof value != 'undefined') {\n+          if (typeof value !== 'undefined') {\n             expected += quoteRegexp(value) + '[\\\\\\'\"]';\n           }\n           return this.actual.match(new RegExp(expected));\n@@ -323,13 +323,13 @@ describe('ReactDOMComponent', function() {\n       });\n     });\n \n-    it(\"should generate the correct markup with className\", function() {\n+    it('should generate the correct markup with className', function() {\n       expect(genMarkup({className: 'a'})).toHaveAttribute('class', 'a');\n       expect(genMarkup({className: 'a b'})).toHaveAttribute('class', 'a b');\n       expect(genMarkup({className: ''})).toHaveAttribute('class', '');\n     });\n \n-    it(\"should escape style names and values\", function() {\n+    it('should escape style names and values', function() {\n       expect(genMarkup({\n         style: {'b&ckground': '<3'}\n       })).toHaveAttribute('style', 'b&ckground:<3;');\n@@ -340,7 +340,7 @@ describe('ReactDOMComponent', function() {\n     var genMarkup;\n \n     function quoteRegexp(str) {\n-      return (str + '').replace(\/([.?*+\\^$\\[\\]\\\\(){}|-])\/g, \"\\\\$1\");\n+      return (str + '').replace(\/([.?*+\\^$\\[\\]\\\\(){}|-])\/g, '\\\\$1');\n     }\n \n     beforeEach(function() {\n@@ -368,7 +368,7 @@ describe('ReactDOMComponent', function() {\n       });\n     });\n \n-    it(\"should handle dangerouslySetInnerHTML\", function() {\n+    it('should handle dangerouslySetInnerHTML', function() {\n       var innerHTML = {__html: 'testContent'};\n       expect(\n         genMarkup({dangerouslySetInnerHTML: innerHTML})\n@@ -399,13 +399,13 @@ describe('ReactDOMComponent', function() {\n         var stubComponent = new StubNativeComponent({\n           type: StubNativeComponent,\n           props: props,\n-          _owner: null,\n+          _owner: null\n         });\n         return stubComponent.mountComponent('test', transaction, {});\n       };\n     });\n \n-    it(\"should warn against children for void elements\", function() {\n+    it('should warn against children for void elements', function() {\n       spyOn(console, 'error');\n \n       var container = document.createElement('div');\n@@ -416,7 +416,7 @@ describe('ReactDOMComponent', function() {\n       expect(console.error.argsForCall[0][0]).toContain('void element');\n     });\n \n-    it(\"should warn against dangerouslySetInnerHTML for void elements\", function() {\n+    it('should warn against dangerouslySetInnerHTML for void elements', function() {\n       spyOn(console, 'error');\n \n       var container = document.createElement('div');\n@@ -430,7 +430,7 @@ describe('ReactDOMComponent', function() {\n       expect(console.error.argsForCall[0][0]).toContain('void element');\n     });\n \n-    it(\"should treat menuitem as a void element but still create the closing tag\", function() {\n+    it('should treat menuitem as a void element but still create the closing tag', function() {\n       spyOn(console, 'error');\n \n       var container = document.createElement('div');\n@@ -445,7 +445,7 @@ describe('ReactDOMComponent', function() {\n       expect(console.error.argsForCall[0][0]).toContain('void element');\n     });\n \n-    it(\"should validate against multiple children props\", function() {\n+    it('should validate against multiple children props', function() {\n       expect(function() {\n         mountComponent({children: '', dangerouslySetInnerHTML: ''});\n       }).toThrow(\n@@ -490,14 +490,14 @@ describe('ReactDOMComponent', function() {\n       }).not.toThrow();\n     });\n \n-    it(\"should warn about contentEditable and children\", function() {\n+    it('should warn about contentEditable and children', function() {\n       spyOn(console, 'error');\n       mountComponent({contentEditable: true, children: ''});\n       expect(console.error.argsForCall.length).toBe(1);\n       expect(console.error.argsForCall[0][0]).toContain('contentEditable');\n     });\n \n-    it(\"should validate against invalid styles\", function() {\n+    it('should validate against invalid styles', function() {\n       expect(function() {\n         mountComponent({style: 'display: none'});\n       }).toThrow(\n@@ -507,8 +507,7 @@ describe('ReactDOMComponent', function() {\n       );\n     });\n \n-    it(\"should execute custom event plugin listening behavior\", function() {\n-      var React = require('React');\n+    it('should execute custom event plugin listening behavior', function() {\n       var SimpleEventPlugin = require('SimpleEventPlugin');\n \n       SimpleEventPlugin.didPutListener = mocks.getMockFunction();\n@@ -527,8 +526,7 @@ describe('ReactDOMComponent', function() {\n       expect(SimpleEventPlugin.willDeleteListener.mock.calls.length).toBe(1);\n     });\n \n-    it(\"should handle null and missing properly with event hooks\", function() {\n-      var React = require('React');\n+    it('should handle null and missing properly with event hooks', function() {\n       var SimpleEventPlugin = require('SimpleEventPlugin');\n \n       SimpleEventPlugin.didPutListener = mocks.getMockFunction();\n@@ -570,7 +568,7 @@ describe('ReactDOMComponent', function() {\n       container = document.createElement('div');\n     });\n \n-    it(\"should warn against children for void elements\", function() {\n+    it('should warn against children for void elements', function() {\n       spyOn(console, 'error');\n \n       React.render(<input \/>, container);\n@@ -580,7 +578,7 @@ describe('ReactDOMComponent', function() {\n       expect(console.error.argsForCall[0][0]).toContain('void element');\n     });\n \n-    it(\"should warn against dangerouslySetInnerHTML for void elements\", function() {\n+    it('should warn against dangerouslySetInnerHTML for void elements', function() {\n       spyOn(console, 'error');\n \n       React.render(<input \/>, container);\n@@ -593,7 +591,7 @@ describe('ReactDOMComponent', function() {\n       expect(console.error.argsForCall[0][0]).toContain('void element');\n     });\n \n-    it(\"should validate against multiple children props\", function() {\n+    it('should validate against multiple children props', function() {\n       React.render(<div><\/div>, container);\n \n       expect(function() {\n@@ -607,7 +605,7 @@ describe('ReactDOMComponent', function() {\n       );\n     });\n \n-    it(\"should warn about contentEditable and children\", function() {\n+    it('should warn about contentEditable and children', function() {\n       spyOn(console, 'error');\n       React.render(\n         <div contentEditable><div \/><\/div>,\n@@ -617,7 +615,7 @@ describe('ReactDOMComponent', function() {\n       expect(console.error.argsForCall[0][0]).toContain('contentEditable');\n     });\n \n-    it(\"should validate against invalid styles\", function() {\n+    it('should validate against invalid styles', function() {\n       React.render(<div><\/div>, container);\n \n       expect(function() {\n@@ -629,7 +627,7 @@ describe('ReactDOMComponent', function() {\n       );\n     });\n \n-    it(\"should properly escape text content and attributes values\", function() {\n+    it('should properly escape text content and attributes values', function() {\n       expect(\n         React.renderToStaticMarkup(\n           React.DOM.div({\n@@ -648,7 +646,7 @@ describe('ReactDOMComponent', function() {\n   });\n \n   describe('unmountComponent', function() {\n-    it(\"should clean up listeners\", function() {\n+    it('should clean up listeners', function() {\n       var React = require('React');\n       var ReactBrowserEventEmitter = require('ReactBrowserEventEmitter');\n       var ReactMount = require('ReactMount');\n@@ -784,10 +782,14 @@ describe('ReactDOMComponent', function() {\n         render: () => <Row \/>\n       });\n       var Table = React.createClass({\n-        render: function() { return <table>{this.props.children}<\/table>; }\n+        render: function() {\n+          return <table>{this.props.children}<\/table>;\n+        }\n       });\n       var FancyTable = React.createClass({\n-        render: function() { return <Table>{this.props.children}<\/Table>; }\n+        render: function() {\n+          return <Table>{this.props.children}<\/Table>;\n+        }\n       });\n \n       var Viz1 = React.createClass({\n@@ -833,7 +835,9 @@ describe('ReactDOMComponent', function() {\n       );\n \n       var Link = React.createClass({\n-        render: function() { return <a>{this.props.children}<\/a>; }\n+        render: function() {\n+          return <a>{this.props.children}<\/a>;\n+        }\n       });\n       ReactTestUtils.renderIntoDocument(<Link><div><Link \/><\/div><\/Link>);\n       expect(console.error.calls.length).toBe(6);\ndiff --git a\/src\/renderers\/shared\/event\/__tests__\/EventPluginHub-test.js b\/src\/renderers\/shared\/event\/__tests__\/EventPluginHub-test.js\nindex 9a54b7b245..9caf5896ad 100644\n--- a\/src\/renderers\/shared\/event\/__tests__\/EventPluginHub-test.js\n+++ b\/src\/renderers\/shared\/event\/__tests__\/EventPluginHub-test.js\n@@ -26,7 +26,7 @@ describe('EventPluginHub', function() {\n     isEventSupported.mockReturnValueOnce(false);\n   });\n \n-  it(\"should prevent non-function listeners\", function() {\n+  it('should prevent non-function listeners', function() {\n     expect(function() {\n       EventPluginHub.putListener(1, 'onClick', 'not a function');\n     }).toThrow(\ndiff --git a\/src\/renderers\/shared\/event\/eventPlugins\/__tests__\/ResponderEventPlugin-test.js b\/src\/renderers\/shared\/event\/eventPlugins\/__tests__\/ResponderEventPlugin-test.js\nindex 57ffd5d323..fef1859867 100644\n--- a\/src\/renderers\/shared\/event\/eventPlugins\/__tests__\/ResponderEventPlugin-test.js\n+++ b\/src\/renderers\/shared\/event\/eventPlugins\/__tests__\/ResponderEventPlugin-test.js\n@@ -13,10 +13,8 @@\n \n var EventPluginHub;\n var EventConstants;\n-var EventPropagators;\n var ReactInstanceHandles;\n var ResponderEventPlugin;\n-var SyntheticEvent;\n var EventPluginUtils;\n \n var GRANDPARENT_ID = '.0';\n@@ -25,7 +23,6 @@ var CHILD_ID = '.0.0.0';\n var CHILD_ID2 = '.0.0.1';\n \n var topLevelTypes;\n-var responderEventTypes;\n \n var touch = function(nodeHandle, i) {\n   return {target: nodeHandle, identifier: i};\n@@ -88,7 +85,7 @@ var _touchConfig =\n     ),\n     topLevelType: topType,\n     target: targetNodeHandle,\n-    targetID: targetNodeHandle,\n+    targetID: targetNodeHandle\n   };\n };\n \n@@ -180,13 +177,13 @@ var oneEventLoopTestConfig = function(readableIDToID) {\n     responderTerminationRequest: {},\n \n     \/\/ Non-negotiation\n-    responderReject:      {}, \/\/ These do not bubble capture.\n+    responderReject:    {}, \/\/ These do not bubble capture.\n     responderGrant:     {},\n     responderStart:     {},\n     responderMove:      {},\n     responderTerminate: {},\n     responderEnd:       {},\n-    responderRelease:   {},\n+    responderRelease:   {}\n   };\n   for (var eventName in ret) {\n     for (var readableNodeName in readableIDToID) {\n@@ -218,23 +215,29 @@ var registerTestHandlers = function(eventTestConfig, readableIDToID) {\n       '\\nFor event test config:\\n' + JSON.stringify(eventTestConfig) + '\\n'\n     );\n   };\n+  \/*eslint-disable no-loop-func, no-shadow *\/\n   var registerOneEventType = function(registrationName, eventTypeTestConfig) {\n     for (var readableID in eventTypeTestConfig) {\n       var nodeConfig = eventTypeTestConfig[readableID];\n       var id = readableIDToID[readableID];\n       var handler = nodeConfig.order === NA ? neverFire.bind(null, readableID, registrationName) :\n-        function(readableID, registrationName, nodeConfig, e) {\n+        \/\/ We partially apply readableID and nodeConfig, as they change in the\n+        \/\/ parent closure across iterations.\n+        function(readableID, nodeConfig, e) {\n           expect(\n             readableID + '->' + registrationName + ' index:' + runs.dispatchCount++\n           ).toBe(\n             readableID + '->' + registrationName + ' index:' + nodeConfig.order\n           );\n-          nodeConfig.assertEvent && nodeConfig.assertEvent(e);\n+          if (nodeConfig.assertEvent) {\n+            nodeConfig.assertEvent(e);\n+          }\n           return nodeConfig.returnVal;\n-        }.bind(null, readableID, registrationName, nodeConfig);\n+        }.bind(null, readableID, nodeConfig);\n       EventPluginHub.putListener(id, registrationName, handler);\n     }\n   };\n+  \/*eslint-enable no-loop-func, no-shadow *\/\n   for (var eventName in eventTestConfig) {\n     var oneEventTypeTestConfig = eventTestConfig[eventName];\n     var hasTwoPhase = !!oneEventTypeTestConfig.bubbled;\n@@ -306,13 +309,13 @@ var run = function(config, hierarchyConfig, nativeEventConfig) {\n var three = {\n   grandParent: GRANDPARENT_ID,\n   parent: PARENT_ID,\n-  child: CHILD_ID,\n+  child: CHILD_ID\n };\n \n var siblings = {\n   parent: PARENT_ID,\n   childOne: CHILD_ID,\n-  childTwo: CHILD_ID2,\n+  childTwo: CHILD_ID2\n };\n \n describe('ResponderEventPlugin', function() {\n@@ -322,10 +325,8 @@ describe('ResponderEventPlugin', function() {\n     EventConstants = require('EventConstants');\n     EventPluginHub = require('EventPluginHub');\n     EventPluginUtils = require('EventPluginUtils');\n-    EventPropagators = require('EventPropagators');\n     ReactInstanceHandles = require('ReactInstanceHandles');\n     ResponderEventPlugin = require('ResponderEventPlugin');\n-    SyntheticEvent = require('SyntheticEvent');\n \n     EventPluginHub.injection.injectInstanceHandle(ReactInstanceHandles);\n \n@@ -341,17 +342,16 @@ describe('ResponderEventPlugin', function() {\n     });\n \n     topLevelTypes = EventConstants.topLevelTypes;\n-    responderEventTypes = ResponderEventPlugin.eventTypes;\n   });\n \n   it('should do nothing when no one wants to respond', function() {\n     var config = oneEventLoopTestConfig(three);\n     config.startShouldSetResponder.captured.grandParent = {order: 0, returnVal: false};\n-    config.startShouldSetResponder.captured.parent =      {order: 1, returnVal: false};\n-    config.startShouldSetResponder.captured.child =       {order: 2, returnVal: false};\n-    config.startShouldSetResponder.bubbled.child =        {order: 3, returnVal: false};\n-    config.startShouldSetResponder.bubbled.parent =       {order: 4, returnVal: false};\n-    config.startShouldSetResponder.bubbled.grandParent =  {order: 5, returnVal: false};\n+    config.startShouldSetResponder.captured.parent = {order: 1, returnVal: false};\n+    config.startShouldSetResponder.captured.child = {order: 2, returnVal: false};\n+    config.startShouldSetResponder.bubbled.child = {order: 3, returnVal: false};\n+    config.startShouldSetResponder.bubbled.parent = {order: 4, returnVal: false};\n+    config.startShouldSetResponder.bubbled.grandParent = {order: 5, returnVal: false};\n     run(config, three, startConfig(three.child, [three.child], [0]));\n     expect(ResponderEventPlugin.getResponderID()).toBe(null);\n \ndiff --git a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactComponent-test.js b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactComponent-test.js\nindex 675672acb5..6b93a951d1 100644\n--- a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactComponent-test.js\n+++ b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactComponent-test.js\n@@ -12,19 +12,15 @@\n 'use strict';\n \n var React;\n-var ReactInstanceMap;\n var ReactTestUtils;\n \n var mocks;\n-var reactComponentExpect;\n \n describe('ReactComponent', function() {\n   beforeEach(function() {\n     mocks = require('mocks');\n     React = require('React');\n-    ReactInstanceMap = require('ReactInstanceMap');\n     ReactTestUtils = require('ReactTestUtils');\n-    reactComponentExpect = require('reactComponentExpect');\n   });\n \n   it('should throw on invalid render targets', function() {\ndiff --git a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactComponentLifeCycle-test.js b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactComponentLifeCycle-test.js\nindex 7d61482da9..9f1d426f26 100644\n--- a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactComponentLifeCycle-test.js\n+++ b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactComponentLifeCycle-test.js\n@@ -207,7 +207,7 @@ describe('ReactComponentLifeCycle', function() {\n   it('throws when accessing state in componentWillMount', function() {\n     var StatefulComponent = React.createClass({\n       componentWillMount: function() {\n-        this.state.yada;\n+        void this.state.yada;\n       },\n       render: function() {\n         return (\n@@ -271,7 +271,7 @@ describe('ReactComponentLifeCycle', function() {\n         expect(this.isMounted()).toBeTruthy();\n       },\n       render: function() {\n-        expect(this.isMounted()).toBeFalsy()\n+        expect(this.isMounted()).toBeFalsy();\n         return <div\/>;\n       }\n     });\n@@ -530,7 +530,7 @@ describe('ReactComponentLifeCycle', function() {\n \n     \/\/ Since `instance` is a root component, we can set its props. This also\n     \/\/ makes Tooltip rerender the tooltip component, which shouldn't throw.\n-    instance.setProps({text: \"dos\", tooltipText: \"two\"});\n+    instance.setProps({text: 'dos', tooltipText: 'two'});\n   });\n \n   it('should not allow setProps() called on an unmounted element',\ndiff --git a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactCompositeComponent-test.js b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactCompositeComponent-test.js\nindex 20f238fdf4..5282a5fcee 100644\n--- a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactCompositeComponent-test.js\n+++ b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactCompositeComponent-test.js\n@@ -326,7 +326,7 @@ describe('ReactCompositeComponent', function() {\n         expect(() => {\n           this.setState({value: 2}, function() {\n             cbCalled = true;\n-          })\n+          });\n         }).not.toThrow();\n       },\n       render: function() {\n@@ -515,7 +515,7 @@ describe('ReactCompositeComponent', function() {\n \n       render: function() {\n         return <div>{this.context.foo}<\/div>;\n-      },\n+      }\n     });\n \n     var component = ReactTestUtils.renderIntoDocument(<Parent \/>);\n@@ -632,7 +632,7 @@ describe('ReactCompositeComponent', function() {\n \n     expect(parentInstance.state.flag).toBe(false);\n     ReactUpdates.batchedUpdates(function() {\n-        parentInstance.setState({flag: true});\n+      parentInstance.setState({flag: true});\n     });\n     expect(parentInstance.state.flag).toBe(true);\n \n@@ -852,7 +852,7 @@ describe('ReactCompositeComponent', function() {\n \n       getChildContext: function() {\n         return {\n-          foo: \"bar\"\n+          foo: 'bar'\n         };\n       },\n \ndiff --git a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactCompositeComponentError-test.js b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactCompositeComponentError-test.js\nindex 053399043f..8599116422 100644\n--- a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactCompositeComponentError-test.js\n+++ b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactCompositeComponentError-test.js\n@@ -12,7 +12,6 @@\n 'use strict';\n \n var React = require('React');\n-var ReactTestUtils = require('ReactTestUtils');\n var ReactErrorUtils = require('ReactErrorUtils');\n \n describe('ReactCompositeComponent-error', function() {\n@@ -26,7 +25,7 @@ describe('ReactCompositeComponent-error', function() {\n       }\n     });\n \n-    new Component();\n+    void new Component();\n \n     expect(ReactErrorUtils.guard.mostRecentCall.args[1])\n       .toEqual('Component.someHandler');\ndiff --git a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactCompositeComponentNestedState-test.js b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactCompositeComponentNestedState-test.js\nindex 0963484d46..310890b874 100644\n--- a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactCompositeComponentNestedState-test.js\n+++ b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactCompositeComponentNestedState-test.js\n@@ -58,7 +58,7 @@ describe('ReactCompositeComponentNestedState-state', function() {\n         this.setState(function(state, props) {\n           this.props.logger('setState-this', this.state.hue, this.props.color);\n           this.props.logger('setState-args', state.hue, props.color);\n-          return {hue: shade + ' ' + props.color}\n+          return {hue: shade + ' ' + props.color};\n         }, function() {\n           this.props.logger('after-setState', this.state.hue, this.props.color);\n         });\n@@ -79,7 +79,7 @@ describe('ReactCompositeComponentNestedState-state', function() {\n           <button onClick={this.handleHue.bind(this, 'light', 'green')}>\n             Light Green\n           <\/button>\n-        <\/div>\n+        <\/div>;\n       }\n     });\n \n@@ -88,7 +88,7 @@ describe('ReactCompositeComponentNestedState-state', function() {\n \n     var logger = mocks.getMockFunction();\n \n-    var instance = React.render(\n+    void React.render(\n       <ParentComponent logger={logger} \/>,\n       container\n     );\ndiff --git a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactCompositeComponentState-test.js b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactCompositeComponentState-test.js\nindex 76a1f923d7..9e788848a2 100644\n--- a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactCompositeComponentState-test.js\n+++ b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactCompositeComponentState-test.js\n@@ -14,9 +14,6 @@\n var mocks = require('mocks');\n \n var React;\n-var ReactInstanceMap;\n-var ReactTestUtils;\n-var reactComponentExpect;\n \n var TestComponent;\n \n@@ -24,9 +21,6 @@ describe('ReactCompositeComponent-state', function() {\n \n   beforeEach(function() {\n     React = require('React');\n-    ReactInstanceMap = require('ReactInstanceMap');\n-    ReactTestUtils = require('ReactTestUtils');\n-    reactComponentExpect = require('reactComponentExpect');\n \n     TestComponent = React.createClass({\n       peekAtState: function(from, state) {\ndiff --git a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactEmptyComponent-test.js b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactEmptyComponent-test.js\nindex 402c0b894c..d3b996c0e8 100644\n--- a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactEmptyComponent-test.js\n+++ b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactEmptyComponent-test.js\n@@ -235,7 +235,9 @@ describe('ReactEmptyComponent', function() {\n   it('does not break when updating during mount', function() {\n     var Child = React.createClass({\n       componentDidMount() {\n-        this.props.onMount && this.props.onMount();\n+        if (this.props.onMount) {\n+          this.props.onMount();\n+        }\n       },\n       render() {\n         if (!this.props.visible) {\n@@ -262,7 +264,7 @@ describe('ReactEmptyComponent', function() {\n     });\n \n     expect(function() {\n-      ReactTestUtils.renderIntoDocument(<Parent\/>)\n+      ReactTestUtils.renderIntoDocument(<Parent \/>);\n     }).not.toThrow();\n   });\n });\ndiff --git a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactIdentity-test.js b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactIdentity-test.js\nindex 2779e80904..e6e95ae85c 100644\n--- a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactIdentity-test.js\n+++ b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactIdentity-test.js\n@@ -107,12 +107,12 @@ describe('ReactIdentity', function() {\n     var Wrapper = React.createClass({\n \n       render: function() {\n-        var span1 = <span ref=\"span1\" key={key} \/>;\n-        var span2 = <span ref=\"span2\" \/>;\n+        var s1 = <span ref=\"span1\" key={key} \/>;\n+        var s2 = <span ref=\"span2\" \/>;\n \n         var map = {};\n-        map[key] = span2;\n-        return <div>{[span1, frag(map)]}<\/div>;\n+        map[key] = s2;\n+        return <div>{[s1, frag(map)]}<\/div>;\n       }\n \n     });\n@@ -132,7 +132,10 @@ describe('ReactIdentity', function() {\n \n   it('should allow any character as a key, in a detached parent', function() {\n     var detachedContainer = document.createElement('div');\n-    renderAComponentWithKeyIntoContainer(\"<'WEIRD\/&\\\\key'>\", detachedContainer);\n+    renderAComponentWithKeyIntoContainer(\n+      \"<'WEIRD\/&\\\\key'>\",\n+      detachedContainer\n+    );\n   });\n \n   it('should allow any character as a key, in an attached parent', function() {\n@@ -141,7 +144,10 @@ describe('ReactIdentity', function() {\n     var attachedContainer = document.createElement('div');\n     document.body.appendChild(attachedContainer);\n \n-    renderAComponentWithKeyIntoContainer(\"<'WEIRD\/&\\\\key'>\", attachedContainer);\n+    renderAComponentWithKeyIntoContainer(\n+      \"<'WEIRD\/&\\\\key'>\",\n+      attachedContainer\n+    );\n \n     document.body.removeChild(attachedContainer);\n   });\ndiff --git a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactInstanceHandles-test.js b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactInstanceHandles-test.js\nindex 58a50cfc8e..aedaed7672 100644\n--- a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactInstanceHandles-test.js\n+++ b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactInstanceHandles-test.js\n@@ -141,7 +141,7 @@ describe('ReactInstanceHandles', function() {\n       expect(function() {\n         ReactMount.findComponentRoot(\n           parentNode,\n-          ReactMount.getID(childNodeB) + \":junk\"\n+          ReactMount.getID(childNodeB) + ':junk'\n         );\n       }).toThrow(\n         'Invariant Violation: findComponentRoot(..., .0.1:0:junk): ' +\n@@ -177,14 +177,14 @@ describe('ReactInstanceHandles', function() {\n   });\n \n   describe('traverseTwoPhase', function() {\n-    it(\"should not traverse when traversing outside DOM\", function() {\n+    it('should not traverse when traversing outside DOM', function() {\n       var targetID = '';\n       var expectedAggregation = [];\n       ReactInstanceHandles.traverseTwoPhase(targetID, argAggregator, ARG);\n       expect(aggregatedArgs).toEqual(expectedAggregation);\n     });\n \n-    it(\"should traverse two phase across component boundary\", function() {\n+    it('should traverse two phase across component boundary', function() {\n       var parent = renderParentIntoDocument();\n       var targetID = getNodeID(parent.refs.P_P1_C1.refs.DIV_1);\n       var expectedAggregation = [\n@@ -202,7 +202,7 @@ describe('ReactInstanceHandles', function() {\n       expect(aggregatedArgs).toEqual(expectedAggregation);\n     });\n \n-    it(\"should traverse two phase at shallowest node\", function() {\n+    it('should traverse two phase at shallowest node', function() {\n       var parent = renderParentIntoDocument();\n       var targetID = getNodeID(parent.refs.P);\n       var expectedAggregation = [\n@@ -215,7 +215,7 @@ describe('ReactInstanceHandles', function() {\n   });\n \n   describe('traverseEnterLeave', function() {\n-    it(\"should not traverse when enter\/leaving outside DOM\", function() {\n+    it('should not traverse when enter\/leaving outside DOM', function() {\n       var targetID = '';\n       var expectedAggregation = [];\n       ReactInstanceHandles.traverseEnterLeave(\n@@ -224,7 +224,7 @@ describe('ReactInstanceHandles', function() {\n       expect(aggregatedArgs).toEqual(expectedAggregation);\n     });\n \n-    it(\"should not traverse if enter\/leave the same node\", function() {\n+    it('should not traverse if enter\/leave the same node', function() {\n       var parent = renderParentIntoDocument();\n       var leaveID = getNodeID(parent.refs.P_P1_C1.refs.DIV_1);\n       var enterID = getNodeID(parent.refs.P_P1_C1.refs.DIV_1);\n@@ -235,7 +235,7 @@ describe('ReactInstanceHandles', function() {\n       expect(aggregatedArgs).toEqual(expectedAggregation);\n     });\n \n-    it(\"should traverse enter\/leave to sibling - avoids parent\", function() {\n+    it('should traverse enter\/leave to sibling - avoids parent', function() {\n       var parent = renderParentIntoDocument();\n       var leaveID = getNodeID(parent.refs.P_P1_C1.refs.DIV_1);\n       var enterID = getNodeID(parent.refs.P_P1_C1.refs.DIV_2);\n@@ -250,7 +250,7 @@ describe('ReactInstanceHandles', function() {\n       expect(aggregatedArgs).toEqual(expectedAggregation);\n     });\n \n-    it(\"should traverse enter\/leave to parent - avoids parent\", function() {\n+    it('should traverse enter\/leave to parent - avoids parent', function() {\n       var parent = renderParentIntoDocument();\n       var leaveID = getNodeID(parent.refs.P_P1_C1.refs.DIV_1);\n       var enterID = getNodeID(parent.refs.P_P1_C1.refs.DIV);\n@@ -263,7 +263,7 @@ describe('ReactInstanceHandles', function() {\n       expect(aggregatedArgs).toEqual(expectedAggregation);\n     });\n \n-    it(\"should enter from the window\", function() {\n+    it('should enter from the window', function() {\n       var parent = renderParentIntoDocument();\n       var leaveID = ''; \/\/ From the window or outside of the React sandbox.\n       var enterID = getNodeID(parent.refs.P_P1_C1.refs.DIV);\n@@ -278,7 +278,7 @@ describe('ReactInstanceHandles', function() {\n       expect(aggregatedArgs).toEqual(expectedAggregation);\n     });\n \n-    it(\"should enter from the window to the shallowest\", function() {\n+    it('should enter from the window to the shallowest', function() {\n       var parent = renderParentIntoDocument();\n       var leaveID = ''; \/\/ From the window or outside of the React sandbox.\n       var enterID = getNodeID(parent.refs.P);\n@@ -291,7 +291,7 @@ describe('ReactInstanceHandles', function() {\n       expect(aggregatedArgs).toEqual(expectedAggregation);\n     });\n \n-    it(\"should leave to the window\", function() {\n+    it('should leave to the window', function() {\n       var parent = renderParentIntoDocument();\n       var enterID = ''; \/\/ From the window or outside of the React sandbox.\n       var leaveID = getNodeID(parent.refs.P_P1_C1.refs.DIV);\n@@ -306,7 +306,7 @@ describe('ReactInstanceHandles', function() {\n       expect(aggregatedArgs).toEqual(expectedAggregation);\n     });\n \n-    it(\"should leave to the window from the shallowest\", function() {\n+    it('should leave to the window from the shallowest', function() {\n       var parent = renderParentIntoDocument();\n       var enterID = ''; \/\/ From the window or outside of the React sandbox.\n       var leaveID = getNodeID(parent.refs.P_P1_C1.refs.DIV);\n@@ -323,7 +323,7 @@ describe('ReactInstanceHandles', function() {\n   });\n \n   describe('getNextDescendantID', function() {\n-    it(\"should return next descendent from window\", function() {\n+    it('should return next descendent from window', function() {\n       var parent = renderParentIntoDocument();\n       expect(\n         ReactInstanceHandles._getNextDescendantID(\n@@ -333,11 +333,11 @@ describe('ReactInstanceHandles', function() {\n       ).toBe(getNodeID(parent.refs.P));\n     });\n \n-    it(\"should return window for next descendent towards window\", function() {\n+    it('should return window for next descendent towards window', function() {\n       expect(ReactInstanceHandles._getNextDescendantID('', '')).toBe('');\n     });\n \n-    it(\"should return self for next descendent towards self\", function() {\n+    it('should return self for next descendent towards self', function() {\n       var parent = renderParentIntoDocument();\n       expect(\n         ReactInstanceHandles._getNextDescendantID(\n@@ -349,7 +349,7 @@ describe('ReactInstanceHandles', function() {\n   });\n \n   describe('getFirstCommonAncestorID', function() {\n-    it(\"should determine the first common ancestor correctly\", function() {\n+    it('should determine the first common ancestor correctly', function() {\n       var parent = renderParentIntoDocument();\n       var ancestors = [\n         \/\/ Common ancestor from window to deep element is ''.\ndiff --git a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactUpdates-test.js b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactUpdates-test.js\nindex b04d455a02..28eb389725 100644\n--- a\/src\/renderers\/shared\/reconciler\/__tests__\/ReactUpdates-test.js\n+++ b\/src\/renderers\/shared\/reconciler\/__tests__\/ReactUpdates-test.js\n@@ -530,7 +530,7 @@ describe('ReactUpdates', function() {\n         return {x: 0};\n       },\n       componentDidUpdate: function() {\n-        expect(React.findDOMNode(b).textContent).toBe(\"B1\");\n+        expect(React.findDOMNode(b).textContent).toBe('B1');\n         aUpdated = true;\n       },\n       render: function() {\n@@ -698,7 +698,9 @@ describe('ReactUpdates', function() {\n     });\n \n     var Z = React.createClass({\n-      render: function() { return <div \/>; },\n+      render: function() {\n+        return <div \/>;\n+      },\n       componentWillUpdate: function() {\n         x.go();\n       }\n@@ -785,7 +787,6 @@ describe('ReactUpdates', function() {\n         return <div \/>;\n       },\n       componentDidUpdate: function() {\n-        var component = this;\n         ReactUpdates.asap(function() {\n           expect(this).toBe(component);\n           callbackCount++;\n@@ -793,7 +794,7 @@ describe('ReactUpdates', function() {\n             callbackCount++;\n           });\n           expect(callbackCount).toBe(1);\n-        }, this);\n+        }, component);\n         expect(callbackCount).toBe(0);\n       }\n     });\ndiff --git a\/src\/renderers\/shared\/reconciler\/__tests__\/refs-destruction-test.js b\/src\/renderers\/shared\/reconciler\/__tests__\/refs-destruction-test.js\nindex 20d32270c8..845b3fb305 100644\n--- a\/src\/renderers\/shared\/reconciler\/__tests__\/refs-destruction-test.js\n+++ b\/src\/renderers\/shared\/reconciler\/__tests__\/refs-destruction-test.js\n@@ -12,7 +12,6 @@\n 'use strict';\n \n var React = require('React');\n-var ReactTestUtils = require('ReactTestUtils');\n var reactComponentExpect = require('reactComponentExpect');\n \n var TestComponent = React.createClass({\n@@ -34,7 +33,7 @@ describe('refs-destruction', function() {\n     require('mock-modules').dumpCache();\n   });\n \n-  it(\"should remove refs when destroying the parent\", function() {\n+  it('should remove refs when destroying the parent', function() {\n     var container = document.createElement('div');\n     var testInstance = React.render(<TestComponent \/>, container);\n     reactComponentExpect(testInstance.refs.theInnerDiv)\n@@ -44,7 +43,7 @@ describe('refs-destruction', function() {\n     expect(Object.keys(testInstance.refs || {}).length).toEqual(0);\n   });\n \n-  it(\"should remove refs when destroying the child\", function() {\n+  it('should remove refs when destroying the child', function() {\n     var container = document.createElement('div');\n     var testInstance = React.render(<TestComponent \/>, container);\n     reactComponentExpect(testInstance.refs.theInnerDiv)\ndiff --git a\/src\/renderers\/shared\/reconciler\/__tests__\/refs-test.js b\/src\/renderers\/shared\/reconciler\/__tests__\/refs-test.js\nindex e6731f5096..9ebeb50736 100644\n--- a\/src\/renderers\/shared\/reconciler\/__tests__\/refs-test.js\n+++ b\/src\/renderers\/shared\/reconciler\/__tests__\/refs-test.js\n@@ -38,8 +38,8 @@ var ClickCounter = React.createClass({\n       children.push(\n         <div\n           className=\"clickLogDiv\"\n-          key={\"clickLog\" + i}\n-          ref={\"clickLog\" + i}\n+          key={'clickLog' + i}\n+          ref={'clickLog' + i}\n         \/>\n       );\n     }\n@@ -122,7 +122,7 @@ describe('reactiverefs', function() {\n    * Ensure that for every click log there is a corresponding ref (from the\n    * perspective of the injected ClickCounter component.\n    *\/\n-  it(\"Should increase refs with an increase in divs\", function() {\n+  it('Should increase refs with an increase in divs', function() {\n     var testRefsComponent = renderTestRefsComponent();\n     var clickIncrementer =\n       ReactTestUtils.findRenderedDOMComponentWithClass(\n@@ -195,7 +195,7 @@ describe('ref swapping', function() {\n     }\n   });\n \n-  it(\"Allow refs to hop around children correctly\", function() {\n+  it('Allow refs to hop around children correctly', function() {\n     var refHopsAround = ReactTestUtils.renderIntoDocument(<RefHopsAround \/>);\n \n     var firstDiv =\ndiff --git a\/src\/shared\/utils\/__tests__\/OrderedMap-test.js b\/src\/shared\/utils\/__tests__\/OrderedMap-test.js\nindex 914417606a..408c985aa7 100644\n--- a\/src\/shared\/utils\/__tests__\/OrderedMap-test.js\n+++ b\/src\/shared\/utils\/__tests__\/OrderedMap-test.js\n@@ -252,10 +252,8 @@ describe('OrderedMap', function() {\n       expect(om.has(0)).toBe(true);\n       expect(om.has('0')).toBe(true);\n     };\n-    var om = OrderedMap.fromArray(hasZeroStringKey, extractUniqueID);\n-    verifyOM(om);\n-    om = OrderedMap.fromArray(hasZeroNumberKey, extractUniqueID);\n-    verifyOM(om);\n+    verifyOM(OrderedMap.fromArray(hasZeroStringKey, extractUniqueID));\n+    verifyOM(OrderedMap.fromArray(hasZeroNumberKey, extractUniqueID));\n   });\n \n   it('should throw when getting invalid public key', function() {\n@@ -494,22 +492,22 @@ describe('OrderedMap', function() {\n   });\n \n   it('should compute indices on array that extracted numeric ids', function() {\n-    var om = OrderedMap.fromArray(hasZeroStringKey, extractUniqueID);\n-    expect(om.keyAtIndex(0)).toBe('greg');\n-    expect(om.keyAtIndex(1)).toBe('0');\n-    expect(om.keyAtIndex(2)).toBe('tom');\n-    expect(om.indexOfKey('greg')).toBe(0);\n-    expect(om.indexOfKey('0')).toBe(1);\n-    expect(om.indexOfKey('tom')).toBe(2);\n-\n-\n-    var verifyNumericKeys = function(om) {\n-      expect(om.keyAtIndex(0)).toBe('0');\n-      expect(om.keyAtIndex(1)).toBe('2');\n-      expect(om.keyAtIndex(2)).toBe('1');\n-      expect(om.indexOfKey('0')).toBe(0);\n-      expect(om.indexOfKey('2')).toBe(1); \/\/ Proove these are not ordered by\n-      expect(om.indexOfKey('1')).toBe(2); \/\/ their keys\n+    var som = OrderedMap.fromArray(hasZeroStringKey, extractUniqueID);\n+    expect(som.keyAtIndex(0)).toBe('greg');\n+    expect(som.keyAtIndex(1)).toBe('0');\n+    expect(som.keyAtIndex(2)).toBe('tom');\n+    expect(som.indexOfKey('greg')).toBe(0);\n+    expect(som.indexOfKey('0')).toBe(1);\n+    expect(som.indexOfKey('tom')).toBe(2);\n+\n+\n+    var verifyNumericKeys = function(nom) {\n+      expect(nom.keyAtIndex(0)).toBe('0');\n+      expect(nom.keyAtIndex(1)).toBe('2');\n+      expect(nom.keyAtIndex(2)).toBe('1');\n+      expect(nom.indexOfKey('0')).toBe(0);\n+      expect(nom.indexOfKey('2')).toBe(1); \/\/ Prove these are not ordered by\n+      expect(nom.indexOfKey('1')).toBe(2); \/\/ their keys\n     };\n     var omStringNumberKeys =\n       OrderedMap.fromArray(hasAllNumericStringKeys, extractUniqueID);\n@@ -731,7 +729,7 @@ describe('OrderedMap', function() {\n       om.forEachRange(duplicate, 0, 3, scope);\n     }).not.toThrow();\n     expect(function() {\n-      om.mapKeyRange(duplicate, 'x' , 3, scope);\n+      om.mapKeyRange(duplicate, 'x', 3, scope);\n     }).toThrow(\n       'Invariant Violation: mapKeyRange must be given keys ' +\n       'that are present.'\ndiff --git a\/src\/shared\/utils\/__tests__\/Transaction-test.js b\/src\/shared\/utils\/__tests__\/Transaction-test.js\nindex e9b1f0bc47..afece9ffc1 100644\n--- a\/src\/shared\/utils\/__tests__\/Transaction-test.js\n+++ b\/src\/shared\/utils\/__tests__\/Transaction-test.js\n@@ -56,7 +56,9 @@ describe('Transaction', function() {\n           }\n         },\n         {\n-          initialize: function() { return 'asdf'; },\n+          initialize: function() {\n+            return 'asdf';\n+          },\n           close: function(initResult) {\n             this.secondCloseParam = initResult;\n           }\n@@ -201,7 +203,7 @@ describe('Transaction', function() {\n       var isTypeError = false;\n       try {\n         transaction.perform(function() {\n-          throw new TypeError(\"Thrown in main wrapped operation\");\n+          throw new TypeError('Thrown in main wrapped operation');\n         });\n       } catch (err) {\n         isTypeError = (err instanceof TypeError);\n","commit":{"authorEmail":"bgw@fb.com","revision":"13823ec218580ad25fe801874e901e7b43114c29","subject":"Merge pull request #3985 from PiPeep\/eslint-tests","shortRevision":"13823ec218","authorName":"Benjamin Woodruff","authoredAt":"2015-06-01T16:05:42-07:00"}}</article></body></html>

dillon1000/react

Files changed