display more useful output on errors #187

[rofe]

Fixes #187

[trieloff]

Please check that the error output is only shown in non-production environments. We have a predicate for that already.

[trieloff]

See also adobe/helix-simulator#17

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (master@5dc0e80). Click here to learn what that means.
The diff coverage is 100%.

@@           Coverage Diff           @@
##             master   #188   +/-   ##
=======================================
  Coverage          ?   100%           
=======================================
  Files             ?     38           
  Lines             ?    837           
  Branches          ?    161           
=======================================
  Hits              ?    837           
  Misses            ?      0           
  Partials          ?      0

Impacted Files	Coverage Δ
src/html/set-status.js	100% <100%> (ø)
src/defaults/html.pipe.js	100% <100%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5dc0e80...7298f9e. Read the comment docs.

[rofe]

Please check that the error output is only shown in non-production environments

Like so?

[rofe]

Readded lost response object (thanks @tripodsan!)

[trieloff]

@rofe it's generally a better idea to pull the if statement out of the function and apply it in the pipeline like this:

.error(detailed-status).when(() => !production())
.error(terse-status).when(() => production())

that way, your detailed-status and terse-status functions can stay simple, and you easily get 100% branch coverage in testing because there is only one branch in each function.

[trieloff]

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