Removing unnecessary element detach logic from Canvas DocumentNodeElement

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

diff --git a/src/smartxml/smartxml.test.js b/src/smartxml/smartxml.test.js
index 0bf5dd5..bb82467 100644 (file)
--- a/src/smartxml/smartxml.test.js
+++ b/src/smartxml/smartxml.test.js
@@ -1,8 +1,9 @@
 define([
     'libs/chai',
     'libs/sinon',
+    'libs/underscore',
     './smartxml.js'
-], function(chai, sinon, smartxml) {
+], function(chai, sinon, _, smartxml) {
     
 'use strict';
 /*jshint expr:true */
@@ -939,7 +940,7 @@ describe('smartxml', function() {
             expect(elementNode.textTestTransformation).to.be.undefined;
         
             expect(textNode.textTestTransformation().sameNode(textNode)).to.be.true;
-            expect(textNode.elementTestTransfomation).to.be.undefined; 
+            expect(textNode.elementTestTransfomation).to.be.undefined;
         });
 
         it('allows text/element node methods and transformations to access node and transormations on document node', function() {
@@ -998,8 +999,7 @@ describe('smartxml', function() {
 
         it('smoke tests', function() {
             var doc = getDocumentFromXML('<div>Alice</div>'),
-                textNode = doc.root.contents()[0],
-                result;
+                textNode = doc.root.contents()[0];
 
             expect(doc.undoStack).to.have.length(0);
             
@@ -1011,7 +1011,6 @@ describe('smartxml', function() {
             expect(doc.undoStack).to.have.length(0, '2');
             expect(doc.toXML()).to.equal('<div>Alice</div>');
 
-            debugger;
             doc.redo();
             expect(doc.undoStack).to.have.length(1, '3');
             expect(doc.toXML()).to.equal('<div>A<span>l</span>ice</div>');
@@ -1028,10 +1027,8 @@ describe('smartxml', function() {
         it('smoke tests 2', function() {
             var doc = getDocumentFromXML('<div>Alice</div>'),
                 textNode = doc.root.contents()[0],
-                path = textNode.getPath(),
-                result;
+                path = textNode.getPath();
 
-            debugger;
             textNode.setText('Alice ');
             textNode.setText('Alice h');
             textNode.setText('Alice ha');
@@ -1147,66 +1144,52 @@ describe('smartxml', function() {
                     expect(doc.root.contents()[0].getAttr('x')).to.equal('10', 'empty redoStack so redo was noop');
                     doc.undo();
                     expect(doc.root.contents()[0].getAttr('x')).to.equal('4', 'undoing additional transformation');
-                    doc.redo()
+                    doc.redo();
                     expect(doc.root.contents()[0].getAttr('x')).to.equal('10', 'redoing additional transformation');
                 });
             });
         });
 
-        // it('does work', function() {
-        //     var doc = getDocumentFromXML('<section><span>Alice</span></section>'),
-        //         span = doc.root.contents()[0];
-
-        //     span.transform('smartxml.detach');
-
-
-        //     doc.undo();
+        it('smoke tests nested transformations', function() {
+            var doc = getDocumentFromXML('<div></div>');
 
-        //     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.registerExtension({elementNode: {transformations: {
+                nested: function(v) {
+                    this._$.attr('innerAttr', v);
+                },
+                outer: function(v) {
+                    this.nested(v);
+                    this._$.attr('outerAttr', v);
+                }
+            }}});
 
-        //     doc.redo();
-        //     expect(doc.root.contents()).to.have.length(0);
+            doc.root.outer('test1');
+            doc.root.outer('test2');
 
-        //     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');
+            expect(doc.root.getAttr('innerAttr')).to.equal('test2');
+            expect(doc.root.getAttr('outerAttr')).to.equal('test2');
 
-        // });
-        // it('does work - merged text nodes case', function() {
-        //     var doc = getDocumentFromXML('<section>Alice <span>has</span> a cat.</section>'),
-        //         span = doc.root.contents()[1];
+            doc.undo();
 
-        //     span.transform('smartxml.detach');
+            expect(doc.root.getAttr('innerAttr')).to.equal('test1');
+            expect(doc.root.getAttr('outerAttr')).to.equal('test1');
 
+            doc.undo();
 
-        //     doc.undo();
+            expect(doc.root.getAttr('innerAttr')).to.equal(undefined);
+            expect(doc.root.getAttr('outerAttr')).to.equal(undefined);
 
-        //     expect(doc.root.contents().length).to.equal(3);
-        //     //console.log(doc.toXML());
-        //     expect(doc.root.contents()[1].contents()[0].getText()).to.equal('has');
+            doc.redo();
 
-        // });
-        // 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];
+            expect(doc.root.getAttr('innerAttr')).to.equal('test1');
+            expect(doc.root.getAttr('outerAttr')).to.equal('test1');
 
-        //     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();
-        //     doc.redo();
-        //     expect(doc.root.contents().length).to.equal(0);
+            expect(doc.root.getAttr('innerAttr')).to.equal('test2');
+            expect(doc.root.getAttr('outerAttr')).to.equal('test2');
 
-        // });
+        });
     });
 
 });