[Secuirty Solution] Fixes alerts flaky test

[MadameSheema]

Summary

The alerts spec started to be flaky and was skipped: #77957

In this PR we are fixing the flakiness modifying some of the code :)

[MadameSheema]

@elasticmachine merge upstream

[kibanamachine]

💛 Build succeeded, but was flaky

• continuous-integration/kibana-ci/pull-request
• Commit: 76563e0
• Flaky suites:
  • xpack-securitySolutionCypress

---

Test Failures

"after each" hook for "inspects the Hosts Stat".Inspect Hosts stats and tables "after each" hook for "inspects the Hosts Stat"

Link to Jenkins

Stack Trace

Failed Tests Reporter:
  - Test has not failed recently on tracked branches

CypressError: Timed out retrying: `cy.click()` failed because this element is detached from the DOM.

`<button class="euiButton euiButton--primary euiButton--fill" type="button" data-test-subj="modal-inspect-close">...</button>`

Cypress requires elements be attached in the DOM to interact with them.

The previous command that ran was:

  > `cy.get()`

This DOM element likely became detached somewhere between the previous and current command.

Common situations why this happens:
  - Your JS framework re-rendered asynchronously
  - Your app code reacted to an event firing and removed the element

You typically need to re-query for the element or add 'guards' which delay Cypress from running new commands.

https://on.cypress.io/element-has-detached-from-dom

Because this error occurred during a `after each` hook we are skipping the remaining tests in the current suite: `Hosts stats and tables`
    at $Cy.ensureAttached (http://elastic:changeme@localhost:6111/__cypress/runner/cypress_runner.js:160835:24)
    at runAllChecks (http://elastic:changeme@localhost:6111/__cypress/runner/cypress_runner.js:149825:12)
    at retryActionability (http://elastic:changeme@localhost:6111/__cypress/runner/cypress_runner.js:149884:16)
    at tryCatcher (http://elastic:changeme@localhost:6111/__cypress/runner/cypress_runner.js:9852:23)
    at Function.Promise.attempt.Promise.try (http://elastic:changeme@localhost:6111/__cypress/runner/cypress_runner.js:7126:29)
    at tryFn (http://elastic:changeme@localhost:6111/__cypress/runner/cypress_runner.js:164075:24)
    at whenStable (http://elastic:changeme@localhost:6111/__cypress/runner/cypress_runner.js:164116:12)
    at http://elastic:changeme@localhost:6111/__cypress/runner/cypress_runner.js:163634:16
    at tryCatcher (http://elastic:changeme@localhost:6111/__cypress/runner/cypress_runner.js:9852:23)
    at Promise._settlePromiseFromHandler (http://elastic:changeme@localhost:6111/__cypress/runner/cypress_runner.js:7787:31)
    at Promise._settlePromise (http://elastic:changeme@localhost:6111/__cypress/runner/cypress_runner.js:7844:18)
    at Promise._settlePromise0 (http://elastic:changeme@localhost:6111/__cypress/runner/cypress_runner.js:7889:10)
    at Promise._settlePromises (http://elastic:changeme@localhost:6111/__cypress/runner/cypress_runner.js:7969:18)
    at Promise._fulfill (http://elastic:changeme@localhost:6111/__cypress/runner/cypress_runner.js:7913:18)
    at http://elastic:changeme@localhost:6111/__cypress/runner/cypress_runner.js:9527:46
From Your Spec Code:
    at Object.exports.closesModal (http://localhost:6111/__cypress/tests?p=cypress/integration/inspect.spec.ts:771:54)
    at Context.eval (http://localhost:6111/__cypress/tests?p=cypress/integration/inspect.spec.ts:723:23)

---

Metrics [docs]

✅ unchanged

History

• 💛 Build #76923 was flaky 8de6772

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

[rylnd]

For posterity: the behavior of should is to retry until successful. .invoke('text').should('eq', expectedText) has nothing to retry since it's a simple comparison with a static value, but .should('have.text', expectedText) and .should(retryableCallback)` will perform retries.

[MadameSheema]

Yes @rylnd you are completely right, this is why we are moving everything to that clause in refactors :)