Skip to content

Commit 4cef6a6

Browse files
Skip activity api test (open-metadata#28068)
Co-authored-by: Shailesh Parmar <shailesh.parmar.webdev@gmail.com>
1 parent 64f49c1 commit 4cef6a6

1 file changed

Lines changed: 7 additions & 22 deletions

File tree

openmetadata-ui/src/main/resources/ui/playwright/e2e/Features/ActivityAPI.spec.ts

Lines changed: 7 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,10 @@ import {
4242
} from '../../utils/entity';
4343
import { test } from '../fixtures/pages';
4444

45-
test.describe(
45+
// Investigation needed:
46+
// 1. Profile actual event propagation time — measure how long it typically takes from a PATCH/PUT call to the event appearing in /api/v1/activity, then tighten the ceiling.
47+
// 2. If synchronous flushing is not feasible, restructure tests to assert entity state directly via the entity API, seed a pre-built activity event, and verify only that the UI renders it decoupling UI assertions from event latency.
48+
test.describe.fixme(
4649
'Activity API - Entity Changes',
4750
{ tag: [DOMAIN_TAGS.DISCOVERY] },
4851
() => {
@@ -76,8 +79,6 @@ test.describe(
7679
test('creates an activity event when the description is updated', async ({
7780
page,
7881
}) => {
79-
// waitForActivityEvent polls the API for up to 5 minutes (ACTIVITY_EVENT_TIMEOUT = 300_000ms)
80-
// because activity events are processed asynchronously in the background.
8182
test.setTimeout(ACTIVITY_TEST_TIMEOUT);
8283

8384
const newDescription = `Test description updated at ${Date.now()}`;
@@ -119,8 +120,6 @@ test.describe(
119120
});
120121

121122
test('creates an activity event when tags are added', async ({ page }) => {
122-
// waitForActivityEvent polls the API for up to 5 minutes (ACTIVITY_EVENT_TIMEOUT = 300_000ms)
123-
// because activity events are processed asynchronously in the background.
124123
test.setTimeout(ACTIVITY_TEST_TIMEOUT);
125124

126125
const entityFqn = getTableFqn(entityChangesTable);
@@ -159,8 +158,6 @@ test.describe(
159158
});
160159

161160
test('creates an activity event when owner is added', async ({ page }) => {
162-
// waitForActivityEvent polls the API for up to 5 minutes (ACTIVITY_EVENT_TIMEOUT = 300_000ms)
163-
// because activity events are processed asynchronously in the background.
164161
test.setTimeout(ACTIVITY_TEST_TIMEOUT);
165162

166163
const entityFqn = getTableFqn(entityChangesTable);
@@ -198,8 +195,6 @@ test.describe(
198195
});
199196

200197
test('shows the actor who made the activity change', async ({ page }) => {
201-
// waitForActivityEvent polls the API for up to 5 minutes (ACTIVITY_EVENT_TIMEOUT = 300_000ms)
202-
// because activity events are processed asynchronously in the background.
203198
test.setTimeout(ACTIVITY_TEST_TIMEOUT);
204199

205200
const entityFqn = getTableFqn(entityChangesTable);
@@ -235,8 +230,6 @@ test.describe(
235230
});
236231

237232
test('links activity items to the correct entity', async ({ page }) => {
238-
// waitForActivityEvent polls the API for up to 5 minutes (ACTIVITY_EVENT_TIMEOUT = 300_000ms)
239-
// because activity events are processed asynchronously in the background.
240233
test.setTimeout(ACTIVITY_TEST_TIMEOUT);
241234

242235
const description = `Entity link description ${uuid()}`;
@@ -266,7 +259,7 @@ test.describe(
266259
}
267260
);
268261

269-
test.describe(
262+
test.describe.fixme(
270263
'Activity API - Reactions',
271264
{ tag: [DOMAIN_TAGS.DISCOVERY] },
272265
() => {
@@ -288,8 +281,6 @@ test.describe(
288281
});
289282

290283
test('adds a reaction to a feed item', async ({ page }) => {
291-
// waitForActivityEvent polls the API for up to 5 minutes (ACTIVITY_EVENT_TIMEOUT = 300_000ms)
292-
// because activity events are processed asynchronously in the background.
293284
test.setTimeout(ACTIVITY_TEST_TIMEOUT);
294285

295286
const description = `Test activity for adding reaction ${uuid()}`;
@@ -314,8 +305,6 @@ test.describe(
314305
});
315306

316307
test('removes an existing reaction from a feed item', async ({ page }) => {
317-
// waitForActivityEvent polls the API for up to 5 minutes (ACTIVITY_EVENT_TIMEOUT = 300_000ms)
318-
// because activity events are processed asynchronously in the background.
319308
test.setTimeout(ACTIVITY_TEST_TIMEOUT);
320309

321310
const description = `Test activity for removing reaction ${uuid()}`;
@@ -346,7 +335,7 @@ test.describe(
346335
}
347336
);
348337

349-
test.describe(
338+
test.describe.fixme(
350339
'Activity API - Comments',
351340
{ tag: [DOMAIN_TAGS.DISCOVERY] },
352341
() => {
@@ -368,8 +357,6 @@ test.describe(
368357
});
369358

370359
test('adds a comment to a feed item', async ({ page }) => {
371-
// waitForActivityEvent polls the API for up to 5 minutes (ACTIVITY_EVENT_TIMEOUT = 300_000ms)
372-
// because activity events are processed asynchronously in the background.
373360
test.setTimeout(ACTIVITY_TEST_TIMEOUT);
374361

375362
const description = `Test activity for comments ${uuid()}`;
@@ -394,8 +381,6 @@ test.describe(
394381
});
395382

396383
test('shows the activity detail layout', async ({ page }) => {
397-
// waitForActivityEvent polls the API for up to 5 minutes (ACTIVITY_EVENT_TIMEOUT = 300_000ms)
398-
// because activity events are processed asynchronously in the background.
399384
test.setTimeout(ACTIVITY_TEST_TIMEOUT);
400385

401386
const description = `Test activity detail layout ${uuid()}`;
@@ -426,7 +411,7 @@ test.describe(
426411
}
427412
);
428413

429-
test.describe(
414+
test.describe.fixme(
430415
'Activity API - Homepage Widget',
431416
{ tag: [DOMAIN_TAGS.DISCOVERY] },
432417
() => {

0 commit comments

Comments
 (0)