-
Notifications
You must be signed in to change notification settings - Fork 669
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
Enable screenshots functionality #104
Comments
I'm working on this |
@inikulin @VasilyStrelyaev please check it and write your opinion regarding the issue:
runner.screenshots (path, takeOnFails = false)
When
act.screenshot('my-screenshots/1.png'); Saves to
And one more thing. Now we always create a thumbnail for each screenshot and put it in the
Maybe should we make an option for thumbnails creating? |
|
With |
Or enclose the OS name in brackets |
Why do I use |
It's actual for CI. The typical scenario is: save screenshots during test run on the testing machine (it's maybe a virtual machine), compare them with original images then clear them or save to some storage |
The studio will works with screenshots as before |
Wrong. You will just specifiy the root dir for your screenshots. Then for each test run dir with the unique ID will be created. So, nothing will be overwritten |
I suppose, @VasilyStrelyaev writes not about |
@VasilyStrelyaev @AlexanderMoskovkin Well, if you explicitly specify the path to the screenshot, then it's uniqueness is your burden now. I don't see any problem here. |
@VasilyStrelyaev @inikulin
Should we highlight the |
1)Let's move all error screenshots to the E.g.:
|
|
I like the third one |
Cool! |
…nctionality Enable screenshot functionality (closes #104)
@AlexanderMoskovkin is there any option to disable creating thumbnails? |
@saeidzebardast Not yet, but we're going to do it in the contex of the #1550 issue. |
I can't seem to find anywhere so I'l ask here. Has there been any talk of supporting screenshot name pattern? I'd love if the created screenshots generated on failure could be named based on the test name...but currently it only seems to be named via the index, as I get: |
Hi @graeme-plangrid, Take a look at the #2152 issue. Does this proposal feet your need. Feel free to add you comments there. We already have a PR for this and we'll review and merge it once we're done with our current tasks |
i would appreciate a clarification: If im running old testcafe tests, which were recorded manually from the browser recorder, is this currently the only was in which screenshots are still supported when Testcafe is integrated in a command line/CI environment |
Hi @peterpfberlin, TestCafe supports screenshot functionality for both tests captured with the browser recorder and written using JavaScript API. |
@AlexanderMoskovkin Yes, this looks good! Thanks for pointing me in the right direction. |
This thread has been automatically locked since it is closed and there has not been any recent activity. Please open a new issue for related bugs or feature requests. We recommend you ask TestCafe API, usage and configuration inquiries on StackOverflow. |
No description provided.
The text was updated successfully, but these errors were encountered: