Redesign and improve reporter error display

packages/driver/src/cy/chai.js

@@ -340,7 +340,7 @@ chai.use((chai, u) => {
                 return `Not enough elements found. Found '${len1}', expected '${len2}'.`
               }
 
-              e1.displayMessage = getLongLengthMessage(obj.length, length)
+              e1.message = getLongLengthMessage(obj.length, length)
               throw e1
             }
 
@@ -397,10 +397,10 @@ chai.use((chai, u) => {
                 return `Expected ${node} not to exist in the DOM, but it was continuously found.`
               }
 
-              return `Expected to find element: '${obj.selector}', but never found it.`
+              return `Expected to find element: \`${obj.selector}\`, but never found it.`
             }
 
-            e1.displayMessage = getLongExistsMessage(obj)
+            e1.message = getLongExistsMessage(obj)
             throw e1
           }
         }

packages/driver/src/cy/commands/actions/check.coffee

@@ -58,7 +58,7 @@ checkOrUncheck = (type, subject, values = [], options = {}) ->
     if not isAcceptableElement($el)
       node   = $dom.stringify($el)
       word   = $utils.plural(options.$el, "contains", "is")
-      phrase = if type is "check" then " and :radio" else ""
+      phrase = if type is "check" then " and `:radio`" else ""
       $errUtils.throwErrByPath("check_uncheck.invalid_element", {
         onFail: options._log
         args: { node, word, phrase, cmd: type }

packages/driver/src/cy/commands/actions/scroll.coffee

@@ -32,7 +32,7 @@ module.exports = (Commands, Cypress, cy, state, config) ->
       ## ensure the subject is not window itself
       ## cause how are you gonna scroll the window into view...
       if subject is state("window")
-        $utils.throwErrByPath("scrollIntoView.subject_is_window")
+        $errUtils.throwErrByPath("scrollIntoView.subject_is_window")
 
       ## throw if we're trying to scroll to multiple elements
       if subject.length > 1

packages/driver/src/cy/commands/angular.coffee

@@ -35,7 +35,7 @@ module.exports = (Commands, Cypress, cy, state, config) ->
         cy.verifyUpcomingAssertions(getEl($elements), options, {
           onRetry: resolveElements
           onFail: (err) ->
-            err.displayMessage = "Could not find element for binding: '#{binding}'."
+            err.message = "Could not find element for binding: '#{binding}'."
         })
 
   findByNgAttr = (name, attr, el, options) ->

packages/driver/src/cy/commands/commands.coffee

@@ -5,12 +5,12 @@ $errUtils = require("../../cypress/error_utils")
 
 command = (ctx, name, args...) ->
   if not ctx[name]
-    cmds = _.keys($Chainer.prototype).join(", ")
+    cmds = "\`#{_.keys($Chainer.prototype).join("`, `")}\`"
     $errUtils.throwErrByPath("miscellaneous.invalid_command", {
       args: { name, cmds }
     })
 
-  ctx[name].apply(window, args)
+  ctx[name].apply(ctx, args)
 
 module.exports = (Commands, Cypress, cy, state, config) ->
   Commands.addChainer({

packages/driver/src/cy/commands/connectors.coffee

@@ -4,6 +4,7 @@ Promise = require("bluebird")
 $dom = require("../../dom")
 $utils = require("../../cypress/utils")
 $errUtils = require("../../cypress/error_utils")
+$errMessages = require("../../cypress/error_messages")
 
 returnFalseIfThenable = (key, args...) ->
   if key is "then" and _.isFunction(args[0]) and _.isFunction(args[1])
@@ -211,11 +212,9 @@ module.exports = (Commands, Cypress, cy, state, config) ->
         args: { cmd: name }
       })
 
-    ## TODO: use the new error utils that are part of
-    ## the error message enhancements PR
     propertyNotOnSubjectErr = (prop) ->
-      $errUtils.cypressErr(
-        $errUtils.errMsgByPath("invoke_its.nonexistent_prop", {
+      $errUtils.cypressErrObj(
+        $errUtils.errObjByPath($errMessages, "invoke_its.nonexistent_prop", {
           prop,
           cmd: name
         })
@@ -224,8 +223,8 @@ module.exports = (Commands, Cypress, cy, state, config) ->
     propertyValueNullOrUndefinedErr = (prop, value) ->
       errMessagePath = if isCmdIts then "its" else "invoke"
 
-      $errUtils.cypressErr(
-        $errUtils.errMsgByPath("#{errMessagePath}.null_or_undefined_prop_value", {
+      $errUtils.cypressErrObj(
+        $errUtils.errObjByPath($errMessages, "#{errMessagePath}.null_or_undefined_prop_value", {
           prop,
           value,
           cmd: name
@@ -235,17 +234,17 @@ module.exports = (Commands, Cypress, cy, state, config) ->
     subjectNullOrUndefinedErr = (prop, value) ->
       errMessagePath = if isCmdIts then "its" else "invoke"
 
-      $errUtils.cypressErr(
-        $errUtils.errMsgByPath("#{errMessagePath}.subject_null_or_undefined", {
+      $errUtils.cypressErrObj(
+        $errUtils.errObjByPath($errMessages, "#{errMessagePath}.subject_null_or_undefined", {
           prop,
           value,
           cmd: name
         })
       )
 
     propertyNotOnPreviousNullOrUndefinedValueErr = (prop, value, previousProp) ->
-      $errUtils.cypressErr(
-        $errUtils.errMsgByPath("invoke_its.previous_prop_null_or_undefined", {
+      $errUtils.cypressErrObj(
+        $errUtils.errObjByPath($errMessages, "invoke_its.previous_prop_null_or_undefined", {
           prop,
           value,
           previousProp,
@@ -359,6 +358,9 @@ module.exports = (Commands, Cypress, cy, state, config) ->
               Yielded: getFormattedElement(value)
             })
 
+            if traversalErr
+              obj.Error = "#{traversalErr.name}: #{traversalErr.message}"
+
             return obj
         })
 

packages/driver/src/cy/commands/cookies.js

@@ -92,7 +92,7 @@ module.exports = function (Commands, Cypress, cy, state, config) {
       $errUtils.throwErrByPath('cookies.backend_error', {
         args: {
           action,
-          command,
+          cmd: command,
           browserDisplayName: Cypress.browser.displayName,
           errMessage: err.message,
           errStack: err.stack,

packages/driver/src/cy/commands/files.coffee

@@ -2,6 +2,7 @@ _ = require("lodash")
 Promise = require("bluebird")
 
 $errUtils = require("../../cypress/error_utils")
+$errMessages = require("../../cypress/error_messages")
 
 module.exports = (Commands, Cypress, cy, state, config) ->
   Commands.addAll({
@@ -50,16 +51,20 @@ module.exports = (Commands, Cypress, cy, state, config) ->
             onFail: (err) ->
               return unless err.type is "existence"
 
-              if contents?
+              { message, docsUrl } = if contents?
                 ## file exists but it shouldn't
-                err.displayMessage = $errUtils.errMsgByPath("files.existent", {
+                $errUtils.errObjByPath($errMessages, "files.existent", {
                   file, filePath
                 })
               else
                 ## file doesn't exist but it should
-                err.displayMessage = $errUtils.errMsgByPath("files.nonexistent", {
+                $errUtils.errObjByPath($errMessages, "files.nonexistent", {
                   file, filePath
                 })
+
+              err.message = message
+              err.docsUrl = docsUrl
+
             onRetry: verifyAssertions
           })
 

packages/driver/src/cy/commands/querying.js

@@ -68,7 +68,7 @@ module.exports = (Commands, Cypress, cy) => {
     get (selector, options = {}) {
       const ctx = this
 
-      if ((options === null) || Array.isArray(options) || (typeof options !== 'object')) {
+      if ((options === null) || _.isArray(options) || !_.isPlainObject(options)) {
         return $errUtils.throwErrByPath('get.invalid_options', {
           args: { options },
         })
@@ -497,7 +497,7 @@ module.exports = (Commands, Cypress, cy) => {
 
                   break
                 case 'existence':
-                  return err.displayMessage = getErr(err)
+                  return err.message = getErr(err)
                 default:
                   break
               }

packages/driver/src/cy/commands/screenshot.coffee

@@ -352,7 +352,7 @@ module.exports = (Commands, Cypress, cy, state, config) ->
       isWin = $dom.isWindow(subject)
 
       screenshotConfig = _.pick(options, "capture", "scale", "disableTimersAndAnimations", "blackout", "waitForCommandSynchronization", "padding", "clip", "onBeforeScreenshot", "onAfterScreenshot")
-      screenshotConfig = $Screenshot.validate(screenshotConfig, "cy.screenshot", options._log)
+      screenshotConfig = $Screenshot.validate(screenshotConfig, "screenshot", options._log)
       screenshotConfig = _.extend($Screenshot.getConfig(), screenshotConfig)
 
       ## set this regardless of options.log b/c its used by the

packages/driver/src/cy/commands/traversals.coffee

@@ -59,5 +59,5 @@ module.exports = (Commands, Cypress, cy, state, config) ->
           onFail: (err) ->
             if err.type is "existence"
               node = $dom.stringify(subject, "short")
-              err.displayMessage += " Queried from element: #{node}"
-        })
+              err.message += " Queried from element: #{node}"
+            })

packages/driver/src/cy/commands/window.coffee

@@ -178,7 +178,7 @@ module.exports = (Commands, Cypress, cy, state, config) ->
 
           orientationIsValidAndLandscape = (orientation) =>
             if orientation not in validOrientations
-              all = validOrientations.join("' or '")
+              all = validOrientations.join("` or `")
               $errUtils.throwErrByPath "viewport.invalid_orientation", {
                 onFail: options._log
                 args: { all, orientation }

packages/driver/src/cy/ensures.coffee

@@ -71,7 +71,9 @@ create = (state, expect) ->
 
       if types.length > 1
         ## append a nice error message telling the user this
-        err = $errUtils.appendErrMsg(err, "All #{types.length} subject validations failed on this subject.")
+        errProps = $errUtils.appendErrMsg(err, "All #{types.length} subject validations failed on this subject.")
+
+        $errUtils.mergeErrProps(err, errProps)
 
       throw err
 

packages/driver/src/cy/errors.coffee

@@ -1,5 +1,7 @@
+_ = require("lodash")
 $dom = require("../dom")
 $errUtils = require("../cypress/error_utils")
+$errorMessages = require('../cypress/error_messages')
 
 crossOriginScriptRe = /^script error/i
 
@@ -37,28 +39,37 @@ create = (state, config, log) ->
       msg = $errUtils.errMsgByPath("uncaught.cross_origin_script")
 
     createErrFromMsg = ->
-      new Error $errUtils.errMsgByPath("uncaught.error", { msg, source, lineno })
+      new Error($errUtils.errMsgByPath("uncaught.error", {
+        msg, source, lineno
+      }))
 
     ## if we have the 5th argument it means we're in a super
     ## modern browser making this super simple to work with.
     err ?= createErrFromMsg()
 
-    err.name = "Uncaught " + err.name
-
-    suffixMsg = switch type
+    uncaughtErrLookup = switch type
       when "app" then "uncaught.fromApp"
       when "spec" then "uncaught.fromSpec"
 
-    err = $errUtils.appendErrMsg(err, $errUtils.errMsgByPath(suffixMsg))
+    uncaughtErrObj = $errUtils.errObjByPath($errorMessages, uncaughtErrLookup)
+
+    err.name = "Uncaught " + err.name
+
+    uncaughtErrProps = $errUtils.modifyErrMsg(err, uncaughtErrObj.message, (msg1, msg2) ->
+      return "#{msg1}\n\n#{msg2}"
+    )
+    _.defaults(uncaughtErrProps, uncaughtErrObj)
+
+    uncaughtErr = $errUtils.mergeErrProps(err, uncaughtErrProps)
 
-    err.onFail = ->
+    uncaughtErr.onFail = ->
       if l = current and current.getLastLog()
-        l.error(err)
+        l.error(uncaughtErr)
 
     ## normalize error message for firefox
-    $errUtils.normalizeErrorStack(err)
+    $errUtils.normalizeErrorStack(uncaughtErr)
 
-    return err
+    return uncaughtErr
 
   commandRunningFailed = (err) ->
     ## allow for our own custom onFail function

packages/driver/src/cy/retries.coffee

@@ -1,6 +1,7 @@
 _ = require("lodash")
 Promise = require("bluebird")
 debug = require('debug')('cypress:driver:retries')
+
 $utils = require("../cypress/utils")
 $errUtils = require("../cypress/error_utils")
 
@@ -58,15 +59,19 @@ create = (Cypress, state, timeout, clearTimeout, whenStable, finishAssertions) -
         if assertions = options.assertions
           finishAssertions(assertions)
 
-        getErrMessage = (err) ->
-          _.get(err, 'displayMessage') or
-            _.get(err, 'message') or
-              err
+        { error, onFail } = options
+
+        prependMsg = $errUtils.errMsgByPath("miscellaneous.retry_timed_out")
+
+        retryErrProps = $errUtils.modifyErrMsg(error, prependMsg, (msg1, msg2) ->
+          return "#{msg2}#{msg1}"
+        )
+
+        retryErr = $errUtils.mergeErrProps(error, retryErrProps)
 
-        $errUtils.throwErrByPath "miscellaneous.retry_timed_out", {
-          onFail: (options.onFail or log)
-          args: { error: getErrMessage(options.error) }
-        }
+        $errUtils.throwErr(retryErr, {
+          onFail: onFail or log
+        })
 
       runnableHasChanged = ->
         ## if we've changed runnables don't retry!

packages/driver/src/cypress.js

@@ -53,7 +53,7 @@ $Log.command = () => {
   return $errUtils.throwErrByPath('miscellaneous.command_log_renamed')
 }
 
-const throwDeprecatedCommandInterface = (key, method) => {
+const throwDeprecatedCommandInterface = (key = 'commandName', method) => {
   let signature = ''
 
   switch (method) {
@@ -502,7 +502,7 @@ class $Cypress {
           // attaching long stace traces
           // which otherwise make this err
           // unusably long
-          const err = $errUtils.cloneErr(e)
+          const err = $errUtils.makeErrFromObj(e)
 
           err.__stackCleaned__ = true
           err.backend = true
@@ -524,7 +524,7 @@ class $Cypress {
         const e = reply.error
 
         if (e) {
-          const err = $errUtils.cloneErr(e)
+          const err = $errUtils.makeErrFromObj(e)
 
           err.automation = true
 

packages/driver/src/cypress/cy.js

@@ -105,9 +105,9 @@ const create = function (specWindow, Cypress, Cookies, state, config, log) {
   const warnMixingPromisesAndCommands = function () {
     const title = state('runnable').fullTitle()
 
-    const msg = $errUtils.errMsgByPath('miscellaneous.mixing_promises_and_commands', title)
-
-    return $utils.warning(msg)
+    $errUtils.warnByPath('miscellaneous.mixing_promises_and_commands', {
+      args: { title },
+    })
   }
 
   const $$ = function (selector, context) {
@@ -678,7 +678,9 @@ const create = function (specWindow, Cypress, Cookies, state, config, log) {
 
     stopped = true
 
-    $errUtils.normalizeErrorStack(err)
+    err = $errUtils.normalizeErrorStack(err)
+
+    err = $errUtils.processErr(err, config)
 
     // store the error on state now
     state('error', err)
@@ -723,7 +725,7 @@ const create = function (specWindow, Cypress, Cookies, state, config, log) {
     }
 
     // bail if we had callbacks attached
-    if (rets.length) {
+    if (rets && rets.length) {
       return
     }
 
@@ -1269,7 +1271,7 @@ const create = function (specWindow, Cypress, Cookies, state, config, log) {
               $utils.stringify(ret)
 
             $errUtils.throwErrByPath('miscellaneous.returned_value_and_commands', {
-              args: ret,
+              args: { returned: ret },
             })
           }