fix: SeaDexBackup.create() now returns BackupFile

Author: Ravencentric

src/seadex/_backup.py

@@ -152,7 +152,7 @@ def download(self, file: StrPath | BackupFile | None = None, *, destination: Str
 
         return outfile
 
-    def create(self, filename: StrPath | None = None) -> str:
+    def create(self, filename: StrPath | None = None) -> BackupFile:
         """
         Creates a new backup with an optional filename.
 
@@ -165,8 +165,8 @@ def create(self, filename: StrPath | None = None) -> str:
 
         Returns
         -------
-        str
-            The name of the newly created backup file.
+        BackupFile
+            The newly created backup file.
         """
         datefmt = "%Y%m%d%H%M%S"
         name = f"{datefmt}-seadex-backup.zip"
@@ -179,7 +179,7 @@ def create(self, filename: StrPath | None = None) -> str:
 
         self.client.backups.create(_filename)
 
-        return _filename
+        return next(filter(lambda member: member.name == _filename, self.backups))
 
     def delete(self, file: StrPath | BackupFile) -> None:
         """

tests/test_backup.py

@@ -85,6 +85,6 @@ def test_backup_download_with_pathlike(tmp_path: Path) -> None:
 def test_backup_create() -> None:
     client = SeaDexBackup(email, password)  # type: ignore
     new_backup = client.create(Path(f"{uuid4()}-made-by-pytest.zip"))
-    assert new_backup in [backup.name for backup in client.backups]
+    assert new_backup in [backup for backup in client.backups]
     client.delete(new_backup)
-    assert new_backup not in [backup.name for backup in client.backups]
+    assert new_backup not in [backup for backup in client.backups]