Skip to content

Commit 5b7bb4a

Browse files
CI: improve QUnit tests stability (#34150)
1 parent a334289 commit 5b7bb4a

67 files changed

Lines changed: 407 additions & 542 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/workflows/qunit_tests.yml

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -129,24 +129,23 @@ jobs:
129129
needs: build
130130
runs-on: devextreme-shr2
131131
name: Constel ${{ matrix.CONSTEL }}${{ matrix.TIMEZONE != '' && format('-{0}', matrix.TIMEZONE) || '' }}${{ matrix.CSP == 'false' && '-no-csp' || '' }}
132-
timeout-minutes: 20
132+
timeout-minutes: 25
133133
strategy:
134134
fail-fast: false
135135
matrix:
136136
CONSTEL: [
137137
misc,
138138
ui,
139139
ui.editors,
140-
ui.grid,
141140
ui.scheduler,
142141
]
143142
TIMEZONE: ['']
144143
CSP: ['true']
145144
useJQuery: ['false']
146145
MAX_WORKERS: ['']
147146
include:
148-
- CONSTEL: export
149-
MAX_WORKERS: '3'
147+
- CONSTEL: ui.grid
148+
MAX_WORKERS: '1'
150149
useJQuery: 'false'
151150
CSP: 'true'
152151
- CONSTEL: ui.scheduler
@@ -158,7 +157,7 @@ jobs:
158157
- CONSTEL: ui.widgets
159158
CSP: 'false'
160159
useJQuery: 'false'
161-
MAX_WORKERS: '3'
160+
MAX_WORKERS: '1'
162161

163162
steps:
164163
- name: Get sources
@@ -203,7 +202,7 @@ jobs:
203202
useJQuery: 'true'
204203
useShadowDom: 'true'
205204
useCsp: 'true'
206-
maxWorkers: ${{ (matrix.constel == 'ui.editors' || matrix.constel == 'ui.widgets') && '3' || '' }}
205+
maxWorkers: ${{ (matrix.constel == 'ui.editors' || matrix.constel == 'ui.widgets') && '1' || '' }}
207206

208207
notify:
209208
runs-on: ubuntu-latest

.github/workflows/testcafe_tests.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -219,11 +219,11 @@ jobs:
219219
220220
base_matrix_end=$(cat <<'JSON'
221221
[
222-
{ "componentFolder": "common", "name": "common" },
222+
{ "componentFolder": "common", "name": "common", "concurrency": 2 },
223223
224-
{ "name": "generic", "theme": "generic.light" },
225-
{ "name": "material", "theme": "material.blue.light" },
226-
{ "name": "material - compact", "theme": "material.blue.light.compact" },
224+
{ "name": "generic", "theme": "generic.light", "concurrency": 2 },
225+
{ "name": "material", "theme": "material.blue.light", "concurrency": 2 },
226+
{ "name": "material - compact", "theme": "material.blue.light.compact", "concurrency": 2 },
227227
228228
{ "componentFolder": "cardView", "name": "cardView" },
229229
{ "componentFolder": "dataGrid", "name": "dataGrid (1/4)", "indices": "1/4", "cache": true, "concurrency": 6 },

e2e/testcafe-devextreme/tests/cardView/columnChooser/functional.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -98,11 +98,13 @@ testsFactory({
9898
await t.click(
9999
cardView.getColumnChooser().getCheckbox(0),
100100
);
101+
await t.expect(cardView.isReady()).ok();
101102
},
102103
async showFirstColumn(t: TestController, cardView: CardView) {
103104
await t.click(
104105
cardView.getColumnChooser().getCheckbox(0),
105106
);
107+
await t.expect(cardView.isReady()).ok();
106108
},
107109
async assertFirstColumnVisible(t: TestController, cardView: CardView) {
108110
await t.expect(cardView.getColumnChooser().getColumnsCount()).eql(3);

e2e/testcafe-devextreme/tests/cardView/headerFilter/common.functional.ts

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ test('should support custom translations', async (t) => {
5959
height: 600,
6060
}));
6161

62-
test.meta({ unstable: true })('Filtering different data types', async (t) => {
62+
test('Filtering different data types', async (t) => {
6363
const cardView = new CardView('#container');
6464
const headerPanel = cardView.getHeaderPanel();
6565

@@ -100,8 +100,14 @@ test.meta({ unstable: true })('Filtering different data types', async (t) => {
100100
// Date type
101101
await t
102102
.click(headerPanel.getHeaderItem(4).getFilterIcon())
103+
.expect(treeView.getNodes().count)
104+
.gte(1)
103105
.click(treeView.getNode(0).getExpandButton())
106+
.expect(treeView.getNodes().count)
107+
.gte(2)
104108
.click(treeView.getNode(1).getExpandButton())
109+
.expect(treeView.getNodes().count)
110+
.gte(3)
105111
.click(treeView.getNode(2).getCheckBox().element)
106112
.click(doneBtn.element)
107113
.expect(cardView.getCards().count)

e2e/testcafe-devextreme/tests/common/pivotGrid/fieldPanel/dragAndDropFieldItems.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import { MouseAction, MouseUpEvents } from '../../../../helpers/mouseUpEvents';
66
import { testScreenshot } from '../../../../helpers/themeUtils';
77
import { DRAG_MOUSE_OPTIONS } from '../const';
88

9-
fixture.disablePageReloads`pivotGrid_fieldPanel_drag-n-drop`
9+
fixture`pivotGrid_fieldPanel_drag-n-drop`
1010
.page(url(__dirname, '../../../container.html'));
1111

1212
const PIVOT_GRID_SELECTOR = '#container';
@@ -75,7 +75,7 @@ test('Field panel items markup in the middle of the drag-n-drop', async (t) => {
7575
});
7676
});
7777

78-
test.meta({ unstable: true })('Should show d-n-d indicator during drag to first place in columns fields', async (t) => {
78+
test('Should show d-n-d indicator during drag to first place in columns fields', async (t) => {
7979
const { takeScreenshot, compareResults } = createScreenshotsComparer(t);
8080

8181
const pivotGrid = new PivotGrid(PIVOT_GRID_SELECTOR);
-75 Bytes
Loading
203 Bytes
Loading
231 Bytes
Loading
233 Bytes
Loading

e2e/testcafe-devextreme/tests/dataGrid/common/aiColumn/virtualScrolling.functional.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ const deleteGlobalVariables = ClientFunction((): void => {
4141
delete (window as any).aiResolve;
4242
});
4343

44-
test('DataGrid should send an AI request for rendered rows after scrolling without changing the page index', async (t) => {
44+
test.meta({ unstable: true })('DataGrid should send an AI request for rendered rows after scrolling without changing the page index', async (t) => {
4545
// arrange
4646
const dataGrid = new DataGrid(DATA_GRID_SELECTOR);
4747

0 commit comments

Comments
 (0)