Cleaning up test cases and adding a few more

riknoll · riknoll · commit e11d5e9de632 · 2016-08-05T17:53:12.000-07:00
diff --git a/tests/cases/fourslash/completionForStringLiteralImport1.ts b/tests/cases/fourslash/completionForStringLiteralImport1.ts
@@ -1,5 +1,7 @@
 /// <reference path='fourslash.ts' />
 
+// Should define spans for replacement that appear after the last directory seperator in import statements
+
 // @typeRoots: my_typings
 
 // @Filename: test.ts
diff --git a/tests/cases/fourslash/completionForStringLiteralImport2.ts b/tests/cases/fourslash/completionForStringLiteralImport2.ts
@@ -1,5 +1,7 @@
 /// <reference path='fourslash.ts' />
 
+// Should define spans for replacement that appear after the last directory seperator in triple slash references
+
 // @typeRoots: my_typings
 
 // @Filename: test.ts
diff --git a/tests/cases/fourslash/completionForStringLiteralNonrelativeImport1.ts b/tests/cases/fourslash/completionForStringLiteralNonrelativeImport1.ts
@@ -1,5 +1,7 @@
 /// <reference path='fourslash.ts' />
 
+// Should give completions for node modules and files within those modules with ts file extensions
+
 // @Filename: tests/test0.ts
 //// import * as foo1 from "f/*import_as0*/
 //// import * as foo2 from "fake-module//*import_as1*/
diff --git a/tests/cases/fourslash/completionForStringLiteralNonrelativeImport10.ts b/tests/cases/fourslash/completionForStringLiteralNonrelativeImport10.ts
@@ -1,5 +1,7 @@
 /// <reference path='fourslash.ts' />
 
+// Should not give node module completions if classic module resolution is enabled
+
 // @moduleResolution: classic
 
 // @Filename: dir1/dir2/dir3/dir4/test0.ts
diff --git a/tests/cases/fourslash/completionForStringLiteralNonrelativeImport11.ts b/tests/cases/fourslash/completionForStringLiteralNonrelativeImport11.ts
@@ -0,0 +1,38 @@
+/// <reference path='fourslash.ts' />
+
+// Should handle nested files in folders discovered via the baseUrl compiler option
+
+// @baseUrl: tests/cases/fourslash/modules
+
+// @Filename: tests/test0.ts
+//// import * as foo1 from "subfolder//*import_as0*/
+//// import foo2 = require("subfolder//*import_equals0*/
+//// var foo3 = require("subfolder//*require0*/
+
+//// import * as foo1 from "module-from-node//*import_as1*/
+//// import foo2 = require("module-from-node//*import_equals1*/
+//// var foo3 = require("module-from-node//*require1*/
+
+// @Filename: modules/subfolder/module.ts
+//// export var x = 5;
+
+// @Filename: package.json
+//// { "dependencies": { "module-from-node": "latest" } }
+// @Filename: node_modules/module-from-node/index.ts
+//// /*module1*/
+
+
+
+const kinds = ["import_as", "import_equals", "require"];
+
+for (const kind of kinds) {
+    goTo.marker(kind + "0");
+
+    verify.importModuleCompletionListContains("module");
+    verify.not.importModuleCompletionListItemsCountIsGreaterThan(1);
+
+    goTo.marker(kind + "1");
+
+    verify.importModuleCompletionListContains("index");
+    verify.not.importModuleCompletionListItemsCountIsGreaterThan(1);
+}
diff --git a/tests/cases/fourslash/completionForStringLiteralNonrelativeImport2.ts b/tests/cases/fourslash/completionForStringLiteralNonrelativeImport2.ts
@@ -1,5 +1,7 @@
 /// <reference path='fourslash.ts' />
 
+// Should not give duplicate entries for similarly named files with different extensions
+
 // @Filename: tests/test0.ts
 //// import * as foo1 from "fake-module//*import_as0*/
 //// import foo2 = require("fake-module//*import_equals0*/
diff --git a/tests/cases/fourslash/completionForStringLiteralNonrelativeImport3.ts b/tests/cases/fourslash/completionForStringLiteralNonrelativeImport3.ts
@@ -1,4 +1,7 @@
 /// <reference path='fourslash.ts' />
