Skip to content
This repository has been archived by the owner on Sep 7, 2018. It is now read-only.

Buster is modifying the global Error object (via old JSDOM) #410

Closed
grncdr opened this issue Jun 6, 2014 · 5 comments
Closed

Buster is modifying the global Error object (via old JSDOM) #410

grncdr opened this issue Jun 6, 2014 · 5 comments

Comments

@grncdr
Copy link

grncdr commented Jun 6, 2014

cat <<EOF | node
console.log(Error.prototype.__proto__);
require('buster');
console.log(Error.prototype.__proto__);
EOF
{}
{ [Function: XPathException] INVALID_EXPRESSION_ERR: 51, TYPE_ERR: 52 }

I haven't dug down to figure out where in JSDOM the bug is, but is it really required for running tests in the 'node' environment??? 👎

@grncdr
Copy link
Author

grncdr commented Jun 6, 2014

Some further investigation has turned up jsdom/jsdom#688 which fixes this bug. However buster-test and buster-syntax are both using quite old versions of jsdom (~0.5.3 and ~0.2.14 respectively).

@grncdr grncdr changed the title Buster is modifying the global Error object (via JSDOM) Buster is modifying the global Error object (via old JSDOM) Jun 6, 2014
@dwittner
Copy link
Member

dwittner commented Jun 6, 2014

JsDom is used by the html reporter, which can also be used for node.js test runs. I will update the version of JsDom for buster-test and buster-syntax.

@dwittner
Copy link
Member

dwittner commented Jun 6, 2014

@grncdr, i have published new version 0.7.7 of buster-test and 0.4.3 of buster-syntax right now, both with new version ~0.10 of JsDom. Please check if your issue is solved now.

@grncdr
Copy link
Author

grncdr commented Jun 8, 2014

Yes, looks good to me now. Thanks for the quick response.

-Stephen

On June 6, 2014 at 4:47:21 PM, Daniel Wittner (notifications@github.com) wrote:

@grncdr, i have published new version 0.7.7 of buster-test and 0.4.3 of buster-syntax right now, both with new version ~0.10 of JsDom. Please check if your issue is solved now.


Reply to this email directly or view it on GitHub.

@dwittner
Copy link
Member

dwittner commented Jun 8, 2014

You're welcome!

@dwittner dwittner closed this as completed Jun 8, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants