Fixes Failing test: Chrome X-Pack UI Functional Tests.x-pack/test/functional_with_es_ssl/apps/triggers_actions_ui/details·ts - Actions and Triggers app Alert Details Alert Instances renders the active alert instances

[ymao1]

Resolves #57426

Summary

The latest flakey test failures were related to showing the action group id as part of the status column on the alert details instance list. Updated the test to explicitly retrieve the action group id from the instance summary and map to action group name to determine what the displayed text should be.

Ran through the flaky test runner 42 times here
I tried to run through it 100 times but the job gets cut off if it runs for too long

[elasticmachine]

Pinging @elastic/kibana-alerting-services (Team:Alerting Services)

[YulNaumenko]

LGTM

[pmuellr]

LGTM, but my understanding from the issue is that there was some kind of date rounding issue - but that report was back from Feb, and the last build that showed the error is no longer available, so ??? hehe.

In general, I'd say this test still feels a little flaky in that it's comparing the instance summary API to the UI, and in theory there could be differences, given when they are run. But I expect the always firing alert to produce pretty stable results :-), so seems fine, especially after running through the flaky test runner. And we're using that instance summary API elsewhere in these tests, so ... it must be working out ok, in practice.

[ymao1]

LGTM, but my understanding from the issue is that there was some kind of date rounding issue - but that report was back from Feb, and the last build that showed the error is no longer available, so ??? hehe.

The original flaky test failures were due to date issues and those were fixed by previous PRs. This was a failure of the same test, but for a different reason (but because it was the same test, it attached itself to the same original issue). Here is the console output that I copied from Jenkins for posterity.

[00:05:55]                 │       Error: expected [ { instance: 'us-central',
[00:05:55]                 │     status: 'Active (Default)',
[00:05:55]                 │     start: '9 Nov 2020 @ 07:18:34' },
[00:05:55]                 │   { instance: 'us-east',
[00:05:55]                 │     status: 'Active',
[00:05:55]                 │     start: '9 Nov 2020 @ 07:18:34' },
[00:05:55]                 │   { instance: 'us-west',
[00:05:55]                 │     status: 'Active (Default)',
[00:05:55]                 │     start: '9 Nov 2020 @ 07:18:34' } ] to sort of equal [ { instance: 'us-central',
[00:05:55]                 │     status: 'Active (Default)',
[00:05:55]                 │     start: '9 Nov 2020 @ 07:18:34' },
[00:05:55]                 │   { instance: 'us-east',
[00:05:55]                 │     status: 'Active (Default)',
[00:05:55]                 │     start: '9 Nov 2020 @ 07:18:34' },
[00:05:55]                 │   { instance: 'us-west',
[00:05:55]                 │     status: 'Active (Default)',
[00:05:55]                 │     start: '9 Nov 2020 @ 07:18:34' } ]
[00:05:55]                 │       + expected - actual
[00:05:55]                 │ 
[00:05:55]                 │          }
[00:05:55]                 │          {
[00:05:55]                 │            "instance": "us-east"
[00:05:55]                 │            "start": "9 Nov 2020 @ 07:18:34"
[00:05:55]                 │       -    "status": "Active"
[00:05:55]                 │       +    "status": "Active (Default)"
[00:05:55]                 │          }
[00:05:55]                 │          {
[00:05:55]                 │            "instance": "us-west"
[00:05:55]                 │            "start": "9 Nov 2020 @ 07:18:34"

[ymao1]

@elasticmachine merge upstream

[ymao1]

@elasticmachine merge upstream

[kibanamachine]

💚 Build Succeeded

• continuous-integration/kibana-ci/pull-request
• Commit: 08d2125

Metrics [docs]

✅ unchanged

History

• 💚 Build #88743 succeeded b4f2fc3
• 💔 Build #88727 failed ae97cfc
• 💚 Build #88347 succeeded ae97cfc
• 💚 Build #88203 succeeded 7e71e07
• 💚 Build #88138 succeeded d76d44c

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