Skip to content

Move more test data to test scope#283

Merged
pbchase merged 1 commit intoctsit:developfrom
pbchase:move_more_test_data
Sep 29, 2025
Merged

Move more test data to test scope#283
pbchase merged 1 commit intoctsit:developfrom
pbchase:move_more_test_data

Conversation

@pbchase
Copy link
Copy Markdown
Contributor

@pbchase pbchase commented Sep 29, 2025

Reconstruct make_test_data.R.
Convert rda to rds as we move them to test scope.
Delete 3 more man pages.

Reconstruct make_test_data.R.
Convert rda to rds as we move them to test scope.
Delete 3 more man pages.
@pbchase pbchase requested a review from Copilot September 29, 2025 20:48
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR refactors test data management by moving test datasets from R package data (.rda files) to test-scoped RDS files. The changes improve test isolation by making test data local to the test suite rather than part of the package's exported data objects.

Key changes:

  • Convert .rda test data files to .rds format and move to test scope
  • Update data loading from load() to readRDS() calls in test files
  • Remove exported data documentation from the package

Reviewed Changes

Copilot reviewed 7 out of 14 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/testthat/test-redcap_projects.R Updates test data loading to use readRDS() instead of load()
tests/testthat/redcap_projects/make_test_data.R Refactors test data generation script to use RDS format and test paths
tests/testthat/redcap_entity_project_ownership/make_test_data.R Updates dependency loading to use RDS format
man/projects_table_fragment.Rd Removes documentation for test data no longer exported
man/one_deleted_project_record.Rd Removes documentation for test data no longer exported
man/invoice_line_item_test_data.Rd Removes documentation for test data no longer exported
R/data.R Removes roxygen documentation for test datasets no longer part of package data

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@pbchase pbchase merged commit 3c9ef2f into ctsit:develop Sep 29, 2025
1 check passed
@pbchase pbchase deleted the move_more_test_data branch September 30, 2025 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants