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 e045788..c7cb6ad 100644 (file)
--- a/src/smartxml/smartxml.test.js
+++ b/src/smartxml/smartxml.test.js
@@ -715,6 +715,41 @@ describe('smartxml', function() {
             expect(event2.type).to.equal('nodeAdded');
             expect(event2.meta.node.sameNode(doc.root)).to.equal(true, 'new root node in nodelAdded event meta');
         });
+
+
+        ['append', 'prepend', 'before', 'after'].forEach(function(insertionMethod) {
+            it('emits nodeDetached for node moved from a document tree 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'}),
+                    newNodeInner = newNode.append({tagName:'c'});
+
+                newNodeInner[insertionMethod](a);
+
+                var event = spy.args[0][0];
+                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);
+            });
+        });
+
+
     });
 
     describe('Traversing', function() {
@@ -792,6 +827,64 @@ describe('smartxml', function() {
         });
     });
 
+    describe('Undo/redo', function() {
+
+        it('does work', function() {
+            var doc = getDocumentFromXML('<section><span>Alice</span></section>'),
+                span = doc.root.contents()[0];
+
+            span.transform('smartxml.detach');
+
+
+            doc.undo();
+
+            expect(doc.root.contents()).to.have.length(1);
+            expect(doc.root.contents()[0].getTagName()).to.equal('span');
+            expect(doc.root.contents()[0].contents()[0].getText()).to.equal('Alice');
+
+            doc.redo();
+            expect(doc.root.contents()).to.have.length(0);
+
+            doc.undo();
+            expect(doc.root.contents()).to.have.length(1);
+            expect(doc.root.contents()[0].getTagName()).to.equal('span');
+            expect(doc.root.contents()[0].contents()[0].getText()).to.equal('Alice');
+
+        });
+        it('does work - merged text nodes case', function() {
+            var doc = getDocumentFromXML('<section>Alice <span>has</span> a cat.</section>'),
+                span = doc.root.contents()[1];
+
+            span.transform('smartxml.detach');
+
+
+            doc.undo();
+
+            expect(doc.root.contents().length).to.equal(3);
+            //console.log(doc.toXML());
+            expect(doc.root.contents()[1].contents()[0].getText()).to.equal('has');
+
+        });
+        it('dbg - don not store nodes in tranformation state!', function() {
+            var doc = getDocumentFromXML('<section><a></a><b></b></section>'),
+                a = doc.root.contents()[0],
+                b = doc.root.contents()[1];
+
+            a.transform('smartxml.detach');
+            b.transform('smartxml.detach');
+            doc.undo();
+            doc.undo();
+            expect(doc.root.contents().length).to.equal(2);
+            expect(doc.root.contents()[0].getTagName()).to.equal('a');
+            expect(doc.root.contents()[1].getTagName()).to.equal('b');
+
+            doc.redo();
+            doc.redo();
+            expect(doc.root.contents().length).to.equal(0);
+
+        });
+    });
+
 });
 
 });
\ No newline at end of file