fix: throw when expected file is ENOENT (#1062)

forcedotcom · Aug 3, 2023 · ad3f441 · ad3f441
1 parent 2001a0b
commit ad3f441
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/resolve/treeContainers.ts b/src/resolve/treeContainers.ts
@@ -113,6 +113,9 @@ export class NodeFSTreeContainer extends TreeContainer {
   }
 
   public stream(fsPath: SourcePath): Readable {
+    if (!this.exists(fsPath)) {
+      throw new Error(`File not found: ${fsPath}`);
+    }
     return createReadStream(fsPath);
   }
 }

diff --git a/test/resolve/treeContainers.test.ts b/test/resolve/treeContainers.test.ts
@@ -117,9 +117,9 @@ describe('Tree Containers', () => {
 
     it('should use expected Node API for stream', () => {
       const readable = new Readable();
-      const createReadStreamStub = env.stub(fs, 'createReadStream');
       // @ts-ignore wants ReadStream but Readable works for testing
-      createReadStreamStub.withArgs(path).returns(readable);
+      env.stub(fs, 'createReadStream').returns(readable);
+      env.stub(fs, 'existsSync').returns(true);
       expect(tree.stream(path)).to.deep.equal(readable);
     });
   });