LiveScript icon indicating copy to clipboard operation
LiveScript copied to clipboard

Fixing test script for windows

Open othelarian opened this issue 1 year ago • 0 comments

The issue:

When running the test script in scripts/test on Windows some tests failed because of some paths. The two paths are:

  • for runtime-error.ls, there is a carriage return (\r) before the \n, and the path is absolute with \ (in th script it's .*/test/data/runtime-error.ls)
  • for data.json.ls, the output of lsc -c --debug test/data/data.json.ls used \ on windows
  • same for data.ls

Fix:

All the regex are in test/cli.ls.

  • The runtime-error.ls regex is fixed by adding a check on the new line mess ([\r\n]{1,2}) and the absolute path is abstracted. The code still checks if it's runtime-error.ls that is throwing the error, and if it's line 2, char 17.
  • For data.json.ls and data.ls the path is now normalized.

NOTE: I haven't any linux on hand to check if the new regex and normalize will fail on it. If someone can check on it, it would be great. I'll try tomorrow to run it on linux, but another pair of eyes is better than one ;-)

othelarian avatar Mar 03 '24 23:03 othelarian