using mocha by default

Author: amejiarosario

package.json

@@ -4,18 +4,18 @@
   "description": "",
   "main": "index.js",
   "dependencies": {
-    "jasmine": "2.5.2",
-    "jasmine-spec-reporter": "2.7.0",
-    "karma": "1.3.0",
-    "karma-mocha": "1.3.0",
-    "karma-chrome-launcher": "2.0.0"
   },
   "devDependencies": {
     "jasmine-spec-reporter": "2.7.0",
-    "mocha": "3.2.0"
+    "mocha": "3.2.0",
+    "chai": "3.5.0",
+    "jasmine": "2.5.2",
+    "karma": "1.3.0",
+    "karma-chrome-launcher": "2.0.0",
+    "karma-mocha": "1.3.0"
   },
   "scripts": {
-    "test": "jasmine JASMINE_CONFIG_PATH=jasmine.json # node jasmine-runner.js"
+    "test": "mocha src/**/*spec.js # jasmine JASMINE_CONFIG_PATH=jasmine.json # node jasmine-runner.js"
   },
   "keywords": [],
   "author": "",

src/linked-list/linkedlist.js

@@ -1,6 +1,8 @@
 const Node = require('./node');
+
 /**
  * Singly LinkedList data structure
+ * Trying to keep this interface https://docs.oracle.com/javase/7/docs/api/java/util/LinkedList.html
  */
 class LinkedList {
   constructor() {

src/linked-list/linkedlist.spec.js

@@ -1,3 +1,4 @@
+const expect = require('chai').expect;
 const LinkedList = require('./linkedlist');
 
 describe('LinkedList', function () {
@@ -13,11 +14,11 @@ describe('LinkedList', function () {
     });
 
     it('should add elements to the tail and count size', function () {
-      expect(list.toString()).toEqual('1 -> 2 -> 3 -> 4');
+      expect(list.toString()).to.equal('1 -> 2 -> 3 -> 4');
     });
 
     it('should have a size of 4', function () {
-      expect(list.size()).toBe(4);
+      expect(list.size()).to.equal(4);
     });
   });
 
@@ -31,11 +32,11 @@ describe('LinkedList', function () {
     });
 
     it('should add elements to the tail and count size', function () {
-      expect(list.toString()).toEqual('4 -> 3 -> 2 -> 1');
+      expect(list.toString()).to.equal('4 -> 3 -> 2 -> 1');
     });
 
     it('should have a size of 4', function () {
-      expect(list.size()).toBe(4);
+      expect(list.size()).to.equal(4);
     });
   });
 
@@ -49,11 +50,11 @@ describe('LinkedList', function () {
     });
 
     it('should add elements to the tail and count size', function () {
-      expect(list.toString()).toEqual('4 -> 3 -> 2 -> 1');
+      expect(list.toString()).to.equal('4 -> 3 -> 2 -> 1');
     });
 
     it('should have a size of 4', function () {
-      expect(list.size()).toBe(4);
+      expect(list.size()).to.equal(4);
     });
   });
 
@@ -68,20 +69,20 @@ describe('LinkedList', function () {
 
     it('should start with size of 3 after deleting one', function () {
       list.delete(4);
-      expect(list.size()).toBe(3);
+      expect(list.size()).to.equal(3);
     });
 
     it('should have a size of 4', function () {
       list.delete(4);
       list.delete(2);
       list.delete(1);
       list.delete(3);
-      expect(list.size()).toBe(0);
+      expect(list.size()).to.equal(0);
     });
 
     it('should not change if element not found', function () {
       list.delete(7);
-      expect(list.size()).toBe(4);
+      expect(list.size()).to.equal(4);
     });
   });
 

src/linked-list/palindrome.js

@@ -12,7 +12,7 @@ LinkedList.prototype.isPalindrome = function() {
   queue.push(i1.data);
 
   // collect element up to the middle
-  while(i2.next.next) {
+  while(i2.next && i2.next.next) {
     i1 = i1.next;
     i2 = i2.next.next;
     if(i2.next) { queue.push(i1.data); }

src/linked-list/palindrome.spec.js

@@ -1,3 +1,4 @@
+const expect = require('chai').expect;
 const LinkedList = require('./palindrome');
 
 describe('LinkedList: isPalindrome', function () {
@@ -7,8 +8,40 @@ describe('LinkedList: isPalindrome', function () {
     list = new LinkedList();
   });
 
-  it('should pass', function () {
+  it('should return true with "a"', function () {
     list.addLast('a');
-    expect(list.isPalindrome()).toBe(true);
+    expect(list.isPalindrome()).to.equal(true);
+  });
+
+  it('should return true with "aba"', function () {
+    list.addLast('a');
+    list.addLast('b');
+    list.addLast('a');
+    expect(list.isPalindrome()).to.equal(true);
+  });
+
+  it('should return true with "abba"', function () {
+    list.addLast('a');
+    list.addLast('b');
+    list.addLast('b');
+    list.addLast('a');
+    expect(list.isPalindrome()).to.equal(true);
+  });
+
+  it('should return true with "abcba"', function () {
+    list.addLast('a');
+    list.addLast('b');
+    list.addLast('c');
+    list.addLast('b');
+    list.addLast('a');
+    expect(list.isPalindrome()).to.equal(true);
+  });
+
+  it('should return false with "abcab"', function () {
+    list.addLast('a');
+    list.addLast('b');
+    list.addLast('a');
+    list.addLast('b');
+    expect(list.isPalindrome()).to.equal(false);
   });
 });