Revert "Fix and updated tests"

armanio123 · armanio123 · commit e1a2b9ed19a7 · 2021-08-23T16:27:51.000-07:00
This reverts commit 33829fa.
diff --git a/src/services/completions.ts b/src/services/completions.ts
@@ -416,10 +416,6 @@ namespace ts.Completions {
             }
         }
 
-        if (!isNewIdentifierLocation && (!symbols || symbols.length === 0) && keywordFilters === KeywordCompletionFilters.None) {
-            return undefined;
-        }
-
         const entries: CompletionEntry[] = [];
 
         if (isUncheckedFile(sourceFile, compilerOptions)) {
@@ -447,6 +443,10 @@ namespace ts.Completions {
             getJSCompletionEntries(sourceFile, location.pos, uniqueNames, compilerOptions.target!, entries); // TODO: GH#18217
         }
         else {
+            if (!isNewIdentifierLocation && (!symbols || symbols.length === 0) && keywordFilters === KeywordCompletionFilters.None) {
+                return undefined;
+            }
+
             getCompletionEntriesFromSymbols(
                 symbols,
                 entries,
diff --git a/tests/cases/fourslash/completionInUncheckedJSFile.ts b/tests/cases/fourslash/completionInUncheckedJSFile.ts
@@ -13,5 +13,16 @@
 
 verify.completions({
     marker: "0",
-    exact: undefined
+    includes: [
+        {
+            name: "hello",
+            sortText: completion.SortText.JavascriptIdentifiers,
+            isFromUncheckedFile: true
+        },
+        {
+            name: "goodbye",
+            sortText: completion.SortText.JavascriptIdentifiers,
+            isFromUncheckedFile: true
+        }
+    ]
 });
diff --git a/tests/cases/fourslash/getJavaScriptCompletions12.ts b/tests/cases/fourslash/getJavaScriptCompletions12.ts
@@ -26,5 +26,5 @@
 verify.completions(
     { marker: "1", includes: { name: "charCodeAt", kind: "method", kindModifiers: "declare" } },
     { marker: ["2", "3", "4"], includes: { name: "toExponential", kind: "method", kindModifiers: "declare" } },
-    { marker: "5", exact: undefined },
+    { marker: "5", includes: { name: "test1", kind: "warning", sortText: completion.SortText.JavascriptIdentifiers } },
 );
diff --git a/tests/cases/fourslash/javaScriptPrototype2.ts b/tests/cases/fourslash/javaScriptPrototype2.ts
@@ -31,4 +31,4 @@ verify.completions({ includes: { name: "toFixed", kind: "method", kindModifiers:
 goTo.marker('3');
 edit.insert('.');
 // Make sure symbols don't leak out into the constructor
-verify.completions({ exact: undefined });
+verify.completions({ includes: ["qua", "foo", "bar"].map(name => ({ name, kind: "warning", sortText: completion.SortText.JavascriptIdentifiers })) });
diff --git a/tests/cases/fourslash/server/jsdocTypedefTag2.ts b/tests/cases/fourslash/server/jsdocTypedefTag2.ts
@@ -16,6 +16,14 @@
 ////     my2.yes./*1*/
 //// }
 
+//// /**
+////  * @param {MyType} my2
+////  */
+//// function b(my2) {
+////     my2.yes./*2*/
+//// }
+
 verify.completions(
-    { marker: "1", includes: "charAt" }
+    { marker: "1", includes: "charAt" },
+    { marker: "2", excludes: "charAt" },
 );

Original file line number	Diff line number	Diff line change
`@@ -416,10 +416,6 @@ namespace ts.Completions {`
`416`	`416`	`}`
`417`	`417`	`}`
`418`	`418`
`419`		`- if (!isNewIdentifierLocation && (!symbols \|\| symbols.length === 0) && keywordFilters === KeywordCompletionFilters.None) {`
`420`		`- return undefined;`
`421`		`- }`
`422`		`-`
`423`	`419`	`const entries: CompletionEntry[] = [];`
`424`	`420`
`425`	`421`	`if (isUncheckedFile(sourceFile, compilerOptions)) {`
`@@ -447,6 +443,10 @@ namespace ts.Completions {`
`447`	`443`	`getJSCompletionEntries(sourceFile, location.pos, uniqueNames, compilerOptions.target!, entries); // TODO: GH#18217`
`448`	`444`	`}`
`449`	`445`	`else {`
	`446`	`+ if (!isNewIdentifierLocation && (!symbols \|\| symbols.length === 0) && keywordFilters === KeywordCompletionFilters.None) {`
	`447`	`+ return undefined;`
	`448`	`+ }`
	`449`	`+`
`450`	`450`	`getCompletionEntriesFromSymbols(`
`451`	`451`	`symbols,`
`452`	`452`	`entries,`