Fix issue #1753 Reject promise when workbook reader is writing to tem…

…porary file stream and error occurs (#1756)

Co-authored-by: Paulius Grabauskas <paulius.grabauskas@devbridge.com>
Co-authored-by: Siemienik Pawel <hi@siemienik.com>

lib/stream/xlsx/workbook-reader.js

@@ -119,6 +119,7 @@ class WorkbookReader extends EventEmitter {
                   waitingWorkSheets.push({sheetNo, path, tempFileCleanupCallback});
 
                   const tempStream = fs.createWriteStream(path);
+                  tempStream.on('error', reject);
                   entry.pipe(tempStream);
                   return tempStream.on('finish', () => {
                     return resolve();
@@ -298,11 +299,8 @@ class WorkbookReader extends EventEmitter {
       options: this.options,
     });
 
-    const matchingRel = (this.workbookRels || []).find(
-      rel => rel.Target === `worksheets/sheet${sheetNo}.xml`
-    );
-    const matchingSheet =
-      matchingRel && (this.model.sheets || []).find(sheet => sheet.rId === matchingRel.Id);
+    const matchingRel = (this.workbookRels || []).find(rel => rel.Target === `worksheets/sheet${sheetNo}.xml`);
+    const matchingSheet = matchingRel && (this.model.sheets || []).find(sheet => sheet.rId === matchingRel.Id);
     if (matchingSheet) {
       worksheetReader.id = matchingSheet.id;
       worksheetReader.name = matchingSheet.name;