Autodesk-AutoCAD / AutoLispExt

Visual Studio Code Extension for AutoCAD® AutoLISP
https://marketplace.visualstudio.com/items?itemName=Autodesk.autolispext
Apache License 2.0
87 stars 30 forks source link

Fix test failure on Mac about end-of-line #115

Closed Sen-real closed 3 years ago

Sen-real commented 3 years ago
Objective

A potential bug is discovered by running automation tests on Mac. Error "failed to read string on top level" is printed in console.

Abstractions

It's caused by an error in a line of code. Unlike Windows, when running tests on Mac, the calls to split and join methods (which was added for an edge case, and not needed with the given test data) will change the original string, and this is why the error is discovered. On Windows, the original string is untouched, and that original string is already in correct format, so the test case passes.

Tests performed

Tested on Mac. The error is gone. All 40 tests pass.

Screen shot