-
Notifications
You must be signed in to change notification settings - Fork 14
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.
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
feat(PPDSC-2313): script to run applitools locally #321
Conversation
- Shell script to set env variables (inc asking for api key) - Add scripts to package json to run comp and doc tests
You can preview these changes on: |
- Add local applitools scripts to scripts.md - Guide user to only run a subset of tests
How this script will be fitting the development flow? In which scenario |
Yep. The idea is that if you know you will have to resolve some tests because you have made visual changes, you don't have to run the whole pipeline (and all >1,600 tests) before being able to resolve them. You can run them locally, resolve any changes, then when your pipeline runs they should all pass. Alternatively, if you want to make sure that you have not made any visual changes that are going to cause the pipeline to fail, this script is a quick way to check. The general idea is that this will make resolving and debugging Applitools quicker for devs and will reduce our usage. |
* feat(PPDSC-2313): script to run applitools locally - Shell script to set env variables (inc asking for api key) - Add scripts to package json to run comp and doc tests * feat(PPDSC-2313): update readme and encourage fewer tests - Add local applitools scripts to scripts.md - Guide user to only run a subset of tests * feat(PPDSC-2313): warn user about unclosed batches * feat(PPDSC-2313): fix break error
PPDSC-2313
What
The script will guide the user to run fewer tests by using the
include
config option, then ask for an api key:Then when the suite has finished the results and a link to the batch in the Applitools UI are shown:
I have done:
I have tested manually:
Before:
After:
Who should review this PR:
How to test: