Skip to content

Commit b3a8a12

Browse files
committed
fixup! front: reduce and remove unnecessary timeouts for e2e tests
1 parent d753c93 commit b3a8a12

4 files changed

+11
-23
lines changed

front/tests/006-stdcm.spec.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ test.describe('Verify stdcm simulation page', () => {
1313
test.use({ viewport: { width: 1920, height: 1080 } });
1414

1515
let stdcmPage: STDCMPage;
16-
let stdcmLinkedTrainPage = STDCMLinkedTrainPage;
16+
let stdcmLinkedTrainPage: STDCMLinkedTrainPage;
1717

1818
let infra: Infra;
1919
let createdTowedRollingStock: TowedRollingStock;

front/tests/013-stdcm-simulation-sheet.spec.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ test.describe('Verify stdcm simulation page', () => {
8383
const [newPage] = await Promise.all([context.waitForEvent('page'), stdcmPage.startNewQuery()]);
8484
await newPage.waitForLoadState();
8585
const newStdcmPage = new STDCMPage(newPage);
86-
await newStdcmPage.verifyAllFieldsEmpty();
86+
await newStdcmPage.verifyAllDefaultPageFields();
8787
});
8888

8989
/** *************** Test 2 **************** */

front/tests/014-stdcm-linked-train.spec.ts

+9-18
Original file line numberDiff line numberDiff line change
@@ -1,25 +1,20 @@
11
import type { Infra, TowedRollingStock } from 'common/api/osrdEditoastApi';
22

33
import { fastRollingStockName } from './assets/project-const';
4-
import HomePage from './pages/home-page-model';
4+
import test from './logging-fixture';
55
import STDCMLinkedTrainPage from './pages/stdcm-linked-train-page-model';
6-
import STDCMPage, { type ConsistFields } from './pages/stdcm-page-model';
7-
import test from './test-logger';
6+
import STDCMPage from './pages/stdcm-page-model';
87
import { waitForInfraStateToBeCached } from './utils';
98
import { getInfra, setTowedRollingStock } from './utils/api-setup';
9+
import type { ConsistFields } from './utils/types';
1010

11-
test.use({
12-
launchOptions: {
13-
slowMo: 500, // Give the interface time to update between actions
14-
},
15-
});
1611
test.describe('Verify stdcm simulation page', () => {
1712
test.slow(); // Mark test as slow due to multiple steps
18-
1913
test.use({ viewport: { width: 1920, height: 1080 } });
20-
let homePage: HomePage;
21-
let stdcmLinkedTrainPage: STDCMLinkedTrainPage;
14+
2215
let stdcmPage: STDCMPage;
16+
let stdcmLinkedTrainPage: STDCMLinkedTrainPage;
17+
2318
let infra: Infra;
2419
let createdTowedRollingStock: TowedRollingStock;
2520
let towedConsistDetails: ConsistFields;
@@ -45,15 +40,11 @@ test.describe('Verify stdcm simulation page', () => {
4540
});
4641

4742
test.beforeEach('Navigate to the STDCM page', async ({ page }) => {
48-
[homePage, stdcmPage, stdcmLinkedTrainPage] = [
49-
new HomePage(page),
50-
new STDCMPage(page),
51-
new STDCMLinkedTrainPage(page),
52-
];
43+
[stdcmPage, stdcmLinkedTrainPage] = [new STDCMPage(page), new STDCMLinkedTrainPage(page)];
5344
// Navigate to STDCM page
5445
await page.goto('/stdcm');
55-
await page.waitForLoadState('domcontentloaded', { timeout: 30_000 });
56-
await homePage.removeViteOverlay();
46+
await page.waitForLoadState('networkidle');
47+
await stdcmPage.removeViteOverlay();
5748

5849
// Wait for infra to be in 'CACHED' state before proceeding
5950
await waitForInfraStateToBeCached(infra.id);

front/tests/pages/stdcm-page-model.ts

-3
Original file line numberDiff line numberDiff line change
@@ -163,8 +163,6 @@ class STDCMPage extends HomePage {
163163

164164
readonly helpButton: Locator;
165165

166-
readonly downloadLink: Locator;
167-
168166
constructor(page: Page) {
169167
super(page);
170168
this.notificationHeader = page.locator('#notification');
@@ -258,7 +256,6 @@ class STDCMPage extends HomePage {
258256
this.simulationLengthAndDuration = page.locator(
259257
'.simulation-metadata .total-length-trip-duration'
260258
);
261-
this.downloadLink = page.locator('.download-simulation a');
262259
}
263260

264261
// Dynamic selectors for via cards

0 commit comments

Comments
 (0)