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

read/write repository regression tests #481

Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open

Conversation

ryanSoley
Copy link
Member

What

  • adds regression tests to validate all interactions between the repository layer and the filesystem itself
    • test_read_regression tests that the repository accurately retrieves data from the filesystem
    • test_write_regression tests that the repository accurately writes data to the filesystem
    • test_delete_regression tests that the repository accurately removes data from the filesystem
    • test_read_write_regression tests that the repository accurately retrieves data written by the repository itself

How to Test

  • python -m pytest tests/regression

@ryanSoley ryanSoley requested review from a team as code owners August 29, 2024 18:03
@ryanSoley ryanSoley self-assigned this Aug 29, 2024
@ryanSoley ryanSoley added the enhancement Issues & PRs that capture enhancements to existing features. Issues require associated PRs. label Aug 29, 2024
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we split these tests into either parameterized versions or just smaller ones that test one (or a few) things at a time?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Issues & PRs that capture enhancements to existing features. Issues require associated PRs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants