[Cloud Security] Disable/Enable rule takes too long

[seanrathier]

Summary

This change resolves the Benchmark Rules page's performance when enabling or disabling rules. Most of the work was converting the change_csp_rule_state to use the ReactQuery useMutation hook. This change introduces optimistic updating of the cached data, while at the same time performing the HTTP request to bulk update to mute or unmute the rules.

Should anything fail with the HTP request, the snapshot of the data taken before the cache was optimistically updated will replace the mutated data.

Other changes:

• Removed the refetch call in the rules_table and instead opted to use the invalidateQueries
• Removed the await from the original invalidateQueries as these can be executed in the background.
• Removed useMemo hook from the table columns because there was no visual difference and it introduces more complexity for the execution.

Screen.Recording.2024-05-09.at.11.41.33.AM.mov

Checklisthttps://github.com//pull/182768/files

• Unit or functional tests were updated or added to match the most common scenarios
• Flaky Test Runner was used on any tests changed

For maintainers

• This was checked for breaking API changes and was labeled appropriately

[seanrathier]

/ci

[elasticmachine]

Pinging @elastic/kibana-cloud-security-posture (Team:Cloud Security)

[seanrathier]

/ci

[opauloh]

overall LGTM, just some comments,

Great test coverage 🚀

[opauloh]

LGTM

[JordanSh]

Looks great, Sean 👑

[kibana-ci]

💛 Build succeeded, but was flaky

• Buildkite Build
• Commit: 55bfd3d

Failed CI Steps

• FTR Configs #55

Test Failures

• [job] [logs] FTR Configs #55 / endpoint For each artifact list under management When on the Trusted applications entries list "after each" hook for "should not show page title if there is no Trusted applications entry"

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id	before	after	diff
cloudSecurityPosture	448.5KB	448.7KB	+190.0B

History

• 💛 Build #210850 was flaky 4003b14
• 💚 Build #209562 succeeded 309b446
• 💔 Build #209315 failed a63b679
• 💚 Build #209034 succeeded 630c546
• 💛 Build #209001 was flaky 630c546
• 💚 Build #208809 succeeded 6ada528

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