eslint auto-fixes

lib/rules/assertion-before-screenshot.js

@@ -17,7 +17,7 @@ const assertionCommands = [
   // not an assertion, but unlikely to require waiting for render
   'scrollIntoView',
   'scrollTo',
-];
+]
 
 module.exports = {
   meta: {
@@ -42,19 +42,22 @@ module.exports = {
   },
 }
 
-function isRootCypress(node) {
-  while(node.type === 'CallExpression') {
+function isRootCypress (node) {
+  while (node.type === 'CallExpression') {
     if (node.callee.type !== 'MemberExpression') return false
+
     if (node.callee.object.type === 'Identifier' &&
         node.callee.object.name === 'cy') {
       return true
     }
+
     node = node.callee.object
   }
+
   return false
 }
 
-function getPreviousInChain(node) {
+function getPreviousInChain (node) {
   return node.type === 'CallExpression' &&
          node.callee.type === 'MemberExpression' &&
          node.callee.object.type === 'CallExpression' &&
@@ -63,7 +66,7 @@ function getPreviousInChain(node) {
          node.callee.object.callee.property.name
 }
 
-function getCallExpressionCypressCommand(node) {
+function getCallExpressionCypressCommand (node) {
   return isRootCypress(node) &&
          node.callee.property.type === 'Identifier' &&
          node.callee.property.name
@@ -73,14 +76,14 @@ function isCallingCyScreenshot (node) {
   return getCallExpressionCypressCommand(node) === 'screenshot'
 }
 
-function getPreviousCypressCommand(node) {
+function getPreviousCypressCommand (node) {
   const previousInChain = getPreviousInChain(node)
 
   if (previousInChain) {
     return previousInChain
   }
 
-  while(node.parent && !node.parent.body) {
+  while (node.parent && !node.parent.body) {
     node = node.parent
   }
 
@@ -93,18 +96,20 @@ function getPreviousCypressCommand(node) {
   // in the case of a function declaration it won't be found
   if (index < 0) return null
 
-  if (index === 0) return getPreviousCypressCommand(node.parent);
+  if (index === 0) return getPreviousCypressCommand(node.parent)
 
   const previousStatement = body[index - 1]
 
   if (previousStatement.type !== 'ExpressionStatement' ||
-      previousStatement.expression.type !== 'CallExpression')
+      previousStatement.expression.type !== 'CallExpression') {
     return null
+  }
 
   return getCallExpressionCypressCommand(previousStatement.expression)
 }
 
 function isPreviousAnAssertion (node) {
   const previousCypressCommand = getPreviousCypressCommand(node)
+
   return assertionCommands.indexOf(previousCypressCommand) >= 0
 }

lib/rules/no-assigning-return-values.js

@@ -8,11 +8,15 @@
 // safely get nested object property
 function get (obj, propertyString = '') {
   const properties = propertyString.split('.')
+
   for (let i = 0; i < properties.length; i++) {
     const value = (obj || {})[properties[i]]
+
     if (value == null) return value
+
     obj = value
   }
+
   return obj
 }
 
@@ -57,6 +61,7 @@ function isCypressCommandDeclaration (declarator) {
 
     if (!object) return
   }
+
   const commandName = get(declarator, 'init.callee.property.name')
 
   if (commandName && whitelistedCommands[commandName]) return