BackstopJS: Failing to create bitmaps_test folder in GitHub Actions after upgrade to 6.2.0
We have successfully been using backstopjs in our GitHub Actions with 6.1.4, using npx backstop --docker test. On upgrading to 6.2.0 the backstop run in GitHub Actions fails, erroring with a bunch of:
Browser Console Log 0: JSHandle:BackstopTools have been installed.
Error: EACCES: permission denied, mkdir 'backstop_data/bitmaps_test'
We suspect the regression may be stemming from the changes in user introduced in https://github.com/garris/BackstopJS/commit/6d221a275686bad7c7b58fd7ff1325e7067e1bcb.
About this issue
- Original URL
- State: closed
- Created a year ago
- Comments: 18 (14 by maintainers)
Commits related to this issue
- fix permission problem with local bind mount (#1466) — committed to tkrah/BackstopJS by tkrah a year ago
- fix permission problem with local bind mount (#1466) (#1467) — committed to garris/BackstopJS by tkrah a year ago
- Rollback docker user change - defaults to root now again - added a recommendation to the README.md to configure a more appropriate user to run the test by changing the dockerCommandTemplate confi... — committed to tkrah/BackstopJS by tkrah a year ago
- Rollback docker user change - defaults to root now again - added a recommendation to the README.md to configure a more appropriate user to run the test by changing the dockerCommandTemplate confi... — committed to tkrah/BackstopJS by tkrah a year ago
I would have removed the user command from the docker file all together, so I don’t know if we would have that template still in use. Let’s just document it (windows user have no chance anyway to change it, which would again will be against your rule 2). I’ll prepare a PR and we can discuss it there further. 😉
backstopjs runs as user node now, not anymore as root user, but better care about the user mapping than having root owned files which a user can’t delete anymore 😉.
I’m part of the same ensemble as @davidcmoulton and can confirm that the patch works for us:
https://github.com/sciety/sciety/pull/2168