Fixes a few bugs affecting the tests. (#92)

GoogleChromeLabs · Jun 27, 2017 · e7e3e13 · e7e3e13
1 parent 7540da3
commit e7e3e13
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 11 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/download-manager.js b/src/download-manager.js
@@ -250,7 +250,8 @@ class DownloadManager {
 
               fse.copySync(
                 path.join(mountedPath, chromeOSXAppName),
-                path.join(installDir, 'chrome', release, chromeOSXAppName)
+                path.join(installDir, 'chrome', release, chromeOSXAppName),
+                {dereference: true}
               );
 
               dmg.unmount(mountedPath, (err) => {
@@ -267,7 +268,9 @@ class DownloadManager {
       }
     })
     .then((filePath) => {
-      return del(filePath, {force: true});
+      if (filePath) {
+        return del(filePath, {force: true});
+      }
     });
   }
 
@@ -375,7 +378,8 @@ class DownloadManager {
 
             fse.copySync(
               path.join(mountedPath, firefoxMacApp),
-              path.join(installDir, 'firefox', release, firefoxMacApp)
+              path.join(installDir, 'firefox', release, firefoxMacApp),
+              {dereference: true}
             );
 
             dmg.unmount(mountedPath, (err) => {
@@ -392,7 +396,9 @@ class DownloadManager {
       throw new Error('Unable to handle downloaded file: ', downloadUrl);
     })
     .then((filePath) => {
-      return del(filePath, {force: true});
+      if (filePath) {
+        return del(filePath, {force: true});
+      }
     });
   }
 
@@ -608,7 +614,9 @@ class DownloadManager {
       }
     })
     .then((filePath) => {
-      return del(filePath, {force: true});
+      if (filePath) {
+        return del(filePath, {force: true});
+      }
     });
   }
 }