From ef102a5952353676982f9853400bab618c6e90b5 Mon Sep 17 00:00:00 2001 From: Aleksey Semikozov Date: Mon, 16 Feb 2026 14:06:00 -0300 Subject: [PATCH 1/3] PivotGrid: enable tests for horizontal dragging between lines --- .../DevExpress.ui.widgets.pivotGrid/sortable.tests.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/packages/devextreme/testing/tests/DevExpress.ui.widgets.pivotGrid/sortable.tests.js b/packages/devextreme/testing/tests/DevExpress.ui.widgets.pivotGrid/sortable.tests.js index 9d72eb885e22..baa886618e03 100644 --- a/packages/devextreme/testing/tests/DevExpress.ui.widgets.pivotGrid/sortable.tests.js +++ b/packages/devextreme/testing/tests/DevExpress.ui.widgets.pivotGrid/sortable.tests.js @@ -197,7 +197,7 @@ QUnit.module('sortable without containers', () => { assert.equal($items.eq(3).text(), '4', 'item 3 text'); }); - QUnit.skip('horizontal dragging between lines', function(assert) { + QUnit.test('horizontal dragging between lines', function(assert) { createHorizontalMarkUp(HORIZONTAL_WIDTH_SMALL, true); const $sortable = $('#sortable').dxSortableOld({ @@ -229,7 +229,7 @@ QUnit.module('sortable without containers', () => { assert.equal($items.eq(5).text(), '6', 'item 5 text'); }); - QUnit.skip('horizontal dragging between lines to the end of the first line', function(assert) { + QUnit.test('horizontal dragging between lines to the end of the first line', function(assert) { createHorizontalMarkUp(HORIZONTAL_WIDTH_SMALL, true); const $sortable = $('#sortable').dxSortableOld({ @@ -490,7 +490,7 @@ QUnit.module('sortable without containers', () => { assert.ok(!changedArgs, 'changed called'); }); - QUnit.skip('dragging not allowed item', function(assert) { + QUnit.test('dragging not allowed item', function(assert) { let changedArgs; const $sortable = $('#sortable').dxSortableOld({ itemSelector: '.test-item', @@ -1768,7 +1768,7 @@ QUnit.module('Horizontal direction. RTL', { assert.strictEqual(this.sortable.option('onChanged').lastCall.args[0].targetIndex, 2); }); - QUnit.skip('horizontal dragging between lines', function(assert) { + QUnit.test('horizontal dragging between lines', function(assert) { createHorizontalMarkUp(HORIZONTAL_WIDTH_SMALL, true); const $sortable = this.createSortable({}); From f201fe10a375f6a44a4ca54dd0e86347f41921d8 Mon Sep 17 00:00:00 2001 From: Aleksey Semikozov Date: Mon, 16 Feb 2026 14:49:19 -0300 Subject: [PATCH 2/3] PivotGrid: fix item order in sortable tests for horizontal dragging --- .../DevExpress.ui.widgets.pivotGrid/sortable.tests.js | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/packages/devextreme/testing/tests/DevExpress.ui.widgets.pivotGrid/sortable.tests.js b/packages/devextreme/testing/tests/DevExpress.ui.widgets.pivotGrid/sortable.tests.js index baa886618e03..ee232f97df69 100644 --- a/packages/devextreme/testing/tests/DevExpress.ui.widgets.pivotGrid/sortable.tests.js +++ b/packages/devextreme/testing/tests/DevExpress.ui.widgets.pivotGrid/sortable.tests.js @@ -225,8 +225,8 @@ QUnit.module('sortable without containers', () => { assert.equal($items.eq(1).text(), '3', 'item 1 text'); assert.equal($items.eq(2).text(), '4', 'item 2 text'); assert.equal($items.eq(3).text(), '5', 'item 3 text'); - assert.equal($items.eq(4).text(), '1', 'item 4 text'); - assert.equal($items.eq(5).text(), '6', 'item 5 text'); + assert.equal($items.eq(4).text(), '6', 'item 4 text'); + assert.equal($items.eq(5).text(), '1', 'item 5 text'); }); QUnit.test('horizontal dragging between lines to the end of the first line', function(assert) { @@ -500,6 +500,8 @@ QUnit.module('sortable without containers', () => { } }); + $('
').addClass('not-test-item').text('X').appendTo('.test-container'); + const $item = $sortable.find('.not-test-item').eq(0); const offset = $item.offset(); @@ -1790,8 +1792,8 @@ QUnit.module('Horizontal direction. RTL', { assert.equal($items.eq(1).text(), '3', 'item 1 text'); assert.equal($items.eq(2).text(), '4', 'item 2 text'); assert.equal($items.eq(3).text(), '5', 'item 3 text'); - assert.equal($items.eq(4).text(), '1', 'item 4 text'); - assert.equal($items.eq(5).text(), '6', 'item 5 text'); + assert.equal($items.eq(4).text(), '6', 'item 4 text'); + assert.equal($items.eq(5).text(), '1', 'item 5 text'); assert.strictEqual(this.sortable.option('onChanged').lastCall.args[0].sourceIndex, 0); assert.strictEqual(this.sortable.option('onChanged').lastCall.args[0].targetIndex, 5); From c945d1261f3241d8c83da5d469bae2912045f08f Mon Sep 17 00:00:00 2001 From: Aleksey Semikozov Date: Mon, 16 Feb 2026 15:03:32 -0300 Subject: [PATCH 3/3] PivotGrid: fix target index in RTL horizontal dragging test --- .../tests/DevExpress.ui.widgets.pivotGrid/sortable.tests.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/devextreme/testing/tests/DevExpress.ui.widgets.pivotGrid/sortable.tests.js b/packages/devextreme/testing/tests/DevExpress.ui.widgets.pivotGrid/sortable.tests.js index ee232f97df69..f9d0bd43a64f 100644 --- a/packages/devextreme/testing/tests/DevExpress.ui.widgets.pivotGrid/sortable.tests.js +++ b/packages/devextreme/testing/tests/DevExpress.ui.widgets.pivotGrid/sortable.tests.js @@ -1796,7 +1796,7 @@ QUnit.module('Horizontal direction. RTL', { assert.equal($items.eq(5).text(), '1', 'item 5 text'); assert.strictEqual(this.sortable.option('onChanged').lastCall.args[0].sourceIndex, 0); - assert.strictEqual(this.sortable.option('onChanged').lastCall.args[0].targetIndex, 5); + assert.strictEqual(this.sortable.option('onChanged').lastCall.args[0].targetIndex, 6); }); QUnit.test('drag to the end of the container', function(assert) {