fix: better error messages for download errors

Author: aminya

src/download.ts

@@ -36,9 +36,10 @@ type DownloadResult = {
  * Downloads a file to a temporary location and returns the file path and hash
  */
 async function download(url: string, opts: DownloadOptions) {
-  const filePath = opts.path ?? join(tmpdir(), basename(url));
-  const fileName = basename(filePath);
-  const fileDir = dirname(filePath);
+  try {
+    const filePath = opts.path ?? join(tmpdir(), basename(url));
+    const fileName = basename(filePath);
+    const fileDir = dirname(filePath);
 
   await ensureDir(fileDir);
   const downloader = new DownloaderHelper(url, fileDir, {
@@ -60,7 +61,10 @@ async function download(url: string, opts: DownloadOptions) {
   // calculate hash after download is complete
   result.hash = opts.hashType !== undefined ? await calculateHash(filePath, opts.hashType) : undefined;
 
-  return result;
+    return result;
+  } catch (err) {
+    throw new Error(`Failed to download ${url}: ${err}`);
+  }
 }
 
 /**

test/download.test.ts

@@ -33,10 +33,6 @@ suite('Download Module', { timeout: 20_000 }, () => {
             expect(content).toBeTruthy();
             expect(JSON.parse(content)).toHaveProperty('fs.readFileSync');
         });
-
-        test.fails('should throw an error for non-existent files', async () => {
-            await downloadToString(`${nodeBaseUrl}/nonexistent.txt`, { timeout: 100 })
-        });
     });
 
     suite('downloadFile', () => {