Handle options errors in server-manager

And consider that an exception thrown in "server" is a port issue
'use strict';

const Server = require('./server');

class ServersManager {
  constructor() {
    this.servers = [];
  }

  addServer(options) {
    if (this.servers.filter(s => s.port === options.port).length > 0) {
      throw new Error('This port is already in use.');
    }
    // Path and port are mandatory
    if (!options.path && !options.port) {
      throw new Error('You have to chose a directory and a port');
    }
    if (!options.path) {
      throw new Error('You have to chose a directory');
    }
    if (!options.port) {
      throw new Error('You have to chose a port'); 
    }
    try {
      let server = new Server(options);
      this.servers.push(server);
    } catch(e) {
      console.log(e);
      throw new Error('This port is already in use.');
    }
  }

  removeServer(server) {
    server.stop();
    const indexToRemove = this.servers.indexOf(server);
    if (indexToRemove > -1) {
      this.servers.splice(indexToRemove, 1);
    }
  }
}


module.exports = ServersManager;

'use strict';

const Server = require('./server');

class ServersManager {
  constructor() {
    this.servers = [];
  }

  addServer(options) {
    if (this.servers.filter(s => s.port === options.port).length > 0) {
      throw new Error('This port is already in use.');
    }
    try {
      let server = new Server(options);
      this.servers.push(server);
    } catch(_) {
      throw new Error('This port is already in use.');
    }
  }

  removeServer(server) {
    server.stop();
    const indexToRemove = this.servers.indexOf(server);
    if (indexToRemove > -1) {
      this.servers.splice(indexToRemove, 1);
    }
  }
}


module.exports = ServersManager;
