smartxml: tests refactoring
authorAleksander Łukasz <aleksander.lukasz@nowoczesnapolska.org.pl>
Wed, 23 Oct 2013 10:56:28 +0000 (12:56 +0200)
committerAleksander Łukasz <aleksander.lukasz@nowoczesnapolska.org.pl>
Wed, 23 Oct 2013 10:56:28 +0000 (12:56 +0200)
src/smartxml/smartxml.test.js

index 7fd408e..938e5f7 100644 (file)
@@ -77,28 +77,37 @@ 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('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'});
-            });
         });
     });