insightsengineering / insightsengineering/teal.picks
Tests are skipped
- Dominant language
- R
- Stars
- 4
- Forks
- 0
- Avg merge
- 6d 11h
- Merged PRs (30d)
- 3
Description
Some tests have `skip("todo: tests can't trigger data()")` because the reactivity is hard to simulate.
However, in some other tests we rely on the experimental feature of shiny to test those cases:
```
shiny::reactiveConsole(TRUE)
on.exit(reactiveConsole(FALSE))
```
We should activate those test and verify them. Activating them as above will show they are currently failing.
Contributor guide
Research direction
The issue does not name specific files; start by locating tests containing `skip("todo: tests can't trigger data()")` and compare them with tests that use `shiny::reactiveConsole(TRUE)`. Enable the skipped tests with the demonstrated reactive-console setup, investigate their current failures, and finish when the cases run successfully without being skipped.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- r
- Domain
- testing
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100