Skip to content
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

Jump to bottom

Flaky Cypress test Spotlight > should be able to navigate results via keyboard #26161

Closed
andybalaam opened this issue Sep 13, 2023 · 6 comments
Closed

Flaky Cypress test Spotlight > should be able to navigate results via keyboard #26161

andybalaam opened this issue Sep 13, 2023 · 6 comments
Labels
A-Developer-Experience A-New-Search-Experience The new search dialog available in Labs O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Z-Flaky-Test A test is raising false alarms

Comments

@andybalaam
Copy link
Member

https://github.com/matrix-org/matrix-js-sdk/actions/runs/6173326658/job/16755418278

       cy:command ✔  get	.mx_SpotlightDialog_section.mx_SpotlightDialog_results .mx_SpotlightDialog_option
      cy:command ✘  assert	expected **<li#mx_SpotlightDialog_button_result_@bot_293:localhost.mx_AccessibleButton.mx_SpotlightDialog_option>** to have a length of **2** but got **1**


  1) Spotlight
       should be able to navigate results via keyboard:

      AssertionError: Timed out retrying after 10000ms: Not enough elements found. Found '1', expected '2'.
@andybalaam andybalaam added S-Minor Impairs non-critical functionality or suitable workarounds exist A-Developer-Experience O-Occasional Affects or can be seen by some users regularly or most users rarely Z-Flaky-Test A test is raising false alarms labels Sep 13, 2023
@robintown robintown added T-Defect A-New-Search-Experience The new search dialog available in Labs labels Sep 13, 2023
@andybalaam
Copy link
Member Author

This passes consistently locally for me, so hard to debug

@richvdh
Copy link
Member

richvdh commented Sep 18, 2023

seems to be a continuation of #26140

@richvdh
Copy link
Member

richvdh commented Sep 18, 2023

https://cloud.cypress.io/projects/ppvnzg/runs/19150/test-results/35a5b72c-43a7-4ba2-9fc2-03ce814cdba6

@t3chguy
Copy link
Member

t3chguy commented Sep 18, 2023

This passes consistently locally for me, so hard to debug

Have you tried Test Replay? https://docs.cypress.io/guides/cloud/debugging/test-replay

@richvdh
Copy link
Member

richvdh commented Sep 20, 2023

https://cloud.cypress.io/projects/ppvnzg/runs/19225/test-results/4809b546-8ed5-446f-aa86-10d0b6384bcd

@richvdh richvdh changed the title Flaky Cypress test Spotlight > should be able to navigate results via keyboard Flaky Cypress test Spotlight > should be able to navigate results via keyboard Sep 22, 2023
@richvdh
Copy link
Member

richvdh commented Jan 19, 2024

since this hasn't been reported for several months, let's assume for now that it is fixed.

@richvdh richvdh closed this as completed Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Developer-Experience A-New-Search-Experience The new search dialog available in Labs O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Z-Flaky-Test A test is raising false alarms
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@andybalaam @richvdh @t3chguy @robintown