瀏覽代碼

Fix link protocol tests.

Maciej Gołaszewski 5 年之前
父節點
當前提交
1029b25f4a
共有 1 個文件被更改,包括 22 次插入14 次删除
  1. 22 14
      packages/ckeditor5-link/tests/linkui.js

+ 22 - 14
packages/ckeditor5-link/tests/linkui.js

@@ -8,7 +8,7 @@
 import ClassicTestEditor from '@ckeditor/ckeditor5-core/tests/_utils/classictesteditor';
 import testUtils from '@ckeditor/ckeditor5-core/tests/_utils/utils';
 import { keyCodes } from '@ckeditor/ckeditor5-utils/src/keyboard';
-import { setData as setModelData, getData as getModelData } from '@ckeditor/ckeditor5-engine/src/dev-utils/model';
+import { getData as getModelData, setData as setModelData } from '@ckeditor/ckeditor5-engine/src/dev-utils/model';
 import { getData as getViewData } from '@ckeditor/ckeditor5-engine/src/dev-utils/view';
 
 import Paragraph from '@ckeditor/ckeditor5-paragraph/src/paragraph';
@@ -939,7 +939,7 @@ describe( 'LinkUI', () => {
 
 						expect( defaultProtocol ).to.equal( 'https://' );
 
-						editor.destroy();
+						return editor.destroy();
 					} );
 			} );
 
@@ -952,56 +952,64 @@ describe( 'LinkUI', () => {
 
 			it( 'should not add a protocol to the local links even when `config.link.defaultProtocol` configured', () => {
 				return createEditorWithDefaultProtocol( 'http://' ).then( ( { editor, formView } ) => {
+					const linkCommandSpy = sinon.spy( editor.commands.get( 'link' ), 'execute' );
 					formView.urlInputView.fieldView.value = '#test';
 					formView.fire( 'submit' );
 
-					expect( formView.urlInputView.fieldView.value ).to.equal( '#test' );
+					sinon.assert.calledWith( linkCommandSpy, '#test', sinon.match.any );
 
-					editor.destroy();
+					return editor.destroy();
 				} );
 			} );
 
 			it( 'should not add a protocol to the relative links even when `config.link.defaultProtocol` configured', () => {
 				return createEditorWithDefaultProtocol( 'http://' ).then( ( { editor, formView } ) => {
+					const linkCommandSpy = sinon.spy( editor.commands.get( 'link' ), 'execute' );
 					formView.urlInputView.fieldView.value = '/test.html';
 					formView.fire( 'submit' );
 
-					expect( formView.urlInputView.fieldView.value ).to.equal( '/test.html' );
+					sinon.assert.calledWith( linkCommandSpy, '/test.html', sinon.match.any );
 
-					editor.destroy();
+					return editor.destroy();
 				} );
 			} );
 
 			it( 'should not add a protocol when given provided within the value even when `config.link.defaultProtocol` configured', () => {
 				return createEditorWithDefaultProtocol( 'http://' ).then( ( { editor, formView } ) => {
+					const linkCommandSpy = sinon.spy( editor.commands.get( 'link' ), 'execute' );
 					formView.urlInputView.fieldView.value = 'http://example.com';
 					formView.fire( 'submit' );
 
 					expect( formView.urlInputView.fieldView.value ).to.equal( 'http://example.com' );
+					sinon.assert.calledWith( linkCommandSpy, 'http://example.com', sinon.match.any );
 
-					editor.destroy();
+					return editor.destroy();
 				} );
 			} );
 
 			it( 'should use the "http://" protocol when it\'s configured', () => {
 				return createEditorWithDefaultProtocol( 'http://' ).then( ( { editor, formView } ) => {
+					const linkCommandSpy = sinon.spy( editor.commands.get( 'link' ), 'execute' );
+
 					formView.urlInputView.fieldView.value = 'ckeditor.com';
 					formView.fire( 'submit' );
 
-					expect( formView.urlInputView.fieldView.value ).to.equal( 'http://ckeditor.com' );
+					sinon.assert.calledWith( linkCommandSpy, 'http://ckeditor.com', sinon.match.any );
 
-					editor.destroy();
+					return editor.destroy();
 				} );
 			} );
 
 			it( 'should use the "http://" protocol when it\'s configured and form input value contains "www."', () => {
 				return createEditorWithDefaultProtocol( 'http://' ).then( ( { editor, formView } ) => {
+					const linkCommandSpy = sinon.spy( editor.commands.get( 'link' ), 'execute' );
+
 					formView.urlInputView.fieldView.value = 'www.ckeditor.com';
 					formView.fire( 'submit' );
 
-					expect( formView.urlInputView.fieldView.value ).to.equal( 'http://www.ckeditor.com' );
+					sinon.assert.calledWith( linkCommandSpy, 'http://www.ckeditor.com', sinon.match.any );
 
-					editor.destroy();
+					return editor.destroy();
 				} );
 			} );
 
@@ -1016,7 +1024,7 @@ describe( 'LinkUI', () => {
 						'[<$text linkHref="http://ckeditor.com">ckeditor.com</$text>]'
 					);
 
-					editor.destroy();
+					return editor.destroy();
 				} );
 			} );
 
@@ -1032,7 +1040,7 @@ describe( 'LinkUI', () => {
 						'[<$text linkHref="mailto:email@example.com">email@example.com</$text>]'
 					);
 
-					editor.destroy();
+					return editor.destroy();
 				} );
 			} );
 
@@ -1044,7 +1052,7 @@ describe( 'LinkUI', () => {
 
 					expect( formView.urlInputView.fieldView.value ).to.equal( 'mailto:test@example.com' );
 
-					editor.destroy();
+					return editor.destroy();
 				} );
 			} );
 		} );