Skip to content

Commit e84fe39

Browse files
committed
add test for deleting blob with generation
1 parent 9cb9786 commit e84fe39

2 files changed

Lines changed: 14 additions & 2 deletions

File tree

storage/tests/unit/test_blob.py

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -543,6 +543,18 @@ def test_delete(self):
543543
self.assertFalse(blob.exists())
544544
self.assertEqual(bucket._deleted, [(BLOB_NAME, None)])
545545

546+
def test_delete_with_generation(self):
547+
BLOB_NAME = 'blob-name'
548+
not_found_response = ({'status': http_client.NOT_FOUND}, b'')
549+
connection = _Connection(not_found_response)
550+
client = _Client(connection)
551+
bucket = _Bucket(client)
552+
blob = self._make_one(BLOB_NAME, bucket=bucket)
553+
bucket._blobs[BLOB_NAME] = 1
554+
blob.delete(generation=blob.generation)
555+
self.assertFalse(blob.exists())
556+
self.assertEqual(bucket._deleted, [(BLOB_NAME, None)])
557+
546558
def test__get_transport(self):
547559
client = mock.Mock(spec=[u'_credentials', '_http'])
548560
client._http = mock.sentinel.transport
@@ -3027,7 +3039,7 @@ def __init__(self, client=None, name='name', user_project=None):
30273039
self.path = '/b/' + name
30283040
self.user_project = user_project
30293041

3030-
def delete_blob(self, blob_name, client=None):
3042+
def delete_blob(self, blob_name, client=None, generation=None):
30313043
del self._blobs[blob_name]
30323044
self._deleted.append((blob_name, client))
30333045

storage/tests/unit/test_bucket.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -874,7 +874,7 @@ def __init__(self, name, bucket_name):
874874
self.path = '/b/%s/o/%s' % (bucket_name, name)
875875
self._deleted = []
876876

877-
def delete(self, client=None):
877+
def delete(self, client=None, generation=None):
878878
self._deleted.append(client)
879879

880880
blob = _Blob(BLOB_NAME, BUCKET_NAME)

0 commit comments

Comments
 (0)