Skip to content

Commit 8801091

Browse files
LorisFriedelclaude
andcommitted
refactor(forwarder): remove unnecessary backward-compatible Storage alias
The alias is no longer needed since all production code now uses the factory method and base class pattern. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent f545686 commit 8801091

File tree

2 files changed

+1
-8
lines changed

2 files changed

+1
-8
lines changed

aws/logs_monitoring/retry/storage.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,3 @@ def _serialize(self, data):
8686

8787
def _deserialize(self, data):
8888
return json.loads(data.decode("UTF-8"))
89-
90-
91-
# Backward-compatible alias
92-
Storage = S3Storage

aws/logs_monitoring/tests/test_s3_storage.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
from botocore.exceptions import ClientError
66

7-
from retry.storage import S3Storage, Storage
7+
from retry.storage import S3Storage
88

99

1010
class TestS3Storage(unittest.TestCase):
@@ -15,9 +15,6 @@ def setUp(self):
1515
with patch("retry.storage.DD_S3_BUCKET_NAME", "test-bucket"):
1616
self.storage = S3Storage("test_function_prefix")
1717

18-
def test_backward_compatible_alias(self):
19-
self.assertIs(Storage, S3Storage)
20-
2118
def test_store_data_puts_object(self):
2219
self.storage.store_data("logs", [{"message": "hello"}])
2320
self.mock_s3.put_object.assert_called_once()

0 commit comments

Comments
 (0)