avajs/ava

Do you want to work on this issue?

You can request for a bounty in order to promote it!


1 active bounty request

$100.00
Solve someone else's bounty request by clicking on it and completing the funding to earn additional credits

Timeouts shouldn't hide assertion failures #1997

Marsup posted onGitHub

Prerequisites

I checked other issues and couldn't find something similar, although #1485 is slightly related and is genuinely both surprising and completely counter-intuitive to me, but I'm not going to start this debate again.

Description

Test timeouts are hiding failed assertions.

Test Source

test('foo', t => {
  t.is(1, 2);
  return new Promise(() => {
    function f() {
      setTimeout(f, 10);
    }
    f();
  });
});

Error Message & Stack Trace

✖ Exited because no new tests completed within the last 2000ms of inactivity

That's all there is, no trace of the assertion.

Command-Line Arguments

ava --timeout=2s

Environment

Node.js v10.14.1
linux 4.18.0-12-generic
1.0.0-rc.2
6.4.1

Agreed. This is ultimately a limitation of the reporters, though we currently don't send the results of individual assertions to the main process either. I'm adding the appropriate labels.

posted by novemberborn about 6 years ago

@issuehunt has funded $40.00 to this issue.


posted by IssueHuntBot almost 6 years ago

I want to contribute and fix this bug.

posted by yagyadeep786 12 months ago

@yagyadeep786 by all means. I don’t know if this is still an issue though, you’ll want to verify that first.

posted by novemberborn 12 months ago

Fund this Issue

$40.00
Funded
Only logged in users can fund an issue

Pull requests

Recent activities

issuehunt funded 40.00 for avajs/ava# 1997
almost 6 years ago