TheProjecter / phantomjs-qunit-runner

Automatically exported from code.google.com/p/phantomjs-qunit-runner
0 stars 0 forks source link

Patch for /trunk/src/main/java/net/kennychua/phantomjs_qunit_runner/PhantomJsQunitRunnerMojo.java #14

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
Places user defined dependencies before code that uses those dependencies.

Original issue reported on code.google.com by kierans...@gmail.com on 29 Oct 2012 at 4:20

Attachments:

GoogleCodeExporter commented 8 years ago
This patch addresses Issue 13.

Original comment by kierans...@gmail.com on 29 Oct 2012 at 4:21

GoogleCodeExporter commented 8 years ago
I was having the same problem: by listing the file under test before the 
libraries (jquery in my case) the tests were failing under maven (but worked ok 
in the generated html). The tests were failing with "ReferenceError: Can't find 
variable: $,".

Applying this patch such that the file under test and the test file are listed 
last in the phantomjs command line makes things work as expected.

Original comment by one...@energyhub.net on 4 Jan 2013 at 7:25