ChildProcess change

[kimmolinna]

With the latest version of Zig I have to delete try and deinit.

[leecannon]

Duplicate of #481

But the other PR has been open for 9 days and still is not ready to merge, if you want this merged you need to fix the tests as well (zig build test must pass)

[kimmolinna]

I updated tests too but I'm not sure what will cause the following errors:

==> Expected:
{"isIncomplete":false,"items":[{"label":"w","kind":3,"textEdit":null,"filterText":null,"insertText":"w","insertTextFormat":1,"detail":"fn","documentation":null}]}
==> Got: (Mismatch in position 102)
{"isIncomplete":false,"items":[{"label":"w","kind":3,"textEdit":null,"filterText":null,"insertText":"w()","insertTextFormat":2,"detail":"fn","documentation":null}]}
Test [6/8] test "Missing return type"... FAIL (InvalidResponse)
[debug-main] Took 0ms to process method shutdown
/home/kimmo/zls/tests/sessions.zig:82:17: 0x20dcce in Server.request (test)
                return error.InvalidResponse;
                ^
/home/kimmo/zls/tests/sessions.zig:227:5: 0x20ca3e in test "Missing return type" (test)
    try server.request("textDocument/completion",
    ^

[leecannon]

I have no idea why it now expects the below line to be ..,"insertText":"w()","insertTextFormat":2,..

zls/tests/sessions.zig

Line 231 in ed3d383

\\{"isIncomplete":false,"items":[{"label":"w","kind":3,"textEdit":null,"filterText":null,"insertText":"w","insertTextFormat":1,"detail":"fn","documentation":null}]}

[kimmolinna]

And is this related to ChildProcess or something else...

[Manuzor]

I tried running the tests on windows after applying the changes from this PR. Some test failed with a file-not-found error, so I changed all occurrences of file://./ to file:///./ (3 slashes instead of just 2). After that, all tests passed.