update ignore tests

microsoft · Jun 25, 2021 · edd70e3 · edd70e3
1 parent 9d7c1bd
commit edd70e3
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 7 deletions.
diff --git a/src/test/fixtures/vscodeignore/.vscodeignore b/src/test/fixtures/vscodeignore/.vscodeignore
@@ -5,4 +5,8 @@
 out/
 
 # pattern of file name
-*.log
+*.log
+
+# https://github.com/microsoft/vscode-vsce/issues/588
+foo
+!foo/bar/hello.txt
diff --git a/src/test/fixtures/vscodeignore/foo/bar/hello.txt b/src/test/fixtures/vscodeignore/foo/bar/hello.txt
@@ -0,0 +1 @@
+hi there
diff --git a/src/test/package.test.ts b/src/test/package.test.ts
@@ -150,14 +150,13 @@ describe('collect', function () {
 			});
 	});
 
-	it('should ignore content of .vscodeignore', () => {
+	it('should ignore content of .vscodeignore', async () => {
 		const cwd = fixture('vscodeignore');
+		const manifest = await readManifest(cwd);
+		const files = await collect(manifest, { cwd });
+		const names = files.map(f => f.path).sort();
 
-		return readManifest(cwd)
-			.then(manifest => collect(manifest, { cwd }))
-			.then(files => {
-				assert.equal(files.length, 3);
-			});
+		assert.deepStrictEqual(names, ['[Content_Types].xml', 'extension.vsixmanifest', 'extension/package.json']);
 	});
 
 	it('should ignore devDependencies', () => {