Skip to content

Commit 83ae768

Browse files
Simplify some test setup.
1 parent bf37dfb commit 83ae768

File tree

1 file changed

+16
-14
lines changed

1 file changed

+16
-14
lines changed

src/test/common/platform/filesystem.unit.test.ts

Lines changed: 16 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -382,23 +382,25 @@ suite('Raw FileSystem', () => {
382382
if (ft === FileType.File) {
383383
stat.setup(s => s.isFile())
384384
.returns(() => true);
385+
} else if (ft === FileType.Directory) {
386+
stat.setup(s => s.isFile())
387+
.returns(() => false);
388+
stat.setup(s => s.isDirectory())
389+
.returns(() => true);
390+
} else if (ft === FileType.SymbolicLink) {
391+
stat.setup(s => s.isFile())
392+
.returns(() => false);
393+
stat.setup(s => s.isDirectory())
394+
.returns(() => false);
395+
stat.setup(s => s.isSymbolicLink())
396+
.returns(() => true);
385397
} else {
386398
stat.setup(s => s.isFile())
387399
.returns(() => false);
388-
if (ft === FileType.Directory) {
389-
stat.setup(s => s.isDirectory())
390-
.returns(() => true);
391-
} else {
392-
stat.setup(s => s.isDirectory())
393-
.returns(() => false);
394-
if (ft === FileType.SymbolicLink) {
395-
stat.setup(s => s.isSymbolicLink())
396-
.returns(() => true);
397-
} else {
398-
stat.setup(s => s.isSymbolicLink())
399-
.returns(() => false);
400-
}
401-
}
400+
stat.setup(s => s.isDirectory())
401+
.returns(() => false);
402+
stat.setup(s => s.isSymbolicLink())
403+
.returns(() => false);
402404
}
403405
// This is necessary because passing "stat.object" to
404406
// Promise.resolve() triggers the lookup.

0 commit comments

Comments
 (0)