From 2b51c39af841727a52c0bc5b49dde5e56854b5ed Mon Sep 17 00:00:00 2001 From: Aleksander Nowodzinski Date: Thu, 9 Mar 2017 12:56:08 +0100 Subject: [PATCH] Tests: Removed dangerous * import from InlineEditorUI test. --- tests/inlineeditorui.js | 19 ++++++++++++------- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/tests/inlineeditorui.js b/tests/inlineeditorui.js index 5f7c3d6..f8a6dae 100644 --- a/tests/inlineeditorui.js +++ b/tests/inlineeditorui.js @@ -13,8 +13,8 @@ import InlineEditorUIView from '../src/inlineeditoruiview'; import ClassicTestEditor from '@ckeditor/ckeditor5-core/tests/_utils/classictesteditor'; import FocusTracker from '@ckeditor/ckeditor5-utils/src/focustracker'; -import * as enableToolbarKeyboardFocus from '@ckeditor/ckeditor5-ui/src/toolbar/enabletoolbarkeyboardfocus'; +import { keyCodes } from '@ckeditor/ckeditor5-utils/src/keyboard'; import testUtils from '@ckeditor/ckeditor5-core/tests/_utils/utils'; import utils from '@ckeditor/ckeditor5-utils/tests/_utils/utils'; @@ -136,15 +136,20 @@ describe( 'InlineEditorUI', () => { } ); it( 'initializes keyboard navigation between view#toolbar and view#editable', () => { - const spy = testUtils.sinon.spy( enableToolbarKeyboardFocus, 'default' ); + const spy = testUtils.sinon.spy( view.toolbar, 'focus' ); return ui.init().then( () => { - sinon.assert.calledWithExactly( spy, { - origin: editor.editing.view, - originFocusTracker: ui.focusTracker, - originKeystrokeHandler: editor.keystrokes, - toolbar: view.toolbar + ui.focusTracker.isFocused = true; + ui.view.toolbar.focusTracker.isFocused = false; + + editor.keystrokes.press( { + keyCode: keyCodes.f10, + altKey: true, + preventDefault: sinon.spy(), + stopPropagation: sinon.spy() } ); + + sinon.assert.calledOnce( spy ); } ); } ); } );