Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[synthetics] Update docs to reflect the changes to expect #3297

Merged
merged 3 commits into from
Oct 19, 2023

Conversation

colleenmcginnis
Copy link
Contributor

@colleenmcginnis colleenmcginnis commented Oct 18, 2023

Closes #3206

Currently the docs list the supported Jest expect methods. As of @elastic/synthetics v1.5.0, we use Playwright Test expect assertions. This replaces the list of supported Jest methods with links to the Playwright assertion docs.

👋 @vigneshshanmugam I'm not sure if I've captured everything correctly.

cc @paulb-elastic

@colleenmcginnis colleenmcginnis added the backport-8.11 Automated backport with mergify label Oct 18, 2023
@colleenmcginnis colleenmcginnis self-assigned this Oct 18, 2023
@github-actions
Copy link
Contributor

A documentation preview will be available soon:

Copy link
Member

@vigneshshanmugam vigneshshanmugam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Thanks @colleenmcginnis

docs/en/observability/synthetics-create-test.asciidoc Outdated Show resolved Hide resolved
@colleenmcginnis colleenmcginnis marked this pull request as ready for review October 19, 2023 16:42
@colleenmcginnis colleenmcginnis requested a review from a team as a code owner October 19, 2023 16:42
@colleenmcginnis colleenmcginnis merged commit a3e45c0 into elastic:main Oct 19, 2023
2 checks passed
@colleenmcginnis colleenmcginnis deleted the issue-3206 branch October 19, 2023 18:09
mergify bot pushed a commit that referenced this pull request Oct 19, 2023
* first draft switch to playwright expect

* adjust based on release notes

* clarify what's not supported

(cherry picked from commit a3e45c0)
colleenmcginnis added a commit that referenced this pull request Oct 19, 2023
…3301)

* first draft switch to playwright expect

* adjust based on release notes

* clarify what's not supported

(cherry picked from commit a3e45c0)

Co-authored-by: Colleen McGinnis <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-8.11 Automated backport with mergify
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Synthetics] Update docs to reflect the change to expect from @playwright/test
3 participants