Chore(UI): Add test.slow() to customProperties spec to cover for slow responses in AUT runs#27782
Chore(UI): Add test.slow() to customProperties spec to cover for slow responses in AUT runs#27782aniketkatkar97 merged 3 commits intomainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR improves Playwright E2E stability for Custom Properties by marking several long-running tests as slow, giving them extended timeouts during execution (notably for AUT runs where responses can be slower).
Changes:
- Marked basic CRUD custom property tests as slow.
- Marked entityReferenceList-related custom property tests as slow.
- Marked hyperlink and table custom property tests as slow to reduce timeout-related flakes.
🟡 Playwright Results — all passed (13 flaky)✅ 3962 passed · ❌ 0 failed · 🟡 13 flaky · ⏭️ 86 skipped
🟡 13 flaky test(s) (passed on retry)
How to debug locally# Download playwright-test-results-<shard> artifact and unzip
npx playwright show-trace path/to/trace.zip # view trace |
Code Review ✅ ApprovedIncreases timeout threshold for customProperties spec using test.slow() to accommodate sluggish response times during automated runs. No issues found. OptionsDisplay: compact → Showing less information. Comment with these commands to change:
Was this helpful? React with 👍 / 👎 | Gitar |
|
|
Failed to cherry-pick changes to the 1.12.7 branch. |
… 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
… 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
… responses in AUT runs (open-metadata#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
… responses in AUT runs (open-metadata#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
…ver slow responses in 1.13 AUT runs (#27885) The test consistently lands at the 60s test-timeout boundary on 1.13 AUT (image n56+) where the backend CSV-import flow takes ~50s end-to-end — matching the precedent set by #27782 for customProperties. test.slow() triples the timeout to 180s so transient AUT jitter no longer flakes the test. Co-authored-by: Siddhant <siddhant@MacBook-Pro-621.local> Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>



This pull request marks several Playwright tests as "slow" in the
CustomProperties.spec.tsfile. This helps the test runner handle potentially long-running tests more gracefully, reducing the chance of false negatives due to timeouts.The most important changes are:
Test Stability Improvements:
test.slow();to basic property tests to indicate they may take longer to execute.Hyperlink Custom Property Tests:
Table Custom Property Tests:
Summary by Gitar
test.slow(true)with an explicit 5-minute timeout viatest.setTimeout(300000)inCustomProperties.spec.ts.This will update automatically on new commits.