+
+// Should give completions for js files in node modules when allowJs is set to true
+
 // @allowJs: true
 
 // @Filename: tests/test0.ts
diff --git a/tests/cases/fourslash/completionForStringLiteralNonrelativeImport4.ts b/tests/cases/fourslash/completionForStringLiteralNonrelativeImport4.ts
@@ -1,5 +1,7 @@
 /// <reference path='fourslash.ts' />
 
+// Should give completions for all node modules visible to the script
+
 // @Filename: dir1/dir2/dir3/dir4/test0.ts
 //// import * as foo1 from "f/*import_as0*/
 //// import foo4 = require("f/*import_equals0*/
diff --git a/tests/cases/fourslash/completionForStringLiteralNonrelativeImport5.ts b/tests/cases/fourslash/completionForStringLiteralNonrelativeImport5.ts
@@ -1,5 +1,7 @@
 /// <reference path='fourslash.ts' />
 
+// Should give completions for ambiently declared modules
+
 // @Filename: test0.ts
 //// /// <reference path="./ambientModules.d.ts" />
 //// /// <reference path="./ambientModules2.d.ts" />
diff --git a/tests/cases/fourslash/completionForStringLiteralNonrelativeImport6.ts b/tests/cases/fourslash/completionForStringLiteralNonrelativeImport6.ts
diff --git a/tests/cases/fourslash/completionForStringLiteralNonrelativeImport7.ts b/tests/cases/fourslash/completionForStringLiteralNonrelativeImport7.ts
@@ -1,4 +1,7 @@
 /// <reference path='fourslash.ts' />
+
+// Should give completions for files that are discovered via the baseUrl compiler option
+
 // @baseUrl: tests/cases/fourslash/modules
 
 // @Filename: tests/test0.ts
diff --git a/tests/cases/fourslash/completionForStringLiteralNonrelativeImport8.ts b/tests/cases/fourslash/completionForStringLiteralNonrelativeImport8.ts
@@ -1,5 +1,7 @@
 /// <reference path='fourslash.ts' />
 
+// Should give completions for modules referenced via baseUrl and paths compiler options with wildcards
+
 // @Filename: tsconfig.json
 //// {
 ////     "compilerOptions": {
diff --git a/tests/cases/fourslash/completionForStringLiteralNonrelativeImport9.ts b/tests/cases/fourslash/completionForStringLiteralNonrelativeImport9.ts
@@ -1,5 +1,7 @@
 /// <reference path='fourslash.ts' />
 
+// Should give completions for modules referenced via baseUrl and paths compiler options with explicit name mappings
+
 // @Filename: tsconfig.json
 //// {
 ////     "compilerOptions": {
diff --git a/tests/cases/fourslash/completionForStringLiteralNonrelativeImportTypings1.ts b/tests/cases/fourslash/completionForStringLiteralNonrelativeImportTypings1.ts
@@ -1,5 +1,7 @@
 /// <reference path='fourslash.ts' />
 
+// Should give completions for typings discovered via the typeRoots compiler option
+
 // @typeRoots: my_typings,my_other_typings
 
 
diff --git a/tests/cases/fourslash/completionForStringLiteralNonrelativeImportTypings2.ts b/tests/cases/fourslash/completionForStringLiteralNonrelativeImportTypings2.ts
@@ -1,5 +1,7 @@
 /// <reference path='fourslash.ts' />
 
+// Should respect the types compiler option when giving completions
+
 // @typeRoots: my_typings,my_other_typings
 // @types: module-x,module-z
 
diff --git a/tests/cases/fourslash/completionForStringLiteralNonrelativeImportTypings3.ts b/tests/cases/fourslash/completionForStringLiteralNonrelativeImportTypings3.ts
@@ -1,5 +1,7 @@
 /// <reference path='fourslash.ts' />
 
+// Should give completions for typings discovered in all visible @types directories
+
 // @Filename: subdirectory/test0.ts
 //// /// <reference types="m/*types_ref0*/" />
 //// import * as foo1 from "m/*import_as0*/
diff --git a/tests/cases/fourslash/completionForStringLiteralRelativeImport1.ts b/tests/cases/fourslash/completionForStringLiteralRelativeImport1.ts
@@ -1,5 +1,7 @@
 /// <reference path='fourslash.ts' />
 
