-
-
Notifications
You must be signed in to change notification settings - Fork 15
changed TealAppDriver constructor #927
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Unit Tests Summary 1 files 23 suites 17m 5s ⏱️ Results for commit 7a8e0d7. ♻️ This comment has been updated with latest results. |
Code Coverage SummaryDiff against mainResults for commit: 7a8e0d7 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
|
Remember to install teal before, and then run tests in tmg. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey, no issues after reinstalling teal
> options(TESTING_DEPTH = 5);devtools::test()
ℹ Testing teal.modules.general
Registered S3 method overwritten by 'teal':
method from
c.teal_slices teal.slice
Registered S3 methods overwritten by 'ggpp':
method from
heightDetails.titleGrob ggplot2
widthDetails.titleGrob ggplot2
Registered S3 method overwritten by 'tern':
method from
tidy.glm broom
✔ | F W S OK | Context
✔ | 37 0 | examples
✔ | 11 0 | shinytest2-tm_a_pca
✔ | 7 0 | shinytest2-tm_a_regression
✔ | 4 0 | shinytest2-tm_data_table
✔ | 4 0 | shinytest2-tm_file_viewer
✔ | 3 0 | shinytest2-tm_front_page
✔ | 4 0 | shinytest2-tm_g_association
✔ | 4 0 | shinytest2-tm_g_bivariate
✔ | 3 0 | shinytest2-tm_g_distribution
✔ | 4 0 | shinytest2-tm_g_response
✔ | 5 0 | shinytest2-tm_g_scatterplot
✔ | 4 0 | shinytest2-tm_g_scatterplotmatrix
✔ | 5 0 | shinytest2-tm_misssing_data
✔ | 10 0 | shinytest2-tm_outliers
✔ | 4 0 | shinytest2-tm_t_crosstable
✔ | 6 0 | shinytest2-tm_variable_browser
✔ | 18 | tm_g_bivariate
✔ | 12 | utils
✔ | 21 | bivariate_ggplot_call
✔ | 4 | front_page
✔ | 15 | g_expert_cl
✔ | 6 | scatterplotmatrix_get_stats
✔ | 3 | variable_browser
══ Results ═════════════════════════════════════════════════════════════════════════════
Duration: 1.6 s
── Skipped tests (115) ─────────────────────────────────────────────────────────────────
• chromium (115): test-examples.R:144:7, test-examples.R:144:7, test-examples.R:144:7,
test-examples.R:144:7, test-examples.R:144:7, test-examples.R:144:7,
test-examples.R:144:7, test-examples.R:144:7, test-examples.R:144:7,
test-examples.R:144:7, test-examples.R:144:7, test-examples.R:144:7,
test-examples.R:144:7, test-examples.R:144:7, test-examples.R:144:7,
test-examples.R:144:7, test-examples.R:144:7, test-examples.R:144:7,
test-examples.R:144:7, test-examples.R:144:7, test-examples.R:144:7,
test-examples.R:144:7, test-examples.R:144:7, test-examples.R:144:7,
test-examples.R:144:7, test-examples.R:144:7, test-examples.R:144:7,
test-examples.R:144:7, test-examples.R:144:7, test-examples.R:144:7,
test-examples.R:144:7, test-examples.R:144:7, test-examples.R:144:7,
test-examples.R:144:7, test-examples.R:144:7, test-examples.R:144:7,
test-examples.R:144:7, test-shinytest2-tm_a_pca.R:35:3,
test-shinytest2-tm_a_pca.R:60:3, test-shinytest2-tm_a_pca.R:78:3,
test-shinytest2-tm_a_pca.R:96:3, test-shinytest2-tm_a_pca.R:114:3,
test-shinytest2-tm_a_pca.R:136:3, test-shinytest2-tm_a_pca.R:160:3,
test-shinytest2-tm_a_pca.R:179:3, test-shinytest2-tm_a_pca.R:194:3,
test-shinytest2-tm_a_pca.R:227:3, test-shinytest2-tm_a_pca.R:246:3,
test-shinytest2-tm_a_regression.R:49:3, test-shinytest2-tm_a_regression.R:68:3,
test-shinytest2-tm_a_regression.R:88:3, test-shinytest2-tm_a_regression.R:102:3,
test-shinytest2-tm_a_regression.R:130:3, test-shinytest2-tm_a_regression.R:143:3,
test-shinytest2-tm_a_regression.R:172:3, test-shinytest2-tm_data_table.R:24:3,
test-shinytest2-tm_data_table.R:39:3, test-shinytest2-tm_data_table.R:49:3,
test-shinytest2-tm_data_table.R:86:3, test-shinytest2-tm_file_viewer.R:20:3,
test-shinytest2-tm_file_viewer.R:46:3, test-shinytest2-tm_file_viewer.R:63:3,
test-shinytest2-tm_file_viewer.R:86:3, test-shinytest2-tm_front_page.R:25:3,
test-shinytest2-tm_front_page.R:50:3, test-shinytest2-tm_front_page.R:63:3,
test-shinytest2-tm_g_association.R:48:3, test-shinytest2-tm_g_association.R:67:3,
test-shinytest2-tm_g_association.R:89:3, test-shinytest2-tm_g_association.R:99:3,
test-shinytest2-tm_g_bivariate.R:88:3, test-shinytest2-tm_g_bivariate.R:117:3,
test-shinytest2-tm_g_bivariate.R:159:3, test-shinytest2-tm_g_bivariate.R:196:3,
test-shinytest2-tm_g_distribution.R:48:3, test-shinytest2-tm_g_distribution.R:73:3,
test-shinytest2-tm_g_distribution.R:111:3, test-shinytest2-tm_g_response.R:43:3,
test-shinytest2-tm_g_response.R:62:3, test-shinytest2-tm_g_response.R:93:3,
test-shinytest2-tm_g_response.R:105:3, test-shinytest2-tm_g_scatterplot.R:87:3,
test-shinytest2-tm_g_scatterplot.R:114:3, test-shinytest2-tm_g_scatterplot.R:135:3,
test-shinytest2-tm_g_scatterplot.R:155:3, test-shinytest2-tm_g_scatterplot.R:168:3,
test-shinytest2-tm_g_scatterplotmatrix.R:48:3,
test-shinytest2-tm_g_scatterplotmatrix.R:66:3,
test-shinytest2-tm_g_scatterplotmatrix.R:79:3,
test-shinytest2-tm_g_scatterplotmatrix.R:107:3,
test-shinytest2-tm_misssing_data.R:39:3, test-shinytest2-tm_misssing_data.R:61:3,
test-shinytest2-tm_misssing_data.R:94:3, test-shinytest2-tm_misssing_data.R:132:3,
test-shinytest2-tm_misssing_data.R:163:3, test-shinytest2-tm_outliers.R:52:3,
test-shinytest2-tm_outliers.R:73:3, test-shinytest2-tm_outliers.R:110:3,
test-shinytest2-tm_outliers.R:128:3, test-shinytest2-tm_outliers.R:144:3,
test-shinytest2-tm_outliers.R:162:3, test-shinytest2-tm_outliers.R:179:3,
test-shinytest2-tm_outliers.R:195:3, test-shinytest2-tm_outliers.R:247:3,
test-shinytest2-tm_outliers.R:296:3, test-shinytest2-tm_t_crosstable.R:47:3,
test-shinytest2-tm_t_crosstable.R:65:3, test-shinytest2-tm_t_crosstable.R:76:3,
test-shinytest2-tm_t_crosstable.R:100:3, test-shinytest2-tm_variable_browser.R:30:3,
test-shinytest2-tm_variable_browser.R:71:3,
test-shinytest2-tm_variable_browser.R:104:3,
test-shinytest2-tm_variable_browser.R:159:3,
test-shinytest2-tm_variable_browser.R:175:3,
test-shinytest2-tm_variable_browser.R:203:3
[ FAIL 0 | WARN 0 | SKIP 115 | PASS 79 ]|
sorry, looks like my testing strategy skips shinytests. will try again |
|
There are |
Unit Test Performance Difference
Additional test case details
Results for commit 777f843 ♻️ This comment has been updated with latest results. |
Co-authored-by: Vedha Viyash <[email protected]> Signed-off-by: Dawid Kałędkowski <[email protected]>
Please check also: - insightsengineering/teal.modules.general#927 - Removed `$active_` methods in favour of one `$namespaces()` - `namespaces(FALSE)` (default) gives a list of `ns` functions, so that: - `namespaces(FALSE)$module("element-id-in-module")` will return an id of element - `namespaces(TRUE)$module("element-id-in-module")` will return an id of element prefixed by `#` - `namespaces()` returns following namespaces: `module`, `filter_panel`, `data_summary`, `wrapper` --------- Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Vedha Viyash <[email protected]>
Followup after insightsengineering/teal#1623
TealAppDrivernow acceptsteal_appinstead ofdata, modules, ...TealAppDriverhas methodnamespaces(is_selector = FALSE)in favour ofactive_module_id,active_module_ns, ... Teal app driver changes teal#1633options(TESTING_DEPTH=5)