linting & cleaning

[fnpeditor.git] / src / smartxml / smartxml.test.js

diff --git a/src/smartxml/smartxml.test.js b/src/smartxml/smartxml.test.js
index 7fd408e..86f16de 100644 (file)
--- a/src/smartxml/smartxml.test.js
+++ b/src/smartxml/smartxml.test.js
@@ -77,28 +77,49 @@ describe('smartxml', function() {
         });
 
         describe('Changing node tag', function() {
+
             it('can change tag name', function() {
-                var doc = getDocumentFromXML('<div><header></header></div>'),
-                    header = doc.root.contents()[0];
-                header.setTag('span');
-                expect(header.getTagName()).to.equal('span');
-                
+                var node = elementNodeFromXML('<div></div>');
+                node.setTag('span');
+                expect(node.getTagName()).to.equal('span');
+            });
+
+            describe('Implementation specific expectations', function() {
                 // DOM specifies ElementNode tag as a read-only property, so
                 // changing it in a seamless way is a little bit tricky. For this reason
                 // the folowing expectations are required, despite the fact that they actually are
-                // motivated by implemetation detail.
-                expect(header.parent().sameNode(doc.root)).to.equal(true, 'ensure we stayed in a document');
+                // motivated by implemetation details.
+
+                it('keeps node in the document', function() {
+                    var doc = getDocumentFromXML('<div><header></header></div>'),
+                        header = doc.root.contents()[0];
+                    header.setTag('span');
+                    expect(header.parent().sameNode(doc.root)).to.be.true;
+                });
+                it('keeps custom data', function() {
+                    var node = elementNodeFromXML('<div></div>');
+
+                    node.setData('key', 'value');
+                    node.setTag('header');
+                    
+                    expect(node.getTagName()).to.equal('header');
+                    expect(node.getData()).to.eql({key: 'value'});
+                });
+
+                it('can change document root tag name', function() {
+                    var doc = getDocumentFromXML('<div></div>');
+                    doc.root.setTag('span');
+                    expect(doc.root.getTagName()).to.equal('span');
+                });
+
+                it('keeps contents', function() {
+                    var node = elementNodeFromXML('<div><div></div></div>');
+                    node.setTag('header');
+                    expect(node.contents()).to.have.length(1);
+                });
             });
 
-            it('keeps custom data', function() {
-                var node = elementNodeFromXML('<div></div>');
 
-                node.setData('key', 'value');
-                node.setTag('header');
-                
-                expect(node.getTagName()).to.equal('header');
-                expect(node.getData()).to.eql({key: 'value'});
-            });
         });
     });
 
@@ -155,6 +176,19 @@ describe('smartxml', function() {
             
             expect(input.isEqualNode(output)).to.be.true;
         });
+
+        it('keeps entities intact', function() {
+            var xmlIn = '<section>&lt; &gt;</section>',
+                doc = getDocumentFromXML(xmlIn),
+                xmlOut = doc.toXML();
+            expect(xmlOut).to.equal(xmlIn);
+        });
+        it('keeps entities intact when they form html/xml', function() {
+            var xmlIn = '<section>&lt;abc&gt;</section>',
+                doc = getDocumentFromXML(xmlIn),
+                xmlOut = doc.toXML();
+            expect(xmlOut).to.equal(xmlIn);
+        });
     });
 
 });