-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Screenshots saved under 'All Specs' directory when running all specs #2319
Comments
This has been a major pain point for me. Related (or the root cause), |
This is really part of the refactor that needs to happen to In fact this is hangover from when we implemented spec isolation and there is an open issue that discusses possible avenues for handling it better. TLDR: when running all the specs it will "transition" between specs so that each still loads in isolation. That would fix this issue as well. |
I see that the assigned people were removed, is this on hold? Or do we have a timeline for this issue? |
This issue is still in the 'ready for work' stage, which means no work has been done on this issue as of today, so we do not have an estimate on when this will be delivered. |
I'd like to contribute a fix for this bug. Where would I start? |
We have also seen a issue where in when we run a particular set of tests with spec for eg: if all tests under UI folder then if tests have been stored under multiple subfolders then path from mochawesome doesnt report correctly leading to screenshot not displaying in the mochawesome reports , My code looks like this // //adding screenshots to the report const addContext = require('mochawesome/addContext');
Cypress.on('test:after:run', (test, runnable) => {
if (test.state === 'failed') {
const screenshotFileName = `${runnable.parent.title} -- ${test.title} (failed).png`;
addContext(
{
test,
},
`assets/${Cypress.spec.name}/${screenshotFileName}`
);
} folder structure is |
@jennifer-shehane can I get some response from a project manager or engineer about the status of this? I am willing to contribute to this, with the proper guidance |
@vegerot This is not a priority for our team currently, but we would review a PR to fix this. Check out our contributing doc. |
Run All specs functionality was removed in Cypress 10.0.0. See #21628 |
We want to save screenshots under a directory with the spec path, but when running all specs, all screenshots end up under an
All Specs
directory. We need to ensure each screenshot gets saved under its own spec's directory when running all specs.See #2091 (comment)
The text was updated successfully, but these errors were encountered: