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.

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

Failing test: Chrome X-Pack UI Functional Tests.x-pack/test/functional/apps/dashboard/feature_controls/dashboard_security·ts - dashboard feature controls dashboard security global dashboard & embeddable all privileges allow saving via the saved query management component popover with no query loaded #44631

Closed
kibanamachine opened this issue Sep 2, 2019 · 36 comments · Fixed by #44797, #57715 or #71173
Assignees
Labels
blocker failed-test A test failure on a tracked branch, potentially flaky-test skipped-test Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@kibanamachine
Copy link
Contributor

kibanamachine commented Sep 2, 2019

A test failed on a tracked branch


        Error: retry.try timeout: Error: retry.try timeout: TimeoutError: Waiting for element to be located By(css selector, [data-test-subj~="saveQueryFormTitle"])
Wait timed out after 10059ms
    at /var/lib/jenkins/workspace/elastic+kibana+master/JOB/x-pack-ciGroup3/node/linux-immutable/kibana/node_modules/selenium-webdriver/lib/webdriver.js:841:17
    at process._tickCallback (internal/process/next_tick.js:68:7)
    at lastError (/var/lib/jenkins/workspace/elastic+kibana+master/JOB/x-pack-ciGroup3/node/linux-immutable/kibana/test/common/services/retry/retry_for_success.ts:28:9)
    at onFailure (/var/lib/jenkins/workspace/elastic+kibana+master/JOB/x-pack-ciGroup3/node/linux-immutable/kibana/test/common/services/retry/retry_for_success.ts:68:13)
    at lastError (/var/lib/jenkins/workspace/elastic+kibana+master/JOB/x-pack-ciGroup3/node/linux-immutable/kibana/test/common/services/retry/retry_for_success.ts:28:9)
    at onFailure (/var/lib/jenkins/workspace/elastic+kibana+master/JOB/x-pack-ciGroup3/node/linux-immutable/kibana/test/common/services/retry/retry_for_success.ts:68:13)
      

First failure: Jenkins Build

@kibanamachine kibanamachine added the failed-test A test failure on a tracked branch, potentially flaky-test label Sep 2, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-test-triage

@spalger spalger added the Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! label Sep 3, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-security

@spalger
Copy link
Contributor

spalger commented Sep 3, 2019

This is replicated in several places, so I'd rather not disable them all, can someone from @elastic/kibana-security please take a look?

image

@legrego
Copy link
Member

legrego commented Sep 3, 2019

Possibly related to #39140? -- @Bargs do you have any insights here? Looks like you graciously added this test, and the failure itself doesn't seem related to Security/Spaces/Feature Controls

@Bargs
Copy link
Contributor

Bargs commented Sep 3, 2019

I'll take a look

@Bargs
Copy link
Contributor

Bargs commented Sep 3, 2019

So it's failing because the save query modal hasn't opened up, even though the "Save current query" button was successfully clicked. Maybe it's just clicking the button before the popover component is fully rendered. I could add a retry that clicks it until the modal appears, but that might cover up a real issue. @spalger is there some way to capture the browser console output in a functional test so we could see if there's an error?

@Bargs
Copy link
Contributor

Bargs commented Sep 3, 2019

Ah nice, thanks. I'm not seeing any errors so for now I'll try to fix this with a retry.

@kibanamachine
Copy link
Contributor Author

New failure: Jenkins Build

@kibanamachine kibanamachine reopened this Sep 10, 2019
@kobelb kobelb removed the Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! label Sep 10, 2019
@spalger spalger added the Team:Visualizations Visualization editors, elastic-charts and infrastructure label Sep 11, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app

@spalger
Copy link
Contributor

spalger commented Sep 11, 2019

@Bargs looks like there must be some other underlying issue here, the situation in #45317 looks very similar... not sure if that helps... @dmlemeshko do you think that any of the changes we've made to the driver could be interfering with click actions?

@spalger
Copy link
Contributor

spalger commented Sep 11, 2019

This has failed several times in PRs this week too
image

@flash1293
Copy link
Contributor

flash1293 commented Sep 11, 2019

For reference - this is also the problem in the other failures in Graph functional tests
graph1
graph2
graph3

The other failures from this batch are probably all the same underlying problem - e.g. #45105 (here the button to disable an aggregation wasn't actually clicked)

visualize

You can see the button being highlighted in all cases, but somehow the click is not processed. I could not reproduce this behavior locally.

Things in common:

  • React is handling the click
  • The elements are EUI buttons (specific html structure with nested spans inside of the button)
  • They are all targeted with data-test-subj

@legrego
Copy link
Member

legrego commented Sep 11, 2019

hmm, the underlying root cause here might also be the cause of #44575. It looks like I'm missing a button click too...

@flash1293
Copy link
Contributor

flash1293 commented Sep 11, 2019

I wonder whether it's this thing:

buttonflickering

I don't know how exactly selenium is clicking this button, but if you hover at the wrong place, you fall in and out of the clickable area because of the animation.

Maybe we should disable these things for the functional tests or make sure the button gets clicked in the center.

Related issue in EUI: elastic/eui#519

EDIT: It's probably not the cause, just noticed some of the failures also happen on non-animated buttons.

@spalger
Copy link
Contributor

spalger commented Sep 11, 2019

I don't know how exactly selenium is clicking this button, but if you hover at the wrong place, you fall in and out of the clickable area because of the animation.

It's unlikely that is causing the problem because the functional tests disable animations, but also in situations like that we should really be treating the unclickable region as a bug in the button I think.

It's totally reasonable to me that the element exists for a moment before it's clickable, though I'm not sure how it's rendered and if that's actually possible.

@spalger
Copy link
Contributor

spalger commented Sep 11, 2019

Related #45348

@flash1293
Copy link
Contributor

flash1293 commented Sep 11, 2019

It's totally reasonable to me that the element exists for a moment before it's clickable, though I'm not sure how it's rendered and if that's actually possible.

Not sure whether that’s the case here. I have to check to be 100% sure but some of the buttons have been around for some time before they are clicked.

@flash1293
Copy link
Contributor

@LeeDr do you have an idea what the issue could be here? Seems to be something lowlevel.

@dmlemeshko
Copy link
Member

do you think that any of the changes we've made to the driver could be interfering with click actions?

@spalger I doubt it is related to recent FTR changes: we added retry on error, but based on the logs it never retries. We still run Chrome tests the old way, so click is happening as before. But I noticed that we started to have an issue with selecting combobox with click action as I explained in #45751

@LeeDr
Copy link

LeeDr commented Sep 17, 2019

It could be a similar problem to this issue;
"Discover New button takes forever or doesn't work" #45701

Fixed by;
"Fix scope digest issue with new button in discover" #45913

