fix(doctor): improve error handling in system stack checks

closes #260 - handle execa errors specifically in the checks for nginx and systemd
TryGhost · Jul 4, 2017 · 3f508a0 · 3f508a0
1 parent 19d79e7
commit 3f508a0
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/lib/commands/doctor/checks/setup.js b/lib/commands/doctor/checks/setup.js
@@ -26,14 +26,12 @@ module.exports = [{
 
                 return context.ui.listr([{
                     title: 'Systemd',
-                    task: (ctx) => execa.shell('dpkg -l | grep systemd').then(() => {
-                        ctx.systemd = true;
-                    })
+                    task: () => execa.shell('dpkg -l | grep systemd')
+                        .catch(() => Promise.reject(new errors.SystemError('systemd is not installed')))
                 }, {
                     title: 'Nginx',
-                    task: (ctx) => execa.shell('dpkg -l | grep nginx').then(() => {
-                        ctx.nginx = true;
-                    })
+                    task: () => execa.shell('dpkg -l | grep nginx')
+                        .catch(() => Promise.reject(new errors.SystemError('nginx is not installed')))
                 }], context, {concurrent: true, renderer: context.ui.verbose ? 'verbose' : 'silent', exitOnError: false})
             });
         }