Job fails with `Cannot open display`

[stefanbuck]

Job fails with `Cannot open display`

[jcblw]

Hey there, @stefanbuck

This is the place I am using this action
https://github.com/mujo-code/source/blob/master/.github/workflows/push.yml#L32

and here is the configuration of puppeteer
https://github.com/mujo-code/source/blob/master/packages/extension/src/e2e.test.js#L19

The only thing that jumps out to me is the different passing of args. I pass it in a with block. Let me know if that working and I can update the documentation around this.

[stefanbuck]

Right after opening this issue I spotted this as well. Thanks for confirming this and I'll let you know if this works.

[stefanbuck]

This works, but only if I use test to issue a npm test command. In my case I want to use npm run e2e so set args to run e2e. This seems to be cut off on the whitespace.

[jcblw]

yup! left a comment over on the other repo.

OctoLinker/OctoLinker#662 (review)

I think your use case is more common so I am open to changing the way the command is executed.

[stefanbuck]

I just pushed a commit with args: '"run e2e"' which should work, but it looks super weird 😆

[stefanbuck]

What about decoupling this from npm so you can pass every command like npm test, npm run whatever or even run.sh?

[jcblw]

Seems like that run failed. I am also up for that. I am not sure who all is using it. I know if we switch to just a running a sh script that will bust any existing. that being said I do thing that is probably the right direction.

[stefanbuck]

Here we go #9

[jcblw]

@stefanbuck I merged it thanks again, I think you can run it off master. Let me know if you need a release setup to try.