[clang-tools-extra] r318339 - [clangd] Loosen more brittle tests

Sam McCall via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 15 13:50:53 PST 2017


Author: sammccall
Date: Wed Nov 15 13:50:53 2017
New Revision: 318339

URL: http://llvm.org/viewvc/llvm-project?rev=318339&view=rev
Log:
[clangd] Loosen more brittle tests

Modified:
    clang-tools-extra/trunk/test/clangd/completion-snippet.test
    clang-tools-extra/trunk/test/clangd/completion.test

Modified: clang-tools-extra/trunk/test/clangd/completion-snippet.test
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clangd/completion-snippet.test?rev=318339&r1=318338&r2=318339&view=diff
==============================================================================
--- clang-tools-extra/trunk/test/clangd/completion-snippet.test (original)
+++ clang-tools-extra/trunk/test/clangd/completion-snippet.test Wed Nov 15 13:50:53 2017
@@ -61,16 +61,6 @@ Content-Length: 148
 # CHECK-NEXT:        "label": "fake::",
 # CHECK-NEXT:        "sortText": "000075fake"
 # CHECK-NEXT:      },
-# CHECK-NEXT:      {
-# CHECK-NEXT:        "detail": "fake &",
-# CHECK-NEXT:        "filterText": "operator=",
-# CHECK-NEXT:        "insertText": "operator=(${1:const fake &})",
-# CHECK-NEXT:        "insertTextFormat": 2,
-# CHECK-NEXT:        "kind": 2,
-# CHECK-NEXT:        "label": "operator=(const fake &)",
-# CHECK-NEXT:        "sortText": "000079operator="
-# CHECK-NEXT:      },
-# FIXME: Why do some buildbots show an extra operator==(fake&&) here?
 #      CHECK:      {
 #      CHECK:        "detail": "void",
 # CHECK-NEXT:        "filterText": "~fake",

Modified: clang-tools-extra/trunk/test/clangd/completion.test
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clangd/completion.test?rev=318339&r1=318338&r2=318339&view=diff
==============================================================================
--- clang-tools-extra/trunk/test/clangd/completion.test (original)
+++ clang-tools-extra/trunk/test/clangd/completion.test Wed Nov 15 13:50:53 2017
@@ -61,16 +61,7 @@ Content-Length: 148
 # CHECK-NEXT:      "label": "fake::",
 # CHECK-NEXT:      "sortText": "000075fake"
 # CHECK-NEXT:    },
-# CHECK-NEXT:    {
-# CHECK-NEXT:      "detail": "fake &",
-# CHECK-NEXT:      "filterText": "operator=",
-# CHECK-NEXT:      "insertText": "operator=",
-# CHECK-NEXT:      "insertTextFormat": 1,
-# CHECK-NEXT:      "kind": 2,
-# CHECK-NEXT:      "label": "operator=(const fake &)",
-# CHECK-NEXT:      "sortText": "000079operator="
-# CHECK-NEXT:    },
-# FIXME: Why do some buildbots show an extra operator==(fake&&) here?
+# FIXME: Why do buildbots show different operator=s here?
 #      CHECK:    {
 #      CHECK:      "detail": "void",
 # CHECK-NEXT:      "filterText": "~fake",
@@ -133,15 +124,6 @@ Content-Length: 148
 # CHECK-NEXT:      "label": "fake::",
 # CHECK-NEXT:      "sortText": "000075fake"
 # CHECK-NEXT:    },
-# CHECK-NEXT:    {
-# CHECK-NEXT:      "detail": "fake &",
-# CHECK-NEXT:      "filterText": "operator=",
-# CHECK-NEXT:      "insertText": "operator=",
-# CHECK-NEXT:      "insertTextFormat": 1,
-# CHECK-NEXT:      "kind": 2,
-# CHECK-NEXT:      "label": "operator=(const fake &)",
-# CHECK-NEXT:      "sortText": "000079operator="
-# CHECK-NEXT:    },
 #      CHECK:    {
 #      CHECK:      "detail": "void",
 # CHECK-NEXT:      "filterText": "~fake",
@@ -182,15 +164,6 @@ Content-Length: 148
 # CHECK-NEXT:      "label": "fancy::",
 # CHECK-NEXT:      "sortText": "000075fancy"
 # CHECK-NEXT:    },
-# CHECK-NEXT:    {
-# CHECK-NEXT:      "detail": "fancy &",
-# CHECK-NEXT:      "filterText": "operator=",
-# CHECK-NEXT:      "insertText": "operator=",
-# CHECK-NEXT:      "insertTextFormat": 1,
-# CHECK-NEXT:      "kind": 2,
-# CHECK-NEXT:      "label": "operator=(const fancy &)",
-# CHECK-NEXT:      "sortText": "000079operator="
-# CHECK-NEXT:    },
 #      CHECK:    {
 #      CHECK:      "detail": "void",
 # CHECK-NEXT:      "filterText": "~fancy",




More information about the cfe-commits mailing list