8000 fix tests · brandonwillard/python-lsp-server@b786754 · GitHub
[go: up one dir, main page]

Skip to content

Commit b786754

Browse files
author
forozco
committed
fix tests
1 parent 3b72899 commit b786754

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

pyls/rpc_manager.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -149,9 +149,10 @@ def did_finish_callback(completed_future):
149149
except JSONRPCDispatchException as e:
150150
output = _make_response(request, error=e.error._data)
151151
except Exception: # pylint: disable=broad-except
152-
# TODO(forozco): add more descriptive error
153152
log.exception('asynchronous method handler exception for request: %s', request)
154-
output = _make_response(request, error={'message': traceback.format_exc()})
153+
output = _make_response(request, error={
154+
'code': JSONRPCServerError.CODE, 'message': traceback.format_exc()
155+
})
155156
else:
156157
output = _make_response(request, result=result)
157158
finally:

test/test_rpc_manager.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,10 @@ def wrapper():
5353
rpc_manager._sent_requests.values()[0].result(timeout=1)
5454
message_manager.write_message.assert_called_once()
5555
(sent_message, ), _ = message_manager.write_message.call_args
56-
assert sent_message.data == JSONRPC20Response(_id=1, error=JSONRPCServerError()._data).data
56+
data = sent_message.data
57+
assert data['id'] == 1
58+
assert data['error']['code'] == JSONRPCServerError.CODE
59+
assert 'something bad happened' in data['error']['message']
5760

5861

5962
def test_handle_request_async_error(rpc_management):

0 commit comments

Comments
 (0)
0