break(linkedlist) rename removeAt for remove

Author: amejiarosario

lib/data-structures/linked-lists/linked-list.js

@@ -123,7 +123,7 @@ class LinkedList {
    * Runtime: O(n)
    * @param {any} index
    */
-  removeAt(index) {
+  remove(index = 0) {
     if(index === 0) {
       return this.removeFirst();
     }
@@ -172,7 +172,7 @@ class LinkedList {
 
 // Aliases
 LinkedList.prototype.push = LinkedList.prototype.addLast;
-LinkedList.prototype.remove = LinkedList.prototype.pop = LinkedList.prototype.removeLast;
+LinkedList.prototype.pop = LinkedList.prototype.removeLast;
 LinkedList.prototype.unshift = LinkedList.prototype.addFirst;
 LinkedList.prototype.shift = LinkedList.prototype.removeFirst;
 LinkedList.prototype.search = LinkedList.prototype.contains;

lib/data-structures/linked-lists/linked-list.spec.js

@@ -107,25 +107,25 @@ describe('LinkedList', function () {
       });
     });
 
-    describe('#removeAt', () => {
+    describe('#remove', () => {
       it('should remove element and return value', () => {
         expect(linkedList.size).toBe(2);
-        expect(linkedList.removeAt(1)).toBe('found');
+        expect(linkedList.remove(1)).toBe('found');
         expect(linkedList.size).toBe(1);
 
-        expect(linkedList.removeAt(0)).toBe(0);
+        expect(linkedList.remove(0)).toBe(0);
         expect(linkedList.size).toBe(0);
       });
 
       it('should return undefined if not found', () => {
-        expect(linkedList.removeAt(2)).toBe(undefined);
-        expect(linkedList.removeAt(-2)).toBe(undefined);
+        expect(linkedList.remove(2)).toBe(undefined);
+        expect(linkedList.remove(-2)).toBe(undefined);
       });
 
       it('should update size, last and first', () => {
-        expect(linkedList.removeAt(0)).toBe(0);
-        expect(linkedList.removeAt(0)).toBe('found');
-        expect(linkedList.removeAt(0)).toBe(undefined);
+        expect(linkedList.remove(0)).toBe(0);
+        expect(linkedList.remove(0)).toBe('found');
+        expect(linkedList.remove(0)).toBe(undefined);
         expect(linkedList.size).toBe(0);
         expect(linkedList.first).toBe(null);
         expect(linkedList.last).toBe(null);