Add test cases for empty lines

src/analyze.js

@@ -11,31 +11,27 @@ const analyze = lines => {
 
   let model = {}
   for (let line of lines) {
-    try {
-      let tokens = line.split(' ')
+    let tokens = line.split(' ')
 
-      for (let i = 0; i < tokens.length; i++) {
-        let token = tokens[i]
-        let next = tokens[i + 1]
+    for (let i = 0; i < tokens.length; i++) {
+      let token = tokens[i]
+      let next = tokens[i + 1]
 
-        if (!next) {
-          token = token.replace('\n', '')
-          next = '\n'
-        }
+      if (!next) {
+        token = token.replace('\n', '')
+        next = '\n'
+      }
 
-        if (!model[token]) model[token] = { start: false, next: {} }
-        if (i === 0) model[token].start = true
+      if (!model[token]) model[token] = { start: false, next: {} }
+      if (i === 0) model[token].start = true
 
-        let count = model[token].count || 0
-        if (i === 0) count++
-        model[token].count = count
+      let count = model[token].count || 0
+      if (i === 0) count++
+      model[token].count = count
 
-        let nextCount = model[token].next[next] || 0
-        nextCount++
-        model[token].next[next] = nextCount
-      }
-    } catch (err) {
-      continue
+      let nextCount = model[token].next[next] || 0
+      nextCount++
+      model[token].next[next] = nextCount
     }
   }
 

test/tests.js

@@ -35,6 +35,22 @@ describe('analyze()', () => {
       expect(() => { analyze({ object: true }) }).to.throw(analyzeErrors.invalidInput)
     })
   })
+
+  describe('return types', () => {
+    it('should return an Object', () => {
+      expect(analyze(testInput)).to.be.an('object')
+    })
+  })
+
+  describe('handling empty lines', () => {
+    it('should not throw', () => {
+      expect(() => { analyze(['a b c', '', 'd e f']) }).to.not.throw()
+    })
+
+    it('should return an Object', () => {
+      expect(analyze(['a b c', '', 'd e f'])).to.be.an('object')
+    })
+  })
 })
 
 describe('generate()', () => {