search

TelluIoT / ThingML

The ThingML modelling language
https://github.com/TelluIoT/ThingML
Apache License 2.0
102 stars 33 forks source link

Local tests are broken #130

Closed brice-morin closed 7 years ago

brice-morin commented 7 years ago

It seems the local execution of the tests is broken. The ThingML files produced all contains error. Please fix ASAP

nharrand commented 7 years ago

I don't have the problem on my machine with the latest commit... What kind of errors are you dealing with?

brice-morin commented 7 years ago

Seems like the problem is that it generate one absolute path with \ on Windows, for one of the import, and that it does not parse correctly. Something like: import "C:\Users\xxx\ThingML\testJar\src\main\resources\tests\StateMachine\testCompositeStates.thingml"

brice-morin commented 7 years ago

Something like import "file://C:/Users/xxx/ThingML/testJar/src/main/resources/tests/StateMachine/testCompositeStates.thingml" seems to work. Just try on Linux if file:// works.

nharrand commented 7 years ago

No it doesn't seem to. But we can just use it for windows if needed.

brice-morin commented 7 years ago

OK, then import "/C:/Users/xxx/ThingML/testJar/src/main/resources/tests/StateMachine/testCompositeStates.thingml" seems to work on Windows. So it is basically to replace \ with / and add a leading / if it is not there. That should be generic enough to work everywhere.