430 use expect_no_error in tests instead of expect_error(, NA)

[m7pr]

Part of https://github.com/insightsengineering/coredev-tasks/issues/430

[averissimo]

LGTM, I added some prefixes and bumped {testthat}

edit: Nevermind my comment. The original branch was outdated, once you update from main this problem is resolved

However, there's a weird issue with tests as there's a teal.transform:: missing that fails a test.

#> ── Failed tests ────────────────────────────────────────
#> Error (test-utils.R:270:3): cs_to_filter_spec creates filter_spec from choices_selected
#> Error in `value_choices(data, "a")`: could not find function "value_choices"

[m7pr]

Thanks for verification! Would you mind accepting?

[github-actions]

Unit Tests Summary

  1 files   33 suites   2s ⏱️
149 tests 149 ✅   0 💤 0 ❌
280 runs  168 ✅ 112 💤 0 ❌

Results for commit b97400b.

♻️ This comment has been updated with latest results.

[averissimo]

teal.data is already defined above. Please make that change before merging