Incrementally initializing the test apps so it becomes easier to review the tests.

[vedhav]

Part of #1108

[m7pr]

@vedhav we start the test description with a capital letter, and end with a dot

"e2e - tm_a_pca: Module is initialised with the specified defaults in function call."

[vedhav]

App drivers are only initialized for 9/30 modules.
Hopefully after the app drivers are created and merged to main, it will be easier to start working on these apps and focus more of the energy on test specs.

[m7pr]

@vedhav if there is one module only, do not use modules = teal::modules( in init_teal_app_driver but specify the module directly modules = tm_module_name

[m7pr]

Hey, when it comes for the motivation why this PR is not necessary - I just started with tm_g_km module, and needed to review the driver anyway and found out that we can test few more parameters that are missing because there are some default values. bc9165a

[m7pr]

I think you have tests/testthat/test-shinytest2-tm_g_forest_tte copy.R and tests/testthat/test-shinytest2-tm_g_forest_tte.R

[m7pr]

This is stupid but ui_g_adverse_events uses teal.widgets::plot_with_settings_ui(id = ns("chart"))

Suggested change

	app_driver$active_module_element("myplot-plot_main > img"),
	app_driver$active_module_element("chart-plot_main > img"),

Checkout the discussion about discrepancy in plot namespace name in here : ) insightsengineering/teal#1208 (comment)

[m7pr]

Same reasoning as here https://github.com/insightsengineering/teal.modules.clinical/pull/1132/files#r1576318046

IMHO it would be better if we can go module by module, so we can focus on one module and catch such things.

Suggested change

	app_driver$active_module_element("myplot-plot_main > img"),
	app_driver$active_module_element("patient_timeline_plot-plot_main > img"),

[m7pr]

Suggested change

	app_driver$active_module_element("myplot-plot_main > img"),
	app_driver$active_module_element("therapy_plot-plot_main > img"),

[m7pr]

Suggested change

	app_driver$active_module_element("myplot-plot_main > img"),
	app_driver$active_module_element("vitals_plot-plot_main > img"),

[vedhav]

Closing the PR as we have independent PRs raised