Skip to content
This repository has been archived by the owner on Jun 26, 2020. It is now read-only.

Commit

Permalink
Tests: Enabled unit test that was blocked by a bug that has been rece…
Browse files Browse the repository at this point in the history
…ntly resolved.
  • Loading branch information
mlewand committed Mar 11, 2020
1 parent 700fbb1 commit 3ec288a
Showing 1 changed file with 22 additions and 24 deletions.
46 changes: 22 additions & 24 deletions tests/commands/removerowcommand.js
Original file line number Diff line number Diff line change
Expand Up @@ -151,30 +151,28 @@ describe( 'RemoveRowCommand', () => {
] ) );
} );

// This test is blocked by (#6370).
//
// it( 'should properly remove tailing rows', () => {
// setData( model, modelTable( [
// [ '00', '01' ],
// [ '10', '11' ],
// [ '20', '21' ],
// [ '30', '31' ]
// ] ) );

// const tableSelection = editor.plugins.get( TableSelection );
// const modelRoot = model.document.getRoot();
// tableSelection._setCellSelection(
// modelRoot.getNodeByPath( [ 0, 2, 0 ] ),
// modelRoot.getNodeByPath( [ 0, 36, 0 ] )
// );

// command.execute();

// assertEqualMarkup( getData( model ), modelTable( [
// [ '00', '01' ],
// [ '10', '11[]' ]
// ] ) );
// } );
it( 'should properly remove tailing rows', () => {
setData( model, modelTable( [
[ '00', '01' ],
[ '10', '11' ],
[ '20', '21' ],
[ '30', '31' ]
] ) );

const tableSelection = editor.plugins.get( TableSelection );
const modelRoot = model.document.getRoot();
tableSelection._setCellSelection(
modelRoot.getNodeByPath( [ 0, 2, 0 ] ),
modelRoot.getNodeByPath( [ 0, 3, 0 ] )
);

command.execute();

assertEqualMarkup( getData( model ), modelTable( [
[ '00', '01' ],
[ '[]10', '11' ]
] ) );
} );

it( 'should properly remove beginning rows', () => {
setData( model, modelTable( [
Expand Down

0 comments on commit 3ec288a

Please sign in to comment.