Multiple app directories doesn't split app-dir input (on Windows)

Open
#337 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
50/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Stale
Tech stack
github-actions, r
Domain
ci-cd

Research direction

Start at actions/test-app/action.yml line 22 and compare how the multiline app-dir input is split for Windows-generated files. Reproduce the issue with the shown simple_app, first_dummy, and second_dummy values, then verify that each directory is passed separately on Windows.

Written by the indexing model from the issue text.

Description

When including multiple-line app-dirs in the yaml file:

      - uses: rstudio/shinytest2/actions/test-app@v1
        with:
          app-dir: |
            simple_app
            first_dummy
            second_dummy

Returns:

Error: An error occurred trying to start process '/usr/local/bin/Rscript' with working directory '/home/runner/work/test_shinytests/test_shinytests/simple_app
first_dummy
second_dummy
'. No such file or directory

https://github.com/andrewbaxter439/test_shinytests/actions/runs/4926230322/jobs/8801491068#step:6:52

This split doesn't seem to be working on Windows-generated files at least:

https://github.com/rstudio/shinytest2/blob/6850de4a421fd592873e638d6deda8a4fd176660/actions/test-app/action.yml#L22

Dominant language
R
Stars
123
Forks
23
PR merge metrics
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from rstudio/shinytest2

All issues in rstudio/shinytest2

Similar issues

More R issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.