Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add unit test for get_imputation_target_time() #2409

Closed
Tracked by #2369
zdz2101 opened this issue Apr 16, 2024 · 0 comments · Fixed by #2439
Closed
Tracked by #2369

Add unit test for get_imputation_target_time() #2409

zdz2101 opened this issue Apr 16, 2024 · 0 comments · Fixed by #2439
Assignees
@millerg23
Labels
general issue

Comments

@zdz2101
Copy link
Collaborator

zdz2101 commented Apr 16, 2024 •
edited

The function get_imputation_target_time() which is used in the underpinnings of derive_vars_dtm() needs unit tests

Definition of done:
Add appropriate unit tests in the test-derive_dt_dtm_utils file

can check out #2386 and respective PR for inspiration of what needs to be done
@zdz2101 zdz2101 mentioned this issue Apr 16, 2024
Open
16 tasks
@millerg23 millerg23 linked a pull request May 21, 2024 that will close this issue
Closes #2409 unit test created for get_imputation_target_time #2439
Merged
15 tasks
bms63 added a commit that referenced this issue May 23, 2024
@millerg23 @bms63
Closes #2409 unit test created for get_imputation_target_time (#2439)
be7f93e 
* 2409 unit test created for get_imputation_target_time

* 2409 fix STYLER

* Update tests/testthat/test-derive_vars_dt_dtm_utils.R

---------

Co-authored-by: Ben Straub <ben.x.straub@gsk.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@millerg23 millerg23

Labels
general issue
Projects
admiral (sdtm/adam, dev, ci, template, core)
Status: Archive
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Closes #2409 unit test created for get_imputation_target_time pharmaverse/admiral
2 participants
@zdz2101 @millerg23