cypress-plugins: Slack video link is incorrect for --custom-url

Hi @YOU54F, I’m also seeing this issue, which doesn’t seem to be resolved?

When using

--ci-provider custom --custom-url "${GOCDURL}"

the resulting video artifact path in Slack is

https://xxxx/files/test_DOCKER_core_cypress_debug/7/run_all/1/run_all//var/go/rg_cypress/cypress/videos/PR/xx.spec.js.mp4

note the //var/go/rg_cypress is not needed, the relative path should work.

Also if --video-dir 'rg_cypress/cypress/videos' is added, there are no video urls output to Slack at all.

Please could this be looked into? Thanks

About this issue

Original URL
State: closed
Created 3 years ago
Comments: 25 (12 by maintainers)

Most upvoted comments

@YOU54F we had some permission issues that were preventing the /mochawesome-report being created in CI following an upgrade. So 1.1.9-alpha seems to work for report and artefacts. Sorry for any confusion, glad it’s working.

you1anna on May 5, 2021

Cool beans. If you fork the repo and put your changes on a branch you will be able to create an pr again the main repo. You can always delete your fork once the pr is complete. 🤙 On Fri, 23 Apr 2021 at 17:40, Samantha Swartz @.***> wrote:

@YOU54F https://github.com/YOU54F happy to put up a PR - I cloned and have the fix + some additional tests locally, but I need permission to push to the repo

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/YOU54F/cypress-slack-reporter/issues/979#issuecomment-825779706, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEYCJ4MCJV4QG6ZP5AQMGPTTKGPGZANCNFSM4ZLIIQ6A .

YOU54F on Apr 24, 2021

@YOU54F happy to put up a PR - I cloned and have the fix + some additional tests locally, but I need permission to push to the repo

samswartz13 on Apr 23, 2021

1.1.7-alpha just released now @you1anna

PR here https://github.com/YOU54F/cypress-slack-reporter/pull/988

YOU54F on Apr 12, 2021

Ace, ill push a fix up after work matey thanks for your time testing

On Mon, 12 Apr 2021 at 11:02, Robin Miklinski @.***> wrote:

Looking at 1.1.6-alpha the output is run_all/1/run_allcypress/videos/PR/ecards.spec.js.mp4 The correct relative path is /cypress/videos/PR/ecards.spec.js.mp4 so if the missing slash is added, it looks like it work work.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/YOU54F/cypress-slack-reporter/issues/979#issuecomment-817673867, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEYCJ4NIZI55H6ZQ6O7GB33TILALFANCNFSM4ZLIIQ6A .

YOU54F on Apr 12, 2021