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.

Sign up for GitHub

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

Jump to bottom

Clarify e2e dependency on yarn in contributin docs #43287

Merged
merged 8 commits into from
Dec 9, 2022
Merged

Clarify e2e dependency on yarn in contributin docs #43287

Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
50aab6c
Clarify e2e dependency on yarn in contributin docs
jankaifer Nov 23, 2022
c6b620b
Reformulated sentence in docs due to a linter rule
jankaifer Nov 23, 2022
93b8cbf
Add COREPACK_ENABLE_STRICT=0 before yarn pack
jankaifer Nov 23, 2022
f20bc63
fixed docs
jankaifer Nov 28, 2022
f661636
made yarn pack invocation work in non-bash shells
jankaifer Dec 1, 2022
342cf7a
Merge branch 'canary' into contributing-docs-yarn
jankaifer Dec 1, 2022
7eae25b
Merge branch 'canary' into contributing-docs-yarn
ijjk Dec 8, 2022
e6ae50f
Merge branch 'canary' into contributing-docs-yarn
jankaifer Dec 9, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion .github/actions/next-stats-action/src/prepare/repo-setup.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,13 @@ module.exports = (actionInfo) => {
// to the correct versions
for (const pkgName of pkgDatas.keys()) {
const { pkg, pkgPath } = pkgDatas.get(pkgName)
await exec(`cd ${pkgPath} && yarn pack -f ${pkg}-packed.tgz`, true)
await exec(`cd ${pkgPath} && yarn pack -f ${pkg}-packed.tgz`, true, {
env: {
// Yarn installed through corepack will not run in pnpm project without this env var set
// This var works for corepack >=0.15.0
COREPACK_ENABLE_STRICT: '0',
},
})
}
return pkgPaths
},
Expand Down
7 changes: 7 additions & 0 deletions contributing/core/testing.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,13 @@ When the test runs it will open the browser that is in the background by default
pnpm testonly test/integration/production/ -t "should allow etag header support"
```

**End-to-end (e2e)** tests are run in complete isolation from the repository.
When you run an e2e test, a local version of next will be created inside your system's temp folder (eg. /tmp),
which is then linked to the app, also created inside a temp folder. A server is started on a random port, against which the tests will run.
After all tests have finished, the server is destroyed and all remaining files are deleted from the temp folder.

You will need `yarn` for running e2e tests. Installing it `corepack` won't work because `next.js` is `pnpm` workspace.

## Writing tests for Next.js

### Getting Started
Expand Down