We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
PlaywrightEnvironment
This line could be changed and it'd help users of PlaywrightEnvironment to leverage the context that's passed to jest environments
context
From
constructor(config: JestEnvironmentConfig) { super(config) this._config = config.projectConfig as JestPlaywrightProjectConfig }
to
constructor(config: JestEnvironmentConfig, context: EnvironmentContext) { super(config, context) this._config = config.projectConfig as JestPlaywrightProjectConfig }
This is actually how jest recommends creating your own test environment in their docs:
// my-custom-environment const NodeEnvironment = require('jest-environment-node').TestEnvironment; class CustomEnvironment extends NodeEnvironment { constructor(config, context) { super(config, context); console.log(config.globalConfig); console.log(config.projectConfig); this.testPath = context.testPath; this.docblockPragmas = context.docblockPragmas; } ...
Describe the solution you'd like Passing context when calling super
super
Additional context This is an issue specifically for Datadog's test visibility.
If you're OK with this change I can contribute it myself 😄
The text was updated successfully, but these errors were encountered:
RootEnv
I created a PR just for this: #820
Sorry, something went wrong.
Successfully merging a pull request may close this issue.
This line could be changed and it'd help users of
PlaywrightEnvironment
to leverage thecontext
that's passed to jest environmentsFrom
to
This is actually how jest recommends creating your own test environment in their docs:
Describe the solution you'd like
Passing
context
when callingsuper
Additional context
This is an issue specifically for Datadog's test visibility.
If you're OK with this change I can contribute it myself 😄
The text was updated successfully, but these errors were encountered: