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

Broken module, broken tests #80

Closed
ferlores opened this issue Apr 6, 2015 · 5 comments
Closed

Broken module, broken tests #80

ferlores opened this issue Apr 6, 2015 · 5 comments

Comments

@ferlores
Copy link
Contributor

ferlores commented Apr 6, 2015

Hi @mantoni
I just realized that if you delete your node_module folder and npm install the dependencies again mochify doesn't report back the result of the tests run in phantom. It looks like a dependency got broken.

I discover this through one of the projects where I use mochify, and I confirmed it by cloning the repo from scratch, npm install and run the tests. Half of them fail.

Do you mind looking at it?

Thanks!
F

@mantoni
Copy link
Owner

mantoni commented Apr 7, 2015

Thanks for the heads up. Will look into it as soon as I get near a computer.

-Max

Am 06.04.2015 um 23:13 schrieb Fernando Lores notifications@github.com:

Hi @mantoni
I just realized that if you delete your node_module folder and npm install the dependencies again mochify doesn't report back the result of the tests run in phantom. It looks like a dependency got broken.

I discover this through one of the projects where I use mochify, and I confirming by cloning the repo from scratch, npm install and run the tests. Half of them fail.

Do you mind looking at it?

Thanks!
F


Reply to this email directly or view it on GitHub.

@tjunghans
Copy link

I noticed the same issue. I don't see any test results. The only output is #phantomjs:. I'm using phantomjs v1.9.7 and mochify v2.7.0.

@mantoni
Copy link
Owner

mantoni commented Apr 7, 2015

This was introduce in browserify 9.0.5. As a quick fix, I'll nail browserify to 9.0.4.

Apparently, streams that are pushed to pipeling.get('wrap') are not part of the output stream anymore. E.g. inserting n.stdout.pipe(process.stdout); at this line shows the expected output on the console.

If anybody can look further into this, I'd be very thankful.

@mantoni
Copy link
Owner

mantoni commented Apr 7, 2015

Worked around in Mochify v2.7.1. Please re-install. I'll keep this issue open until there is a proper solution.

@mantoni
Copy link
Owner

mantoni commented Apr 14, 2015

I still don't understand the issue 100%, but v2.7.2 has a "good enough" workaround.

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

No branches or pull requests

3 participants