From 7dd872fdebc9d5bd98829b1a4ff826a1339004d8 Mon Sep 17 00:00:00 2001 From: "Kevin W. Beam" Date: Wed, 27 Nov 2024 14:35:16 -0700 Subject: [PATCH] Applies ruff (black) code formatting rules --- tests/test_aws.py | 60 +++++++++++++++++++++++++++++++---------------- 1 file changed, 40 insertions(+), 20 deletions(-) diff --git a/tests/test_aws.py b/tests/test_aws.py index 5f5dae6..0c668e4 100644 --- a/tests/test_aws.py +++ b/tests/test_aws.py @@ -17,6 +17,7 @@ # correct parameters, correctly handle their return values, and handle any # exceptions they may throw. + @pytest.fixture(scope="module") def aws_credentials(): """Mocked AWS Credentials for moto.""" @@ -26,26 +27,27 @@ def aws_credentials(): os.environ["AWS_SESSION_TOKEN"] = "testing" os.environ["AWS_DEFAULT_REGION"] = "us-west-2" + @pytest.fixture def kinesis(aws_credentials): """A mocked Kinesis client.""" with mock_aws(): yield boto3.client("kinesis", region_name="us-west-2") + @pytest.fixture def kinesis_stream_summary(kinesis): """Create a Kinesis stream and return its summary info.""" kinesis.create_stream(StreamName="duck-test-stream", ShardCount=1) summary = kinesis.describe_stream_summary(StreamName="duck-test-stream") - return summary['StreamDescriptionSummary'] + return summary["StreamDescriptionSummary"] + @pytest.fixture def test_message(): """Returns a JSON string for testing.""" - return json.dumps({ - 'foo': 333, - 'bar': 'xyzzy' - }) + return json.dumps({"foo": 333, "bar": "xyzzy"}) + @pytest.fixture def s3(aws_credentials): @@ -53,18 +55,18 @@ def s3(aws_credentials): with mock_aws(): yield boto3.client("s3") + @pytest.fixture def s3_bucket(s3): """Create an S3 buket and return the bucket name.""" bucket_name = "duck-test-bucket" s3.create_bucket( - Bucket=bucket_name, - CreateBucketConfiguration={ - 'LocationConstraint': 'us-west-2' - }, + Bucket=bucket_name, + CreateBucketConfiguration={"LocationConstraint": "us-west-2"}, ) return bucket_name + @pytest.fixture def science_data(): return """ @@ -73,89 +75,107 @@ def science_data(): bar """ + def test_kinesis_stream_exists_for_valid_name(kinesis_stream_summary): stream_name = "duck-test-stream" assert aws.kinesis_stream_exists(stream_name) + def test_kinesis_stream_exists_for_invalid_name(kinesis_stream_summary): stream_name = "xyzzy" assert not aws.kinesis_stream_exists(stream_name) + def test_post_to_kinesis(kinesis_stream_summary, test_message): """Given a Kinesis stream name and a message, it should post successfully.""" - stream_name = kinesis_stream_summary['StreamName'] + stream_name = kinesis_stream_summary["StreamName"] success = aws.post_to_kinesis(stream_name, test_message) assert type(success) is str + def test_post_to_kinesis_returns_shard_id(kinesis_stream_summary, test_message): """Given a Kinesis stream name and a test message, the response should include the shard id.""" - stream_name = kinesis_stream_summary['StreamName'] + stream_name = kinesis_stream_summary["StreamName"] result = aws.post_to_kinesis(stream_name, test_message) assert "shardId" in result + def test_post_to_kinesis_with_invalid_stream_name(kinesis_stream_summary, test_message): """Given an invalid Kinesis stream name and a message, it should raise an exception.""" invalid_stream_name = "abcd-1234-wxyz-0987" with pytest.raises(Exception): aws.post_to_kinesis(invalid_stream_name, test_message) + def test_post_to_kinesis_with_empty_message(kinesis_stream_summary): """Given a Kinesis stream name, it should raise an exception when posting an empty message.""" - stream_name = kinesis_stream_summary['StreamName'] + stream_name = kinesis_stream_summary["StreamName"] with pytest.raises(Exception): aws.post_to_kinesis(stream_name, None) + def test_stage_data_to_s3(s3, s3_bucket, science_data): object_name = "/external/NSIDC-TEST666/3/abcd-1234-wxyz-0987/science-data.bin" aws.stage_file(s3_bucket, object_name, data=science_data) s3_object = s3.get_object( - Bucket=s3_bucket, - Key=object_name, + Bucket=s3_bucket, + Key=object_name, ) - object_lines = [line.decode(encoding="utf-8") for line in s3_object['Body'].readlines()] + object_lines = [ + line.decode(encoding="utf-8") for line in s3_object["Body"].readlines() + ] object_data = "".join(object_lines) assert object_data == science_data + def test_stage_data_to_s3_with_invalid_bucket_name(s3_bucket, science_data): bucket_name = "xyzzy" object_name = "/external/NSIDC-TEST666/3/abcd-1234-wxyz-0987/science-data.bin" with pytest.raises(Exception): aws.stage_file(bucket_name, object_name, data=science_data) + def test_stage_data_to_s3_with_missing_object_name(s3, s3_bucket, science_data): with pytest.raises(Exception): aws.stage_file(s3_bucket, None, data=science_data) + def test_stage_data_to_s3_with_no_data(s3, s3_bucket): object_name = "/external/NSIDC-TEST666/3/abcd-1234-wxyz-0987/science-data.bin" with pytest.raises(Exception): aws.stage_file(s3_bucket, object_name, data=None) + def test_stage_file_to_s3(s3, s3_bucket, science_data): with TemporaryFile() as source_file: - source_file.write(science_data.encode('UTF-8')) + source_file.write(science_data.encode("UTF-8")) source_file.seek(0) object_name = "/external/NSIDC-TEST666/3/abcd-1234-wxyz-0987/science-data.bin" aws.stage_file(s3_bucket, object_name, file=source_file) s3_object = s3.get_object( - Bucket=s3_bucket, - Key=object_name, + Bucket=s3_bucket, + Key=object_name, ) - object_lines = [line.decode(encoding="utf-8") for line in s3_object['Body'].readlines()] + object_lines = [ + line.decode(encoding="utf-8") for line in s3_object["Body"].readlines() + ] object_data = "".join(object_lines) assert object_data == science_data + def test_stage_file_requires_data_or_file(s3_bucket): with pytest.raises(Exception): - aws.stage_file(s3_bucket, 'foo') + aws.stage_file(s3_bucket, "foo") + def test_staging_bucket_exists_for_valid_name(s3_bucket): bucket_name = "duck-test-bucket" assert aws.staging_bucket_exists(bucket_name) + def test_staging_bucket_exists_for_invalid_name(s3_bucket): bucket_name = "xyzzy" assert not aws.staging_bucket_exists(bucket_name)