From: Dominik Csapak <d.csapak@proxmox.com>
To: pbs-devel@lists.proxmox.com
Subject: [pbs-devel] [PATCH proxmox-backup v3 2/5] datastore: test DataBlob encode/decode roundtrip
Date: Mon, 5 Aug 2024 11:24:11 +0200 [thread overview]
Message-ID: <20240805092414.1178930-3-d.csapak@proxmox.com> (raw)
In-Reply-To: <20240805092414.1178930-1-d.csapak@proxmox.com>
so that we can be sure we can decode an encoded blob again
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
changes from v2:
* separate into individual tests
pbs-datastore/src/data_blob.rs | 79 ++++++++++++++++++++++++++++++++++
1 file changed, 79 insertions(+)
diff --git a/pbs-datastore/src/data_blob.rs b/pbs-datastore/src/data_blob.rs
index a7a55fb7..a3a41c5e 100644
--- a/pbs-datastore/src/data_blob.rs
+++ b/pbs-datastore/src/data_blob.rs
@@ -562,3 +562,82 @@ impl<'a, 'b> DataChunkBuilder<'a, 'b> {
chunk_builder.build()
}
}
+
+#[cfg(test)]
+mod test {
+ use pbs_tools::crypt_config::CryptConfig;
+
+ use super::DataChunkBuilder;
+
+ const TEST_DATA_LEN: usize = 50;
+
+ fn build_test_data() -> Vec<u8> {
+ let mut data = Vec::with_capacity(TEST_DATA_LEN);
+ for i in 0..TEST_DATA_LEN / 10 {
+ for _ in 0..10 {
+ data.push(i as u8);
+ }
+ }
+ data
+ }
+
+ #[test]
+ fn unencrypted_uncompressed() {
+ let data = build_test_data();
+ let (chunk, digest) = DataChunkBuilder::new(&data)
+ .compress(false)
+ .build()
+ .expect("could not create unencrypted, uncompressed chunk");
+
+ let data_decoded = chunk
+ .decode(None, Some(&digest))
+ .expect("cannot decode unencrypted, uncompressed chunk");
+ assert_eq!(data, data_decoded);
+ }
+
+ #[test]
+ fn unencrypted_compressed() {
+ let data = build_test_data();
+ let (chunk, digest) = DataChunkBuilder::new(&data)
+ .compress(true)
+ .build()
+ .expect("could not create unencrypted, compressed chunk");
+
+ let data_decoded = chunk
+ .decode(None, Some(&digest))
+ .expect("cannot decode unencrypted, compressed chunk");
+ assert_eq!(data, data_decoded);
+ }
+
+ #[test]
+ fn encrypted_uncompressed() {
+ let data = build_test_data();
+ let crypt_config = CryptConfig::new([9; 32]).expect("could not create crypt config");
+ let (chunk, digest) = DataChunkBuilder::new(&data)
+ .compress(false)
+ .crypt_config(&crypt_config)
+ .build()
+ .expect("could not create encrypted, uncompressed chunk");
+
+ let data_decoded = chunk
+ .decode(Some(&crypt_config), Some(&digest))
+ .expect("cannot decode encrypted, uncompressed chunk");
+ assert_eq!(data, data_decoded);
+ }
+
+ #[test]
+ fn encrypted_compressed() {
+ let data = build_test_data();
+ let crypt_config = CryptConfig::new([9; 32]).expect("could not create crypt config");
+ let (chunk, digest) = DataChunkBuilder::new(&data)
+ .compress(true)
+ .crypt_config(&crypt_config)
+ .build()
+ .expect("could not create encrypted, compressed chunk");
+
+ let data_decoded = chunk
+ .decode(Some(&crypt_config), Some(&digest))
+ .expect("cannot decode encrypted, compressed chunk");
+ assert_eq!(data, data_decoded);
+ }
+}
--
2.39.2
_______________________________________________
pbs-devel mailing list
pbs-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel
next prev parent reply other threads:[~2024-08-05 9:24 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-08-05 9:24 [pbs-devel] [PATCH proxmox-backup v3 0/5] improve compression throughput Dominik Csapak
2024-08-05 9:24 ` [pbs-devel] [PATCH proxmox-backup v3 1/5] remove data blob writer Dominik Csapak
2024-08-05 9:24 ` Dominik Csapak [this message]
2024-08-05 9:24 ` [pbs-devel] [PATCH proxmox-backup v3 3/5] datastore: data blob: add helper and test for checking zstd_safe error code Dominik Csapak
2024-08-05 9:24 ` [pbs-devel] [PATCH proxmox-backup v3 4/5] datastore: data blob: increase compression throughput Dominik Csapak
2024-08-05 9:32 ` Dominik Csapak
2024-08-05 9:24 ` [pbs-devel] [PATCH proxmox-backup v3 5/5] datastore: DataBlob encode: simplify code Dominik Csapak
2024-08-07 17:06 ` [pbs-devel] applied-series: [PATCH proxmox-backup v3 0/5] improve compression throughput Thomas Lamprecht
2024-08-08 6:53 ` Dominik Csapak
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240805092414.1178930-3-d.csapak@proxmox.com \
--to=d.csapak@proxmox.com \
--cc=pbs-devel@lists.proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.