summaryrefslogtreecommitdiff
path: root/test
diff options
context:
space:
mode:
authorJon Gjengset <jon@thesquareplanet.com>2017-05-31 13:16:49 -0400
committerw0rp <w0rp@users.noreply.github.com>2017-05-31 18:16:49 +0100
commit88948e0ee3729b9b31b7cfd7e0efd5fe15143621 (patch)
treeb92942c9a33cf6f23d1b7d053cff5d3cc3b28f7a /test
parent42efd517232c3f62289910e53f1d2fe2549196cf (diff)
downloadale-88948e0ee3729b9b31b7cfd7e0efd5fe15143621.zip
Include span label in rust lints (#601)
* Include span label in rust lints This turns relatively unhelpful error messages like mismatched types into more expressive messages along the lines of mismatched types: expected bool, found integral variable Fixes #597. * Exclude rust lint span label if empty * Use single-quoted strings in vimscript * Add test for detailed rust errors * Prune Cargo JSON * Use matching error file name * Byte offsets not char offsets
Diffstat (limited to 'test')
-rw-r--r--test/handler/test_rust_handler.vader19
1 files changed, 19 insertions, 0 deletions
diff --git a/test/handler/test_rust_handler.vader b/test/handler/test_rust_handler.vader
index 38228f38..11dcf17f 100644
--- a/test/handler/test_rust_handler.vader
+++ b/test/handler/test_rust_handler.vader
@@ -55,6 +55,25 @@ Execute(The Rust handler should handle cargo output):
\ '{"message":{"children":[],"code":null,"level":"error","message":"aborting due to previous error","rendered":null,"spans":[]},"package_id":"update 0.0.1 (path+file:///home/w0rp/Downloads/rust-by-example)","reason":"compiler-message","target":{"kind":["bin"],"name":"update","src_path":"/home/w0rp/Downloads/rust-by-example/src/main.rs"}}',
\ ])
+Execute(The Rust handler should show detailed errors):
+ AssertEqual
+ \ [
+ \ {
+ \ 'lnum': 4,
+ \ 'end_lnum': 4,
+ \ 'type': 'E',
+ \ 'col': 52,
+ \ 'end_col': 54,
+ \ 'text': 'mismatched types: expected bool, found integral variable',
+ \ },
+ \ ],
+ \ ale#handlers#rust#HandleRustErrorsForFile(347, 'src/playpen.rs', [
+ \ '',
+ \ 'ignore this',
+ \ '{"message":{"children":[],"code":null,"level":"error","message":"mismatched types","rendered":null,"spans":[{"byte_end":54,"byte_start":52,"column_end":23,"column_start":21,"expansion":null,"file_name":"src/playpen.rs","is_primary":true,"label":"expected bool, found integral variable","line_end":4,"line_start":4,"suggested_replacement":null,"text":[{"highlight_end":23,"highlight_start":21,"text":" let foo: bool = 42;"}]}]},"package_id":"ale-rust-details 0.1.1 (path+file:///home/jon/tmp/ale-rust-details)","reason":"compiler-message","target":{"crate_types":["bin"],"kind":["bin"],"name":"ale-rust-details","src_path":"/home/jon/tmp/ale-rust-details/src/main.rs"}}',
+ \ '{"message":{"children":[],"code":null,"level":"error","message":"aborting due to previous error(s)","rendered":null,"spans":[]},"package_id":"ale-rust-details 0.1.1 (path+file:///home/jon/tmp/ale-rust-details)","reason":"compiler-message","target":{"crate_types":["bin"],"kind":["bin"],"name":"ale-rust-details","src_path":"/home/jon/tmp/ale-rust-details/src/main.rs"}}',
+ \ ])
+
Execute(The Rust handler should find correct files):
AssertEqual
\ [],