remove warning on concatention in tm_g_pp_adverse_events

[m7pr]

Part of

• add verbose to c.teal_report and c.teal_card teal.reporter#421

Fixes

• [Bug]: Warning in tm_g_pp_adverse_events #1427

[github-actions]

Unit Tests Summary

  1 files   71 suites   11s ⏱️
734 tests 155 ✅ 579 💤 0 ❌
874 runs  176 ✅ 698 💤 0 ❌

Results for commit e96db94.

♻️ This comment has been updated with latest results.

[github-actions]

Unit Test Performance Difference

Test Suite	$Status$	Time on main	$±Time$	$±Tests$	$±Skipped$	$±Failures$	$±Errors$
examples	💚	$6.90$	$-5.24$	$0$	$+115$	$0$	$0$
tm_t_glm_counts	💚	$24.73$	$-24.52$	$-3$	$0$	$0$	$+2$

Additional test case details

Test Suite	$Status$	Time on main	$±Time$	Test Case
tm_t_glm_counts	💚	$12.24$	$-12.11$	e2e_tm_t_glm_counts_Module_initializes_in_teal_without_errors_and_produces_table_output.
tm_t_glm_counts	💚	$12.49$	$-12.41$	e2e_tm_t_glm_counts_Selecting_arm_var_changes_the_table_and_does_not_throw_validation_errors.

Results for commit 5af1202

♻️ This comment has been updated with latest results.

[llrs-roche]

Simple change. Looks good.

Locally some examples fail (6) but I don't expect this to be related to this change. There are also some other failures with checkmate::assert_class(app, "teal_app"): argument "app" is missing, with no default, as reported by the CI. I've seen that error message before but I am not sure what I did to fix it.

[m7pr]

@llrs-roche tests are failiing becuase we changed TealAppDriver in teal and in tmg, but not anywhere else

[m7pr]

Oh, but the PR for teal changes in TealAppDriver is not yet merged insightsengineering/teal#1633
So I think we need to wait for it