Fixed a bug. Parsing was not in a try block

Author: craftybones

src/machine.js

@@ -30,10 +30,10 @@ class Machine {
  this.lines = new Lines();
  let instructions = program.split(/\n/);
  instructions.forEach((instruction, index) => {
- let { lineNumber, command, args, nonExecutableLine } = parse(instruction);
- if (nonExecutableLine) return;
  let line;
  try {
+ let { lineNumber, command, args, nonExecutableLine } = parse(instruction);
+ if (nonExecutableLine) return;
  line = Line.create(lineNumber, command, args, index + 1, instruction);
  } catch (e) {
  e.setLineNumber(index + 1);

test/testMachine.js

@@ -17,6 +17,14 @@ describe('Machine loading', function() {
  const program = ['10 STAR'];
  assert.throws(() => machine.load(stitch(program)));
  });
+ 
+ it('should throw an exception when there is a parse error in the arguments',() => {
+ const machine = new Machine();
+ const missingArgumentProg = ['10 MOV A,'];
+ const missingQuoteProg = ['10 PRN "HELLO'];
+ assert.throws(() => machine.load(stitch(missingArgumentProg)), InvalidInstructionException);
+ assert.throws(() => machine.load(stitch(missingQuoteProg)), InvalidInstructionException);
+ });
 
  it('should ignore empty lines', function() {
  const machine = new Machine();

test/testParser.js

@@ -71,5 +71,6 @@ describe('should not parse illegal forms', function() {
  assert.throws(() => parse('10 ADD A,-2'), InvalidInstructionException);
  assert.throws(() => parse('10 ADD A -2'), InvalidInstructionException);
  assert.throws(() => parse('-10 ADD A -2'), InvalidInstructionException);
+ assert.throws(() => parse('10 MOV A,'), InvalidInstructionException);
  });
 });