Skip to content

Fix minor test errors#727

Merged
jonahgraham merged 2 commits into
eclipse-lsp4j:mainfrom
jonahgraham:fix_minor_test_errors
May 2, 2023
Merged

Fix minor test errors#727
jonahgraham merged 2 commits into
eclipse-lsp4j:mainfrom
jonahgraham:fix_minor_test_errors

Conversation

@jonahgraham

Copy link
Copy Markdown
Contributor

No description provided.

Gson was parsing this malformed JSON, but it wasn't actually correct
without this escaping.

Before the message would look like:

```json
{"seq":10,"type":"response","request_seq":4,"command":"foo","success":true,body:{"allThreadsContinued":false}}
```

Now it looks like:

```json
{"seq":10,"type":"response","request_seq":4,"command":"foo","success":true,"body":{"allThreadsContinued":false}}

```
@jonahgraham jonahgraham merged commit 3d55fa0 into eclipse-lsp4j:main May 2, 2023
@jonahgraham jonahgraham deleted the fix_minor_test_errors branch May 2, 2023 15:42
@jonahgraham jonahgraham added this to the 0.21.0 milestone May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant