getVersionSync test fails in appveyor

[Almenon]

It says that the version is a empty string. No idea why - passes locally. I should try using a different computer and seeing if it passes there.

The other interesting thing is that al the tests passed on ubuntu. Why would the test only fail on windows? I'm using a windows computer currently so it should fail locally if that's the case.

[Almenon]

I just tried on a different computer and it passes locally there too :/

[Almenon]

maybe it's because I'm on a different version of node. Ideally I should be doing my tests against a matrix of different node & python versions.

[Almenon]

it looks like getverysionsync also fails with python 2

https://ci.appveyor.com/project/Almenon/python-shell/build/job/7h74qfvwjq2gpe57

[Almenon]

and fails with python 3 and node 10 too. So it's not related to the python/node version

https://ci.appveyor.com/project/Almenon/python-shell/build/job/do630jekia3ayplq

[Almenon]

Not sure how I didn't notice it but all the tests are failing :/

[Almenon]

All of the ubuntu tests are passing now with https://github.com/extrabacon/python-shell/commit/61a8fc9455f27e33a89bd632f51e0b7921b1edb1

[Almenon]

I've decided to comment them out on master untill someone fixes them. The methods the tests test are not very important so if the unit tests don't run it's not that big of a deal. Having a 100% passing unit test suite is more important.

[ghost]

Uhm maybe they also should be commented on the README file because I was trying to use them haha

https://github.com/extrabacon/python-shell/tree/5f39ad4d1b59f88f6450651754085c37d5782696#getversionpythonpathstring

Anyway running a small python script like this does the trick

import platform
print(platform.python_version())

[Almenon]

They still work, just not in appveyor