Skip to content

test: add installed-package smoke test for Shiny app - #1412

Draft
Gero1999 wants to merge 1 commit into
mainfrom
test/installed-app-smoke
Draft

test: add installed-package smoke test for Shiny app#1412
Gero1999 wants to merge 1 commit into
mainfrom
test/installed-app-smoke

Conversation

@Gero1999

Copy link
Copy Markdown
Collaborator

Launches the app against a real R CMD INSTALL (exported-only namespace, build-filtered files) from a temp dir, without load_all. This is the only context that surfaces install-only failures such as bare-name internal calls and resource paths pointing at .Rbuildignore'd files, which the existing load_all-based suite cannot catch.

Hosted in the existing tests job to reuse its Chrome and dependency setup; job pinned to contents: read.

Issue

Closes #

Description

Change description.

Definition of Done

Definition of done, preferably copied from the issue.

How to test

How to test features not covered by unit tests.

Contributor checklist

  • Code passes lintr checks
  • Code passes all unit tests
  • New logic covered by unit tests
  • New logic is documented
  • App or package changes are reflected in NEWS
  • Package version is incremented
  • R script works with the new implementation (if applicable)
  • Settings upload works with the new implementation (if applicable)
  • If any .scss change was done, run data-raw/compile_css.R
  • If a package dependency was added/changed, run data-raw/test_suggests_hidden.R

Notes to reviewer

Anything that the reviewer should know before tacking the pull request?

@Gero1999
Gero1999 force-pushed the test/installed-app-smoke branch from bbb3d87 to d824426 Compare July 27, 2026 12:49
@Gero1999

Copy link
Copy Markdown
Collaborator Author

As expected here the CI tests failed, which might mean that the test is detecting the issue

@Gero1999
Gero1999 force-pushed the test/installed-app-smoke branch from 47312d2 to 7673b2d Compare July 27, 2026 16:51
@Gero1999
Gero1999 force-pushed the test/installed-app-smoke branch from 7673b2d to f63a82d Compare July 27, 2026 16:58
@Gero1999

Copy link
Copy Markdown
Collaborator Author

we need to merge agian the other branch and check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant