nridge created this revision. nridge added a reviewer: ayzhao. Herald added subscribers: kadircet, arphaman. Herald added a project: All. nridge requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang-tools-extra.
Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D137674 Files: clang-tools-extra/clangd/test/system-include-extractor.test Index: clang-tools-extra/clangd/test/system-include-extractor.test =================================================================== --- clang-tools-extra/clangd/test/system-include-extractor.test +++ clang-tools-extra/clangd/test/system-include-extractor.test @@ -63,7 +63,9 @@ # CHECK: "method": "textDocument/publishDiagnostics", # CHECK-NEXT: "params": { # CHECK-NEXT: "diagnostics": [], -# CHECK-NEXT: "uri": "file://INPUT_DIR/the-file.cpp", +# Double-check that the diagnostics pertain to the intended file. +# We don't match the path portion exactly to avoid issues around URL escaping. +# CHECK-NEXT: "uri": "file://{{.*}}/the-file.cpp", --- {"jsonrpc":"2.0","id":10000,"method":"shutdown"} ---
Index: clang-tools-extra/clangd/test/system-include-extractor.test =================================================================== --- clang-tools-extra/clangd/test/system-include-extractor.test +++ clang-tools-extra/clangd/test/system-include-extractor.test @@ -63,7 +63,9 @@ # CHECK: "method": "textDocument/publishDiagnostics", # CHECK-NEXT: "params": { # CHECK-NEXT: "diagnostics": [], -# CHECK-NEXT: "uri": "file://INPUT_DIR/the-file.cpp", +# Double-check that the diagnostics pertain to the intended file. +# We don't match the path portion exactly to avoid issues around URL escaping. +# CHECK-NEXT: "uri": "file://{{.*}}/the-file.cpp", --- {"jsonrpc":"2.0","id":10000,"method":"shutdown"} ---
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits