Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

S3: Sending notification to EventBridge LIFECYCLE_TRANSITION_EVENT #7448

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
+6 −5
Draft

S3: Sending notification to EventBridge LIFECYCLE_TRANSITION_EVENT #7448

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
4919790
S3: S3: Sending notification to EventBridge `LIFECYCLE_TRANSITION_EVENT`
tsugumi-sys Mar 9, 2024
8cd8170
use setter for storage class
Mar 10, 2024
892ff9c
Merge branch 'master' into s3-send-eventbridge-notification-Lifecycle…
tsugumi-sys Mar 11, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
11 changes: 6 additions & 5 deletions moto/s3/models.py
View file
Open in desktop
Expand Up @@ -222,11 +222,6 @@
self._metadata = {} # type: ignore
self._metadata.update(metadata)

def set_storage_class(self, storage: Optional[str]) -> None:
if storage is not None and storage not in STORAGE_CLASS:
raise InvalidStorageClass(storage=storage)
self._storage_class = storage

def set_expiry(self, expiry: Optional[datetime.datetime]) -> None:
self._expiry = expiry

Expand Down Expand Up @@ -332,6 +327,12 @@
def storage_class(self) -> Optional[str]:
return self._storage_class

@storage_class.setter
def storage_class(self, storage: Optional[str]) -> None:
if storage is not None and storage not in STORAGE_CLASS:
raise InvalidStorageClass(storage=storage)
self._storage_class = storage

Check warning on line 334 in moto/s3/models.py

View check run for this annotation

Codecov / codecov/patch

moto/s3/models.py#L332-L334 

Added lines #L332 - L334 were not covered by tests

@property
def expiry_date(self) -> Optional[str]:
if self._expiry is not None:
Expand Down