Skip to content

Commit b053fec

Browse files
Chore(UI): Add test.slow() to customProperties spec to cover for slow responses in AUT runs (#27782)
* Add test.slow() to customProperties spec to cover for slow responses in AUT runs * Increase the timeout for custom properties common test as well
1 parent 6019031 commit b053fec

1 file changed

Lines changed: 12 additions & 1 deletion

File tree

openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/CustomProperties.spec.ts

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -335,6 +335,7 @@ ALL_ENTITIES.forEach(({ key, makeInstance }) => {
335335

336336
BASIC_PROPERTIES.forEach((property) => {
337337
test(property, async ({ page }) => {
338+
test.slow();
338339
const propertyName = `cp-${uuid()}-${entity.name}`;
339340

340341
await settingClick(
@@ -426,7 +427,9 @@ ALL_ENTITIES.forEach(({ key, makeInstance }) => {
426427

427428
if (makeInstance !== null) {
428429
test(`Set & Update all CP types on ${entity.name}`, async ({ page }) => {
429-
test.slow(true);
430+
// 5 minutes timeout since the test handles set->update operation on all
431+
// custom property types sequentially
432+
test.setTimeout(300000);
430433
const properties = Object.values(CustomPropertyTypeByName);
431434

432435
await test.step('Set all CP types', async () => {
@@ -607,6 +610,7 @@ ALL_ENTITIES.forEach(({ key, makeInstance }) => {
607610
test('User visible in right panel when added as entityReferenceList custom property', async ({
608611
page,
609612
}) => {
613+
test.slow();
610614
const { apiContext, afterAction } = await getApiContext(page);
611615
const propertyName =
612616
mainEntity.customPropertyValue[
@@ -1062,6 +1066,7 @@ ALL_ENTITIES.forEach(({ key, makeInstance }) => {
10621066
test('should show No Data placeholder when hyperlink has no value', async ({
10631067
page,
10641068
}) => {
1069+
test.slow();
10651070
const propertyName =
10661071
mainEntity.customPropertyValue[CustomPropertyTypeByName.HYPERLINK_CP]
10671072
.property.name;
@@ -1082,6 +1087,7 @@ ALL_ENTITIES.forEach(({ key, makeInstance }) => {
10821087
test('should reject javascript: protocol URLs for XSS protection', async ({
10831088
page,
10841089
}) => {
1090+
test.slow();
10851091
const propertyName =
10861092
mainEntity.customPropertyValue[CustomPropertyTypeByName.HYPERLINK_CP]
10871093
.property.name;
@@ -1107,6 +1113,7 @@ ALL_ENTITIES.forEach(({ key, makeInstance }) => {
11071113
});
11081114

11091115
test('should accept valid http and https URLs', async ({ page }) => {
1116+
test.slow();
11101117
const propertyName =
11111118
mainEntity.customPropertyValue[CustomPropertyTypeByName.HYPERLINK_CP]
11121119
.property.name;
@@ -1146,6 +1153,7 @@ ALL_ENTITIES.forEach(({ key, makeInstance }) => {
11461153
test('should display URL when no display text is provided', async ({
11471154
page,
11481155
}) => {
1156+
test.slow();
11491157
const propertyName =
11501158
mainEntity.customPropertyValue[CustomPropertyTypeByName.HYPERLINK_CP]
11511159
.property.name;
@@ -2771,6 +2779,7 @@ ALL_ENTITIES.forEach(({ key, makeInstance }) => {
27712779
});
27722780

27732781
test('Table CP - Name column with all operators', async ({ page }) => {
2782+
test.slow();
27742783
const value = CP_BASE_VALUES.tableCp.rows[0]['Name'];
27752784
const partialValue = value.substring(1, 4);
27762785
const basePropertyName = propertyNames['table-cp'];
@@ -2865,6 +2874,7 @@ ALL_ENTITIES.forEach(({ key, makeInstance }) => {
28652874
});
28662875

28672876
test('Table CP - Role column with all operators', async ({ page }) => {
2877+
test.slow();
28682878
const value = CP_BASE_VALUES.tableCp.rows[0]['Role'];
28692879
const partialValue = value.substring(1, 4);
28702880
const basePropertyName = propertyNames['table-cp'];
@@ -2959,6 +2969,7 @@ ALL_ENTITIES.forEach(({ key, makeInstance }) => {
29592969
});
29602970

29612971
test('Table CP - Sr No column with all operators', async ({ page }) => {
2972+
test.slow();
29622973
const value = CP_BASE_VALUES.tableCp.rows[1]['Sr No'];
29632974
const basePropertyName = propertyNames['table-cp'];
29642975
const columnPropertyName = `${basePropertyName} - Sr No`;

0 commit comments

Comments
 (0)