editor - fix missing canvas references

[fnpeditor.git] / src / editor / modules / documentCanvas / canvas / canvas.test.js

diff --git a/src/editor/modules/documentCanvas/canvas/canvas.test.js b/src/editor/modules/documentCanvas/canvas/canvas.test.js
index 2ee505d..bb795ec 100644 (file)
--- a/src/editor/modules/documentCanvas/canvas/canvas.test.js
+++ b/src/editor/modules/documentCanvas/canvas/canvas.test.js
@@ -19,9 +19,10 @@ var getCanvasFromXML = function(xml) {
 
 var getDocumentFromXML = function(xml) {
     return wlxml.WLXMLDocumentFromXML(xml);
-}
+};
 
 var wait = function(callback, timeout) {
+    /* globals window */
     return window.setTimeout(callback, timeout || 0.5);
 };
 
@@ -56,7 +57,7 @@ describe('Handling changes to the document', function() {
             c = canvas.fromXMLDocument(doc);
 
         var header = doc.root.replaceWith({tagName: 'header'});
-        expect(c.doc().data('wlxmlNode').sameNode(header)).to.be.true;
+        expect(c.doc().data('wlxmlNode').sameNode(header)).to.equal(true);
     });
 });
 
@@ -87,10 +88,27 @@ describe('Listening to document changes', function() {
         expect(sectionChildren[0].getText()).to.equal('Alice');
         expect(sectionChildren[1].getWlxmlTag()).to.equal('a');
     });
+
+    it('Handles nodeTagChange event', function() {
+
+        var doc = wlxml.WLXMLDocumentFromXML('<section><div>Alice</div></section>'),
+            c = canvas.fromXMLDocument(doc);
+
+        doc.root.contents()[0].setTag('header');
+
+        var headerNode = doc.root.contents()[0],
+            headerElement = c.doc().children()[0];
+
+        expect(headerElement.getWlxmlTag()).to.equal('header', 'element ok');
+
+        /* Make sure we handle invalidation of reference to wlxmlNode after changing its tag */
+        expect(headerNode.getData('canvasElement').sameNode(headerElement)).to.equal(true, 'node->element');
+        expect(headerElement.data('wlxmlNode').sameNode(headerNode)).to.equal(true, 'element->node');
+    });
 });
 
 describe('Cursor', function() {
-
+    /* globals Node */
     var getSelection;
 
     var findTextNode = function(inside, text) {
@@ -104,6 +122,7 @@ describe('Cursor', function() {
     };
 
     beforeEach(function() {
+        /* globals window */
         getSelection = sinon.stub(window, 'getSelection');
     });