refactoring: moving create(wlxmlNode) to constructor, caching element instance

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

diff --git a/src/smartxml/smartxml.test.js b/src/smartxml/smartxml.test.js
index c792cf9..fb420db 100644 (file)
--- a/src/smartxml/smartxml.test.js
+++ b/src/smartxml/smartxml.test.js
@@ -1660,6 +1660,21 @@ describe('smartxml', function() {
                 expect(doc.undoStack.length).to.equal(0, 'nothing to undo');
                 expect(doc.root.contents().length).to.equal(0);
             });
                 expect(doc.undoStack.length).to.equal(0, 'nothing to undo');
                 expect(doc.root.contents().length).to.equal(0);
             });

+
+            it('rollbacks and calls error handleor if error gets thrown', function() {
+                var doc = getDocumentFromXML('<root></root>'),
+                    err = new Error(),
+                    spy = sinon.spy();
+                
+                doc.transaction(function() {
+                    doc.root.append({tagName: 'div'});
+                    throw err;
+                }, {error: spy});
+
+                expect(spy.args[0][0]).to.equal(err);
+                expect(doc.root.contents().length).to.equal(0);
+                expect(doc.undoStack.length).to.equal(0);
+            });

         });
 
         describe('Regression tests', function() {
         });
 
         describe('Regression tests', function() {