Error: The string ";.rhfWrapper .shoveler .price-per-unit", is not a valid CSS selectorat emit (/project_dir/node_modules/jsdom/node_modules/nwmatcher/src/nwmatcher.js:830:37)at Object.match (/project_dir/node_modules/jsdom/node_modules/nwmatcher/src/nwmatcher.js:1303:11)at Object.exports.applyQuerySelector.doc.createElement.element.matchesSelector (/project_dir/node_modules/jsdom/lib/jsdom/selectors/index.js:32:49)at exports.createWindow.DOMWindow.getComputedStyle (/project_dir/node_modules/jsdom/lib/jsdom/browser/index.js:243:20)at Array.forEach (native)at exports.createWindow.DOMWindow.getComputedStyle (/project_dir/node_modules/jsdom/lib/jsdom/browser/index.js:242:17)at Object.forEach (native)at exports.createWindow.DOMWindow.getComputedStyle (/project_dir/node_modules/jsdom/lib/jsdom/browser/index.js:241:15)
Thanks for finding this! 0.3.1 pushed with that fixed.
Eric,
in nwmatcher this can be fixed by setting a simple flag (fx: in production):
Dom.configure({ VERBOSITY: false });
In any case I would prefer to leave this "true" if the developer is NOT interested into catching these validation errors and still maintain a log of the wrong behavior. The latter can be fixed at a higher level (as showed). Current fix/patch seems OK to me.
Wrong ... is still wrong. However with the above infos everybody can make notifications fit their needs (break processing or not) at different stages of dev/dep.
Being able to only return an empty Array was not the right solution, also because the goal is to match Query Selector API which doesn't work like that. Developers should match that (QSA), with the required flexibility when feasible :)
PS: there are currently other invalid selectors in the current sizzle tests like "!=" and several unquoted "href" values.
https://github.com/tmpvar/jsdom#test-compliance
Curious which extra test you're seeing fail...
________________________________________
From: js...@googlegroups.com [js...@googlegroups.com] on behalf of Godmar Back [god...@gmail.com]
Sent: Tuesday, December 11, 2012 13:50
To: js...@googlegroups.com