HuddleEng / PhantomFlow

Describe and visualise user flows through tests with PhantomJS
MIT License
682 stars 60 forks source link

Enquote path args to handle paths with spaces. #54

Closed SeijiSuenaga closed 7 years ago

SeijiSuenaga commented 7 years ago

Prior to this change, running tests from a path that contains spaces produces output like this:

Parallelising 2 test files on 2 processes.

Picking up job: example1.test.js
Picking up job: example2.test.js
[example1.test.js] It broke, sorry. Process aborted. Non-zero code (1) returned.
[example1.test.js] It broke, sorry. Process aborted. Non-zero code (1) returned.

 Please take a look at the error log for more info 'C:\Example Directory\Tests\test-results/log/error_1.log'
[example2.test.js] It broke, sorry. Process aborted. Non-zero code (1) returned.
[example2.test.js] It broke, sorry. Process aborted. Non-zero code (1) returned.

 Please take a look at the error log for more info 'C:\Example Directory\Tests\test-results/log/error_2.log'

error_1.log contains:

[example1.test.js] (1788ms) Unable to open file: C:/Example
[example1.test.js] (0ms)   phantomjs://code/bootstrap.js:110 in __die
[example1.test.js] (0ms) Unable to load script C:/Example; check file syntax
[example1.test.js] (0ms)   phantomjs://code/bootstrap.js:110 in __die