Update test commands (#1205)
* As per @Yun Feng: everyone has npm installed globally but maybe not yarn * Add command to enable test result updating * Default to running tests HEADLESS on rrweb * Add command to build:all in a low memory environment * Add a 'retest' command for when the code hasn't changed, but you are working on the test cases * Add commands to reformat according to prettier. Named 'reformat' to indicate that we are doing a `--write` * Update package.json Co-authored-by: Yun Feng <yun.feng0817@gmail.com> * Apply suggestions from code review Co-authored-by: Yun Feng <yun.feng0817@gmail.com> * Create few-turkeys-reflect.md * Apply formatting changes --------- Co-authored-by: Justin Halsall <Juice10@users.noreply.github.com> Co-authored-by: Yun Feng <yun.feng0817@gmail.com>
This commit is contained in:
@@ -64,8 +64,9 @@ Since we want the record and replay sides to share a strongly typed data structu
|
||||
2. Run `yarn install` in the root to install required dependencies for all sub-packages (note: `npm install` is _not_ recommended).
|
||||
3. Run `yarn dev` in the root to get auto-building for all the sub-packages whenever you modify anything.
|
||||
4. Navigate to one of the sub-packages (in the `packages` folder) where you'd like to make a change.
|
||||
5. Patch the code and run `yarn test` to run the tests, make sure they pass before you commit anything.
|
||||
6. Push the code and create a pull request.
|
||||
5. Patch the code and run `yarn test` to run the tests, make sure they pass before you commit anything. Add test cases in order to avoid future regression.
|
||||
6. If tests are failing, but the change in output is desirable, run `yarn test:update` and carefully commit the changes in test output.
|
||||
7. Push the code and create a pull request.
|
||||
|
||||
Protip: You can run `yarn test` in the root folder to run all the tests.
|
||||
|
||||
|
||||
Reference in New Issue
Block a user