[complete-test] Stop inserting \0 in complete-test

Using \0 as a representation of the cursor position is a implementation
detail and is done by SourceKit. Client including complete-test should
not do it.

This inserted test used to be treated as EOF in compiler so the code
after the completion position is completely ignored.

Author: rintaro

test/SourceKit/CodeComplete/complete_group_overloads.swift

@@ -89,6 +89,7 @@ func test005() {
 // BAR_INIT_0: ()
 // BAR_INIT_0-NEXT: (x: A)
 // BAR_INIT_0-NEXT: (x: B)
+// BAR_INIT_0-NEXT: .foo(self: Bar)
 // BAR_INIT_0-NEXT: .self
 
 extension Bar {

test/SourceKit/CodeComplete/complete_requestlimit.swift

@@ -32,6 +32,8 @@ func test001() {
 // TOP_LEVEL_0_ALL-NEXT: z
 // TOP_LEVEL_0_ALL-NEXT: A
 // TOP_LEVEL_0_ALL-NEXT: B
+// TOP_LEVEL_0_ALL-NEXT: C
+// TOP_LEVEL_0_ALL-NEXT: D
 // TOP_LEVEL_0_ALL-NEXT: test
 
 // TOP_LEVEL_0_3: let

test/SourceKit/CodeComplete/complete_sort_order.swift

@@ -190,7 +190,7 @@ func test6() {
 // VOID_1_RAW-NEXT: key.sourcetext: "foo1()",
 // VOID_1_RAW-NEXT: key.description: "foo1()",
 // VOID_1_RAW-NEXT: key.typename: "Void",
-// VOID_1_RAW-NEXT: key.context: source.codecompletion.context.thismodule,
+// VOID_1_RAW-NEXT: key.context: source.codecompletion.context.local,
 // VOID_1_RAW-NEXT: key.num_bytes_to_erase: 0,
 // VOID_1_RAW-NEXT: key.not_recommended: 1,
 
@@ -205,8 +205,8 @@ func test7() {
   #^CASE_0,caseSensitiveCheck,CaseSensitiveCheck^#
 }
 // CASE_0: Results for filterText: caseSensitiveCheck [
-// CASE_0: caseSensitiveCheck
 // CASE_0: CaseSensitiveCheck
+// CASE_0: caseSensitiveCheck
 // CASE_0: caseSensitiveCheck.
 // CASE_0: ]
 // CASE_0: Results for filterText: CaseSensitiveCheck [

tools/SourceKit/tools/complete-test/complete-test.cpp

@@ -392,7 +392,6 @@ removeCodeCompletionTokens(StringRef Input, StringRef TokenName,
     if (match[1].str() != TokenName)
       continue;
     *CompletionOffset = CleanFile.size();
-    CleanFile.push_back('\0');
     if (match.size() == 2 || !match[2].matched)
       continue;