Added additional tests for AbstractCommand.

jasmine/commands/AbstractCommand.js

@@ -2,6 +2,10 @@
 
 class AbstractCommand {
     constructor (caller, switches, args) {
+        if (this.constructor === AbstractCommand) {
+            throw new TypeError("Cannot construct Abstract instances directly.");
+        }
+
         this.caller = caller;
         if (switches) this.switches = switches.split('/');
         if (args) this.args = args.split(' ');
@@ -10,9 +14,7 @@ class AbstractCommand {
     static get command () { throw new Error('Command must define syntax'); }
     static get aliases () { return []; }
 
-    execute () {
-        throw new Error('Command must implement execute method.');
-    }
+    execute () {}
 }
 
 module.exports = AbstractCommand;

jasmine/test/Session.tests.js

@@ -5,17 +5,9 @@ const net = require('net');
 const proxyquire = require('proxyquire');
 const queue = require('jasmine/Queue').instance;
 
-const AbstractCommand = require('jasmine/commands/AbstractCommand');
+const TestCommand = require('jasmine/test/mock/commands/TestCommand');
 const Mitm = require('mitm');
 
-class TestCommand extends AbstractCommand {
-    static get command () { return 'test'; }
-    static get aliases () { return ['foo']; }
-    execute () {
-        process.emit('executed TestCommand');
-    }
-}
-
 describe("jasmine.Session", function () {
     let mitm, serverSocket, clientSocket;
     const Session = proxyquire('jasmine/Session', {});

jasmine/test/commands/AbstractCommand.tests.js

@@ -0,0 +1,29 @@
+'use strict';
+
+const assert = require('assert');
+const proxyquire = require('proxyquire');
+const TestCommand = require('jasmine/test/mock/commands/TestCommand');
+
+describe("jasmine.commands.AbstractCommand", function () {
+    const AbstractCommand = proxyquire('jasmine/commands/AbstractCommand', {});
+
+    it('should not allow you to instantiate it directly', function () {
+        assert.throws(function () {
+            new AbstractCommand;
+        }, TypeError);
+    });
+
+    it('should allow extending classes to be instantiated', function () {
+        let instance;
+        let caller = {};
+        let switches = 'foo/bar/baz';
+        let args = 'foo bar baz';
+        assert.doesNotThrow(function () {
+            instance = new TestCommand(caller, switches, args);
+        });
+
+        assert.strictEqual(instance.caller, caller);
+        assert.deepEqual(instance.switches, switches.split('/'));
+        assert.deepEqual(instance.args, args.split(' '));
+    });
+});

jasmine/test/mock/commands/TestCommand.js

@@ -0,0 +1,13 @@
+'use strict';
+
+const AbstractCommand = require('jasmine/commands/AbstractCommand');
+
+class TestCommand extends AbstractCommand {
+    static get command () { return 'test'; }
+    static get aliases () { return ['foo']; }
+    execute () {
+        process.emit('executed TestCommand');
+    }
+}
+
+module.exports = TestCommand;