dcmjs - npm Package Compare versions

Comparing version 0.32.0 to 0.33.0

package.json

 {
   "name": "dcmjs",
-  "version": "0.32.0",
+  "version": "0.33.0",
   "description": "Javascript implementation of DICOM manipulation",
@@ -5,0 +5,0 @@ "main": "build/dcmjs.js",

test/testUtils.js

@@ -58,5 +58,5 @@ import fs from "fs";
 async function getZippedTestDataset(url, filename, unpackDirectory) {
-    var dir = ensureTestDataDir();
-    var targetPath = path.join(dir, filename);
-    var unpackPath = path.join(dir, unpackDirectory);
+    const dir = ensureTestDataDir();
+    const targetPath = path.join(dir, filename);
+    const unpackPath = path.join(dir, unpackDirectory);
     if (!fs.existsSync(unpackPath)) {
@@ -69,8 +69,17 @@ await downloadToFile(url, targetPath);
 
+/**
+ * Stores the required downloads to prevent async reading before download completed.
+ */
+const asyncDownloadMap = new Map();
+
 async function getTestDataset(url, filename) {
-    var dir = ensureTestDataDir();
-    var targetPath = path.join(dir, filename);
-    if (!fs.existsSync(targetPath)) {
-        await downloadToFile(url, targetPath);
+    const dir = ensureTestDataDir();
+    const targetPath = path.join(dir, filename);
+    let filePromise = asyncDownloadMap.get(targetPath);
+    if (!filePromise && !fs.existsSync(targetPath)) {
+        filePromise = downloadToFile(url, targetPath);
+        asyncDownloadMap.set(targetPath,filePromise);
     }
+    // This returns immediately if filePromise is undefined - eg if the file already downloaded.
+    await filePromise;
     return targetPath;
@@ -77,0 +86,0 @@ }

New alerts

URL strings

Supply chain risk

Package contains fragments of external URLs or IP addresses, which the package may be accessing at runtime.

Found 1 instance in 1 package

Fixed alerts

URL strings

Supply chain risk

Package contains fragments of external URLs or IP addresses, which the package may be accessing at runtime.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

12432389

0

Lines of code

58087

0.01%

No dependency changes