Clean up AtRule nodes on insert hook

ctxhou · Dec 15, 2015 · b5a4e89 · b5a4e89
1 parent 74ea133
commit b5a4e89
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 10 deletions.
diff --git a/lib/at-rule.es6 b/lib/at-rule.es6
@@ -19,16 +19,6 @@ export default class AtRule extends Container {
         return super.prepend(...children);
     }
 
-    insertBefore(exist, add) {
-        if ( !this.nodes ) this.nodes = [];
-        return super.insertBefore(exist, add);
-    }
-
-    insertAfter(exist, add) {
-        if ( !this.nodes ) this.nodes = [];
-        return super.insertAfter(exist, add);
-    }
-
     /* istanbul ignore next */
     get afterName() {
         warnOnce('AtRule#afterName was deprecated. Use AtRule#raws.afterName');

diff --git a/test/at-rule.es6 b/test/at-rule.es6
@@ -14,6 +14,30 @@ describe('AtRule', () => {
         expect(rule.toString()).to.eql('@encoding "utf-8"');
     });
 
+    describe('prepend()', () => {
+
+        it('creates nodes property', () => {
+            let rule = new AtRule();
+            expect(rule.nodes).to.not.exist;
+
+            rule.prepend('color: black');
+            expect(rule.nodes).to.have.length(1);
+        });
+
+    });
+
+    describe('append()', () => {
+
+        it('creates nodes property', () => {
+            let rule = new AtRule();
+            expect(rule.nodes).to.not.exist;
+
+            rule.append('color: black');
+            expect(rule.nodes).to.have.length(1);
+        });
+
+    });
+
     describe('toString()', () => {
 
         it('inserts default spaces', () => {