Make sure TarballFetcher error messages contain names of problematic files (#5964)

* Make sure name of broken file is mentioned in error message

* Pass when creating stream

This isn't required to open the file (`fd` is used), but it allows the
stream to have a useful `path` attribute.

Author: sth

__tests__/fetchers.js

@@ -313,7 +313,10 @@ test('TarballFetcher.fetch throws on truncated tar data', async () => {
     },
     await Config.create({}, reporter),
   );
-  await expect(fetcher.fetch()).rejects.toThrow(new RegExp(reporter.lang('errorExtractingTarball', '.*', '.*')));
+  await expect(fetcher.fetch()).rejects.toThrow(
+    // The "." in ".tgz" should be escaped, but that doesn't work with reporter.lang
+    new RegExp(reporter.lang('errorExtractingTarball', '.*', '.*broken-tar-data.tgz')),
+  );
 });
 
 test('TarballFetcher.fetch throws on truncated tar header', async () => {
@@ -329,5 +332,8 @@ test('TarballFetcher.fetch throws on truncated tar header', async () => {
     },
     await Config.create({}, reporter),
   );
-  await expect(fetcher.fetch()).rejects.toThrow(new RegExp(reporter.lang('errorExtractingTarball', '.*', '.*')));
+  await expect(fetcher.fetch()).rejects.toThrow(
+    // The "." in ".tgz" should be escaped, but that doesn't work with reporter.lang
+    new RegExp(reporter.lang('errorExtractingTarball', '.*', '.*broken-tar-header.tgz')),
+  );
 });

src/util/fs.js

@@ -823,7 +823,7 @@ export async function readFirstAvailableStream(
     if (tarballPath) {
       try {
         const fd = await open(tarballPath, 'r');
-        stream = fs.createReadStream('', {fd});
+        stream = fs.createReadStream(tarballPath, {fd});
         break;
       } catch (err) {
         // Try the next one