Issue/137 playwright setup #138
Conversation
| This contains information on how to run different tests and add new tests for Lyra. | ||
|
|
||
| We are using Playwright to test integration of features. | ||
| Read the documentation at [playwright.dev/docs](https://playwright.dev/docs) |
There was a problem hiding this comment.
Unfortunately, the URL does not lead to any documentation.
Screencast.from.2024-10-27.10-10-49.webm
Let's update the URL to something that resolves in documentation.
There was a problem hiding this comment.
Will change to correct URL
| @@ -5,4 +5,5 @@ module.exports = { | |||
| moduleNameMapper: { | |||
| '@/(.*)': '<rootDir>/src/$1', | |||
| }, | |||
| modulePathIgnorePatterns: ['<rootDir>/integrationTesting'], | |||
There was a problem hiding this comment.
modulePathIgnorePatterns is for making modules non-require()-able in the test environment. Why do we want to do that?
https://jestjs.io/docs/configuration#modulepathignorepatterns-arraystring
If it is for ignoring Playwright tests, there is a testPathIgnorePatterns that we can use: https://jestjs.io/docs/configuration#testpathignorepatterns-arraystring
There was a problem hiding this comment.
You understood my intent correctly, It is to exclude integration test from running when running jest. :)
Updating this too.
| "@playwright/test": "^1.48.2", | ||
| "@types/node": "^22.8.1" |
There was a problem hiding this comment.
These two new devDependencies are what npm init playwright@latest adds.
We already have a devDependency on @types/node declared in the root package so we do not need to add a new one here.
Lyra does not require Node 22 yet so we should not to update types to types for Node 22.
Playwright Setup
Resolves #137