search

ctsit / redcapcustodian

Simplified, automated data management on REDCap systems
Other
12 stars 6 forks source link

Add `get_redcap_emails()` #20

Closed pbchase closed 2 years ago

pbchase commented 2 years ago

Addresses Issue #8. Builds on PR #18

ChemiKyle commented 2 years ago

Testing tools throw warnings due to create_test_table calls in test-redcap and test-devtools

Warning (test-redcap.R:3:1): (code run outside of `test_that()`)
Closing open result set, pending rows

Backtrace shows the warning cascades from DBI::dbAppendTable. A bit of digging shows the solution may be to use DBI::dbClearResult, but it's not entirely clear what should be passed to this function (it's not the result of dbAppendTable); furthermore this process should be abstracted away by dbAppendTable as the dbClearResult step looks like it's used in more low level pipelines (i.e. using dbSendQuery and dbFetch).

~Other than this warning~ this is all fine. ~I'd like to get this to go away before I merge though.~ my ability to look up a few lines is in question though, since dbSendQuery was right before my eyes.