fix(doctor) skip validate config check for running instances

no issue

- skips the validate config check when the tested instance is currently running, as it would fail for testing the server port

Author: aileen

lib/commands/doctor/checks/validate-config.js

@@ -49,5 +49,10 @@ function validateConfig(ctx) {
 module.exports = {
     title: taskTitle,
     task: validateConfig,
+    skip: (ctx) => {
+        const instance = ctx.system.getInstance();
+
+        return instance.process.isRunning(instance.dir);
+    },
     category: ['start']
 }

test/unit/commands/doctor/checks/validate-config-spec.js

@@ -21,6 +21,12 @@ describe('Unit: Doctor Checks > validateConfig', function () {
         }
     });
 
+    it('skips check, when instance is currently running', function () {
+        const isRunningStub = sinon.stub().returns(true);
+        const getInstanceStub = sinon.stub().returns({process: {isRunning: isRunningStub}});
+        expect(validateConfig.skip({system: {getInstance: getInstanceStub}}), 'true if current instance is running').to.be.true;
+    });
+
     it('rejects if environment is passed and no config exists for that environment', function () {
         env = setupEnv();
         const cwdStub = sandbox.stub(process, 'cwd').returns(env.dir);

test/unit/utils/resolve-version-spec.js

@@ -46,7 +46,6 @@ describe('Unit: resolveVersion', function () {
         });
     });
 
-
     it('rejects if no versions are found', function () {
         stubYarn('{"data": []}');