From d7decda3f4423602e2a8064fa398181736b3a3ab Mon Sep 17 00:00:00 2001 From: Jill Date: Fri, 4 Feb 2022 17:55:54 +0100 Subject: garage(tests): Add random suffix to created buckets. --- src/garage/tests/common/mod.rs | 14 +++++++++++--- src/garage/tests/common/util.rs | 13 +++++++++++++ src/garage/tests/simple.rs | 12 ++++++------ 3 files changed, 30 insertions(+), 9 deletions(-) create mode 100644 src/garage/tests/common/util.rs (limited to 'src/garage/tests') diff --git a/src/garage/tests/common/mod.rs b/src/garage/tests/common/mod.rs index b5f7f0b2..c40fde0f 100644 --- a/src/garage/tests/common/mod.rs +++ b/src/garage/tests/common/mod.rs @@ -7,6 +7,7 @@ pub mod macros; pub mod client; pub mod ext; pub mod garage; +pub mod util; const REGION: Region = Region::from_static("garage-integ-test"); @@ -23,20 +24,27 @@ impl Context { Context { garage, client } } - pub fn create_bucket(&self, name: &str) { + /// Create an unique bucket with a random suffix. + /// + /// Return the created bucket full name. + pub fn create_bucket(&self, name: &str) -> String { + let bucket_name = format!("{}-{}", name, util::random_id(6)); + self.garage .command() - .args(["bucket", "create", name]) + .args(["bucket", "create", &bucket_name]) .quiet() .expect_success_status("Could not create bucket"); self.garage .command() .args(["bucket", "allow"]) .args(["--owner", "--read", "--write"]) - .arg(name) + .arg(&bucket_name) .args(["--key", &self.garage.key.name]) .quiet() .expect_success_status("Could not allow key for bucket"); + + bucket_name } } diff --git a/src/garage/tests/common/util.rs b/src/garage/tests/common/util.rs new file mode 100644 index 00000000..49c72879 --- /dev/null +++ b/src/garage/tests/common/util.rs @@ -0,0 +1,13 @@ +pub fn random_id(len: usize) -> String { + use rand::distributions::Slice; + use rand::Rng; + + static ALPHABET: &[u8] = b"abcdefghijklmnopqrstuvwxyz0123456789."; + + let rng = rand::thread_rng(); + rng.sample_iter(Slice::new(ALPHABET).unwrap()) + .map(|b| char::from(*b)) + .filter(|c| c.is_ascii_lowercase() || c.is_ascii_digit()) + .take(len) + .collect() +} diff --git a/src/garage/tests/simple.rs b/src/garage/tests/simple.rs index a627c770..bf88d8c2 100644 --- a/src/garage/tests/simple.rs +++ b/src/garage/tests/simple.rs @@ -5,13 +5,13 @@ async fn test_simple() { use aws_sdk_s3::ByteStream; let ctx = common::context(); - ctx.create_bucket("test-simple"); + let bucket = ctx.create_bucket("test-simple"); let data = ByteStream::from_static(b"Hello world!"); ctx.client .put_object() - .bucket("test-simple") + .bucket(&bucket) .key("test") .body(data) .send() @@ -21,7 +21,7 @@ async fn test_simple() { let res = ctx .client .get_object() - .bucket("test-simple") + .bucket(&bucket) .key("test") .send() .await @@ -35,13 +35,13 @@ async fn test_simple_2() { use aws_sdk_s3::ByteStream; let ctx = common::context(); - ctx.create_bucket("test-simple-2"); + let bucket = ctx.create_bucket("test-simple-2"); let data = ByteStream::from_static(b"Hello world!"); ctx.client .put_object() - .bucket("test-simple-2") + .bucket(&bucket) .key("test") .body(data) .send() @@ -51,7 +51,7 @@ async fn test_simple_2() { let res = ctx .client .get_object() - .bucket("test-simple-2") + .bucket(&bucket) .key("test") .send() .await -- cgit v1.2.3