smartxml: tests refactoring

diff --git a/src/smartxml/smartxml.test.js b/src/smartxml/smartxml.test.js
index 7fd408e..938e5f7 100644 (file)
--- a/src/smartxml/smartxml.test.js
+++ b/src/smartxml/smartxml.test.js
@@ -77,28 +77,37 @@ describe('smartxml', function() {
         });
 
         describe('Changing node tag', function() {
         });
 
         describe('Changing node tag', function() {

+

             it('can change tag name', 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
                 // 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'});
-            });

         });
     });
 
         });
     });