Other: Aligned the implementation to the new Command API (see https:/…

…/github.com/ckeditor/ckeditor5-core/issues/88). BREAKING CHANGES: The command API has been changed.
ckeditor · Jun 13, 2017 · 36a976e · 36a976e
2 parents 9d1d5e2 + 93373a8
commit 36a976e
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/src/boldengine.js b/src/boldengine.js
@@ -47,6 +47,6 @@ export default class BoldEngine extends Plugin {
 			.toAttribute( BOLD, true );
 
 		// Create bold command.
-		editor.commands.set( BOLD, new ToggleAttributeCommand( editor, BOLD ) );
+		editor.commands.add( BOLD, new ToggleAttributeCommand( editor, BOLD ) );
 	}
 }
diff --git a/src/italicengine.js b/src/italicengine.js
@@ -47,6 +47,6 @@ export default class ItalicEngine extends Plugin {
 			.toAttribute( ITALIC, true );
 
 		// Create italic command.
-		editor.commands.set( ITALIC, new ToggleAttributeCommand( editor, ITALIC ) );
+		editor.commands.add( ITALIC, new ToggleAttributeCommand( editor, ITALIC ) );
 	}
 }
diff --git a/tests/boldengine.js b/tests/boldengine.js
@@ -37,8 +37,6 @@ describe( 'BoldEngine', () => {
 
 	describe( 'command', () => {
 		it( 'should register bold command', () => {
-			expect( editor.commands.has( 'bold' ) ).to.be.true;
-
 			const command = editor.commands.get( 'bold' );
 
 			expect( command ).to.be.instanceOf( ToggleAttributeCommand );

diff --git a/tests/italicengine.js b/tests/italicengine.js
@@ -26,6 +26,10 @@ describe( 'ItalicEngine', () => {
 		} );
 	} );
 
+	afterEach( () => {
+		return editor.destroy();
+	} );
+
 	it( 'should be loaded', () => {
 		expect( editor.plugins.get( ItalicEngine ) ).to.be.instanceOf( ItalicEngine );
 	} );
@@ -37,8 +41,6 @@ describe( 'ItalicEngine', () => {
 
 	describe( 'command', () => {
 		it( 'should register italic command', () => {
-			expect( editor.commands.has( 'italic' ) ).to.be.true;
-
 			const command = editor.commands.get( 'italic' );
 
 			expect( command ).to.be.instanceOf( ToggleAttributeCommand );