Foo 
Foo 
Foo
Foo
Foo
Foo
1.Item 1
'; - const view = htmlDataProcessor.toView( html ); + describe( 'transformListItemLikeElementsIntoLists()', () => { + it( 'replaces list-like elements with semantic lists', () => { + const html = '1.Item 1
'; + const view = htmlDataProcessor.toView( html ); - transformListItemLikeElementsIntoLists( view, '', new View() ); + transformListItemLikeElementsIntoLists( view, '', new View() ); - expect( view.childCount ).to.equal( 1 ); - expect( view.getChild( 0 ).name ).to.equal( 'ol' ); - expect( stringify( view ) ).to.equal( '- Item 1
- Item 1
1.Item 1
'; - const view = htmlDataProcessor.toView( html ); + it( 'replaces list-like elements with semantic lists with proper bullet type based on styles', () => { + const html = '1.Item 1
'; + const view = htmlDataProcessor.toView( html ); - transformListItemLikeElementsIntoLists( view, '@list l0:level1 { mso-level-number-format: bullet; }', new View() ); + transformListItemLikeElementsIntoLists( view, '@list l0:level1 { mso-level-number-format: bullet; }', new View() ); - expect( view.childCount ).to.equal( 1 ); - expect( view.getChild( 0 ).name ).to.equal( 'ul' ); - expect( stringify( view ) ).to.equal( '- Item 1
- Item 1
H1
Foo Bar
'; - const view = htmlDataProcessor.toView( html ); + it( 'does not modify the view if there are no list-like elements', () => { + const html = 'H1
Foo Bar
'; + const view = htmlDataProcessor.toView( html ); - transformListItemLikeElementsIntoLists( view, '', new View() ); + transformListItemLikeElementsIntoLists( view, '', new View() ); - expect( view.childCount ).to.equal( 2 ); - expect( stringify( view ) ).to.equal( html ); - } ); + expect( view.childCount ).to.equal( 2 ); + expect( stringify( view ) ).to.equal( html ); + } ); - it( 'handles empty `mso-list` style correctly', () => { - const html = '1.Item 1
'; - const view = htmlDataProcessor.toView( html ); + it( 'handles empty `mso-list` style correctly', () => { + const html = '1.Item 1
'; + const view = htmlDataProcessor.toView( html ); - transformListItemLikeElementsIntoLists( view, '', new View() ); + transformListItemLikeElementsIntoLists( view, '', new View() ); - expect( view.childCount ).to.equal( 1 ); - expect( view.getChild( 0 ).name ).to.equal( 'ol' ); - expect( stringify( view ) ).to.equal( '- Item 1
- Item 1
Foo Bar
'; - const { body, bodyString, styles, stylesString } = parseHtml( html ); +describe( 'PasteFromOffice - filters', () => { + describe( 'parse', () => { + describe( 'parseHtml()', () => { + it( 'correctly parses HTML with body and one style tag', () => { + const html = 'Foo Bar
'; + const { body, bodyString, styles, stylesString } = parseHtml( html ); - expect( body ).to.instanceof( DocumentFragment ); - expect( body.childCount ).to.equal( 1, 'body.childCount' ); + expect( body ).to.instanceof( DocumentFragment ); + expect( body.childCount ).to.equal( 1, 'body.childCount' ); - expect( bodyString ).to.equal( 'Foo Bar
' ); + expect( bodyString ).to.equal( 'Foo Bar
' ); - expect( styles.length ).to.equal( 1, 'styles.length' ); - expect( styles[ 0 ] ).to.instanceof( CSSStyleSheet ); - expect( styles[ 0 ].cssRules.length ).to.equal( 2 ); - expect( styles[ 0 ].cssRules[ 0 ].style.color ).to.equal( 'red' ); - expect( styles[ 0 ].cssRules[ 1 ].style[ 'font-size' ] ).to.equal( '12px' ); + expect( styles.length ).to.equal( 1, 'styles.length' ); + expect( styles[ 0 ] ).to.instanceof( CSSStyleSheet ); + expect( styles[ 0 ].cssRules.length ).to.equal( 2 ); + expect( styles[ 0 ].cssRules[ 0 ].style.color ).to.equal( 'red' ); + expect( styles[ 0 ].cssRules[ 1 ].style[ 'font-size' ] ).to.equal( '12px' ); - expect( stylesString ).to.equal( 'p { color: red; } a { font-size: 12px; }' ); - } ); + expect( stylesString ).to.equal( 'p { color: red; } a { font-size: 12px; }' ); + } ); - it( 'correctly parses HTML with body contents only', () => { - const html = 'Foo Bar
'; - const { body, bodyString, styles, stylesString } = parseHtml( html ); + it( 'correctly parses HTML with body contents only', () => { + const html = 'Foo Bar
'; + const { body, bodyString, styles, stylesString } = parseHtml( html ); - expect( body ).to.instanceof( DocumentFragment ); - expect( body.childCount ).to.equal( 1 ); + expect( body ).to.instanceof( DocumentFragment ); + expect( body.childCount ).to.equal( 1 ); - expect( bodyString ).to.equal( 'Foo Bar
' ); + expect( bodyString ).to.equal( 'Foo Bar
' ); - expect( styles.length ).to.equal( 0 ); + expect( styles.length ).to.equal( 0 ); - expect( stylesString ).to.equal( '' ); - } ); + expect( stylesString ).to.equal( '' ); + } ); - it( 'correctly parses HTML with no body and multiple style tags', () => { - const html = ''; - const { body, bodyString, styles, stylesString } = parseHtml( html ); + it( 'correctly parses HTML with no body and multiple style tags', () => { + const html = ''; + const { body, bodyString, styles, stylesString } = parseHtml( html ); - expect( body ).to.instanceof( DocumentFragment ); - expect( body.childCount ).to.equal( 0 ); + expect( body ).to.instanceof( DocumentFragment ); + expect( body.childCount ).to.equal( 0 ); - expect( bodyString ).to.equal( '' ); + expect( bodyString ).to.equal( '' ); - expect( styles.length ).to.equal( 2 ); - expect( styles[ 0 ] ).to.instanceof( CSSStyleSheet ); - expect( styles[ 1 ] ).to.instanceof( CSSStyleSheet ); - expect( styles[ 0 ].cssRules.length ).to.equal( 1 ); - expect( styles[ 1 ].cssRules.length ).to.equal( 1 ); - expect( styles[ 0 ].cssRules[ 0 ].style.color ).to.equal( 'blue' ); - expect( styles[ 1 ].cssRules[ 0 ].style.color ).to.equal( 'green' ); + expect( styles.length ).to.equal( 2 ); + expect( styles[ 0 ] ).to.instanceof( CSSStyleSheet ); + expect( styles[ 1 ] ).to.instanceof( CSSStyleSheet ); + expect( styles[ 0 ].cssRules.length ).to.equal( 1 ); + expect( styles[ 1 ].cssRules.length ).to.equal( 1 ); + expect( styles[ 0 ].cssRules[ 0 ].style.color ).to.equal( 'blue' ); + expect( styles[ 1 ].cssRules[ 0 ].style.color ).to.equal( 'green' ); - expect( stylesString ).to.equal( 'p { color: blue; } a { color: green; }' ); - } ); + expect( stylesString ).to.equal( 'p { color: blue; } a { color: green; }' ); + } ); - it( 'correctly parses HTML with no body and no style tags', () => { - const html = ''; - const { body, bodyString, styles, stylesString } = parseHtml( html ); + it( 'correctly parses HTML with no body and no style tags', () => { + const html = ''; + const { body, bodyString, styles, stylesString } = parseHtml( html ); - expect( body ).to.instanceof( DocumentFragment ); - expect( body.childCount ).to.equal( 0 ); + expect( body ).to.instanceof( DocumentFragment ); + expect( body.childCount ).to.equal( 0 ); - expect( bodyString ).to.equal( '' ); + expect( bodyString ).to.equal( '' ); - expect( styles.length ).to.equal( 0 ); + expect( styles.length ).to.equal( 0 ); - expect( stylesString ).to.equal( '' ); - } ); + expect( stylesString ).to.equal( '' ); + } ); - it( 'correctly parses HTML with body contents and empty style tag', () => { - const html = 'Foo Bar
'; - const { body, bodyString, styles, stylesString } = parseHtml( html ); + it( 'correctly parses HTML with body contents and empty style tag', () => { + const html = 'Foo Bar
'; + const { body, bodyString, styles, stylesString } = parseHtml( html ); - expect( body ).to.instanceof( DocumentFragment ); - expect( body.childCount ).to.equal( 1 ); + expect( body ).to.instanceof( DocumentFragment ); + expect( body.childCount ).to.equal( 1 ); - expect( bodyString ).to.equal( 'Foo Bar
' ); + expect( bodyString ).to.equal( 'Foo Bar
' ); - expect( styles.length ).to.equal( 0 ); + expect( styles.length ).to.equal( 0 ); - expect( stylesString ).to.equal( '' ); - } ); + expect( stylesString ).to.equal( '' ); + } ); - it( 'should remove any content after body closing tag - plain', () => { - const html = 'Foo Bar
Ba'; - const { body, bodyString, styles, stylesString } = parseHtml( html ); + it( 'should remove any content after body closing tag - plain', () => { + const html = 'Foo Bar
Ba'; + const { body, bodyString, styles, stylesString } = parseHtml( html ); - expect( body ).to.instanceof( DocumentFragment ); - expect( body.childCount ).to.equal( 1, 'body.childCount' ); + expect( body ).to.instanceof( DocumentFragment ); + expect( body.childCount ).to.equal( 1, 'body.childCount' ); - expect( bodyString ).to.equal( 'Foo Bar
' ); + expect( bodyString ).to.equal( 'Foo Bar
' ); - expect( styles.length ).to.equal( 0 ); + expect( styles.length ).to.equal( 0 ); - expect( stylesString ).to.equal( '' ); - } ); + expect( stylesString ).to.equal( '' ); + } ); - it( 'should remove any content after body closing tag - inline', () => { - const html = 'Foo Bar
Fo'; - const { body, bodyString, styles, stylesString } = parseHtml( html ); + it( 'should remove any content after body closing tag - inline', () => { + const html = 'Foo Bar
Fo'; + const { body, bodyString, styles, stylesString } = parseHtml( html ); - expect( body ).to.instanceof( DocumentFragment ); - expect( body.childCount ).to.equal( 1, 'body.childCount' ); + expect( body ).to.instanceof( DocumentFragment ); + expect( body.childCount ).to.equal( 1, 'body.childCount' ); - expect( bodyString ).to.equal( 'Foo Bar
' ); + expect( bodyString ).to.equal( 'Foo Bar
' ); - expect( styles.length ).to.equal( 0 ); + expect( styles.length ).to.equal( 0 ); - expect( stylesString ).to.equal( '' ); - } ); + expect( stylesString ).to.equal( '' ); + } ); - it( 'should remove any content after body closing tag - block', () => { - const html = 'Foo Bar
ar
'; - const { body, bodyString, styles, stylesString } = parseHtml( html ); + it( 'should remove any content after body closing tag - block', () => { + const html = 'Foo Bar
ar
'; + const { body, bodyString, styles, stylesString } = parseHtml( html ); - expect( body ).to.instanceof( DocumentFragment ); - expect( body.childCount ).to.equal( 1, 'body.childCount' ); + expect( body ).to.instanceof( DocumentFragment ); + expect( body.childCount ).to.equal( 1, 'body.childCount' ); - expect( bodyString ).to.equal( 'Foo Bar
' ); + expect( bodyString ).to.equal( 'Foo Bar
' ); - expect( styles.length ).to.equal( 0 ); + expect( styles.length ).to.equal( 0 ); - expect( stylesString ).to.equal( '' ); - } ); + expect( stylesString ).to.equal( '' ); + } ); - it( 'should remove any content after body closing tag - no html tag', () => { - const html = 'Foo Bar
oo'; - const { body, bodyString, styles, stylesString } = parseHtml( html ); + it( 'should remove any content after body closing tag - no html tag', () => { + const html = 'Foo Bar
oo'; + const { body, bodyString, styles, stylesString } = parseHtml( html ); - expect( body ).to.instanceof( DocumentFragment ); - expect( body.childCount ).to.equal( 1, 'body.childCount' ); + expect( body ).to.instanceof( DocumentFragment ); + expect( body.childCount ).to.equal( 1, 'body.childCount' ); - expect( bodyString ).to.equal( 'Foo Bar
' ); + expect( bodyString ).to.equal( 'Foo Bar
' ); - expect( styles.length ).to.equal( 0 ); + expect( styles.length ).to.equal( 0 ); - expect( stylesString ).to.equal( '' ); - } ); + expect( stylesString ).to.equal( '' ); + } ); - it( 'should not remove any content if no body tag', () => { - const html = 'Foo Bar
Baz'; - const { body, bodyString, styles, stylesString } = parseHtml( html ); + it( 'should not remove any content if no body tag', () => { + const html = 'Foo Bar
Baz'; + const { body, bodyString, styles, stylesString } = parseHtml( html ); - expect( body ).to.instanceof( DocumentFragment ); - expect( body.childCount ).to.equal( 2, 'body.childCount' ); + expect( body ).to.instanceof( DocumentFragment ); + expect( body.childCount ).to.equal( 2, 'body.childCount' ); - expect( bodyString ).to.equal( 'Foo Bar
Baz' ); + expect( bodyString ).to.equal( 'Foo Bar
Baz' ); - expect( styles.length ).to.equal( 0 ); + expect( styles.length ).to.equal( 0 ); - expect( stylesString ).to.equal( '' ); - } ); + expect( stylesString ).to.equal( '' ); } ); } ); } ); diff --git a/tests/filters/reomoveboldwrapper.js b/tests/filters/reomoveboldwrapper.js new file mode 100644 index 0000000..db35333 --- /dev/null +++ b/tests/filters/reomoveboldwrapper.js @@ -0,0 +1,50 @@ +/** + * @license Copyright (c) 2003-2019, CKSource - Frederico Knabben. All rights reserved. + * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license + */ + +import HtmlDataProcessor from '@ckeditor/ckeditor5-engine/src/dataprocessor/htmldataprocessor'; +import removeBoldWrapper from '../../src/filters/removeboldwrapper'; +import UpcastWriter from '@ckeditor/ckeditor5-engine/src/view/upcastwriter'; + +describe( 'PasteFromOffice - filters', () => { + const htmlDataProcessor = new HtmlDataProcessor(); + describe( 'removeBoldWrapper', () => { + let writer; + + before( () => { + writer = new UpcastWriter(); + } ); + + it( 'should remove bold wrapper added by google docs', () => { + const inputData = '' + + 'Hello world
' + + ''; + const documentFragment = htmlDataProcessor.toView( inputData ); + + removeBoldWrapper( documentFragment, writer ); + + expect( htmlDataProcessor.toData( documentFragment ) ).to.equal( 'Hello world
' ); + } ); + + it( 'should not remove non-bold tag with google id', () => { + const inputData = 'Hello world
'; + const documentFragment = htmlDataProcessor.toView( inputData ); + + removeBoldWrapper( documentFragment, writer ); + + expect( htmlDataProcessor.toData( documentFragment ) ).to.equal( + 'Hello world
' ); + } ); + + it( 'should not remove bold tag without google id', () => { + const inputData = 'Hello world'; + const documentFragment = htmlDataProcessor.toView( inputData ); + + removeBoldWrapper( documentFragment, writer ); + + expect( htmlDataProcessor.toData( documentFragment ) ).to.equal( + 'Hello world' ); + } ); + } ); +} ); diff --git a/tests/filters/space.js b/tests/filters/space.js index 93a8607..8898d03 100644 --- a/tests/filters/space.js +++ b/tests/filters/space.js @@ -6,100 +6,99 @@ /* globals DOMParser */ import { normalizeSpacing, normalizeSpacerunSpans } from '../../src/filters/space'; -describe( 'Paste from Office', () => { - describe( 'Filters', () => { - describe( 'space', () => { - describe( 'normalizeSpacing()', () => { - it( 'should replace last space before closing tag with NBSP', () => { - const input = 'Foo
Bar Baz
'; - const expected = 'Foo\u00A0
Bar \u00A0 Baz\u00A0
'; - expect( normalizeSpacing( input ) ).to.equal( expected ); - } ); +describe( 'PasteFromOffice - filters', () => { + describe( 'space', () => { + describe( 'normalizeSpacing()', () => { + it( 'should replace last space before closing tag with NBSP', () => { + const input = 'Foo
Bar Baz
'; + const expected = 'Foo\u00A0
Bar \u00A0 Baz\u00A0
'; - it( 'should replace last space before special "o:p" tag with NBSP', () => { - const input = 'Foo
Foo \u00A0
Foo
Foo \u00A0
Foo
\n\n \nBar
\r\n\r\nBaz
'; - const expected = 'Foo
Bar
Baz
'; + expect( normalizeSpacing( input ) ).to.equal( expected ); + } ); - expect( normalizeSpacing( input ) ).to.equal( expected ); - } ); + it( 'should remove multiline sequences of whitespaces', () => { + const input = 'Foo
\n\n \nBar
\r\n\r\nBaz
'; + const expected = 'Foo
Bar
Baz
'; - it( 'should normalize Safari "space spans"', () => { - const input = 'Foo Baz
'; - const expected = 'Foo \u00A0 \u00A0 Baz \u00A0\u00A0
'; + expect( normalizeSpacing( input ) ).to.equal( expected ); + } ); - expect( normalizeSpacing( input ) ).to.equal( expected ); - } ); + it( 'should normalize Safari "space spans"', () => { + const input = 'Foo Baz
'; + const expected = 'Foo \u00A0 \u00A0 Baz \u00A0\u00A0
'; - it( 'should normalize nested Safari "space spans"', () => { - const input = - 'Foo Baz
'; + expect( normalizeSpacing( input ) ).to.equal( expected ); + } ); - const expected = 'Foo \u00A0 \u00A0 \u00A0 Baz
'; + it( 'should normalize nested Safari "space spans"', () => { + const input = + 'Foo Baz
'; - expect( normalizeSpacing( input ) ).to.equal( expected ); - } ); + const expected = 'Foo \u00A0 \u00A0 \u00A0 Baz
'; + + expect( normalizeSpacing( input ) ).to.equal( expected ); } ); + } ); - describe( 'normalizeSpacerunSpans()', () => { - it( 'should normalize spaces inside special "span.spacerun" elements', () => { - const input = 'Foo
' + - 'Baz
'; + describe( 'normalizeSpacerunSpans()', () => { + it( 'should normalize spaces inside special "span.spacerun" elements', () => { + const input = 'Foo
' + + 'Baz
'; - const expected = 'Foo
' + - 'Baz
'; + const expected = 'Foo
' + + 'Baz
'; - const domParser = new DOMParser(); - const htmlDocument = domParser.parseFromString( input, 'text/html' ); + const domParser = new DOMParser(); + const htmlDocument = domParser.parseFromString( input, 'text/html' ); - expect( htmlDocument.body.innerHTML.replace( /'/g, '"' ).replace( /: /g, ':' ) ).to.not.equal( expected ); + expect( htmlDocument.body.innerHTML.replace( /'/g, '"' ).replace( /: /g, ':' ) ).to.not.equal( expected ); - normalizeSpacerunSpans( htmlDocument ); + normalizeSpacerunSpans( htmlDocument ); - expect( htmlDocument.body.innerHTML.replace( /'/g, '"' ).replace( /: /g, ':' ) ).to.equal( expected ); - } ); + expect( htmlDocument.body.innerHTML.replace( /'/g, '"' ).replace( /: /g, ':' ) ).to.equal( expected ); } ); } ); } ); diff --git a/tests/manual/integration.html b/tests/manual/integration.html index 15955a7..09375e5 100644 --- a/tests/manual/integration.html +++ b/tests/manual/integration.html @@ -1,3 +1,8 @@ +Paste here:
@@ -10,11 +15,11 @@Paste here:
Clipboard HTML
-Clipboard text +Clipboard text
-Clipboard HTML after transformation +Clipboard normalized HTML view after transformation
diff --git a/tests/normalizers/googledocsnormalizer.js b/tests/normalizers/googledocsnormalizer.js new file mode 100644 index 0000000..58b9309 --- /dev/null +++ b/tests/normalizers/googledocsnormalizer.js @@ -0,0 +1,25 @@ +/** + * @license Copyright (c) 2003-2019, CKSource - Frederico Knabben. All rights reserved. + * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license + */ + +import GoogleDocsNormalizer from '../../src/normalizers/googledocsnormalizer'; + +// `execute()` of the google docs normalizer is tested with autogenerated normalization tests. +describe( 'GoogleDocsNormalizer', () => { + const normalizer = new GoogleDocsNormalizer(); + + describe( 'isActive()', () => { + it( 'should return true from google docs content', () => { + expect( normalizer.isActive( '' ) ).to.be.true; + } ); + + it( 'should return false for microsoft word content', () => { + expect( normalizer.isActive( 'Foo bar
' ) ).to.be.false; + } ); + + it( 'should return false for content form other sources', () => { + expect( normalizer.isActive( 'foo
' ) ).to.be.false; + } ); + } ); +} ); diff --git a/tests/normalizers/mswordnormalizer.js b/tests/normalizers/mswordnormalizer.js new file mode 100644 index 0000000..db72e02 --- /dev/null +++ b/tests/normalizers/mswordnormalizer.js @@ -0,0 +1,32 @@ +/** + * @license Copyright (c) 2003-2019, CKSource - Frederico Knabben. All rights reserved. + * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license + */ + +import MSWordNormalizer from '../../src/normalizers/mswordnormalizer'; + +// `execute()` of the msword normalizer is tested with autogenerated normalization tests. +describe( 'MSWordNormalizer', () => { + const normalizer = new MSWordNormalizer(); + + describe( 'isActive()', () => { + it( 'should return true for microsoft word content', () => { + expect( normalizer.isActive( 'Foo bar
' ) ).to.be.true; + } ); + + it( 'should return true for microsoft word content - safari', () => { + expect( normalizer.isActive( '' ) ).to.be.true; + } ); + + it( 'should return false for google docs content', () => { + expect( normalizer.isActive( '' ) ).to.be.false; + } ); + + it( 'should return false for content fromother sources', () => { + expect( normalizer.isActive( 'foo
' ) ).to.be.false; + } ); + } ); +} ); diff --git a/tests/pastefromoffice.js b/tests/pastefromoffice.js index a2692e5..f69f6b2 100644 --- a/tests/pastefromoffice.js +++ b/tests/pastefromoffice.js @@ -3,87 +3,123 @@ * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license */ +import PasteFromOffice from '../src/pastefromoffice'; import Clipboard from '@ckeditor/ckeditor5-clipboard/src/clipboard'; import VirtualTestEditor from '@ckeditor/ckeditor5-core/tests/_utils/virtualtesteditor'; -import DocumentFragment from '@ckeditor/ckeditor5-engine/src/view/documentfragment'; -import testUtils from '@ckeditor/ckeditor5-core/tests/_utils/utils'; - -import PasteFromOffice from '../src/pastefromoffice'; +import HtmlDataProcessor from '@ckeditor/ckeditor5-engine/src/dataprocessor/htmldataprocessor'; import { createDataTransfer } from './_utils/utils'; +import testUtils from '@ckeditor/ckeditor5-core/tests/_utils/utils'; -describe( 'Paste from Office plugin', () => { - let editor, content, normalizeSpy; +describe( 'PasteFromOffice', () => { + const htmlDataProcessor = new HtmlDataProcessor(); + let editor, pasteFromOffice, clipboard; testUtils.createSinonSandbox(); - before( () => { - content = new DocumentFragment(); - } ); - beforeEach( () => { - return VirtualTestEditor - .create( { - plugins: [ Clipboard, PasteFromOffice ] - } ) - .then( newEditor => { - editor = newEditor; - normalizeSpy = testUtils.sinon.spy( editor.plugins.get( 'PasteFromOffice' ), '_normalizeWordInput' ); + return VirtualTestEditor.create( { + plugins: [ PasteFromOffice ] + } ) + .then( _editor => { + editor = _editor; + pasteFromOffice = editor.plugins.get( 'PasteFromOffice' ); + clipboard = editor.plugins.get( 'Clipboard' ); } ); } ); - it( 'runs normalizations if Word meta tag detected #1', () => { - const dataTransfer = createDataTransfer( { - 'text/html': '' - } ); - - editor.plugins.get( 'Clipboard' ).fire( 'inputTransformation', { content, dataTransfer } ); - - expect( normalizeSpy.calledOnce ).to.true; + it( 'should be loaded', () => { + expect( pasteFromOffice ).to.be.instanceOf( PasteFromOffice ); } ); - it( 'runs normalizations if Word meta tag detected #2', () => { - const dataTransfer = createDataTransfer( { - 'text/html': '' - } ); - - editor.plugins.get( 'Clipboard' ).fire( 'inputTransformation', { content, dataTransfer } ); + it( 'has proper name', () => { + expect( PasteFromOffice.pluginName ).to.equal( 'PasteFromOffice' ); + } ); - expect( normalizeSpy.calledOnce ).to.true; + it( 'should load Clipboard plugin', () => { + expect( editor.plugins.get( Clipboard ) ).to.be.instanceOf( Clipboard ); } ); - it( 'does not normalize the content without Word meta tag', () => { - const dataTransfer = createDataTransfer( { - 'text/html': '' - } ); + describe( 'isTransformedWithPasteFromOffice - flag', () => { + describe( 'data which should be marked with flag', () => { + it( 'should process data with microsoft word header', () => { + checkCorrectData( '' ); + } ); - editor.plugins.get( 'Clipboard' ).fire( 'inputTransformation', { content, dataTransfer } ); + it( 'should process data with nested microsoft header', () => { + checkCorrectData( '' ); + } ); - expect( normalizeSpy.called ).to.false; - } ); + it( 'should process data from google docs', () => { + checkCorrectData( '' ); + } ); + + function checkCorrectData( inputString ) { + const data = setUpData( inputString ); + const getDataSpy = sinon.spy( data.dataTransfer, 'getData' ); - it( 'does not process content many times for the same `inputTransformation` event', () => { - const clipboard = editor.plugins.get( 'Clipboard' ); + clipboard.fire( 'inputTransformation', data ); - const dataTransfer = createDataTransfer( { - 'text/html': '' + expect( data.isTransformedWithPasteFromOffice ).to.be.true; + sinon.assert.called( getDataSpy ); + } } ); - let eventRefired = false; - clipboard.on( 'inputTransformation', ( evt, data ) => { - if ( !eventRefired ) { - eventRefired = true; + describe( 'data which should not be marked with flag', () => { + it( 'should not process data with regular html', () => { + checkInvalidData( 'Hello world
' ); + } ); - evt.stop(); + it( 'should not process data with similar headers to MS Word', () => { + checkInvalidData( '' ); + } ); + + function checkInvalidData( inputString ) { + const data = setUpData( inputString ); + const getDataSpy = sinon.spy( data.dataTransfer, 'getData' ); clipboard.fire( 'inputTransformation', data ); + + expect( data.isTransformedWithPasteFromOffice ).to.be.undefined; + sinon.assert.called( getDataSpy ); } + } ); - expect( data.pasteFromOfficeProcessed ).to.true; - expect( normalizeSpy.calledOnce ).to.true; - }, { priority: 'low' } ); + describe( 'data which already have the flag', () => { + it( 'should not process again ms word data containing a flag', () => { + checkAlreadyProcessedData( '' + + 'Hello world
Hello world
' ); + } ); + + function checkAlreadyProcessedData( inputString ) { + const data = setUpData( inputString, true ); + const getDataSpy = sinon.spy( data.dataTransfer, 'getData' ); - editor.plugins.get( 'Clipboard' ).fire( 'inputTransformation', { content, dataTransfer } ); + clipboard.fire( 'inputTransformation', data ); - expect( normalizeSpy.calledOnce ).to.true; + expect( data.isTransformedWithPasteFromOffice ).to.be.true; + sinon.assert.notCalled( getDataSpy ); + } + } ); } ); + + // @param {String} inputString html to be processed by paste from office + // @param {Boolean} [isTransformedWithPasteFromOffice=false] if set, marks output data with isTransformedWithPasteFromOffice flag + // @returns {Object} data object simulating content obtained from the clipboard + function setUpData( inputString, isTransformedWithPasteFromOffice = false ) { + const data = { + content: htmlDataProcessor.toView( inputString ), + dataTransfer: createDataTransfer( { 'text/html': inputString } ) + }; + + if ( isTransformedWithPasteFromOffice ) { + data.isTransformedWithPasteFromOffice = true; + } + + return data; + } } );