smartxml: Fixing single text node unwrap
authorAleksander Łukasz <aleksander.lukasz@nowoczesnapolska.org.pl>
Tue, 19 Nov 2013 14:29:51 +0000 (15:29 +0100)
committerAleksander Łukasz <aleksander.lukasz@nowoczesnapolska.org.pl>
Tue, 19 Nov 2013 14:29:51 +0000 (15:29 +0100)
src/smartxml/smartxml.js
src/smartxml/smartxml.test.js

index 40f3aad..b150cde 100644 (file)
@@ -100,6 +100,22 @@ $.extend(DocumentNode.prototype, {
         return node;
     },
 
         return node;
     },
 
+    /**
+    * Removes parent of a node if node has no siblings.
+    */
+    unwrap: function() {
+        if(this.isRoot()) {
+            return;
+        }
+        var parent = this.parent(),
+            grandParent;
+        if(parent.contents().length === 1) {
+            grandParent = parent.parent();
+            parent.unwrapContent();
+            return grandParent;
+        }
+    },
+
     triggerChangeEvent: function(type, metaData) {
         var event = new events.ChangeEvent(type, $.extend({node: this}, metaData || {}));
         if(type === 'nodeDetached' || this.document.containsNode(event.meta.node)) {
     triggerChangeEvent: function(type, metaData) {
         var event = new events.ChangeEvent(type, $.extend({node: this}, metaData || {}));
         if(type === 'nodeDetached' || this.document.containsNode(event.meta.node)) {
@@ -275,22 +291,6 @@ $.extend(ElementNode.prototype, {
         };
     },
 
         };
     },
 
-    /**
-    * Removes parent of a node if node has no siblings.
-    */
-    unwrap: function() {
-        if(this.isRoot()) {
-            return;
-        }
-        var parent = this.parent(),
-            grandParent;
-        if(parent.contents().length === 1) {
-            grandParent = parent.parent();
-            parent.unwrapContent();
-            return grandParent;
-        }
-    },
-
     wrapText: function(params) {
         return this.document._wrapText(_.extend({inside: this}, params));
     },
     wrapText: function(params) {
         return this.document._wrapText(_.extend({inside: this}, params));
     },
index 67ada78..ccaf0ef 100644 (file)
@@ -313,7 +313,7 @@ describe('smartxml', function() {
             expect(node.contents()[2].getText()).to.equal(' a cat!');
         });
 
             expect(node.contents()[2].getText()).to.equal(' a cat!');
         });
 
-        it('unwrap single node from its parent', function() {
+        it('unwrap single element node from its parent', function() {
             var doc = getDocumentFromXML('<div><a><b></b></a></div>'),
                 div = doc.root,
                 a = div.contents()[0],
             var doc = getDocumentFromXML('<div><a><b></b></a></div>'),
                 div = doc.root,
                 a = div.contents()[0],
@@ -326,6 +326,19 @@ describe('smartxml', function() {
             expect(div.contents()[0].sameNode(b)).to.equal(true, 'node got unwrapped');
         });
 
             expect(div.contents()[0].sameNode(b)).to.equal(true, 'node got unwrapped');
         });
 
+        it('unwrap single text node from its parent', function() {
+            var doc = getDocumentFromXML('<div>Some <span>text</span>!</div>'),
+                div = doc.root,
+                span = div.contents()[1],
+                text = span.contents()[0];
+
+            var parent = text.unwrap();
+
+            expect(parent.sameNode(div)).to.equal(true, 'returns new parent');
+            expect(div.contents()).to.have.length(1, 'root contains only one node');
+            expect(div.contents()[0].getText()).to.equal('Some text!');
+        });
+
         describe('Wrapping text', function() {
             it('wraps text spanning multiple sibling TextNodes', function() {
                 var section = elementNodeFromXML('<section>Alice has a <span>small</span> cat</section>'),
         describe('Wrapping text', function() {
             it('wraps text spanning multiple sibling TextNodes', function() {
                 var section = elementNodeFromXML('<section>Alice has a <span>small</span> cat</section>'),