Canvas list API fix: removing unnecessary lists if all items extracted

[fnpeditor.git] / modules / documentCanvas / canvas / canvas.test3.js

diff --git a/modules/documentCanvas/canvas/canvas.test3.js b/modules/documentCanvas/canvas/canvas.test3.js
index 3c25cf5..597fec2 100644 (file)
--- a/modules/documentCanvas/canvas/canvas.test3.js
+++ b/modules/documentCanvas/canvas/canvas.test3.js
@@ -11,6 +11,30 @@ var expect = chai.expect;
 
 describe('Canvas', function() {
 
 
 describe('Canvas', function() {
 

+    describe('Internal HTML representation of a sample document', function() {
+        it('works', function() {
+            var c = canvas.fromXML('\
+                <section>\
+                    This is some text without its own wrapping tag.\
+                    <div class="p.subclass">\
+                        This is a paragraph.\
+                    </div>\
+                    <div>\
+                        This is text in a div <span>with some inline text</span>.\
+                    </div>\
+                    This is some text without its own wrapping tag.\
+                </section>\
+            ');
+            var expected = '<div wlxml-tag="section">'
+                            + 'This is some text without its own wrapping tag.'
+                            + '<div wlxml-tag="div" wlxml-class="p-subclass">This is a paragraph.</div>'
+                            + '<div wlxml-tag="div">This is text in a div <div wlxml-tag="span">with some inline text</div>.</div>'
+                            + 'This is some text without its own wrapping tag.'
+                            + '</div>';
+            expect(c.doc().dom()[0].isEqualNode($(expected)[0])).to.be.true;
+        });
+    });
+

     describe('Internal HTML representation of a DocumentNodeElement', function() {
         it('is always a div tag', function() {
             ['section', 'header', 'span', 'aside', 'figure'].forEach(function(tagName) {
     describe('Internal HTML representation of a DocumentNodeElement', function() {
         it('is always a div tag', function() {
             ['section', 'header', 'span', 'aside', 'figure'].forEach(function(tagName) {


@@ -79,6 +103,8 @@ describe('Canvas', function() {
                 expect(section.getWlxmlClass()).to.equal('some.class');
                 section.setWlxmlClass('some.other.class');
                 expect(section.getWlxmlClass()).to.equal('some.other.class');
                 expect(section.getWlxmlClass()).to.equal('some.class');
                 section.setWlxmlClass('some.other.class');
                 expect(section.getWlxmlClass()).to.equal('some.other.class');

+                section.setWlxmlClass(null);
+                expect(section.getWlxmlClass()).to.be.undefined;

             });
         });
     });
             });
         });
     });


@@ -162,6 +188,7 @@ describe('Canvas', function() {
                     var c = canvas.fromXML('<section> </section>');
                     expect(c.doc().children().length).to.equal(1);
                     expect(c.doc().children()[0]).to.be.instanceOf(documentElement.DocumentTextElement);
                     var c = canvas.fromXML('<section> </section>');
                     expect(c.doc().children().length).to.equal(1);
                     expect(c.doc().children()[0]).to.be.instanceOf(documentElement.DocumentTextElement);

+                    expect(c.doc().children()[0].getText()).to.equal(' ');

                 });
                 it('ignores white space surrounding block elements', function() {
                     var c = canvas.fromXML('<section> <div></div> </section>');
                 });
                 it('ignores white space surrounding block elements', function() {
                     var c = canvas.fromXML('<section> <div></div> </section>');


@@ -415,6 +442,69 @@ describe('Canvas', function() {
                 var list = section.children()[0];
                 expect(list.is('list')).to.equal(true, 'section\'s only child is a list');
                 expect(list.children().length).to.equal(4, 'list contains four elements');
                 var list = section.children()[0];
                 expect(list.is('list')).to.equal(true, 'section\'s only child is a list');
                 expect(list.children().length).to.equal(4, 'list contains four elements');

+                list.children().forEach(function(child) {
+                    expect(child.getWlxmlClass()).to.equal('item', 'list childs have wlxml class of item');
+                });
+            });
+
+            it('allows extracting continuous list of list items from its list', function() {
+                var c = canvas.fromXML('\
+                    <section>\
+                        <div class="list.items">\
+                            <div class="item">0</div>\
+                            <div class="item">1</div>\
+                            <div class="item">2</div>\
+                            <div class="item">3</div>\
+                        </div>\
+                    </section>'),
+                    list = c.doc().children()[0],
+                    item1 = list.children()[1],
+                    item2 = list.children()[2];
+
+                c.list.extractItems({element1: item1, element2: item2});
+
+                var section = c.doc(),
+                    list1 = section.children()[0],
+                    oldItem1 = section.children()[1],
+                    oldItem2 = section.children()[2],
+                    list2 = section.children()[3];
+
+                expect(section.children().length).to.equal(4, 'section contains four children');
+                
+                expect(list1.is('list')).to.equal(true, 'first section child is a list');
+                expect(list1.children().length).to.equal(1, 'first list has one child');
+                expect(list1.children()[0].children()[0].getText()).to.equal('0', 'first item of the first list is a first item of the original list');
+
+                expect(oldItem1.children()[0].getText()).to.equal('1', 'first item got extracted');
+                expect(oldItem1.getWlxmlClass() === undefined).to.equal(true, 'first extracted element has no wlxml class');
+
+                expect(oldItem2.children()[0].getText()).to.equal('2', 'second item got extracted');
+                expect(oldItem2.getWlxmlClass() === undefined).to.equal(true, 'second extracted element has no wlxml class');
+
+                expect(list2.is('list')).to.equal(true, 'last section child is a list');
+                expect(list2.children().length).to.equal(1, 'second list has one child');
+                expect(list2.children()[0].children()[0].getText()).to.equal('3', 'first item of the second list is a last item of the original list');
+            });
+
+            it('removes list if all its items are extracted', function() {
+                var c = canvas.fromXML('\
+                    <section>\
+                        <div class="list.items">\
+                            <div class="item">some item</div>\
+                        </div>\
+                    </section>'),
+                    list = c.doc().children()[0],
+                    item = list.children()[0];
+
+                c.list.extractItems({element1: item, element2: item});
+
+                var section = c.doc(),
+                    list1 = section.children()[0],
+                    oldItem1 = section.children()[1],
+                    oldItem2 = section.children()[2],
+                    list2 = section.children()[3];
+
+                expect(section.children().length).to.equal(1, 'section contains one child');

             });
         });
 
             });
         });