insightsengineering / insightsengineering/teal.picks

Tests are skipped

Open
#111 0 comments 0 reactions 0 assignees View on GitHub
core enhancement
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.