+// Should give completions for ts files when allowJs is false
+
 // @Filename: test0.ts
 //// import * as foo1 from "./*import_as0*/
 //// import * as foo2 from ".//*import_as1*/
diff --git a/tests/cases/fourslash/completionForStringLiteralRelativeImport2.ts b/tests/cases/fourslash/completionForStringLiteralRelativeImport2.ts
@@ -1,4 +1,7 @@
 /// <reference path='fourslash.ts' />
+
+// Should give completions for ts and js files when allowJs is true
+
 // @allowJs: true
 
 // @Filename: test0.ts
diff --git a/tests/cases/fourslash/completionForStringLiteralRelativeImport3.ts b/tests/cases/fourslash/completionForStringLiteralRelativeImport3.ts
@@ -1,5 +1,7 @@
 /// <reference path='fourslash.ts' />
 
+// Should give completions for absolute paths
+
 // @Filename: tests/test0.ts
 //// import * as foo1 from "c:/tests/cases/f/*import_as0*/
 //// import * as foo2 from "c:/tests/cases/fourslash/*import_as1*/
diff --git a/tests/cases/fourslash/completionForStringLiteralRelativeImport4.ts b/tests/cases/fourslash/completionForStringLiteralRelativeImport4.ts
@@ -1,5 +1,7 @@
 /// <reference path='fourslash.ts' />
 
+// Should give completions for directories that are merged via the rootDirs compiler option
+
 // @rootDirs: sub/src1,src2
 
 // @Filename: src2/test0.ts
diff --git a/tests/cases/fourslash/completionForTripleSlashReference1.ts b/tests/cases/fourslash/completionForTripleSlashReference1.ts
@@ -1,5 +1,7 @@
 /// <reference path='fourslash.ts' />
 
+// Should give completions for relative references to ts files when allowJs is false
+
 // @Filename: test0.ts
 //// /// <reference path="/*0*/
 //// /// <reference path="./*1*/
diff --git a/tests/cases/fourslash/completionForTripleSlashReference2.ts b/tests/cases/fourslash/completionForTripleSlashReference2.ts
@@ -1,4 +1,7 @@
 /// <reference path='fourslash.ts' />
+
+// Should give completions for relative references to js and ts files when allowJs is true
+
 // @allowJs: true
 
 // @Filename: test0.ts
diff --git a/tests/cases/fourslash/completionForTripleSlashReference3.ts b/tests/cases/fourslash/completionForTripleSlashReference3.ts
@@ -1,5 +1,7 @@
 /// <reference path='fourslash.ts' />
 
+// Should give completions for absolute paths
+
 // @Filename: tests/test0.ts
 //// /// <reference path="c:/tests/cases/f/*0*/
 
diff --git a/tests/cases/fourslash/completionForTripleSlashReference4.ts b/tests/cases/fourslash/completionForTripleSlashReference4.ts
@@ -0,0 +1,43 @@
+/// <reference path='fourslash.ts' />
+
+// Should NOT give completions for directories that are merged via the rootDirs compiler option
+
+// @rootDirs: sub/src1,src2
+
+// @Filename: src2/test0.ts
+//// /// <reference path="./mo/*0*/
+
+// @Filename: src2/module0.ts
+//// export var w = 0;
+
+// @Filename: sub/src1/module1.ts
+//// export var x = 0;
+
+// @Filename: sub/src1/module2.ts
+//// export var y = 0;
+
+// @Filename: sub/src1/more/module3.ts
+//// export var z = 0;
+
+
+// @Filename: f1.ts
+//// /*f1*/
+// @Filename: f2.tsx
+//// /*f2*/
+// @Filename: folder/f1.ts
+//// /*subf1*/
+// @Filename: f3.js
+//// /*f3*/
+// @Filename: f4.jsx
+//// /*f4*/
+// @Filename: e1.ts
+//// /*e1*/
+// @Filename: e2.js
+//// /*e2*/
+
+
+goTo.marker("0");
+
+verify.importModuleCompletionListContains("module0.ts");
+
+verify.not.importModuleCompletionListItemsCountIsGreaterThan(1);
