We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
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
✖ | 8 7 | derive_blfl [2.6s] ───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────── Failure (test-derive_blfl.R:47:3): derive_blfl sdmt_in validations work Snapshot of derive_blfl(...) has changed: old: "Required variable DOMAIN is missing" new: "Required variable DOMAIN is missing in sdtm_in"
derive_blfl(...)
old
DOMAIN
new
sdtm_in
testthat::snapshot_accept('derive_blfl')
testthat::snapshot_review('derive_blfl')
No response
Run devtools::test()
The text was updated successfully, but these errors were encountered:
@galachad - I think I am half way through fixing this test case. Removing the snapshot works as expected. I will finish it tomorrow.
Sorry, something went wrong.
rammprasad
Successfully merging a pull request may close this issue.
What happened?
✖ | 8 7 | derive_blfl [2.6s]
─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
Failure (test-derive_blfl.R:47:3): derive_blfl sdmt_in validations work
Snapshot of
derive_blfl(...)
has changed:old
: "Required variableDOMAIN
is missing"new
: "Required variableDOMAIN
is missing insdtm_in
"testthat::snapshot_accept('derive_blfl')
to accept the change.testthat::snapshot_review('derive_blfl')
to interactively review the change.Session Information
No response
Reproducible Example
Run devtools::test()
The text was updated successfully, but these errors were encountered: