feat(node): log discovered node version

Author: acburdine

lib/commands/doctor/checks/node-version.js

@@ -1,4 +1,3 @@
-'use strict';
 const chalk = require('chalk');
 const semver = require('semver');
 
@@ -8,19 +7,22 @@ const checkDirectoryAndAbove = require('./check-directory');
 
 const taskTitle = 'Checking system Node.js version';
 
-function nodeVersion(ctx) {
-    if (process.env.GHOST_NODE_VERSION_CHECK !== 'false' && !semver.satisfies(process.versions.node, cliPackage.engines.node)) {
-        return Promise.reject(new errors.SystemError({
+async function nodeVersion(ctx, task) {
+    const {node} = process.versions;
+    task.title = `${taskTitle} - found v${node}`;
+
+    if (process.env.GHOST_NODE_VERSION_CHECK !== 'false' && !semver.satisfies(node, cliPackage.engines.node)) {
+        throw new errors.SystemError({
             message: `${chalk.red('The version of Node.js you are using is not supported.')}
 ${chalk.gray('Supported: ')}${cliPackage.engines.node}
 ${chalk.gray('Installed: ')}${process.versions.node}
 See ${chalk.underline.blue('https://ghost.org/docs/faq/node-versions/')} for more information`,
             task: taskTitle
-        }));
+        });
     }
 
     if (ctx.local || !ctx.system.platform.linux || (ctx.argv && ctx.argv['setup-linux-user'] === false)) {
-        return Promise.resolve();
+        return;
     }
 
     return checkDirectoryAndAbove(process.argv[0], 'install node and Ghost-CLI', taskTitle);
@@ -29,5 +31,5 @@ See ${chalk.underline.blue('https://ghost.org/docs/faq/node-versions/')} for mor
 module.exports = {
     title: taskTitle,
     task: nodeVersion,
-    category: ['install', 'update']
+    category: ['install', 'update', 'start']
 };

test/unit/commands/doctor/checks/node-version-spec.js

@@ -1,4 +1,3 @@
-'use strict';
 const expect = require('chai').expect;
 const sinon = require('sinon');
 const stripAnsi = require('strip-ansi');
@@ -37,7 +36,7 @@ describe('Unit: Doctor Checks > nodeVersion', function () {
             '../../../../package': cliPackage
         }).task;
 
-        return nodeVersion().then(() => {
+        return nodeVersion({}, {}).then(() => {
             expect(false, 'error should be thrown').to.be.true;
         }).catch((error) => {
             expect(error).to.be.an.instanceof(errors.SystemError);
@@ -62,7 +61,7 @@ describe('Unit: Doctor Checks > nodeVersion', function () {
             './check-directory': checkDirectoryStub
         }).task;
 
-        return nodeVersion({local: true}).then(() => {
+        return nodeVersion({local: true}, {}).then(() => {
             expect(checkDirectoryStub.called).to.be.false;
         });
     });
@@ -81,7 +80,7 @@ describe('Unit: Doctor Checks > nodeVersion', function () {
             './check-directory': checkDirectoryStub
         }).task;
 
-        return nodeVersion({local: true}).then(() => {
+        return nodeVersion({local: true}, {}).then(() => {
             expect(checkDirectoryStub.called).to.be.false;
         });
     });
@@ -100,7 +99,7 @@ describe('Unit: Doctor Checks > nodeVersion', function () {
             './check-directory': checkDirectoryStub
         }).task;
 
-        return nodeVersion({local: true}).then(() => {
+        return nodeVersion({local: true}, {}).then(() => {
             expect(checkDirectoryStub.called).to.be.false;
         });
     });
@@ -118,7 +117,7 @@ describe('Unit: Doctor Checks > nodeVersion', function () {
             './check-directory': checkDirectoryStub
         }).task;
 
-        return nodeVersion({local: true}).then(() => {
+        return nodeVersion({local: true}, {}).then(() => {
             expect(checkDirectoryStub.called).to.be.false;
         });
     });
@@ -137,7 +136,7 @@ describe('Unit: Doctor Checks > nodeVersion', function () {
             './check-directory': checkDirectoryStub
         }).task;
 
-        return nodeVersion(ctx).then(() => {
+        return nodeVersion(ctx, {}).then(() => {
             expect(checkDirectoryStub.called).to.be.false;
         });
     });
@@ -156,7 +155,7 @@ describe('Unit: Doctor Checks > nodeVersion', function () {
             './check-directory': checkDirectoryStub
         }).task;
 
-        return nodeVersion(ctx).then(() => {
+        return nodeVersion(ctx, {}).then(() => {
             expect(checkDirectoryStub.called).to.be.false;
         });
     });
@@ -175,7 +174,7 @@ describe('Unit: Doctor Checks > nodeVersion', function () {
             './check-directory': checkDirectoryStub
         }).task;
 
-        return nodeVersion(ctx).then(() => {
+        return nodeVersion(ctx, {}).then(() => {
             expect(checkDirectoryStub.calledOnce).to.be.true;
             expect(checkDirectoryStub.calledWith(process.argv[0])).to.be.true;
         });