Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Proposal: error type #61

Closed
Marsup opened this issue Apr 5, 2016 · 2 comments
Closed

Proposal: error type #61

Marsup opened this issue Apr 5, 2016 · 2 comments
Assignees
@Marsup
Labels
feature New functionality or improvement
Milestone
2.2.0

Comments

@Marsup
Copy link
Contributor

Marsup commented Apr 5, 2016

Hi,

I'd like to simplify some of my checks on async errors, so an API like that would help :
asyncTask((err, value) => expect(err).to.be.an.error('with that message'))

Thoughts ?
@Marsup Marsup added the request label Apr 5, 2016
@cjihrig
Copy link
Contributor

cjihrig commented Apr 5, 2016

I'd take a PR for it. I think it should match throws() as much as possible.

Marsup added a commit to Marsup/code that referenced this issue Apr 5, 2016
@Marsup
Add error type (fixes hapijs#61)
2b8ab77
@cjihrig cjihrig added this to the 2.2.0 milestone Apr 6, 2016
@cjihrig cjihrig closed this as completed in 35c0ac7 Apr 6, 2016
@cjihrig cjihrig modified the milestones: 2.2.0, 2.1.1 Apr 6, 2016
@Marsup Marsup added the feature New functionality or improvement label Sep 20, 2019
@lock
Copy link

lock bot commented Jan 9, 2020

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.

@lock lock bot locked as resolved and limited conversation to collaborators Jan 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@Marsup Marsup

Labels
feature New functionality or improvement
Projects
None yet
Milestone
2.2.0
Development

No branches or pull requests
2 participants
@Marsup @cjihrig