mgorny created this revision.
mgorny added reviewers: ilya-biryukov, arphaman, bkramer, gribozavr.
Update expected completions to match output generated by clang-7.0.
Disclaimer: I have no clue if the results are correct. I'm kinda wondering why
the '::' part is no longer included immediately.
Also PR38417 <https://bugs.llvm.org/show_bug.cgi?id=38417>.
Repository:
rC Clang
https://reviews.llvm.org/D50171
Files:
bindings/python/tests/cindex/test_code_completion.py
Index: bindings/python/tests/cindex/test_code_completion.py
===================================================================
--- bindings/python/tests/cindex/test_code_completion.py
+++ bindings/python/tests/cindex/test_code_completion.py
@@ -61,11 +61,11 @@
cr = tu.codeComplete('fake.cpp', 12, 5, unsaved_files=files)
expected = [
- "{'const', TypedText} || Priority: 40 || Availability: Available ||
Brief comment: None",
- "{'volatile', TypedText} || Priority: 40 || Availability: Available
|| Brief comment: None",
+ "{'const', TypedText} || Priority: 50 || Availability: Available ||
Brief comment: None",
+ "{'volatile', TypedText} || Priority: 50 || Availability: Available
|| Brief comment: None",
"{'operator', TypedText} || Priority: 40 || Availability: Available
|| Brief comment: None",
- "{'P', TypedText} | {'::', Text} || Priority: 75 || Availability:
Available || Brief comment: None",
- "{'Q', TypedText} | {'::', Text} || Priority: 75 || Availability:
Available || Brief comment: None"
+ "{'P', TypedText} || Priority: 50 || Availability: Available ||
Brief comment: None",
+ "{'Q', TypedText} || Priority: 50 || Availability: Available ||
Brief comment: None"
]
self.check_completion_results(cr, expected)
Index: bindings/python/tests/cindex/test_code_completion.py
===================================================================
--- bindings/python/tests/cindex/test_code_completion.py
+++ bindings/python/tests/cindex/test_code_completion.py
@@ -61,11 +61,11 @@
cr = tu.codeComplete('fake.cpp', 12, 5, unsaved_files=files)
expected = [
- "{'const', TypedText} || Priority: 40 || Availability: Available || Brief comment: None",
- "{'volatile', TypedText} || Priority: 40 || Availability: Available || Brief comment: None",
+ "{'const', TypedText} || Priority: 50 || Availability: Available || Brief comment: None",
+ "{'volatile', TypedText} || Priority: 50 || Availability: Available || Brief comment: None",
"{'operator', TypedText} || Priority: 40 || Availability: Available || Brief comment: None",
- "{'P', TypedText} | {'::', Text} || Priority: 75 || Availability: Available || Brief comment: None",
- "{'Q', TypedText} | {'::', Text} || Priority: 75 || Availability: Available || Brief comment: None"
+ "{'P', TypedText} || Priority: 50 || Availability: Available || Brief comment: None",
+ "{'Q', TypedText} || Priority: 50 || Availability: Available || Brief comment: None"
]
self.check_completion_results(cr, expected)
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
