Browse Source

Make sure we check the right state.

Piotrek Koszuliński 7 years ago
parent
commit
8f067de790
1 changed files with 11 additions and 5 deletions
  1. 11 5
      packages/ckeditor5-enter/tests/shiftentercommand.js

+ 11 - 5
packages/ckeditor5-enter/tests/shiftentercommand.js

@@ -295,7 +295,7 @@ describe( 'ShiftEnterCommand', () => {
 			model.change( () => {
 				setData( model, '<p><inlineLimit>F[oo.</inlineLimit>B]ar.</p>' );
 
-				// Enforce command refresh because of 'transparent' batch.
+				// Refresh it manually because we're in the middle of a change block.
 				command.refresh();
 
 				expect( command.isEnabled ).to.equal( false );
@@ -307,7 +307,7 @@ describe( 'ShiftEnterCommand', () => {
 			model.change( () => {
 				setData( model, '<p>F[oo<inlineLimit>Bar].</inlineLimit></p>' );
 
-				// Enforce command refresh because of 'transparent' batch.
+				// Refresh it manually because we're in the middle of a change block.
 				command.refresh();
 
 				expect( command.isEnabled ).to.equal( false );
@@ -319,7 +319,7 @@ describe( 'ShiftEnterCommand', () => {
 			model.change( () => {
 				setData( model, '<img>[]</img>' );
 
-				// Enforce command refresh because of 'transparent' batch.
+				// Refresh it manually because we're in the middle of a change block.
 				command.refresh();
 
 				expect( command.isEnabled ).to.equal( false );
@@ -327,17 +327,23 @@ describe( 'ShiftEnterCommand', () => {
 		} );
 
 		it( 'should be disabled for non-collapsed selection which starts in element inside a block limit element', () => {
-			model.enqueueChange( 'transparent', () => {
+			model.change( () => {
 				setData( model, '<blockLimit><p>F[oo.</p></blockLimit><p>B]ar.</p>' );
 
+				// Refresh it manually because we're in the middle of a change block.
+				command.refresh();
+
 				expect( command.isEnabled ).to.equal( false );
 			} );
 		} );
 
 		it( 'should be disabled for non-collapsed selection which ends in element inside a block limit element', () => {
-			model.enqueueChange( 'transparent', () => {
+			model.change( () => {
 				setData( model, '<p>Fo[o.</p><blockLimit><p>Bar].</p></blockLimit>' );
 
+				// Refresh it manually because we're in the middle of a change block.
+				command.refresh();
+
 				expect( command.isEnabled ).to.equal( false );
 			} );
 		} );