RLovelett / langserver-swift

A Swift implementation of the open Language Server Protocol.
Apache License 2.0
177 stars 16 forks source link

Added guard statements and XCTFail declarations. Full Linux coverage. #40

Closed felix91gr closed 6 years ago

felix91gr commented 6 years ago

Added guard statements to replace forced unwraps that would otherwise crash Linux when running certain tests. Now that those tests are safe, I've enabled them on Linux. With this commit, 100% of the tests run on Linux.

The next step will be to make them all pass :)

RLovelett commented 6 years ago

@felix91gr I really do appreciate this. I've changed the branch from master to fedora-linux-swift-4.1 for now.

I'd like to keep this out of master until we can fix all the logging and other issues that I had to comment out just to get it to work on Linux.

We are definitely making progress which is great.