Updates code to new prettier + eslint config.

Author: ctrlplusb

.eslintrc

@@ -1,33 +1,24 @@
 {
   "name": "react-tree-walker",
   "version": "2.1.2",
-  "description": "Walk a React element tree, executing a provided function against each node.",
+  "description":
+    "Walk a React element tree, executing a provided function against each node.",
   "license": "MIT",
   "main": "commonjs/index.js",
-  "files": [
-    "*.js",
-    "*.md",
-    "umd",
-    "commonjs"
-  ],
+  "files": ["*.js", "*.md", "umd", "commonjs"],
   "repository": {
     "type": "git",
     "url": "https://github.com/ctrlplusb/react-tree-walker.git"
   },
   "homepage": "https://github.com/ctrlplusb/react-tree-walker#readme",
   "author": "Sean Matheson <[email protected]>",
-  "keywords": [
-    "react",
-    "react-element",
-    "util",
-    "tree",
-    "visitor"
-  ],
+  "keywords": ["react", "react-element", "util", "tree", "visitor"],
   "scripts": {
     "precommit": "lint-staged && npm run test",
     "build": "babel-node ./tools/scripts/build.js",
     "check": "npm run lint && npm run test",
-    "clean": "rimraf ./commonjs && rimraf ./umd && rimraf ./coverage && rimraf ./umd",
+    "clean":
+      "rimraf ./commonjs && rimraf ./umd && rimraf ./coverage && rimraf ./umd",
     "lint": "eslint src",
     "prepublish": "npm run build",
     "test": "jest",
@@ -79,37 +70,28 @@
     "webpack-hot-middleware": "^2.19.1"
   },
   "jest": {
-    "collectCoverageFrom": [
-      "src/**/*.{js,jsx}"
-    ],
-    "snapshotSerializers": [
-      "<rootDir>/node_modules/enzyme-to-json/serializer"
-    ],
+    "collectCoverageFrom": ["src/**/*.{js,jsx}"],
+    "snapshotSerializers": ["<rootDir>/node_modules/enzyme-to-json/serializer"],
     "testPathIgnorePatterns": [
       "<rootDir>/(commonjs|coverage|flow-typed|node_modules|tools|umd)/"
     ]
   },
   "lint-staged": {
-    "src/**/*.js": [
-      "prettier --write",
-      "git add"
-    ]
+    "src/**/*.js": ["prettier --write", "git add"]
   },
   "eslintConfig": {
     "root": true,
     "parser": "babel-eslint",
     "env": {
       "browser": true,
       "es6": true,
-      "node": true
+      "node": true,
+      "jest": true
     },
     "extends": "airbnb",
     "rules": {
       "array-callback-return": 0,
-      "arrow-parens": [
-        "error",
-        "as-needed"
-      ],
+      "arrow-parens": ["error", "as-needed"],
       "camelcase": 0,
       "import/prefer-default-export": 0,
       "import/no-extraneous-dependencies": 0,
@@ -120,10 +102,7 @@
       "no-nested-ternary": 0,
       "react/no-array-index-key": 0,
       "react/react-in-jsx-scope": 0,
-      "semi": [
-        2,
-        "never"
-      ],
+      "semi": [2, "never"],
       "react/forbid-prop-types": 0,
       "react/jsx-filename-extension": 0,
       "react/sort-comp": 0

package.json

@@ -36,8 +36,8 @@ describe('reactTreeWalker', () => {
       }, 10),
     )
 
-  const createTree = async =>
-    (<div>
+  const createTree = async => (
+    <div>
       <h1>Hello World!</h1>
       <Foo something={async ? () => resolveLater(1) : 1} />
       <Foo something={async ? () => resolveLater(2) : 2}>
@@ -52,7 +52,8 @@ describe('reactTreeWalker', () => {
         </div>
       </Foo>
       <Foo something={async ? () => resolveLater(3) : 3} />
-    </div>)
+    </div>
+  )
 
   it('simple sync visitor', () => {
     const tree = createTree(false)
@@ -76,7 +77,7 @@ describe('reactTreeWalker', () => {
     // eslint-disable-next-line no-unused-vars
     const visitor = (element, instance, context) => {
       if (instance && typeof instance.getSomething === 'function') {
-        return instance.getSomething().then((something) => {
+        return instance.getSomething().then(something => {
           actual.push(something)
           return true
         })
@@ -130,7 +131,9 @@ describe('reactTreeWalker', () => {
       }
     }
 
-    return reactTreeWalker(<Baz />, () => true, null, { componentWillUnmount: true }).then(() => {
+    return reactTreeWalker(<Baz />, () => true, null, {
+      componentWillUnmount: true,
+    }).then(() => {
       expect(called).toBeTruthy()
     })
   })
@@ -153,7 +156,11 @@ describe('reactTreeWalker', () => {
     }
     Qux.contextTypes = { foo: PropTypes.string.isRequired }
 
-    const tree = <Baz><Qux /></Baz>
+    const tree = (
+      <Baz>
+        <Qux />
+      </Baz>
+    )
     return reactTreeWalker(tree, () => true).then(() => {
       const expected = { foo: 'bar' }
       expect(actual).toMatchObject(expected)

src/__tests__/index.test.js

@@ -18,7 +18,7 @@ const pReduce = (iterable, reducer, initVal) =>
     const iterator = iterable[Symbol.iterator]()
     let i = 0
 
-    const next = (total) => {
+    const next = total => {
       const el = iterator.next()
 
       if (el.done) {
@@ -27,7 +27,7 @@ const pReduce = (iterable, reducer, initVal) =>
       }
 
       Promise.all([total, el.value])
-        .then((value) => {
+        .then(value => {
           // eslint-disable-next-line no-plusplus
           next(reducer(value[0], value[1], i++))
         })
@@ -43,7 +43,7 @@ const pMapSeries = (iterable, iterator) => {
   const ret = []
 
   return pReduce(iterable, (a, b, i) =>
-    Promise.resolve(iterator(b, i)).then((val) => {
+    Promise.resolve(iterator(b, i)).then(val => {
       ret.push(val)
     }),
   ).then(() => ret)
@@ -54,17 +54,23 @@ export const isPromise = x => x != null && typeof x.then === 'function'
 // Recurse an React Element tree, running visitor on each element.
 // If visitor returns `false`, don't call the element's render function
 // or recurse into its child elements
-export default function reactTreeWalker(element, visitor, context, options = defaultOptions) {
-  return new Promise((resolve) => {
+export default function reactTreeWalker(
+  element,
+  visitor,
+  context,
+  options = defaultOptions,
+) {
+  return new Promise(resolve => {
     const doVisit = (getChildren, visitorResult, childContext) => {
-      const doTraverse = (shouldContinue) => {
+      const doTraverse = shouldContinue => {
         if (!shouldContinue) {
           // We recieved a false, which indicates a desire to stop traversal.
           resolve()
         }
 
         const child = getChildren()
-        const theChildContext = typeof childContext === 'function' ? childContext() : childContext
+        const theChildContext =
+          typeof childContext === 'function' ? childContext() : childContext
 
         if (child == null) {
           // If no children then we can't traverse.  We've reached the leaf.
@@ -73,12 +79,16 @@ export default function reactTreeWalker(element, visitor, context, options = def
           // If its a react Children collection we need to breadth-first
           // traverse each of them.
           const mapper = aChild =>
-            aChild ? reactTreeWalker(aChild, visitor, theChildContext, options) : undefined
+            aChild
+              ? reactTreeWalker(aChild, visitor, theChildContext, options)
+              : undefined
           // pMapSeries allows us to do depth-first traversal. Thanks @sindresorhus!
           pMapSeries(Children.map(child, cur => cur), mapper).then(resolve)
         } else {
           // Otherwise we pass the individual child to the next recursion.
-          reactTreeWalker(child, visitor, theChildContext, options).then(resolve)
+          reactTreeWalker(child, visitor, theChildContext, options).then(
+            resolve,
+          )
         }
       }
 
@@ -88,7 +98,7 @@ export default function reactTreeWalker(element, visitor, context, options = def
       } else if (isPromise(visitorResult)) {
         // We need to execute the result and pass it's result through to our
         // continuer.
-        visitorResult.then(doTraverse).catch((e) => {
+        visitorResult.then(doTraverse).catch(e => {
           console.log(
             'Error occurred in Promise based visitor result provided to react-tree-walker.',
           )
@@ -112,7 +122,8 @@ export default function reactTreeWalker(element, visitor, context, options = def
       // Is this a class component? (http://bit.ly/2j9Ifk3)
       const isReactClassComponent =
         Component.prototype &&
-        (Component.prototype.isReactComponent || Component.prototype.isPureReactComponent)
+        (Component.prototype.isReactComponent ||
+          Component.prototype.isPureReactComponent)
 
       if (isReactClassComponent) {
         // React class component
@@ -124,7 +135,7 @@ export default function reactTreeWalker(element, visitor, context, options = def
         instance.context = instance.context || context
 
         // Make the setState synchronous.
-        instance.setState = (newState) => {
+        instance.setState = newState => {
           instance.state = Object.assign({}, instance.state, newState)
         }
 
@@ -144,7 +155,9 @@ export default function reactTreeWalker(element, visitor, context, options = def
                 // This is an experimental feature, we don't want to break
                 // the bootstrapping process, but lets warn the user it
                 // occurred.
-                console.warn('Error calling componentWillUnmount whilst walking your react tree')
+                console.warn(
+                  'Error calling componentWillUnmount whilst walking your react tree',
+                )
                 console.warn(err)
               }
             }
@@ -161,17 +174,24 @@ export default function reactTreeWalker(element, visitor, context, options = def
         )
       } else {
         // Stateless Functional Component
-        doVisit(() => Component(props, context), visitor(element, null, context), context)
+        doVisit(
+          () => Component(props, context),
+          visitor(element, null, context),
+          context,
+        )
       }
     } else {
       // This must be a basic element, such as a string or dom node.
       doVisit(
-        () => (element.props && element.props.children ? element.props.children : undefined),
+        () =>
+          element.props && element.props.children
+            ? element.props.children
+            : undefined,
         visitor(element, null, context),
         context,
       )
     }
-  }).catch((err) => {
+  }).catch(err => {
     // We don't want errors to be swallowed!
     console.error('Error walking your react tree')
     console.error(err)