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

False negatives in header/footer test #29

Closed
nCrazed opened this issue May 14, 2014 · 1 comment
Closed

False negatives in header/footer test #29

nCrazed opened this issue May 14, 2014 · 1 comment

Comments

@nCrazed
Copy link
Contributor

nCrazed commented May 14, 2014

As I mentioned in #28 this test was failing even before my modifications and since it's breaking the build of my pull request I decide to dig around a little.

The error is triggered by these two lines:

assert.ok(/Some\s?Header/.test(chunks.join('')))
assert.ok(/Some\s?Footer/.test(chunks.join('')))

I've inspected the the temporary PDF files generated during the test with evince and both Some Header and Some Footer are clearly present in the result. So it looks like the current test for header/footer started throwing false negatives.

I've tried finding the dependency that caused this change by downgrading both node and some of the packages to the ones used in the successful build without success. But since I am not aware of an automated way to perform such checks I ended up doing it manually so it's possible that I missed a package.

@alanshaw
Copy link
Owner

This is due to the rendering differences from phantomjs. On linux it seems as though the pdf created has text in it and on osx it is just a big image - ariya/phantomjs#10373

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

2 participants