updates for teal.slice

[nikolas-burkoff]

Part of insightsengineering/teal.slice#33 requires insightsengineering/teal.slice#49

[github-actions]

Code Coverage Summary

Filename                          Stmts    Miss  Cover    Missing
------------------------------  -------  ------  -------  ---------------------------------------------------------------------
R/all_choices.R                       1       0  100.00%
R/call_utils.R                      156     124  20.51%   14-23, 64, 66, 68, 107-431
R/check_selector.R                   31       0  100.00%
R/choices_labeled.R                 202      61  69.80%   60, 71, 76, 87, 103, 221-225, 229-234, 264-277, 390-391, 393, 425-473
R/choices_selected.R                 81      11  86.42%   201-229, 260
R/column_functions.R                  3       3  0.00%    13-16
R/data_extract_datanames.R           32       8  75.00%   9-13, 64-66
R/data_extract_filter_module.R       91      32  64.84%   68-81, 83, 86-87, 89-105, 142
R/data_extract_module.R             205      20  90.24%   3, 121, 126, 143, 146-151, 153, 172-175, 222, 355, 360, 392, 398
R/data_extract_read_module.R        121      13  89.26%   27, 31-34, 101-106, 115, 132
R/data_extract_select_module.R       32      18  43.75%   31-48
R/data_extract_single_module.R       53       2  96.23%   29, 42
R/data_extract_spec.R                32       0  100.00%
R/data_merge_module.R                44      19  56.82%   101-122
R/filter_spec.R                     186       1  99.46%   373
R/format_data_extract.R              16       1  93.75%   49
R/get_dplyr_call.R                  298       0  100.00%
R/get_merge_call.R                  283      32  88.69%   29-36, 47, 210-219, 362, 369, 385-397, 460
R/include_css_js.R                    5       0  100.00%
R/input_checks.R                     11       2  81.82%   18-19
R/merge_data_utils.R                  2       0  100.00%
R/merge_datasets.R                  154       7  95.45%   71, 123, 251-255
R/resolve_delayed.R                 104      44  57.69%   223-306
R/select_spec.R                      49       8  83.67%   149, 213-220
R/utils.R                            15      11  26.67%   26-39
R/zzz.R                               2       2  0.00%    2-3
TOTAL                              2209     419  81.03%

Results for commit: b0cea59

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

[github-actions]

Unit Tests Summary

    1 files    20 suites   35s ⏱️
146 tests 145 ✔️ 1 💤 0 ❌
543 runs  542 ✔️ 1 💤 0 ❌

Results for commit 9e12a2d.

[kpagacz]

Interesting that isolate was needed before, but it is not now. Not saying there is something wrong with it, just musing over possible performance implications.

[gogonzo]

I just checked on main - wasn't needed before also