Skip to content

[7.x] [Enterprise Search] Small test helper for expected async errors (#88422)#88492

Merged
cee-chen merged 1 commit intoelastic:7.xfrom
cee-chen:backport/7.x/pr-88422
Jan 15, 2021
Merged

[7.x] [Enterprise Search] Small test helper for expected async errors (#88422)#88492
cee-chen merged 1 commit intoelastic:7.xfrom
cee-chen:backport/7.x/pr-88422

Conversation

@cee-chen
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

…tic#88422)

* Add small helper to reduce boilerplate on async tests expected to throw

- Also hopefully a bit more descriptive on what's happening

* Update all test files using the previous try/catch do nothing blocks to use new helper
# Conflicts:
#	x-pack/plugins/enterprise_search/public/applications/workplace_search/views/content_sources/components/add_source/add_source_logic.test.ts
@cee-chen cee-chen added the backport This PR is a backport of another PR label Jan 15, 2021
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

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

@cee-chen cee-chen merged commit a7ebb5f into elastic:7.x Jan 15, 2021
@cee-chen cee-chen deleted the backport/7.x/pr-88422 branch January 15, 2021 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants