fix(ts-morph): surface read file errors instead of ignoring them

dsherret · Nov 21, 2021 · 760fe8c · 760fe8c
1 parent 9a04d32
commit 760fe8c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 13 deletions.
diff --git a/deno/ts_morph.js b/deno/ts_morph.js
@@ -19132,12 +19132,7 @@ class CompilerFactory {
         filePath = this.context.fileSystemWrapper.getStandardizedAbsolutePath(filePath);
         let sourceFile = this.sourceFileCacheByFilePath.get(filePath);
         if (sourceFile == null) {
-            let fileText;
-            try {
-                fileText = this.context.fileSystemWrapper.readFileSync(filePath, this.context.getEncoding());
-            }
-            catch (_a) {
-            }
+            const fileText = this.context.fileSystemWrapper.readFileIfExistsSync(filePath, this.context.getEncoding());
             if (fileText != null) {
                 this.context.logger.log(`Loaded file: ${filePath}`);
                 sourceFile = this.createSourceFileFromTextInternal(filePath, fileText, options);

diff --git a/packages/ts-morph/src/factories/CompilerFactory.ts b/packages/ts-morph/src/factories/CompilerFactory.ts
@@ -233,13 +233,7 @@ export class CompilerFactory {
     filePath = this.context.fileSystemWrapper.getStandardizedAbsolutePath(filePath);
     let sourceFile = this.sourceFileCacheByFilePath.get(filePath);
     if (sourceFile == null) {
-      let fileText: string | undefined;
-      try {
-        fileText = this.context.fileSystemWrapper.readFileSync(filePath, this.context.getEncoding());
-      } catch {
-        // ignore
-      }
-
+      const fileText = this.context.fileSystemWrapper.readFileIfExistsSync(filePath, this.context.getEncoding());
       if (fileText != null) {
         this.context.logger.log(`Loaded file: ${filePath}`);
         sourceFile = this.createSourceFileFromTextInternal(filePath, fileText, options);