Page MenuHomePhabricator

[DRAFT] [services] Tests - Backup Utils - Add BackupData struct
AbandonedPublic

Authored by karol on Jun 7 2022, 6:48 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Oct 26, 2:58 AM
Unknown Object (File)
Oct 13 2024, 7:14 PM
Unknown Object (File)
Oct 13 2024, 7:13 PM
Unknown Object (File)
Oct 13 2024, 7:08 PM
Unknown Object (File)
Aug 27 2024, 4:16 PM
Unknown Object (File)
Aug 7 2024, 5:09 AM
Unknown Object (File)
Aug 6 2024, 1:25 AM
Unknown Object (File)
Aug 5 2024, 11:54 PM

Details

Summary

Depends on D4164

I decided to wrap backup information in the struct. It contains user/backup/device ids and backup/logs sizes.

Test Plan

this doesn't do anything on its own. Tests can be built.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

karol edited the test plan for this revision. (Show Details)
karol added reviewers: tomek, max, varun.
karol retitled this revision from [services] Tests - Backup Utils - Add BackupData struct to [DRAFT] [services] Tests - Backup Utils - Add BackupData struct.Jun 8 2022, 12:54 AM
tomek requested changes to this revision.Jun 8 2022, 2:05 AM
tomek added inline comments.
services/commtest/src/backup_utils.rs
11–12

Maybe we should create a separate type for a chunk, so that it is ensured that for every chunk there's a log vector? Currently it is possible for backup_chunk_sizes and logs_sizes to have different sizes.

This revision now requires changes to proceed.Jun 8 2022, 2:05 AM
services/commtest/src/backup_utils.rs
11–12

The new version of tests will have this (if I understand you correctly).