Added the ability to ignore numbered lines that are empty by treating them as nonExecutable

* Fixed issue of InvalidInstructionException when instruction has only line number and spaces. * Extracted checks of NonExecutable lines into function named isNonExecutable.

Author: yogiraj99

src/parse.js

@@ -1,10 +1,19 @@
 const InvalidInstructionException = require('./commands/invalidInstructionException');
-const parse = instruction => {
+
+function isNonExecutable(instruction) {
  let empty = /^\s*$/;
+ let onlyNumber = /^\s*([0-9]+)+\s*$/;
  let comment = /^\s*;.*$/;
+ return (
+ instruction.match(empty) ||
+ instruction.match(comment) ||
+ instruction.match(onlyNumber)
+ );
+}
+
+const parse = instruction => {
  let components = /^\s*([0-9]+)\s+([a-zA-Z]+)\s*(.*)*$/;
- if (instruction.match(empty) || instruction.match(comment))
- return { nonExecutableLine: true };
+ if (isNonExecutable(instruction)) return { nonExecutableLine: true };
  let matches = instruction.match(components);
  try {
  let lineNumber = matches[1];

test/testParser.js

@@ -45,6 +45,18 @@ describe('should parse all legal forms', function() {
  let { nonExecutableLine } = parse(` ; this is a comment`);
  assert.equal(true, nonExecutableLine);
  });
+ it('should parse line with only line number', function() {
+ let { nonExecutableLine } = parse(`10`);
+ assert.equal(true, nonExecutableLine);
+ });
+ it('should parse line with only line number with spaces', function() {
+ let { nonExecutableLine } = parse(`10 `);
+ assert.equal(true, nonExecutableLine);
+ });
+ it('should parse line with only line number with newLine at end', function() {
+ let { nonExecutableLine } = parse(`10 \n`);
+ assert.equal(true, nonExecutableLine);
+ });
 });
 
 describe('should not parse illegal forms', function() {