Skip to content

test: cover hot/cold check --read-data routing - #1868

Draft
magrathean-uk wants to merge 1 commit into
rustic-rs:mainfrom
magrathean-uk:split/issue-1811-hot-cold-read-data
Draft

test: cover hot/cold check --read-data routing#1868
magrathean-uk wants to merge 1 commit into
rustic-rs:mainfrom
magrathean-uk:split/issue-1811-hot-cold-read-data

Conversation

@magrathean-uk

@magrathean-uk magrathean-uk commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds an integration regression test for check --read-data with hot/cold repositories.

Status

Test-only draft. The test currently fails against the locked rustic_core, proving the bug remains there. It is blocked on rustic-rs/rustic_core#542, which is still open. This PR does not claim to fix runtime behavior.

Related to #1811.

@aawsome

aawsome commented Aug 9, 2026

Copy link
Copy Markdown
Member

Actually #1811 is a bug in rustic_core and should be fixed there.

@aawsome aawsome closed this Aug 9, 2026
@aawsome

aawsome commented Aug 9, 2026

Copy link
Copy Markdown
Member

I think it might be good to have the unit test here, after rustic-rs/rustic_core#542 is used.

@magrathean-uk Can you extract this so we only add the unit test with this PR?

@aawsome aawsome reopened this Aug 9, 2026
@magrathean-uk
magrathean-uk force-pushed the split/issue-1811-hot-cold-read-data branch from 2c3eb11 to 2e268ad Compare August 10, 2026 15:27
@magrathean-uk

Copy link
Copy Markdown
Contributor Author

Makes sense! I've updated the PR to remove the workaround in check.rs so this PR now strictly adds the integration test (tests/check_hot_cold.rs) for hot/cold --read-data. Once rustic_core#542 is pulled in, this test will verify the fix.

@magrathean-uk magrathean-uk changed the title fix: check cold data for hot and cold repositories test: cover hot/cold check --read-data routing Aug 11, 2026
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