Explorar el Código

Extract table properties editing tests.

Maciej Gołaszewski hace 6 años
padre
commit
498db1f5da

+ 18 - 646
packages/ckeditor5-table/tests/tableproperties.js

@@ -3,668 +3,40 @@
  * For licensing, see LICENSE.md or https://ckeditor.com/legal/ckeditor-oss-license
  */
 
+import global from '@ckeditor/ckeditor5-utils/src/dom/global';
+import ClassicTestEditor from '@ckeditor/ckeditor5-core/tests/_utils/classictesteditor';
 import Paragraph from '@ckeditor/ckeditor5-paragraph/src/paragraph';
-import VirtualTestEditor from '@ckeditor/ckeditor5-core/tests/_utils/virtualtesteditor';
 
 import TableEditing from '../src/tableediting';
 import TableProperties from '../src/tableproperties';
-import { assertTableStyle, assertTRBLAttribute } from './_utils/utils';
-import { setData as setModelData } from '@ckeditor/ckeditor5-engine/src/dev-utils/model';
-import { assertEqualMarkup } from '@ckeditor/ckeditor5-utils/tests/_utils/utils';
+import TablePropertiesEditing from '../src/tablepropertiesediting';
 
 describe( 'TableProperties', () => {
-	let editor, model;
+	let editor, editorElement;
 
-	beforeEach( () => {
-		return VirtualTestEditor
-			.create( {
-				plugins: [ TableProperties, Paragraph, TableEditing ]
-			} )
-			.then( newEditor => {
-				editor = newEditor;
+	beforeEach( async () => {
+		editorElement = global.document.createElement( 'div' );
+		global.document.body.appendChild( editorElement );
 
-				model = editor.model;
-			} );
-	} );
-
-	afterEach( () => {
-		editor.destroy();
-	} );
-
-	it( 'should have pluginName', () => {
-		expect( TableProperties.pluginName ).to.equal( 'TableProperties' );
-	} );
-
-	describe( 'border', () => {
-		it( 'should set proper schema rules', () => {
-			expect( model.schema.checkAttribute( [ '$root', 'table' ], 'borderColor' ) ).to.be.true;
-			expect( model.schema.checkAttribute( [ '$root', 'table' ], 'borderStyle' ) ).to.be.true;
-			expect( model.schema.checkAttribute( [ '$root', 'table' ], 'borderWidth' ) ).to.be.true;
-		} );
-
-		describe( 'upcast conversion', () => {
-			it( 'should upcast border shorthand', () => {
-				editor.setData( '<table style="border:1px solid #f00"><tr><td>foo</td></tr></table>' );
-
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				assertTRBLAttribute( table, 'borderColor', '#f00' );
-				assertTRBLAttribute( table, 'borderStyle', 'solid' );
-				assertTRBLAttribute( table, 'borderWidth', '1px' );
-			} );
-
-			it( 'should upcast border-color shorthand', () => {
-				editor.setData( '<table style="border-color:#f00"><tr><td>foo</td></tr></table>' );
-
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				assertTRBLAttribute( table, 'borderColor', '#f00' );
-			} );
-
-			it( 'should upcast border-style shorthand', () => {
-				editor.setData( '<table style="border-style:ridge"><tr><td>foo</td></tr></table>' );
-
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				assertTRBLAttribute( table, 'borderStyle', 'ridge' );
-			} );
-
-			it( 'should upcast border-width shorthand', () => {
-				editor.setData( '<table style="border-width:1px"><tr><td>foo</td></tr></table>' );
-
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				assertTRBLAttribute( table, 'borderWidth', '1px' );
-			} );
-
-			it( 'should upcast border-top shorthand', () => {
-				editor.setData( '<table style="border-top:1px solid #f00"><tr><td>foo</td></tr></table>' );
-
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				assertTRBLAttribute( table, 'borderColor', '#f00', null, null, null );
-				assertTRBLAttribute( table, 'borderStyle', 'solid', null, null, null );
-				assertTRBLAttribute( table, 'borderWidth', '1px', null, null, null );
-			} );
-
-			it( 'should upcast border-right shorthand', () => {
-				editor.setData( '<table style="border-right:1px solid #f00"><tr><td>foo</td></tr></table>' );
-
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				assertTRBLAttribute( table, 'borderColor', null, '#f00', null, null );
-				assertTRBLAttribute( table, 'borderStyle', null, 'solid', null, null );
-				assertTRBLAttribute( table, 'borderWidth', null, '1px', null, null );
-			} );
-
-			it( 'should upcast border-bottom shorthand', () => {
-				editor.setData( '<table style="border-bottom:1px solid #f00"><tr><td>foo</td></tr></table>' );
-
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				assertTRBLAttribute( table, 'borderColor', null, null, '#f00', null );
-				assertTRBLAttribute( table, 'borderStyle', null, null, 'solid', null );
-				assertTRBLAttribute( table, 'borderWidth', null, null, '1px', null );
-			} );
-
-			it( 'should upcast border-left shorthand', () => {
-				editor.setData( '<table style="border-left:1px solid #f00"><tr><td>foo</td></tr></table>' );
-
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				assertTRBLAttribute( table, 'borderColor', null, null, null, '#f00' );
-				assertTRBLAttribute( table, 'borderStyle', null, null, null, 'solid' );
-				assertTRBLAttribute( table, 'borderWidth', null, null, null, '1px' );
-			} );
-
-			it( 'should upcast border-top-* styles', () => {
-				editor.setData(
-					'<table style="border-top-width:1px;border-top-style:solid;border-top-color:#f00"><tr><td>foo</td></tr></table>'
-				);
-
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				assertTRBLAttribute( table, 'borderColor', '#f00', null, null, null );
-				assertTRBLAttribute( table, 'borderStyle', 'solid', null, null, null );
-				assertTRBLAttribute( table, 'borderWidth', '1px', null, null, null );
-			} );
-
-			it( 'should upcast border-right-* styles', () => {
-				editor.setData(
-					'<table style="border-right-width:1px;border-right-style:solid;border-right-color:#f00"><tr><td>foo</td></tr></table>'
-				);
-
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				assertTRBLAttribute( table, 'borderColor', null, '#f00', null, null );
-				assertTRBLAttribute( table, 'borderStyle', null, 'solid', null, null );
-				assertTRBLAttribute( table, 'borderWidth', null, '1px', null, null );
-			} );
-
-			it( 'should upcast border-bottom-* styles', () => {
-				editor.setData(
-					'<table style="border-bottom-width:1px;border-bottom-style:solid;border-bottom-color:#f00">' +
-					'<tr>' +
-					'<td>foo</td>' +
-					'</tr>' +
-					'</table>'
-				);
-
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				assertTRBLAttribute( table, 'borderColor', null, null, '#f00', null );
-				assertTRBLAttribute( table, 'borderStyle', null, null, 'solid', null );
-				assertTRBLAttribute( table, 'borderWidth', null, null, '1px', null );
-			} );
-
-			it( 'should upcast border-left-* styles', () => {
-				editor.setData(
-					'<table style="border-left-width:1px;border-left-style:solid;border-left-color:#f00"><tr><td>foo</td></tr></table>'
-				);
-
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				assertTRBLAttribute( table, 'borderColor', null, null, null, '#f00' );
-				assertTRBLAttribute( table, 'borderStyle', null, null, null, 'solid' );
-				assertTRBLAttribute( table, 'borderWidth', null, null, null, '1px' );
-			} );
-		} );
-
-		describe( 'downcast conversion', () => {
-			let table;
-
-			beforeEach( () => {
-				table = createEmptyTable();
-			} );
-
-			it( 'should downcast borderColor attribute (same top, right, bottom, left)', () => {
-				model.change( writer => writer.setAttribute( 'borderColor', {
-					top: '#f00',
-					right: '#f00',
-					bottom: '#f00',
-					left: '#f00'
-				}, table ) );
-
-				assertTableStyle( editor, 'border-bottom:#f00;border-left:#f00;border-right:#f00;border-top:#f00;' );
-			} );
-
-			it( 'should downcast borderColor attribute (different top, right, bottom, left)', () => {
-				model.change( writer => writer.setAttribute( 'borderColor', {
-					top: '#f00',
-					right: 'hsla(0, 100%, 50%, 0.5)',
-					bottom: 'deeppink',
-					left: 'rgb(255, 0, 0)'
-				}, table ) );
-
-				assertTableStyle( editor,
-					'border-bottom:deeppink;' +
-					'border-left:rgb(255, 0, 0);' +
-					'border-right:hsla(0, 100%, 50%, 0.5);' +
-					'border-top:#f00;'
-				);
-			} );
-
-			it( 'should downcast borderStyle attribute (same top, right, bottom, left)', () => {
-				model.change( writer => writer.setAttribute( 'borderStyle', {
-					top: 'solid',
-					right: 'solid',
-					bottom: 'solid',
-					left: 'solid'
-				}, table ) );
-
-				assertTableStyle( editor, 'border-bottom:solid;border-left:solid;border-right:solid;border-top:solid;' );
-			} );
-
-			it( 'should downcast borderStyle attribute (different top, right, bottom, left)', () => {
-				model.change( writer => writer.setAttribute( 'borderStyle', {
-					top: 'solid',
-					right: 'ridge',
-					bottom: 'dotted',
-					left: 'dashed'
-				}, table ) );
-
-				assertTableStyle( editor, 'border-bottom:dotted;border-left:dashed;border-right:ridge;border-top:solid;' );
-			} );
-
-			it( 'should downcast borderWidth attribute (same top, right, bottom, left)', () => {
-				model.change( writer => writer.setAttribute( 'borderWidth', {
-					top: '42px',
-					right: '.1em',
-					bottom: '1337rem',
-					left: 'thick'
-				}, table ) );
-
-				assertTableStyle( editor, 'border-bottom:1337rem;border-left:thick;border-right:.1em;border-top:42px;' );
-			} );
-
-			it( 'should downcast borderWidth attribute (different top, right, bottom, left)', () => {
-				model.change( writer => writer.setAttribute( 'borderWidth', {
-					top: '42px',
-					right: '42px',
-					bottom: '42px',
-					left: '42px'
-				}, table ) );
-
-				assertTableStyle( editor, 'border-bottom:42px;border-left:42px;border-right:42px;border-top:42px;' );
-			} );
-
-			it( 'should downcast borderColor, borderStyle and borderWidth attributes together (same top, right, bottom, left)', () => {
-				model.change( writer => {
-					writer.setAttribute( 'borderColor', {
-						top: '#f00',
-						right: '#f00',
-						bottom: '#f00',
-						left: '#f00'
-					}, table );
-
-					writer.setAttribute( 'borderStyle', {
-						top: 'solid',
-						right: 'solid',
-						bottom: 'solid',
-						left: 'solid'
-					}, table );
-
-					writer.setAttribute( 'borderWidth', {
-						top: '42px',
-						right: '42px',
-						bottom: '42px',
-						left: '42px'
-					}, table );
-				} );
-
-				assertTableStyle( editor,
-					'border-bottom:42px solid #f00;' +
-					'border-left:42px solid #f00;' +
-					'border-right:42px solid #f00;' +
-					'border-top:42px solid #f00;'
-				);
-			} );
-
-			it( 'should downcast borderColor, borderStyle and borderWidth attributes together (different top, right, bottom, left)', () => {
-				model.change( writer => {
-					writer.setAttribute( 'borderColor', {
-						top: '#f00',
-						right: 'hsla(0, 100%, 50%, 0.5)',
-						bottom: 'deeppink',
-						left: 'rgb(255, 0, 0)'
-					}, table );
-
-					writer.setAttribute( 'borderStyle', {
-						top: 'solid',
-						right: 'ridge',
-						bottom: 'dotted',
-						left: 'dashed'
-					}, table );
-
-					writer.setAttribute( 'borderWidth', {
-						top: '42px',
-						right: '.1em',
-						bottom: '1337rem',
-						left: 'thick'
-					}, table );
-				} );
-
-				assertTableStyle( editor,
-					'border-bottom:1337rem dotted deeppink;' +
-					'border-left:thick dashed rgb(255, 0, 0);' +
-					'border-right:.1em ridge hsla(0, 100%, 50%, 0.5);' +
-					'border-top:42px solid #f00;'
-				);
-			} );
-
-			describe( 'change attribute', () => {
-				beforeEach( () => {
-					model.change( writer => {
-						writer.setAttribute( 'borderColor', {
-							top: '#f00',
-							right: '#f00',
-							bottom: '#f00',
-							left: '#f00'
-						}, table );
-
-						writer.setAttribute( 'borderStyle', {
-							top: 'solid',
-							right: 'solid',
-							bottom: 'solid',
-							left: 'solid'
-						}, table );
-
-						writer.setAttribute( 'borderWidth', {
-							top: '42px',
-							right: '42px',
-							bottom: '42px',
-							left: '42px'
-						}, table );
-					} );
-				} );
-
-				it( 'should downcast borderColor attribute change', () => {
-					model.change( writer => writer.setAttribute( 'borderColor', {
-						top: 'deeppink',
-						right: 'deeppink',
-						bottom: 'deeppink',
-						left: 'deeppink'
-					}, table ) );
-
-					assertTableStyle( editor,
-						'border-bottom:42px solid deeppink;' +
-						'border-left:42px solid deeppink;' +
-						'border-right:42px solid deeppink;' +
-						'border-top:42px solid deeppink;'
-					);
-				} );
-
-				it( 'should downcast borderStyle attribute change', () => {
-					model.change( writer => writer.setAttribute( 'borderStyle', {
-						top: 'ridge',
-						right: 'ridge',
-						bottom: 'ridge',
-						left: 'ridge'
-					}, table ) );
-
-					assertTableStyle( editor,
-						'border-bottom:42px ridge #f00;' +
-						'border-left:42px ridge #f00;' +
-						'border-right:42px ridge #f00;' +
-						'border-top:42px ridge #f00;'
-					);
-				} );
-
-				it( 'should downcast borderWidth attribute change', () => {
-					model.change( writer => writer.setAttribute( 'borderWidth', {
-						top: 'thick',
-						right: 'thick',
-						bottom: 'thick',
-						left: 'thick'
-					}, table ) );
-
-					assertTableStyle( editor,
-						'border-bottom:thick solid #f00;' +
-						'border-left:thick solid #f00;' +
-						'border-right:thick solid #f00;' +
-						'border-top:thick solid #f00;'
-					);
-				} );
-
-				it( 'should downcast borderColor attribute removal', () => {
-					model.change( writer => writer.removeAttribute( 'borderColor', table ) );
-
-					assertTableStyle( editor,
-						'border-bottom:42px solid;' +
-						'border-left:42px solid;' +
-						'border-right:42px solid;' +
-						'border-top:42px solid;'
-					);
-				} );
-
-				it( 'should downcast borderStyle attribute removal', () => {
-					model.change( writer => writer.removeAttribute( 'borderStyle', table ) );
-
-					assertTableStyle( editor,
-						'border-bottom:42px #f00;' +
-						'border-left:42px #f00;' +
-						'border-right:42px #f00;' +
-						'border-top:42px #f00;'
-					);
-				} );
-
-				it( 'should downcast borderWidth attribute removal', () => {
-					model.change( writer => writer.removeAttribute( 'borderWidth', table ) );
-
-					assertTableStyle( editor,
-						'border-bottom:solid #f00;' +
-						'border-left:solid #f00;' +
-						'border-right:solid #f00;' +
-						'border-top:solid #f00;'
-					);
-				} );
-
-				it( 'should downcast borderColor, borderStyle and borderWidth attributes removal', () => {
-					model.change( writer => {
-						writer.removeAttribute( 'borderColor', table );
-						writer.removeAttribute( 'borderStyle', table );
-						writer.removeAttribute( 'borderWidth', table );
-					} );
-
-					assertEqualMarkup(
-						editor.getData(),
-						'<figure class="table"><table><tbody><tr><td>foo</td></tr></tbody></table></figure>'
-					);
-				} );
-			} );
+		editor = await ClassicTestEditor.create( editorElement, {
+			plugins: [ TableProperties, Paragraph, TableEditing ]
 		} );
 	} );
 
-	describe( 'background color', () => {
-		it( 'should set proper schema rules', () => {
-			expect( model.schema.checkAttribute( [ '$root', 'table' ], 'backgroundColor' ) ).to.be.true;
-		} );
-
-		describe( 'upcast conversion', () => {
-			it( 'should upcast background-color', () => {
-				editor.setData( '<table style="background-color:#f00"><tr><td>foo</td></tr></table>' );
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				expect( table.getAttribute( 'backgroundColor' ) ).to.equal( '#f00' );
-			} );
-
-			it( 'should upcast from background shorthand', () => {
-				editor.setData( '<table style="background:#f00 center center"><tr><td>foo</td></tr></table>' );
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				expect( table.getAttribute( 'backgroundColor' ) ).to.equal( '#f00' );
-			} );
-		} );
-
-		describe( 'downcast conversion', () => {
-			let table;
-
-			beforeEach( () => {
-				table = createEmptyTable();
-			} );
-
-			it( 'should downcast backgroundColor', () => {
-				model.change( writer => writer.setAttribute( 'backgroundColor', '#f00', table ) );
-
-				assertTableStyle( editor, 'background-color:#f00;' );
-			} );
-		} );
+	afterEach( async () => {
+		editorElement.remove();
+		await editor.destroy();
 	} );
 
-	describe( 'width', () => {
-		it( 'should set proper schema rules', () => {
-			expect( model.schema.checkAttribute( [ '$root', 'table' ], 'width' ) ).to.be.true;
-		} );
-
-		describe( 'upcast conversion', () => {
-			it( 'should upcast width', () => {
-				editor.setData( '<table style="width:1337px"><tr><td>foo</td></tr></table>' );
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				expect( table.getAttribute( 'width' ) ).to.equal( '1337px' );
-			} );
-		} );
-
-		describe( 'downcast conversion', () => {
-			let table;
-
-			beforeEach( () => {
-				table = createEmptyTable();
-			} );
-
-			it( 'should downcast width', () => {
-				model.change( writer => writer.setAttribute( 'width', '1337px', table ) );
-
-				assertTableStyle( editor, 'width:1337px;' );
-			} );
-		} );
+	it( 'should be loaded', () => {
+		expect( editor.plugins.get( TableProperties ) ).to.instanceOf( TableProperties );
 	} );
 
-	describe( 'height', () => {
-		it( 'should set proper schema rules', () => {
-			expect( model.schema.checkAttribute( [ '$root', 'table' ], 'height' ) ).to.be.true;
-		} );
-
-		describe( 'upcast conversion', () => {
-			it( 'should upcast height', () => {
-				editor.setData( '<table style="height:1337px"><tr><td>foo</td></tr></table>' );
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				expect( table.getAttribute( 'height' ) ).to.equal( '1337px' );
-			} );
-		} );
-
-		describe( 'downcast conversion', () => {
-			let table;
-
-			beforeEach( () => {
-				table = createEmptyTable();
-			} );
-
-			it( 'should downcast height', () => {
-				model.change( writer => writer.setAttribute( 'height', '1337px', table ) );
-
-				assertTableStyle( editor, 'height:1337px;' );
-			} );
-		} );
+	it( 'should load TablePropertiesEditing plugin', () => {
+		expect( editor.plugins.get( TablePropertiesEditing ) ).to.instanceOf( TablePropertiesEditing );
 	} );
 
-	describe( 'alignment', () => {
-		it( 'should set proper schema rules', () => {
-			expect( model.schema.checkAttribute( [ '$root', 'table' ], 'alignment' ) ).to.be.true;
-		} );
-
-		describe( 'upcast conversion', () => {
-			it( 'should upcast style="margin-left:auto;margin-right:0" to right value', () => {
-				editor.setData( '<table style="margin-left:auto;margin-right:0"><tr><td>foo</td></tr></table>' );
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				expect( table.getAttribute( 'alignment' ) ).to.equal( 'right' );
-			} );
-
-			it( 'should upcast style="margin-left:0;margin-right:auto" to left value', () => {
-				editor.setData( '<table style="margin-left:0;margin-right:auto"><tr><td>foo</td></tr></table>' );
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				expect( table.getAttribute( 'alignment' ) ).to.equal( 'left' );
-			} );
-
-			it( 'should upcast style="margin-left:auto;margin-right:auto" to center value', () => {
-				editor.setData( '<table style="margin-left:auto;margin-right:auto"><tr><td>foo</td></tr></table>' );
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				expect( table.getAttribute( 'alignment' ) ).to.equal( 'center' );
-			} );
-
-			it( 'should upcast style="margin-left:auto;margin-right:0pt" to right value', () => {
-				editor.setData( '<table style="margin-left:auto;margin-right:0pt"><tr><td>foo</td></tr></table>' );
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				expect( table.getAttribute( 'alignment' ) ).to.equal( 'right' );
-			} );
-
-			it( 'should upcast style="margin-left:auto;margin-right:0%" to right value', () => {
-				editor.setData( '<table style="margin-left:auto;margin-right:0%"><tr><td>foo</td></tr></table>' );
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				expect( table.getAttribute( 'alignment' ) ).to.equal( 'right' );
-			} );
-
-			it( 'should not upcast style="margin-left:auto;margin-right:0.23pt" to right value', () => {
-				editor.setData( '<table style="margin-left:auto;margin-right:0.23pt"><tr><td>foo</td></tr></table>' );
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				expect( table.hasAttribute( 'alignment' ) ).to.be.false;
-			} );
-
-			it( 'should upcast align=right attribute', () => {
-				editor.setData( '<table align="right"><tr><td>foo</td></tr></table>' );
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				expect( table.getAttribute( 'alignment' ) ).to.equal( 'right' );
-			} );
-
-			it( 'should upcast align=left attribute', () => {
-				editor.setData( '<table align="left"><tr><td>foo</td></tr></table>' );
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				expect( table.getAttribute( 'alignment' ) ).to.equal( 'left' );
-			} );
-
-			it( 'should upcast align=center attribute', () => {
-				editor.setData( '<table align="center"><tr><td>foo</td></tr></table>' );
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				expect( table.getAttribute( 'alignment' ) ).to.equal( 'center' );
-			} );
-
-			it( 'should discard align=justify attribute', () => {
-				editor.setData( '<table align="justify"><tr><td>foo</td></tr></table>' );
-				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
-
-				expect( table.hasAttribute( 'alignment' ) ).to.be.false;
-			} );
-		} );
-
-		describe( 'downcast conversion', () => {
-			let table;
-
-			beforeEach( () => {
-				table = createEmptyTable();
-			} );
-
-			it( 'should downcast right alignment', () => {
-				model.change( writer => writer.setAttribute( 'alignment', 'right', table ) );
-
-				assertTableStyle( editor, 'margin-left:auto;margin-right:0;' );
-			} );
-
-			it( 'should downcast left alignment', () => {
-				model.change( writer => writer.setAttribute( 'alignment', 'left', table ) );
-
-				assertTableStyle( editor, 'margin-left:0;margin-right:auto;' );
-			} );
-
-			it( 'should downcast centered alignment', () => {
-				model.change( writer => writer.setAttribute( 'alignment', 'center', table ) );
-
-				assertTableStyle( editor, 'margin-left:auto;margin-right:auto;' );
-			} );
-
-			it( 'should downcast changed alignment', () => {
-				model.change( writer => writer.setAttribute( 'alignment', 'center', table ) );
-
-				assertTableStyle( editor, 'margin-left:auto;margin-right:auto;' );
-
-				model.change( writer => writer.setAttribute( 'alignment', 'right', table ) );
-
-				assertTableStyle( editor, 'margin-left:auto;margin-right:0;' );
-			} );
-
-			it( 'should downcast removed alignment', () => {
-				model.change( writer => writer.setAttribute( 'alignment', 'center', table ) );
-
-				assertTableStyle( editor, 'margin-left:auto;margin-right:auto;' );
-
-				model.change( writer => writer.removeAttribute( 'alignment', table ) );
-
-				assertTableStyle( editor );
-			} );
-		} );
+	it( 'should have pluginName', () => {
+		expect( TableProperties.pluginName ).to.equal( 'TableProperties' );
 	} );
-
-	function createEmptyTable() {
-		setModelData(
-			model,
-			'<table headingRows="0" headingColumns="0">' +
-				'<tableRow>' +
-					'<tableCell>' +
-						'<paragraph>foo</paragraph>' +
-					'</tableCell>' +
-				'</tableRow>' +
-			'</table>'
-		);
-
-		return model.document.getRoot().getNodeByPath( [ 0 ] );
-	}
 } );

+ 670 - 0
packages/ckeditor5-table/tests/tablepropertiesediting.js

@@ -0,0 +1,670 @@
+/**
+ * @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 Paragraph from '@ckeditor/ckeditor5-paragraph/src/paragraph';
+import VirtualTestEditor from '@ckeditor/ckeditor5-core/tests/_utils/virtualtesteditor';
+
+import TableEditing from '../src/tableediting';
+import TableProperties from '../src/tableproperties';
+import { assertTableStyle, assertTRBLAttribute } from './_utils/utils';
+import { setData as setModelData } from '@ckeditor/ckeditor5-engine/src/dev-utils/model';
+import { assertEqualMarkup } from '@ckeditor/ckeditor5-utils/tests/_utils/utils';
+
+describe( 'TableProperties', () => {
+	let editor, model;
+
+	beforeEach( () => {
+		return VirtualTestEditor
+			.create( {
+				plugins: [ TableProperties, Paragraph, TableEditing ]
+			} )
+			.then( newEditor => {
+				editor = newEditor;
+
+				model = editor.model;
+			} );
+	} );
+
+	afterEach( () => {
+		editor.destroy();
+	} );
+
+	it( 'should have pluginName', () => {
+		expect( TableProperties.pluginName ).to.equal( 'TableProperties' );
+	} );
+
+	describe( 'border', () => {
+		it( 'should set proper schema rules', () => {
+			expect( model.schema.checkAttribute( [ '$root', 'table' ], 'borderColor' ) ).to.be.true;
+			expect( model.schema.checkAttribute( [ '$root', 'table' ], 'borderStyle' ) ).to.be.true;
+			expect( model.schema.checkAttribute( [ '$root', 'table' ], 'borderWidth' ) ).to.be.true;
+		} );
+
+		describe( 'upcast conversion', () => {
+			it( 'should upcast border shorthand', () => {
+				editor.setData( '<table style="border:1px solid #f00"><tr><td>foo</td></tr></table>' );
+
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				assertTRBLAttribute( table, 'borderColor', '#f00' );
+				assertTRBLAttribute( table, 'borderStyle', 'solid' );
+				assertTRBLAttribute( table, 'borderWidth', '1px' );
+			} );
+
+			it( 'should upcast border-color shorthand', () => {
+				editor.setData( '<table style="border-color:#f00"><tr><td>foo</td></tr></table>' );
+
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				assertTRBLAttribute( table, 'borderColor', '#f00' );
+			} );
+
+			it( 'should upcast border-style shorthand', () => {
+				editor.setData( '<table style="border-style:ridge"><tr><td>foo</td></tr></table>' );
+
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				assertTRBLAttribute( table, 'borderStyle', 'ridge' );
+			} );
+
+			it( 'should upcast border-width shorthand', () => {
+				editor.setData( '<table style="border-width:1px"><tr><td>foo</td></tr></table>' );
+
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				assertTRBLAttribute( table, 'borderWidth', '1px' );
+			} );
+
+			it( 'should upcast border-top shorthand', () => {
+				editor.setData( '<table style="border-top:1px solid #f00"><tr><td>foo</td></tr></table>' );
+
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				assertTRBLAttribute( table, 'borderColor', '#f00', null, null, null );
+				assertTRBLAttribute( table, 'borderStyle', 'solid', null, null, null );
+				assertTRBLAttribute( table, 'borderWidth', '1px', null, null, null );
+			} );
+
+			it( 'should upcast border-right shorthand', () => {
+				editor.setData( '<table style="border-right:1px solid #f00"><tr><td>foo</td></tr></table>' );
+
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				assertTRBLAttribute( table, 'borderColor', null, '#f00', null, null );
+				assertTRBLAttribute( table, 'borderStyle', null, 'solid', null, null );
+				assertTRBLAttribute( table, 'borderWidth', null, '1px', null, null );
+			} );
+
+			it( 'should upcast border-bottom shorthand', () => {
+				editor.setData( '<table style="border-bottom:1px solid #f00"><tr><td>foo</td></tr></table>' );
+
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				assertTRBLAttribute( table, 'borderColor', null, null, '#f00', null );
+				assertTRBLAttribute( table, 'borderStyle', null, null, 'solid', null );
+				assertTRBLAttribute( table, 'borderWidth', null, null, '1px', null );
+			} );
+
+			it( 'should upcast border-left shorthand', () => {
+				editor.setData( '<table style="border-left:1px solid #f00"><tr><td>foo</td></tr></table>' );
+
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				assertTRBLAttribute( table, 'borderColor', null, null, null, '#f00' );
+				assertTRBLAttribute( table, 'borderStyle', null, null, null, 'solid' );
+				assertTRBLAttribute( table, 'borderWidth', null, null, null, '1px' );
+			} );
+
+			it( 'should upcast border-top-* styles', () => {
+				editor.setData(
+					'<table style="border-top-width:1px;border-top-style:solid;border-top-color:#f00"><tr><td>foo</td></tr></table>'
+				);
+
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				assertTRBLAttribute( table, 'borderColor', '#f00', null, null, null );
+				assertTRBLAttribute( table, 'borderStyle', 'solid', null, null, null );
+				assertTRBLAttribute( table, 'borderWidth', '1px', null, null, null );
+			} );
+
+			it( 'should upcast border-right-* styles', () => {
+				editor.setData(
+					'<table style="border-right-width:1px;border-right-style:solid;border-right-color:#f00"><tr><td>foo</td></tr></table>'
+				);
+
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				assertTRBLAttribute( table, 'borderColor', null, '#f00', null, null );
+				assertTRBLAttribute( table, 'borderStyle', null, 'solid', null, null );
+				assertTRBLAttribute( table, 'borderWidth', null, '1px', null, null );
+			} );
+
+			it( 'should upcast border-bottom-* styles', () => {
+				editor.setData(
+					'<table style="border-bottom-width:1px;border-bottom-style:solid;border-bottom-color:#f00">' +
+					'<tr>' +
+					'<td>foo</td>' +
+					'</tr>' +
+					'</table>'
+				);
+
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				assertTRBLAttribute( table, 'borderColor', null, null, '#f00', null );
+				assertTRBLAttribute( table, 'borderStyle', null, null, 'solid', null );
+				assertTRBLAttribute( table, 'borderWidth', null, null, '1px', null );
+			} );
+
+			it( 'should upcast border-left-* styles', () => {
+				editor.setData(
+					'<table style="border-left-width:1px;border-left-style:solid;border-left-color:#f00"><tr><td>foo</td></tr></table>'
+				);
+
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				assertTRBLAttribute( table, 'borderColor', null, null, null, '#f00' );
+				assertTRBLAttribute( table, 'borderStyle', null, null, null, 'solid' );
+				assertTRBLAttribute( table, 'borderWidth', null, null, null, '1px' );
+			} );
+		} );
+
+		describe( 'downcast conversion', () => {
+			let table;
+
+			beforeEach( () => {
+				table = createEmptyTable();
+			} );
+
+			it( 'should downcast borderColor attribute (same top, right, bottom, left)', () => {
+				model.change( writer => writer.setAttribute( 'borderColor', {
+					top: '#f00',
+					right: '#f00',
+					bottom: '#f00',
+					left: '#f00'
+				}, table ) );
+
+				assertTableStyle( editor, 'border-bottom:#f00;border-left:#f00;border-right:#f00;border-top:#f00;' );
+			} );
+
+			it( 'should downcast borderColor attribute (different top, right, bottom, left)', () => {
+				model.change( writer => writer.setAttribute( 'borderColor', {
+					top: '#f00',
+					right: 'hsla(0, 100%, 50%, 0.5)',
+					bottom: 'deeppink',
+					left: 'rgb(255, 0, 0)'
+				}, table ) );
+
+				assertTableStyle( editor,
+					'border-bottom:deeppink;' +
+					'border-left:rgb(255, 0, 0);' +
+					'border-right:hsla(0, 100%, 50%, 0.5);' +
+					'border-top:#f00;'
+				);
+			} );
+
+			it( 'should downcast borderStyle attribute (same top, right, bottom, left)', () => {
+				model.change( writer => writer.setAttribute( 'borderStyle', {
+					top: 'solid',
+					right: 'solid',
+					bottom: 'solid',
+					left: 'solid'
+				}, table ) );
+
+				assertTableStyle( editor, 'border-bottom:solid;border-left:solid;border-right:solid;border-top:solid;' );
+			} );
+
+			it( 'should downcast borderStyle attribute (different top, right, bottom, left)', () => {
+				model.change( writer => writer.setAttribute( 'borderStyle', {
+					top: 'solid',
+					right: 'ridge',
+					bottom: 'dotted',
+					left: 'dashed'
+				}, table ) );
+
+				assertTableStyle( editor, 'border-bottom:dotted;border-left:dashed;border-right:ridge;border-top:solid;' );
+			} );
+
+			it( 'should downcast borderWidth attribute (same top, right, bottom, left)', () => {
+				model.change( writer => writer.setAttribute( 'borderWidth', {
+					top: '42px',
+					right: '.1em',
+					bottom: '1337rem',
+					left: 'thick'
+				}, table ) );
+
+				assertTableStyle( editor, 'border-bottom:1337rem;border-left:thick;border-right:.1em;border-top:42px;' );
+			} );
+
+			it( 'should downcast borderWidth attribute (different top, right, bottom, left)', () => {
+				model.change( writer => writer.setAttribute( 'borderWidth', {
+					top: '42px',
+					right: '42px',
+					bottom: '42px',
+					left: '42px'
+				}, table ) );
+
+				assertTableStyle( editor, 'border-bottom:42px;border-left:42px;border-right:42px;border-top:42px;' );
+			} );
+
+			it( 'should downcast borderColor, borderStyle and borderWidth attributes together (same top, right, bottom, left)', () => {
+				model.change( writer => {
+					writer.setAttribute( 'borderColor', {
+						top: '#f00',
+						right: '#f00',
+						bottom: '#f00',
+						left: '#f00'
+					}, table );
+
+					writer.setAttribute( 'borderStyle', {
+						top: 'solid',
+						right: 'solid',
+						bottom: 'solid',
+						left: 'solid'
+					}, table );
+
+					writer.setAttribute( 'borderWidth', {
+						top: '42px',
+						right: '42px',
+						bottom: '42px',
+						left: '42px'
+					}, table );
+				} );
+
+				assertTableStyle( editor,
+					'border-bottom:42px solid #f00;' +
+					'border-left:42px solid #f00;' +
+					'border-right:42px solid #f00;' +
+					'border-top:42px solid #f00;'
+				);
+			} );
+
+			it( 'should downcast borderColor, borderStyle and borderWidth attributes together (different top, right, bottom, left)', () => {
+				model.change( writer => {
+					writer.setAttribute( 'borderColor', {
+						top: '#f00',
+						right: 'hsla(0, 100%, 50%, 0.5)',
+						bottom: 'deeppink',
+						left: 'rgb(255, 0, 0)'
+					}, table );
+
+					writer.setAttribute( 'borderStyle', {
+						top: 'solid',
+						right: 'ridge',
+						bottom: 'dotted',
+						left: 'dashed'
+					}, table );
+
+					writer.setAttribute( 'borderWidth', {
+						top: '42px',
+						right: '.1em',
+						bottom: '1337rem',
+						left: 'thick'
+					}, table );
+				} );
+
+				assertTableStyle( editor,
+					'border-bottom:1337rem dotted deeppink;' +
+					'border-left:thick dashed rgb(255, 0, 0);' +
+					'border-right:.1em ridge hsla(0, 100%, 50%, 0.5);' +
+					'border-top:42px solid #f00;'
+				);
+			} );
+
+			describe( 'change attribute', () => {
+				beforeEach( () => {
+					model.change( writer => {
+						writer.setAttribute( 'borderColor', {
+							top: '#f00',
+							right: '#f00',
+							bottom: '#f00',
+							left: '#f00'
+						}, table );
+
+						writer.setAttribute( 'borderStyle', {
+							top: 'solid',
+							right: 'solid',
+							bottom: 'solid',
+							left: 'solid'
+						}, table );
+
+						writer.setAttribute( 'borderWidth', {
+							top: '42px',
+							right: '42px',
+							bottom: '42px',
+							left: '42px'
+						}, table );
+					} );
+				} );
+
+				it( 'should downcast borderColor attribute change', () => {
+					model.change( writer => writer.setAttribute( 'borderColor', {
+						top: 'deeppink',
+						right: 'deeppink',
+						bottom: 'deeppink',
+						left: 'deeppink'
+					}, table ) );
+
+					assertTableStyle( editor,
+						'border-bottom:42px solid deeppink;' +
+						'border-left:42px solid deeppink;' +
+						'border-right:42px solid deeppink;' +
+						'border-top:42px solid deeppink;'
+					);
+				} );
+
+				it( 'should downcast borderStyle attribute change', () => {
+					model.change( writer => writer.setAttribute( 'borderStyle', {
+						top: 'ridge',
+						right: 'ridge',
+						bottom: 'ridge',
+						left: 'ridge'
+					}, table ) );
+
+					assertTableStyle( editor,
+						'border-bottom:42px ridge #f00;' +
+						'border-left:42px ridge #f00;' +
+						'border-right:42px ridge #f00;' +
+						'border-top:42px ridge #f00;'
+					);
+				} );
+
+				it( 'should downcast borderWidth attribute change', () => {
+					model.change( writer => writer.setAttribute( 'borderWidth', {
+						top: 'thick',
+						right: 'thick',
+						bottom: 'thick',
+						left: 'thick'
+					}, table ) );
+
+					assertTableStyle( editor,
+						'border-bottom:thick solid #f00;' +
+						'border-left:thick solid #f00;' +
+						'border-right:thick solid #f00;' +
+						'border-top:thick solid #f00;'
+					);
+				} );
+
+				it( 'should downcast borderColor attribute removal', () => {
+					model.change( writer => writer.removeAttribute( 'borderColor', table ) );
+
+					assertTableStyle( editor,
+						'border-bottom:42px solid;' +
+						'border-left:42px solid;' +
+						'border-right:42px solid;' +
+						'border-top:42px solid;'
+					);
+				} );
+
+				it( 'should downcast borderStyle attribute removal', () => {
+					model.change( writer => writer.removeAttribute( 'borderStyle', table ) );
+
+					assertTableStyle( editor,
+						'border-bottom:42px #f00;' +
+						'border-left:42px #f00;' +
+						'border-right:42px #f00;' +
+						'border-top:42px #f00;'
+					);
+				} );
+
+				it( 'should downcast borderWidth attribute removal', () => {
+					model.change( writer => writer.removeAttribute( 'borderWidth', table ) );
+
+					assertTableStyle( editor,
+						'border-bottom:solid #f00;' +
+						'border-left:solid #f00;' +
+						'border-right:solid #f00;' +
+						'border-top:solid #f00;'
+					);
+				} );
+
+				it( 'should downcast borderColor, borderStyle and borderWidth attributes removal', () => {
+					model.change( writer => {
+						writer.removeAttribute( 'borderColor', table );
+						writer.removeAttribute( 'borderStyle', table );
+						writer.removeAttribute( 'borderWidth', table );
+					} );
+
+					assertEqualMarkup(
+						editor.getData(),
+						'<figure class="table"><table><tbody><tr><td>foo</td></tr></tbody></table></figure>'
+					);
+				} );
+			} );
+		} );
+	} );
+
+	describe( 'background color', () => {
+		it( 'should set proper schema rules', () => {
+			expect( model.schema.checkAttribute( [ '$root', 'table' ], 'backgroundColor' ) ).to.be.true;
+		} );
+
+		describe( 'upcast conversion', () => {
+			it( 'should upcast background-color', () => {
+				editor.setData( '<table style="background-color:#f00"><tr><td>foo</td></tr></table>' );
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				expect( table.getAttribute( 'backgroundColor' ) ).to.equal( '#f00' );
+			} );
+
+			it( 'should upcast from background shorthand', () => {
+				editor.setData( '<table style="background:#f00 center center"><tr><td>foo</td></tr></table>' );
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				expect( table.getAttribute( 'backgroundColor' ) ).to.equal( '#f00' );
+			} );
+		} );
+
+		describe( 'downcast conversion', () => {
+			let table;
+
+			beforeEach( () => {
+				table = createEmptyTable();
+			} );
+
+			it( 'should downcast backgroundColor', () => {
+				model.change( writer => writer.setAttribute( 'backgroundColor', '#f00', table ) );
+
+				assertTableStyle( editor, 'background-color:#f00;' );
+			} );
+		} );
+	} );
+
+	describe( 'width', () => {
+		it( 'should set proper schema rules', () => {
+			expect( model.schema.checkAttribute( [ '$root', 'table' ], 'width' ) ).to.be.true;
+		} );
+
+		describe( 'upcast conversion', () => {
+			it( 'should upcast width', () => {
+				editor.setData( '<table style="width:1337px"><tr><td>foo</td></tr></table>' );
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				expect( table.getAttribute( 'width' ) ).to.equal( '1337px' );
+			} );
+		} );
+
+		describe( 'downcast conversion', () => {
+			let table;
+
+			beforeEach( () => {
+				table = createEmptyTable();
+			} );
+
+			it( 'should downcast width', () => {
+				model.change( writer => writer.setAttribute( 'width', '1337px', table ) );
+
+				assertTableStyle( editor, 'width:1337px;' );
+			} );
+		} );
+	} );
+
+	describe( 'height', () => {
+		it( 'should set proper schema rules', () => {
+			expect( model.schema.checkAttribute( [ '$root', 'table' ], 'height' ) ).to.be.true;
+		} );
+
+		describe( 'upcast conversion', () => {
+			it( 'should upcast height', () => {
+				editor.setData( '<table style="height:1337px"><tr><td>foo</td></tr></table>' );
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				expect( table.getAttribute( 'height' ) ).to.equal( '1337px' );
+			} );
+		} );
+
+		describe( 'downcast conversion', () => {
+			let table;
+
+			beforeEach( () => {
+				table = createEmptyTable();
+			} );
+
+			it( 'should downcast height', () => {
+				model.change( writer => writer.setAttribute( 'height', '1337px', table ) );
+
+				assertTableStyle( editor, 'height:1337px;' );
+			} );
+		} );
+	} );
+
+	describe( 'alignment', () => {
+		it( 'should set proper schema rules', () => {
+			expect( model.schema.checkAttribute( [ '$root', 'table' ], 'alignment' ) ).to.be.true;
+		} );
+
+		describe( 'upcast conversion', () => {
+			it( 'should upcast style="margin-left:auto;margin-right:0" to right value', () => {
+				editor.setData( '<table style="margin-left:auto;margin-right:0"><tr><td>foo</td></tr></table>' );
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				expect( table.getAttribute( 'alignment' ) ).to.equal( 'right' );
+			} );
+
+			it( 'should upcast style="margin-left:0;margin-right:auto" to left value', () => {
+				editor.setData( '<table style="margin-left:0;margin-right:auto"><tr><td>foo</td></tr></table>' );
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				expect( table.getAttribute( 'alignment' ) ).to.equal( 'left' );
+			} );
+
+			it( 'should upcast style="margin-left:auto;margin-right:auto" to center value', () => {
+				editor.setData( '<table style="margin-left:auto;margin-right:auto"><tr><td>foo</td></tr></table>' );
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				expect( table.getAttribute( 'alignment' ) ).to.equal( 'center' );
+			} );
+
+			it( 'should upcast style="margin-left:auto;margin-right:0pt" to right value', () => {
+				editor.setData( '<table style="margin-left:auto;margin-right:0pt"><tr><td>foo</td></tr></table>' );
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				expect( table.getAttribute( 'alignment' ) ).to.equal( 'right' );
+			} );
+
+			it( 'should upcast style="margin-left:auto;margin-right:0%" to right value', () => {
+				editor.setData( '<table style="margin-left:auto;margin-right:0%"><tr><td>foo</td></tr></table>' );
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				expect( table.getAttribute( 'alignment' ) ).to.equal( 'right' );
+			} );
+
+			it( 'should not upcast style="margin-left:auto;margin-right:0.23pt" to right value', () => {
+				editor.setData( '<table style="margin-left:auto;margin-right:0.23pt"><tr><td>foo</td></tr></table>' );
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				expect( table.hasAttribute( 'alignment' ) ).to.be.false;
+			} );
+
+			it( 'should upcast align=right attribute', () => {
+				editor.setData( '<table align="right"><tr><td>foo</td></tr></table>' );
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				expect( table.getAttribute( 'alignment' ) ).to.equal( 'right' );
+			} );
+
+			it( 'should upcast align=left attribute', () => {
+				editor.setData( '<table align="left"><tr><td>foo</td></tr></table>' );
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				expect( table.getAttribute( 'alignment' ) ).to.equal( 'left' );
+			} );
+
+			it( 'should upcast align=center attribute', () => {
+				editor.setData( '<table align="center"><tr><td>foo</td></tr></table>' );
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				expect( table.getAttribute( 'alignment' ) ).to.equal( 'center' );
+			} );
+
+			it( 'should discard align=justify attribute', () => {
+				editor.setData( '<table align="justify"><tr><td>foo</td></tr></table>' );
+				const table = model.document.getRoot().getNodeByPath( [ 0 ] );
+
+				expect( table.hasAttribute( 'alignment' ) ).to.be.false;
+			} );
+		} );
+
+		describe( 'downcast conversion', () => {
+			let table;
+
+			beforeEach( () => {
+				table = createEmptyTable();
+			} );
+
+			it( 'should downcast right alignment', () => {
+				model.change( writer => writer.setAttribute( 'alignment', 'right', table ) );
+
+				assertTableStyle( editor, 'margin-left:auto;margin-right:0;' );
+			} );
+
+			it( 'should downcast left alignment', () => {
+				model.change( writer => writer.setAttribute( 'alignment', 'left', table ) );
+
+				assertTableStyle( editor, 'margin-left:0;margin-right:auto;' );
+			} );
+
+			it( 'should downcast centered alignment', () => {
+				model.change( writer => writer.setAttribute( 'alignment', 'center', table ) );
+
+				assertTableStyle( editor, 'margin-left:auto;margin-right:auto;' );
+			} );
+
+			it( 'should downcast changed alignment', () => {
+				model.change( writer => writer.setAttribute( 'alignment', 'center', table ) );
+
+				assertTableStyle( editor, 'margin-left:auto;margin-right:auto;' );
+
+				model.change( writer => writer.setAttribute( 'alignment', 'right', table ) );
+
+				assertTableStyle( editor, 'margin-left:auto;margin-right:0;' );
+			} );
+
+			it( 'should downcast removed alignment', () => {
+				model.change( writer => writer.setAttribute( 'alignment', 'center', table ) );
+
+				assertTableStyle( editor, 'margin-left:auto;margin-right:auto;' );
+
+				model.change( writer => writer.removeAttribute( 'alignment', table ) );
+
+				assertTableStyle( editor );
+			} );
+		} );
+	} );
+
+	function createEmptyTable() {
+		setModelData(
+			model,
+			'<table headingRows="0" headingColumns="0">' +
+				'<tableRow>' +
+					'<tableCell>' +
+						'<paragraph>foo</paragraph>' +
+					'</tableCell>' +
+				'</tableRow>' +
+			'</table>'
+		);
+
+		return model.document.getRoot().getNodeByPath( [ 0 ] );
+	}
+} );