smartxml: fix - do not send nodeDetached event for out of document node

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

diff --git a/src/smartxml/smartxml.test.js b/src/smartxml/smartxml.test.js
index d2ac4a9..c7cb6ad 100644 (file)
--- a/src/smartxml/smartxml.test.js
+++ b/src/smartxml/smartxml.test.js
@@ -734,6 +734,19 @@ describe('smartxml', function() {
                 expect(event.type).to.equal('nodeDetached');
                 expect(event.meta.node.sameNode(a));
             });
+
+            it('doesn\'t emit nodeDetached event for already out of document moved to out of document node: ' + insertionMethod, function() {
+                var doc = getDocumentFromXML('<div><a></a></div>'),
+                    a = doc.root.contents()[0],
+                    spy = sinon.spy();
+
+                doc.on('change', spy);
+
+                var newNode = doc.createDocumentNode({tagName: 'b'});
+                    var newNodeInner = newNode.append({tagName:'c'});
+
+                expect(spy.callCount).to.equal(0);
+            });
         });
 
 
@@ -820,7 +833,7 @@ describe('smartxml', function() {
             var doc = getDocumentFromXML('<section><span>Alice</span></section>'),
                 span = doc.root.contents()[0];
 
-            doc.transform('detach2', {node: span});
+            span.transform('smartxml.detach');
 
 
             doc.undo();
@@ -842,7 +855,7 @@ describe('smartxml', function() {
             var doc = getDocumentFromXML('<section>Alice <span>has</span> a cat.</section>'),
                 span = doc.root.contents()[1];
 
-            doc.transform('detach2', {node: span});
+            span.transform('smartxml.detach');
 
 
             doc.undo();
@@ -857,8 +870,8 @@ describe('smartxml', function() {
                 a = doc.root.contents()[0],
                 b = doc.root.contents()[1];
 
-            doc.transform('detach2', {node: a});
-            doc.transform('detach2', {node: b});
+            a.transform('smartxml.detach');
+            b.transform('smartxml.detach');
             doc.undo();
             doc.undo();
             expect(doc.root.contents().length).to.equal(2);