Rename variables to be more semantic

facebook · Nov 9, 2017 · f5e404c · f5e404c
1 parent bb7ccc3
commit f5e404c
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/packages/react-dom/src/__tests__/ReactDOMComponentTree-test.js b/packages/react-dom/src/__tests__/ReactDOMComponentTree-test.js
@@ -79,18 +79,19 @@ describe('ReactDOMComponentTree', () => {
   });
 
   it('finds closest instance for node when an event happens', () => {
-    const elemID = 'aID';
-    const innerHTML = {__html: `<div id="${elemID}"></div>`};
-    const testID = 'closestInstance';
+    const nonReactElemID = 'aID';
+    const innerHTML = {__html: `<div id="${nonReactElemID}"></div>`};
+    const closestInstanceID = 'closestInstance';
     let currentTargetID = null;
+
     class ClosestInstance extends React.Component {
       _onClick = e => {
         currentTargetID = e.currentTarget.id;
       };
       render() {
         return (
           <div
-            id={testID}
+            id={closestInstanceID}
             onClick={this._onClick}
             dangerouslySetInnerHTML={innerHTML}
           />
@@ -103,8 +104,8 @@ describe('ReactDOMComponentTree', () => {
     ReactDOM.render(<section>{component}</section>, container);
     document.body.appendChild(container);
     expect(currentTargetID).toBe(null);
-    simulateClick(document.getElementById(elemID));
-    expect(currentTargetID).toBe(testID);
+    simulateClick(document.getElementById(nonReactElemID));
+    expect(currentTargetID).toBe(closestInstanceID);
   });
 
   it('finds a controlled instance from node and gets its current fiber props', () => {