Skip to content

Commit 18d6ac8

Browse files
authored
fix(tests): Add missing env vars for shinyapps.io tests (#1663)
1 parent 7851957 commit 18d6ac8

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

.github/workflows/deploy-tests.yaml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,7 @@ jobs:
6464
pypi_shiny: false
6565
pypi_rsconnect: false
6666
base_test_dir: "./tests/playwright/deploys"
67+
app_name: "express_page_sidebar"
6768
test_shinyappsio: true
6869

6970
fail-fast: false
@@ -110,9 +111,9 @@ jobs:
110111
DEPLOY_APPS: "true"
111112
DEPLOY_CONNECT_SERVER_URL: "${{ (matrix.config.released_connect_server && 'https://connect.posit.it/') || 'https://rsc.radixu.com/' }}"
112113
DEPLOY_CONNECT_SERVER_API_KEY: "${{ (matrix.config.released_connect_server && secrets.DEPLOY_CONNECT_POSIT_SERVER_API_KEY) || secrets.DEPLOY_CONNECT_SERVER_API_KEY }}"
113-
DEPLOY_SHINYAPPS_NAME: "${{ matrix.config.test_shinyappsio && matrix.config.shinyapps_name }}"
114-
DEPLOY_SHINYAPPS_TOKEN: "${{ matrix.config.test_shinyappsio && matrix.config.shinyapps_token }}"
115-
DEPLOY_SHINYAPPS_SECRET: "${{ matrix.config.test_shinyappsio && matrix.config.shinyapps_secret }}"
114+
DEPLOY_SHINYAPPS_NAME: "${{ matrix.config.test_shinyappsio && secrets.DEPLOY_SHINYAPPS_NAME }}"
115+
DEPLOY_SHINYAPPS_TOKEN: "${{ matrix.config.test_shinyappsio && secrets.DEPLOY_SHINYAPPS_TOKEN }}"
116+
DEPLOY_SHINYAPPS_SECRET: "${{ matrix.config.test_shinyappsio && secrets.DEPLOY_SHINYAPPS_SECRET }}"
116117
EXPRESS_PAGE_SIDEBAR_NAME: "${{ matrix.config.app_name }}"
117118
DEPLOY_GITHUB_REQUIREMENTS_TXT: "${{ !matrix.config.pypi_shiny }}"
118119
timeout-minutes: 30

tests/playwright/deploys/express-page_sidebar/test_deploys_express_page_sidebar.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,8 @@
1313

1414
app_url = local_deploys_app_url_fixture(
1515
# Possibly use a different app name given by an GHA env var
16-
os.getenv("EXPRESS_PAGE_SIDEBAR_NAME", "express_page_sidebar")
16+
os.getenv("EXPRESS_PAGE_SIDEBAR_NAME")
17+
or "express_page_sidebar"
1718
)
1819

1920

0 commit comments

Comments
 (0)