Some methods that are break the chaining fixed

Author: bcinarli

src/attr.js

@@ -4,7 +4,9 @@
 
 _.extend.attr = function (attr, value) {
     if (value) {
-        return this.el.forEach(item => item[attr] = value);
+        this.el.forEach(item => item[attr] = value);
+
+        return this;
     }
 
     return this.el[0][attr];

src/css.js

@@ -53,7 +53,9 @@ _.extend.css = function (prop, value) {
     options = maybePropIsObject(prop) || options;
 
     if (options.values.length > 0) {
-        return this.el.map(item => options.properties.forEach((p, i) => item.style[p] = options.values[i]));
+        this.el.map(item => options.properties.forEach((p, i) => item.style[p] = options.values[i]));
+
+        return this;
     }
 
     // if no value set then we are asking for getting the values of properties

src/data.js

@@ -13,7 +13,5 @@ _.extend.data = function (name, value) {
         return this;
     }
 
-    else {
-        return item.dataset[domName];
-    }
+    return item.dataset[domName];
 };

src/manipulation.js

@@ -6,34 +6,48 @@
 /* global insertBefore */
 
 _.extend.empty = function () {
-    return this.el.forEach(item => item.innerHTML = '');
+    this.el.forEach(item => item.innerHTML = '');
+
+    return this;
 };
 
 _.extend.remove = function () {
-    return this.el.forEach(item => item.parentNode.removeChild(item));
+    this.el.forEach(item => item.parentNode.removeChild(item));
+
+    return this;
 };
 
 _.extend.append = function (stringOrObject) {
-    return this.el.forEach(item => item.appendChild(getInsertableElement(stringOrObject)));
+    this.el.forEach(item => item.appendChild(getInsertableElement(stringOrObject)));
+
+    return this;
 };
 
 _.extend.prepend = function (stringOrObject) {
-    return this.el.forEach(
+    this.el.forEach(
         item => insertBefore(stringOrObject, item, 'firstChild', false));
+
+    return this;
 };
 
 _.extend.after = function (stringOrObject) {
-    return this.el.forEach(
+    this.el.forEach(
         item => insertBefore(stringOrObject, item, 'nextSibling', true));
+
+    return this;
 };
 
 _.extend.before = function (stringOrObject) {
-    return this.el.forEach(
+    this.el.forEach(
         item => insertBefore(stringOrObject, item, 'self', true));
+
+    return this;
 };
 
 _.extend.replaceWith = function (stringOrObject) {
-    return this.el.map(
+    this.el.map(
         item => item.parentNode.replaceChild(getInsertableElement(stringOrObject), item)
     );
+
+    return this;
 };

test/wrap-test.js

@@ -31,8 +31,8 @@ describe('Element Wrapping', () => {
 
             let newParent = single['0'].parentNode;
 
-            expect(firstParent.matches('#wrap')).to.be.true;
-            expect(newParent.matches('#wrap')).to.be.false;
+            expect(firstParent.matches('#wrap')).to.be.equal(true);
+            expect(newParent.matches('#wrap')).to.be.equal(false);
         });
 
         it('should wrap and replace child if the element is the only child', () => {
@@ -42,14 +42,13 @@ describe('Element Wrapping', () => {
 
             singleInner.wrap('<div />');
 
-            let newParent = singleInner['0'].parentNode;
             let newLength = parent.childNodes.length;
             let newChild = parent.firstChild.classList.contains('wrap-single-inner');
 
             expect(length).to.be.equal(1);
-            expect(child).to.be.true;
+            expect(child).to.be.equal(true);
             expect(newLength).to.be.equal(1);
-            expect(newChild).to.be.false;
+            expect(newChild).to.be.equal(false);
         });
 
         it('should wrap element with "div" has a class value "new-wrap" and id value "wrap-id"', () => {
@@ -59,9 +58,9 @@ describe('Element Wrapping', () => {
 
             let newParent = singleAttr['0'].parentNode;
 
-            expect(firstParent.matches('#wrap')).to.be.true;
-            expect(newParent.matches('#wrap-id')).to.be.true;
-            expect(newParent.classList.contains('new-wrap')).to.be.true;
+            expect(firstParent.matches('#wrap')).to.be.equal(true);
+            expect(newParent.matches('#wrap-id')).to.be.equal(true);
+            expect(newParent.classList.contains('new-wrap')).to.be.equal(true);
         });
     });