Skip to content

Chore(UI): Add test.slow() to customProperties spec to cover for slow responses in AUT runs #27782

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
aniketkatkar97 merged 3 commits into from
Apr 28, 2026
Merged

Chore(UI): Add test.slow() to customProperties spec to cover for slow responses in AUT runs #27782

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
b649ef2
Add test.slow() to customProperties spec to cover for slow responses …
aniketkatkar97 Apr 28, 2026
4a1466a
Increase the timeout for custom properties common test as well
aniketkatkar97 Apr 28, 2026
d436d0d
Merge branch 'main' into customProperty-flaky
aniketkatkar97 Apr 28, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 12 additions & 1 deletion openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/CustomProperties.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -335,6 +335,7 @@ ALL_ENTITIES.forEach(({ key, makeInstance }) => {

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

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

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

await test.step('Set all CP types', async () => {
Expand Down Expand Up @@ -607,6 +610,7 @@ ALL_ENTITIES.forEach(({ key, makeInstance }) => {
test('User visible in right panel when added as entityReferenceList custom property', async ({
page,
}) => {
test.slow();
const { apiContext, afterAction } = await getApiContext(page);
const propertyName =
mainEntity.customPropertyValue[
Expand Down Expand Up @@ -1062,6 +1066,7 @@ ALL_ENTITIES.forEach(({ key, makeInstance }) => {
test('should show No Data placeholder when hyperlink has no value', async ({
page,
}) => {
test.slow();
const propertyName =
mainEntity.customPropertyValue[CustomPropertyTypeByName.HYPERLINK_CP]
.property.name;
Expand All @@ -1082,6 +1087,7 @@ ALL_ENTITIES.forEach(({ key, makeInstance }) => {
test('should reject javascript: protocol URLs for XSS protection', async ({
page,
}) => {
test.slow();
const propertyName =
mainEntity.customPropertyValue[CustomPropertyTypeByName.HYPERLINK_CP]
.property.name;
Expand All @@ -1107,6 +1113,7 @@ ALL_ENTITIES.forEach(({ key, makeInstance }) => {
});

test('should accept valid http and https URLs', async ({ page }) => {
test.slow();
const propertyName =
mainEntity.customPropertyValue[CustomPropertyTypeByName.HYPERLINK_CP]
.property.name;
Expand Down Expand Up @@ -1146,6 +1153,7 @@ ALL_ENTITIES.forEach(({ key, makeInstance }) => {
test('should display URL when no display text is provided', async ({
page,
}) => {
test.slow();
const propertyName =
mainEntity.customPropertyValue[CustomPropertyTypeByName.HYPERLINK_CP]
.property.name;
Expand Down Expand Up @@ -2771,6 +2779,7 @@ ALL_ENTITIES.forEach(({ key, makeInstance }) => {
});

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

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

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