And/or this similar case (but I've only noticed this on IE);
"[IE11] several events aren't triggered until mouse moves off item" #45333

@Bargs
Copy link
Contributor

Bargs commented Sep 17, 2019

I doubt it's a digest issue like #45701. All of these components are in React, and if it were digest related it would be easy to reproduce locally.

spalger pushed a commit that referenced this issue Feb 13, 2020
Several tests using the
savedQueryManagementComponent.saveNewQueryMissingOrFail method have been
failing with 'waiting for element'.  Temproary adding a skip for tests
using this method

Related #50018
Related #44631
Related #45348
spalger pushed a commit that referenced this issue Feb 13, 2020
Several tests using the
savedQueryManagementComponent.saveNewQueryMissingOrFail method have been
failing with 'waiting for element'.  Temproary adding a skip for tests
using this method

Related #50018
Related #44631
Related #45348

(cherry picked from commit 431a1e9)
@spalger

This comment has been minimized.

spalger added a commit that referenced this issue Feb 13, 2020
spalger added a commit that referenced this issue Feb 13, 2020
(cherry picked from commit 6e4efdf)
spalger added a commit that referenced this issue Feb 13, 2020
spalger added a commit that referenced this issue Feb 13, 2020
(cherry picked from commit 8aa718d)
gmmorris added a commit to gmmorris/kibana that referenced this issue Feb 13, 2020
* master: (22 commits)
  skip flaky suite (elastic#50018)
  skip settings tests (elastic#57608)
  skip failing suite (elastic#44631)
  [SIEM] [Case] Initial UI (elastic#57283)
  handle viewing sample dashboards on default dist (elastic#57510)
  Fix detection of "system requests" in plugins (elastic#57149)
  [ML] New Platform server shim: update job service schema (elastic#57614)
  skip flaky suite (elastic#44631)
  [APM] Update monospace font family variable (elastic#57555)
  skip flaky test (elastic#57377)
  Skip save query tests (elastic#57589)
  [Maps] allow simultaneous opening of multiple tooltips (elastic#57226)
  [Uptime] Fix/host connected components (elastic#56969)
  [logs][metrics][docs] Update screenshots for 7.6 (elastic#57254)
  [ML] New Platform server shim: update job service routes to use new platform router (elastic#57403)
  [Maps] Fix document source top hits split by scripted field (elastic#57481)
  Use log4j pattern syntax (elastic#57433)
  [ML] Categorization field example endpoint tests (elastic#57471)
  [Lens] Filter out pinned filters from saved object of Lens (elastic#57197)
  Lens client side shim cleanup (elastic#56976)
  ...
spalger added a commit to spalger/kibana that referenced this issue Feb 14, 2020
spalger added a commit to spalger/kibana that referenced this issue Feb 14, 2020
spalger pushed a commit that referenced this issue Feb 19, 2020
* Revert "Skip flaky test (#57675)"

This reverts commit c965a9e.

* Revert "disable firefox smoke tests so we can fix flakiness out of band"

This reverts commit fe38642.

* Revert "skip flaky tests (#57643)"

This reverts commit b220454.

* Revert "skip flaky suite (#50018)"

This reverts commit b058dc2.

* Revert "skip settings tests (#57608)"

This reverts commit 64625b2.

* Revert "skip failing suite (#44631)"

This reverts commit 8aa718d.

* Revert "skip flaky suite (#44631)"

This reverts commit 6e4efdf.

* Revert "skip flaky test (#57377)"

This reverts commit 59672ab.

* Revert "Skip save query tests (#57589)"

This reverts commit 431a1e9.

* remove hard coded timeouts

* Revert "Revert "disable firefox smoke tests so we can fix flakiness out of band""

This reverts commit 271f881.

* Revert "remove hard coded timeouts"

This reverts commit 8b843d0.

* wait for managementHome to exist, don't bail early if it's missing

* Revert "skip flaky suite (#45244)"

This reverts commit 0cee1a4.

* extend timeouts for common existsOrFail() calls

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
spalger pushed a commit to spalger/kibana that referenced this issue Feb 19, 2020
* Revert "Skip flaky test (elastic#57675)"

This reverts commit c965a9e.

* Revert "disable firefox smoke tests so we can fix flakiness out of band"

This reverts commit fe38642.

* Revert "skip flaky tests (elastic#57643)"

This reverts commit b220454.

* Revert "skip flaky suite (elastic#50018)"

This reverts commit b058dc2.

* Revert "skip settings tests (elastic#57608)"

This reverts commit 64625b2.

* Revert "skip failing suite (elastic#44631)"

This reverts commit 8aa718d.

* Revert "skip flaky suite (elastic#44631)"

This reverts commit 6e4efdf.

* Revert "skip flaky test (elastic#57377)"

This reverts commit 59672ab.

* Revert "Skip save query tests (elastic#57589)"

This reverts commit 431a1e9.

* remove hard coded timeouts

* Revert "Revert "disable firefox smoke tests so we can fix flakiness out of band""

This reverts commit 271f881.

* Revert "remove hard coded timeouts"

This reverts commit 8b843d0.

* wait for managementHome to exist, don't bail early if it's missing

* Revert "skip flaky suite (elastic#45244)"

This reverts commit 0cee1a4.

* extend timeouts for common existsOrFail() calls

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@kibanamachine kibanamachine reopened this Feb 19, 2020
@kibanamachine
Copy link
Contributor Author

New failure: Jenkins Build

@spalger
Copy link
Contributor

spalger commented Feb 19, 2020

Welp... Skipping again...

master: d7c36d0
7.x/7.7: cb3728d

spalger pushed a commit that referenced this issue Feb 19, 2020
* Revert "Skip flaky test (#57675)"

This reverts commit c965a9e.

* Revert "disable firefox smoke tests so we can fix flakiness out of band"

This reverts commit fe38642.

* Revert "skip flaky tests (#57643)"

This reverts commit b220454.

* Revert "skip flaky suite (#50018)"

This reverts commit b058dc2.

* Revert "skip settings tests (#57608)"

This reverts commit 64625b2.

* Revert "skip failing suite (#44631)"

This reverts commit 8aa718d.

* Revert "skip flaky suite (#44631)"

This reverts commit 6e4efdf.

* Revert "skip flaky test (#57377)"

This reverts commit 59672ab.

* Revert "Skip save query tests (#57589)"

This reverts commit 431a1e9.

* remove hard coded timeouts

* Revert "Revert "disable firefox smoke tests so we can fix flakiness out of band""

This reverts commit 271f881.

* Revert "remove hard coded timeouts"

This reverts commit 8b843d0.

* wait for managementHome to exist, don't bail early if it's missing

* Revert "skip flaky suite (#45244)"

This reverts commit 0cee1a4.

* extend timeouts for common existsOrFail() calls

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
spalger added a commit that referenced this issue Feb 19, 2020
spalger added a commit that referenced this issue Feb 19, 2020
(cherry picked from commit d7c36d0)
@wylieconlon wylieconlon self-assigned this Jul 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocker failed-test A test failure on a tracked branch, potentially flaky-test skipped-test Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
10 participants