Skip to content
This repository was archived by the owner on Sep 11, 2025. It is now read-only.

Conversation

mmarkelov
Copy link
Member

close #477

@mmarkelov mmarkelov requested a review from mxschmitt as a code owner December 4, 2020 20:04
@coveralls
Copy link

coveralls commented Dec 4, 2020

Pull Request Test Coverage Report for Build 401436923

  • 7 of 7 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.6%) to 96.407%

Totals Coverage Status
Change from base Build 401380866: 0.6%
Covered Lines: 105
Relevant Lines: 107

💛 - Coveralls

@mmarkelov mmarkelov merged commit e02502d into master Dec 4, 2020
@mmarkelov mmarkelov deleted the Some-improvements branch December 4, 2020 20:07
Comment on lines +135 to +136
jestPlaywrightDebug: JestPlaywrightTestDebug
jestPlaywrightConfig: JestPlaywrightTestConfig
Copy link
Contributor

Choose a reason for hiding this comment

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

@mmarkelov is this correct?
in https://github.com/playwright-community/jest-playwright/blob/master/src/extends.ts#L104
I don't see describe being extended with jestPlaywrightDebug or jestPlaywrightConfig. Only jestPlaywrightSkip.
Or am I missing something?

Copy link
Member Author

Choose a reason for hiding this comment

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

@kenny-f my bad. I'll think about if we need these kind of helpers for describe or we just want to skip only

Copy link
Contributor

Choose a reason for hiding this comment

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

@mmarkelov I think it's useful to have for describe as well as it allows people to group tests differently and not be restricted by what is extended and what isn't.

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.

Tests don't run

3 participants