Gulp PhpUnit Fix for Windows

[pataelmo]

Wrapped the path used to call phpunit from gulp with path.normalize. This makes gulp able to run phpUnit from Windows. Haven't tested to ensure this didn't break anything for mac or linux, but hopefully it should be pretty safe.

[onecentlin]

I tried these modified changes in both Windows and Mac. Gulp phpunit testing works properly.

[pataelmo]

I don't know the easiest way to test as I've just been learning laravel, but it would seem that changes in commit b20e61e make this pull request unnecessary.

Assuming Elixir.Plugins.shell() can take a path with /'s on windows and execute properly then the aforemention commit change should mean that any and all tasks should be fixed for elixir and not just PHPUnit.

@JeffreyWay can you confirm that is true? If so then I'll close this pull request. Also if you don't mind letting me know the best way to see this in a composer based laravel install (or pointing me to docs that show how to do so) would be great. Thanks!

[JeffreyWay]

Not sure yet. Need to try out on Windows when I have a chance.

Try doing npm install git://github.com/pataelmo/elixir.git in a fresh Laravel project to try things out.

[onecentlin]

Without modifying current laravel-elixir, I change phpunit options in gulpfile.js settings.

Here is my gulpfile.js file:

var elixir = require('laravel-elixir');
var path = require('path');

elixir(function(mix) {
    mix.phpUnit([] , path.normalize('vendor/bin/phpunit') + ' --verbose');
});

[pascal08]

@onecentlin Works for me on Windows 10 x64, so I will be using this until this is merged.

[kidtangerine]

@onecentlin works for me too. thanks.

[mbaljeetsingh]

@onecentlin This worked for me too (Windows 10 x64)

[tannakartikey]

@pataelmo Thanks. Working fine on Windows 10