-
Notifications
You must be signed in to change notification settings - Fork 715
test: Dashboard tests stable deployed #9360
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.
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
Conversation
3f12e15 to
8b5c828
Compare
|
Failed to generate code suggestions for PR |
8b5c828 to
5d8d8bb
Compare
|
Failed to generate code suggestions for PR |
Greptile OverviewGreptile SummaryThis PR improves test stability for dashboard-related UI tests by adding explicit wait mechanisms before dashboard interactions. Key changes:
The changes follow consistent patterns from existing code (dashboard-variables.js) and address timing issues by ensuring the UI is stable before interactions. Confidence Score: 5/5
Important Files ChangedFile Analysis
Sequence DiagramsequenceDiagram
participant Test as Test Suite
participant Page as Page Object
participant UI as Dashboard UI
participant API as Backend API
Test->>Page: Navigate to dashboards
Test->>Page: waitForDashboardUIStable()
Page->>UI: Wait for search input (visible)
UI-->>Page: Element visible
Page->>UI: Wait for search input (attached)
UI-->>Page: Element attached
Page->>UI: Wait for New Dashboard button (visible)
UI-->>Page: Element visible
Page->>UI: Wait for New Dashboard button (attached)
UI-->>Page: Element attached
Page->>UI: Wait for Import button (visible)
UI-->>Page: Element visible
Page->>UI: Wait for Import button (attached)
UI-->>Page: Element attached
Page->>Page: Wait 500ms for animations
Page-->>Test: UI stable, ready for interaction
Test->>Page: createDashboard(name)
Page->>UI: Click New Dashboard
UI->>API: Create dashboard request
API-->>UI: Dashboard created
Test->>Page: addVariable(type, name, stream, field)
Page->>UI: Wait for dropdown options
UI-->>Page: Options loaded
alt Exact match found
Page->>UI: Click exact field match
else No exact match
alt Partial match found
Page->>UI: Click first partial match
else No partial match
Page->>UI: Use keyboard navigation (ArrowDown + Enter)
end
end
UI-->>Page: Field selected
Page-->>Test: Variable added successfully
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
4 files reviewed, no comments
5d8d8bb to
b2c3bbc
Compare
2fe2568 to
1f64089
Compare
1f64089 to
84f26f5
Compare
No description provided.