[PATCH] D54833: [clangd] Cleanup: use index file instead of header in workspace symbols lit test.
Eric Liu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Nov 22 06:26:11 PST 2018
ioeric created this revision.
ioeric added a reviewer: sammccall.
Herald added subscribers: cfe-commits, kadircet, arphaman, jkorous, MaskRay, ilya-biryukov.
The full path of the input header depends on the execution environment
and may result in different behavior (e.g. when different URI schemes are used).
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D54833
Files:
test/clangd/Inputs/sstream.h
test/clangd/Inputs/symbols.test.yaml
test/clangd/symbols.test
Index: test/clangd/symbols.test
===================================================================
--- test/clangd/symbols.test
+++ test/clangd/symbols.test
@@ -1,9 +1,9 @@
-# RUN: env CPATH=%S/Inputs clangd -lit-test < %s | FileCheck %s
+# RUN: clangd --index-file=%S/Inputs/symbols.test.yaml -lit-test < %s | FileCheck %s
{"jsonrpc":"2.0","id":0,"method":"initialize","params":{"processId":123,"rootPath":"clangd","capabilities":{"workspace":{"symbol":{"symbolKind":{"valueSet": [1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22,23,24,25,26]}}}},"trace":"off"}}
---
-{"jsonrpc":"2.0","method":"textDocument/didOpen","params":{"textDocument":{"uri":"test:///main.cpp","languageId":"cpp","version":1,"text":"#include <sstream.h>\nvoid foo(); int main() { foo(); }\n"}}}
+{"jsonrpc":"2.0","method":"textDocument/didOpen","params":{"textDocument":{"uri":"test:///main.cpp","languageId":"cpp","version":1,"text":"void foo(); int main() { foo(); }\n"}}}
---
-{"jsonrpc":"2.0","id":1,"method":"workspace/symbol","params":{"query":"std::basic_ostringstream"}}
+{"jsonrpc":"2.0","id":1,"method":"workspace/symbol","params":{"query":"vector"}}
# CHECK: "id": 1,
# CHECK-NEXT: "jsonrpc": "2.0",
# CHECK-NEXT: "result": [
@@ -21,9 +21,9 @@
# CHECK-NEXT: "line": {{.*}}
# CHECK-NEXT: }
# CHECK-NEXT: },
-# CHECK-NEXT: "uri": "file://{{.*}}/sstream.h"
+# CHECK-NEXT: "uri": "file:///vector.h"
# CHECK-NEXT: },
-# CHECK-NEXT: "name": "basic_ostringstream"
+# CHECK-NEXT: "name": "vector"
# CHECK-NEXT: }
# CHECK-NEXT: ]
# CHECK-NEXT:}
Index: test/clangd/Inputs/symbols.test.yaml
===================================================================
--- /dev/null
+++ test/clangd/Inputs/symbols.test.yaml
@@ -0,0 +1,17 @@
+---
+!Symbol
+ID: 057557CEBF6E6B2D
+Name: 'vector'
+Scope: 'std::'
+SymInfo:
+ Kind: Class
+ Lang: Cpp
+CanonicalDeclaration:
+ FileURI: 'file:///vector.h'
+ Start:
+ Line: 215
+ Column: 10
+ End:
+ Line: 215
+ Column: 16
+...
Index: test/clangd/Inputs/sstream.h
===================================================================
--- test/clangd/Inputs/sstream.h
+++ /dev/null
@@ -1,3 +0,0 @@
-namespace std {
-class basic_ostringstream {};
-}
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54833.175047.patch
Type: text/x-patch
Size: 2442 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181122/6e994804/attachment.bin>
More information about the cfe-commits
mailing list