Skip to content

FOLLOW-UP: Replace fixed 15s Sleep with WaitVisible in auto-tables E2E tests (from PR #207) #211

@adnaan

Description

@adnaan

Context

This follow-up task was identified during the review of PR #207.

Source PR: #207
PR Title: feat: auto-tables — infer interactive UI from markdown tables + YAML sources
Suggested by: @claude[bot]

Task Description

Several auto-tables E2E tests use chromedp.Sleep(15*time.Second) to wait for initial render. This is slow and potentially flaky under load. TestAutoTables_SQLiteAdd already uses chromedp.WaitVisible correctly.

Fix: Replace Sleep(15s) with chromedp.WaitVisible or chromedp.Poll in TestAutoTables_SQLiteRenders, TestAutoTables_RESTReadonly, and TestAutoTables_NoMatch.


This issue was automatically created by prmonitor from PR review comments.

Metadata

Metadata

Assignees

No one assigned

    Labels

    P2-mediumMedium: quality, platform coverage, CIfollow-upFollow-up task from PR reviewfrom-reviewIssue originated from PR review

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions