{"patch":"diff --git a\/packages\/react-dom\/src\/__tests__\/CSSPropertyOperations-test.js b\/packages\/react-dom\/src\/__tests__\/CSSPropertyOperations-test.js\nindex 10190db51e..d1f97800dd 100644\n--- a\/packages\/react-dom\/src\/__tests__\/CSSPropertyOperations-test.js\n+++ b\/packages\/react-dom\/src\/__tests__\/CSSPropertyOperations-test.js\n@@ -9,9 +9,9 @@\n \n 'use strict';\n \n-var React = require('react');\n-var ReactDOM = require('react-dom');\n-var ReactDOMServer = require('react-dom\/server');\n+const React = require('react');\n+const ReactDOM = require('react-dom');\n+const ReactDOMServer = require('react-dom\/server');\n \n function normalizeCodeLocInfo(str) {\n return str && str.replace(\/at .+?:\\d+\/g, 'at **');\n@@ -19,66 +19,66 @@ function normalizeCodeLocInfo(str) {\n \n describe('CSSPropertyOperations', () => {\n it('should automatically append `px` to relevant styles', () => {\n- var styles = {\n+ const styles = {\n left: 0,\n margin: 16,\n opacity: 0.5,\n padding: '4px',\n };\n- var div =

;\n- var html = ReactDOMServer.renderToString(div);\n+ const div =

;\n+ const html = ReactDOMServer.renderToString(div);\n expect(html).toContain('\"left:0;margin:16px;opacity:0.5;padding:4px\"');\n });\n \n it('should trim values', () => {\n- var styles = {\n+ const styles = {\n left: '16 ',\n opacity: 0.5,\n right: ' 4 ',\n };\n- var div =

;\n- var html = ReactDOMServer.renderToString(div);\n+ const div =

;\n+ const html = ReactDOMServer.renderToString(div);\n expect(html).toContain('\"left:16;opacity:0.5;right:4\"');\n });\n \n it('should not append `px` to styles that might need a number', () => {\n- var styles = {\n+ const styles = {\n flex: 0,\n opacity: 0.5,\n };\n- var div =

;\n- var html = ReactDOMServer.renderToString(div);\n+ const div =

;\n+ const html = ReactDOMServer.renderToString(div);\n expect(html).toContain('\"flex:0;opacity:0.5\"');\n });\n \n it('should create vendor-prefixed markup correctly', () => {\n- var styles = {\n+ const styles = {\n msTransition: 'none',\n MozTransition: 'none',\n };\n- var div =

;\n- var html = ReactDOMServer.renderToString(div);\n+ const div =

;\n+ const html = ReactDOMServer.renderToString(div);\n expect(html).toContain('\"-ms-transition:none;-moz-transition:none\"');\n });\n \n it('should set style attribute when styles exist', () => {\n- var styles = {\n+ const styles = {\n backgroundColor: '#000',\n display: 'none',\n };\n- var div =

;\n- var root = document.createElement('div');\n+ let div =

;\n+ const root = document.createElement('div');\n div = ReactDOM.render(div, root);\n expect(\/style=\".*\"\/.test(root.innerHTML)).toBe(true);\n });\n \n it('should not set style attribute when no styles exist', () => {\n- var styles = {\n+ const styles = {\n backgroundColor: null,\n display: null,\n };\n- var div =

;\n- var html = ReactDOMServer.renderToString(div);\n+ const div =

;\n+ const html = ReactDOMServer.renderToString(div);\n expect(\/style=\/.test(html)).toBe(false);\n });\n \n@@ -92,7 +92,7 @@ describe('CSSPropertyOperations', () => {\n }\n \n spyOnDev(console, 'error');\n- var root = document.createElement('div');\n+ const root = document.createElement('div');\n ReactDOM.render(, root);\n if (__DEV__) {\n expect(console.error.calls.count()).toBe(1);\n@@ -114,11 +114,11 @@ describe('CSSPropertyOperations', () => {\n }\n \n spyOnDev(console, 'error');\n- var styles = {\n+ const styles = {\n '-ms-transform': 'translate3d(0, 0, 0)',\n '-webkit-transform': 'translate3d(0, 0, 0)',\n };\n- var root = document.createElement('div');\n+ const root = document.createElement('div');\n ReactDOM.render(, root);\n ReactDOM.render(, root);\n \n@@ -155,7 +155,7 @@ describe('CSSPropertyOperations', () => {\n }\n \n spyOnDev(console, 'error');\n- var root = document.createElement('div');\n+ const root = document.createElement('div');\n ReactDOM.render(, root);\n if (__DEV__) {\n \/\/ msTransform is correct already and shouldn't warn\n@@ -194,7 +194,7 @@ describe('CSSPropertyOperations', () => {\n }\n \n spyOnDev(console, 'error');\n- var root = document.createElement('div');\n+ const root = document.createElement('div');\n ReactDOM.render(, root);\n if (__DEV__) {\n expect(console.error.calls.count()).toBe(2);\n@@ -223,7 +223,7 @@ describe('CSSPropertyOperations', () => {\n }\n \n spyOnDev(console, 'error');\n- var root = document.createElement('div');\n+ const root = document.createElement('div');\n ReactDOM.render(, root);\n \n if (__DEV__) {\n@@ -243,7 +243,7 @@ describe('CSSPropertyOperations', () => {\n }\n }\n \n- var root = document.createElement('div');\n+ const root = document.createElement('div');\n ReactDOM.render(, root);\n });\n \n@@ -257,7 +257,7 @@ describe('CSSPropertyOperations', () => {\n }\n \n spyOnDev(console, 'error');\n- var root = document.createElement('div');\n+ const root = document.createElement('div');\n ReactDOM.render(, root);\n \n if (__DEV__) {\n@@ -277,7 +277,7 @@ describe('CSSPropertyOperations', () => {\n }\n }\n \n- var root = document.createElement('div');\n+ const root = document.createElement('div');\n ReactDOM.render(, root);\n \n expect(root.children[0].style.Foo).toEqual('5');\ndiff --git a\/packages\/react-dom\/src\/__tests__\/DOMPropertyOperations-test.js b\/packages\/react-dom\/src\/__tests__\/DOMPropertyOperations-test.js\nindex 7c53588314..8cdf53f1c4 100644\n--- a\/packages\/react-dom\/src\/__tests__\/DOMPropertyOperations-test.js\n+++ b\/packages\/react-dom\/src\/__tests__\/DOMPropertyOperations-test.js\n@@ -10,8 +10,8 @@\n 'use strict';\n \n describe('DOMPropertyOperations', () => {\n- var React;\n- var ReactDOM;\n+ let React;\n+ let ReactDOM;\n \n beforeEach(() => {\n jest.resetModules();\n@@ -21,20 +21,20 @@ describe('DOMPropertyOperations', () => {\n \n describe('setValueForProperty', () => {\n it('should set values as properties by default', () => {\n- var container = document.createElement('div');\n+ const container = document.createElement('div');\n ReactDOM.render(

, container);\n expect(container.firstChild.title).toBe('Tip!');\n });\n \n it('should set values as attributes if necessary', () => {\n- var container = document.createElement('div');\n+ const container = document.createElement('div');\n ReactDOM.render(

, container);\n expect(container.firstChild.getAttribute('role')).toBe('#');\n expect(container.firstChild.role).toBeUndefined();\n });\n \n it('should set values as namespace attributes if necessary', () => {\n- var container = document.createElement('svg');\n+ const container = document.createElement('svg');\n ReactDOM.render(, container);\n expect(\n container.firstChild.getAttributeNS(\n@@ -45,7 +45,7 @@ describe('DOMPropertyOperations', () => {\n });\n \n it('should set values as boolean properties', () => {\n- var container = document.createElement('div');\n+ const container = document.createElement('div');\n ReactDOM.render(

, container);\n expect(container.firstChild.getAttribute('disabled')).toBe('');\n ReactDOM.render(

, container);\n@@ -63,32 +63,32 @@ describe('DOMPropertyOperations', () => {\n it('should convert attribute values to string first', () => {\n \/\/ Browsers default to this behavior, but some test environments do not.\n \/\/ This ensures that we have consistent behavior.\n- var obj = {\n+ const obj = {\n toString: function() {\n return 'css-class';\n },\n };\n \n- var container = document.createElement('div');\n+ const container = document.createElement('div');\n ReactDOM.render(

, container);\n expect(container.firstChild.getAttribute('class')).toBe('css-class');\n });\n \n it('should not remove empty attributes for special properties', () => {\n- var container = document.createElement('div');\n+ const container = document.createElement('div');\n ReactDOM.render(, container);\n expect(container.firstChild.getAttribute('value')).toBe('');\n expect(container.firstChild.value).toBe('');\n });\n \n it('should remove for falsey boolean properties', () => {\n- var container = document.createElement('div');\n+ const container = document.createElement('div');\n ReactDOM.render(

, container);\n expect(container.firstChild.hasAttribute('allowFullScreen')).toBe(false);\n });\n \n it('should remove when setting custom attr to null', () => {\n- var container = document.createElement('div');\n+ const container = document.createElement('div');\n ReactDOM.render(

, container);\n expect(container.firstChild.hasAttribute('data-foo')).toBe(true);\n ReactDOM.render(

, container);\n@@ -96,7 +96,7 @@ describe('DOMPropertyOperations', () => {\n });\n \n it('should set className to empty string instead of null', () => {\n- var container = document.createElement('div');\n+ const container = document.createElement('div');\n ReactDOM.render(

, container);\n expect(container.firstChild.className).toBe('selected');\n ReactDOM.render(

, container);\n@@ -107,7 +107,7 @@ describe('DOMPropertyOperations', () => {\n });\n \n it('should remove property properly for boolean properties', () => {\n- var container = document.createElement('div');\n+ const container = document.createElement('div');\n ReactDOM.render(

dillon1000/react

Files changed