serut / meteor-coverage

Server and client coverage for Meteor
MIT License
54 stars 14 forks source link

Find a way to automatically set the app-path #61

Closed SimonSimCity closed 6 years ago

SimonSimCity commented 6 years ago

I currently don't have a windows environment, but on Mac, process.env.PWD returned the desired value. Could we not get this somewhere tested and replace it by this value?

serut commented 6 years ago

That's the issue. I develop this package on all environments and process.env.PWD inside the package refers to the tmp folder where your app have been compiled on Windows. (something like C/user/yourself/app_bullshit/meteor/tmp/[....]).
That's maybe not an issue anymore on the last version of Meteor.
My favorite workaround is to provide that value from the headless browser runner on runtime as I did with spacejam.

SimonSimCity commented 6 years ago

True ... even https://github.com/VeliovGroup/Meteor-root doesn't work anymore ...

I've opened an issue there and awaiting a response there with which I could get back to this repo.