-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Issue 1281 warning on no write (#7126)
* replace fs write error with warning This change replaces the error with a warning when cypress is executing tests in a readonly fs. * some cleanup - don't need to run test in a specific browser - can combine SS + video test into one - don't seem to need readonly_fs_spec.js * if videos folder cannot be created, just warn * try a non-root e2e test stage * fix comment * FOLDER_NOT_ACCESSIBLE Co-authored-by: Fabian Wiles <fabianwiles@live.com> Co-authored-by: Zach Bloomquist <github@chary.us>
- Loading branch information
3 people
authored
May 12, 2020
1 parent
3558479
commit f5284c6
Showing
10 changed files
with
200 additions
and
34 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
82 changes: 82 additions & 0 deletions
82
packages/server/__snapshots__/non_root_read_only_fs_spec.ts.js
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,82 @@ | ||
| exports['e2e readonly fs / warns when unable to write to disk'] = ` | ||
| Folder /foo/bar/.projects/read-only-project-root is not writable. | ||
| Writing to this directory is required by Cypress in order to store screenshots and videos. | ||
| Enable write permissions to this directory to ensure screenshots and videos are stored. | ||
| If you don't require screenshots or videos to be stored you can safely ignore this warning. | ||
| ✅ not running as root | ||
| ✅ /foo/bar/.projects/read-only-project-root is not writable | ||
| ==================================================================================================== | ||
| (Run Starting) | ||
| ┌────────────────────────────────────────────────────────────────────────────────────────────────┐ | ||
| │ Cypress: 1.2.3 │ | ||
| │ Browser: FooBrowser 88 │ | ||
| │ Specs: 1 found (spec.js) │ | ||
| │ Searched: cypress/integration/spec.js │ | ||
| └────────────────────────────────────────────────────────────────────────────────────────────────┘ | ||
| ──────────────────────────────────────────────────────────────────────────────────────────────────── | ||
| Running: spec.js (1 of 1) | ||
| Warning: We failed to record the video. | ||
| This error will not alter the exit code. | ||
| Error: EACCES: permission denied, mkdir '/foo/bar/.projects/read-only-project-root/cypress/videos' | ||
| 1) fails | ||
| 0 passing | ||
| 1 failing | ||
| 1) fails: | ||
| Error: EACCES: permission denied, mkdir '/foo/bar/.projects/read-only-project-root/cypress/screenshots' | ||
| (Results) | ||
| ┌────────────────────────────────────────────────────────────────────────────────────────────────┐ | ||
| │ Tests: 1 │ | ||
| │ Passing: 0 │ | ||
| │ Failing: 1 │ | ||
| │ Pending: 0 │ | ||
| │ Skipped: 0 │ | ||
| │ Screenshots: 0 │ | ||
| │ Video: true │ | ||
| │ Duration: X seconds │ | ||
| │ Spec Ran: spec.js │ | ||
| └────────────────────────────────────────────────────────────────────────────────────────────────┘ | ||
| Warning: We failed processing this video. | ||
| This error will not alter the exit code. | ||
| Error: ffmpeg exited with code 1: /foo/bar/.projects/read-only-project-root/cypress/videos/spec.js.mp4: No such file or directory | ||
| [stack trace lines] | ||
| ==================================================================================================== | ||
| (Run Finished) | ||
| Spec Tests Passing Failing Pending Skipped | ||
| ┌────────────────────────────────────────────────────────────────────────────────────────────────┐ | ||
| │ ✖ spec.js XX:XX 1 - 1 - - │ | ||
| └────────────────────────────────────────────────────────────────────────────────────────────────┘ | ||
| ✖ 1 of 1 failed (100%) XX:XX 1 - 1 - - | ||
| ` |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
f5284c6There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Circle has built the
linux x64version of the Test Runner.You can install this pre-release platform-specific build using instructions at https://on.cypress.io/installing-cypress#Install-pre-release-version.
You will need to use custom
CYPRESS_INSTALL_BINARYurl and install Cypress using an url instead of the version.f5284c6There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
AppVeyor has built the
win32 ia32version of the Test Runner.You can install this pre-release platform-specific build using instructions at https://on.cypress.io/installing-cypress#Install-pre-release-version.
You will need to use custom
CYPRESS_INSTALL_BINARYurl and install Cypress using an url instead of the version.Instructions are included below, depending on the shell you are using.
In Command Prompt (
cmd.exe):In PowerShell:
In Git Bash:
Using
cross-env:If the above commands do not work for you, you can also try using
cross-env:f5284c6There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
AppVeyor has built the
win32 x64version of the Test Runner.You can install this pre-release platform-specific build using instructions at https://on.cypress.io/installing-cypress#Install-pre-release-version.
You will need to use custom
CYPRESS_INSTALL_BINARYurl and install Cypress using an url instead of the version.Instructions are included below, depending on the shell you are using.
In Command Prompt (
cmd.exe):In PowerShell:
In Git Bash:
Using
cross-env:If the above commands do not work for you, you can also try using
cross-env:f5284c6There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Circle has built the
darwin x64version of the Test Runner.You can install this pre-release platform-specific build using instructions at https://on.cypress.io/installing-cypress#Install-pre-release-version.
You will need to use custom
CYPRESS_INSTALL_BINARYurl and install Cypress using an url instead of the version.