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.

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

Errors in Test syntax not reported to User #177

Closed
ThisIsMissEm opened this issue Aug 28, 2014 · 2 comments
Closed

Errors in Test syntax not reported to User #177

ThisIsMissEm opened this issue Aug 28, 2014 · 2 comments
Assignees
Labels
non issue Issue is not a problem or requires changes

Comments

@ThisIsMissEm
Copy link

Okay, so, here's a nice one that stumped me for a while, and caught someone else as well:

Given the code in this Gist: https://gist.github.com/miksago/d28d3a9491c9dbb9f663

I get the output:
screen shot 2014-08-28 at 11 23 44 pm

I couldn't work out why Lab thought that I didn't have good code coverage. It was only when \malex\ in the IRC channel pointed out that it's actually:

    expect(function(){
      example('/');
    }).to.not.throw();

In the test case.


I've heard that Chai will be going away in the next version, but it's things like this that make Lab really hard to use, and also what I meant by my tweet

@hueniverse
Copy link
Contributor

That's one of the reasons we're dropping Chai.

@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.
Labels
non issue Issue is not a problem or requires changes
Projects
None yet
Development

No branches or pull requests

2 participants