[Watcher] Fix search bug that crashes the app

[sabarasaba]

Fixes #123071

Summary

The table config for the watcher list had a managed state for the search query that wasn't accounting for possible query errors (in our case, using special characters) causing the app to crash. Since it was unecessary for the query state to be managed, I removed that and we now have the default behaviour we have in other tables when these sort of errors happen.

Note to reviewers: in order to test this, just create a regular watcher and try typing or in the search and verify that the app doesnt crash and that the search box turns into error state.

[sabarasaba]

@elasticmachine merge upstream

[sabarasaba]

@elasticmachine merge upstream

[sabarasaba]

@elasticmachine merge upstream

[kibana-ci]

💚 Build Succeeded

• Buildkite Build
• Commit: 4b92673

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
watcher	163.5KB	163.4KB	-69.0B

History

• 💔 Build #145440 failed 879e188
• 💔 Build #145429 failed 522aa52
• 💔 Build #145414 failed 00a2b8e

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

cc @sabarasaba

[elasticmachine]

Pinging @elastic/platform-deployment-management (Team:Deployment Management)

[yuliacech]

Thanks a lot for the fix, @sabarasaba!
Tested locally and the search doesn't crush the page anymore 👍
Code changes LGTM too, would be great to maybe also add a test for the bug if possible.

[alisonelizabeth]

@sabarasaba I think we need to take another look at this one and approach it differently. I re-enabled the watcher jest tests in #162592 and I see this test fails with your change. The list view is refreshed automatically, which causes the query to reset.

[alisonelizabeth]

If you look at the EUI docs, the onChange callback will receive an error object (when applicable). I think you might be able to leverage that.