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.

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

Bug: test case failure test-derive_blfl.R #74

Closed
rammprasad opened this issue Jul 25, 2024 · 1 comment
Closed

Bug: test case failure test-derive_blfl.R #74

rammprasad opened this issue Jul 25, 2024 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@rammprasad
Copy link
Collaborator

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 variable DOMAIN is missing"
new: "Required variable DOMAIN is missing in sdtm_in"

  • Run testthat::snapshot_accept('derive_blfl') to accept the change.
  • Run testthat::snapshot_review('derive_blfl') to interactively review the change.

Session Information

No response

Reproducible Example

Run devtools::test()

@rammprasad rammprasad added the bug Something isn't working label Jul 25, 2024
@github-project-automation github-project-automation bot moved this to Product Backlog in sdtm.oak R package Jul 25, 2024
@rammprasad rammprasad linked a pull request Jul 25, 2024 that will close this issue
14 tasks
@rammprasad rammprasad self-assigned this Jul 25, 2024
@rammprasad
Copy link
Collaborator Author

@galachad - I think I am half way through fixing this test case. Removing the snapshot works as expected. I will finish it tomorrow.

@ramiromagno ramiromagno moved this from Product Backlog to Done in sdtm.oak R package Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

Successfully merging a pull request may close this issue.

2 participants