Skip to content
This repository was archived by the owner on Aug 20, 2024. It is now read-only.

fix: make it compatible with playwright-test #79

Merged
merged 3 commits into from
Jun 8, 2021

Conversation

mxschmitt
Copy link
Collaborator

Gets released soon: https://playwright.dev/docs/next/test-intro/

I tested it locally, seems to work.

@mxschmitt mxschmitt requested a review from mskelton June 8, 2021 19:56
@mxschmitt mxschmitt force-pushed the compatible-with-playwright-test branch from ed930da to 3132d7d Compare June 8, 2021 20:08
@codecov-commenter
Copy link

codecov-commenter commented Jun 8, 2021

Codecov Report

Merging #79 (4cc3665) into master (c531fe1) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #79   +/-   ##
=======================================
  Coverage   90.97%   90.97%           
=======================================
  Files          10       10           
  Lines         144      144           
  Branches       26       26           
=======================================
  Hits          131      131           
  Misses         13       13           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e31b1c3...4cc3665. Read the comment docs.

@mxschmitt mxschmitt force-pushed the compatible-with-playwright-test branch from 4a0c829 to 4cc3665 Compare June 8, 2021 20:22
Copy link
Member

@mskelton mskelton left a comment

Choose a reason for hiding this comment

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

Good stuff!

README.md Outdated
```js
// foo.spec.ts
import { test, expect } from "@playwright/test"
import "expect-playwright"
Copy link
Member

Choose a reason for hiding this comment

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

Wondering if we need this section given the TypeScript section at the bottom of the readme:

https://github.com/playwright-community/expect-playwright#typescript

@mxschmitt mxschmitt merged commit 09ea541 into master Jun 8, 2021
@mxschmitt mxschmitt deleted the compatible-with-playwright-test branch June 8, 2021 21:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants