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

Inconsistent behaviour of tap reporters #137

Closed
icetbr opened this issue Feb 6, 2022 · 2 comments
Closed

Inconsistent behaviour of tap reporters #137

icetbr opened this issue Feb 6, 2022 · 2 comments

Comments

@icetbr
Copy link

icetbr commented Feb 6, 2022

Hi, would you mind creating a sample tap reporter to show its interaction with zora? I'm having a few issues.

I can't get ANY tap reporter to work with nodemon. Using tap-diff, for instance, on first save it works, on the second, it doesn't.

And my custom reporter only the first save works. Here is a demo project: https://github.com/icetbr/tapReporterExample

first save

npm run testTapDiff

> testTapDiff
> nodemon --quiet test/test.js | tap-diff


  reporter
    ✖  should be equivalent at  file:///home/ddv/dev/projects/tapReporterExample/test/test.js:3:30
        45

  skip  0

seconds save

        TAP version 13

  reporter
        not ok 1 - should be equivalent
          ---
            actual: 4
            expected: 5
            operator: "equal"
            at: " file:///home/ddv/dev/projects/tapReporterExample/test/test.js:3:30"
          ...
        1..1

  skip  0
@icetbr icetbr changed the title Inconsistent behaviour tap reporters Inconsistent behaviour of tap reporters Feb 6, 2022
@lorenzofox3
Copy link
Owner

ok let me check. I am not sure how nodemon pass the stdout stream

@lorenzofox3 lorenzofox3 self-assigned this Feb 7, 2022
@lorenzofox3
Copy link
Owner

lorenzofox3 commented Feb 7, 2022

I suspect an issue with a (missing) final end of line which confuses the tap parser.

A easy workaround would be to trigger the test as a command:
nodemon --exec "node test/test.js | tap-diff"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants