insightsengineering / insightsengineering/teal

[Question]: Should decorators be also tested at the modules packages level?

Open
#1,454 1 comment 0 reactions 0 assignees View on GitHub
core question
Dominant language
R
Stars
263
Forks
59
Avg merge
5d 17h
Merged PRs (30d)
5

Description

### What is your question?

Currently there are 3 tests in `teal` and 2 shinytest2 tests in `teal` that cover decorators. Even though the main functionality is implemented in `teal`, actual usage and some customization is provided at modules packages. Should we extend modules packages tests (like in tmg/tmc) to cover decorators usage? Those module packages have some custom functions for decorators specification that do not exist in `teal`, hence the reason to test them there.

Some examples to be used for tmg:
- https://github.com/insightsengineering/teal.modules.general/pull/826/#issuecomment-2598165483
- https://github.com/insightsengineering/teal.modules.general/pull/826/#discussion_r1920380270

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.