fix(storage): set path via 𝘤𝘩𝘪𝘭𝘥 for 𝘥𝘦𝘭𝘦𝘵𝘦

1️⃣  𝘵𝘰𝘬𝘦𝘯 param is now optional
2️⃣  file path is now set via 𝙘𝙝𝙞𝙡𝙙 method, same way used for other methods, instead of 𝘯𝘢𝘮𝘦 param which is removed.

Author: AsifArmanRahman

firebase/storage/__init__.py

@@ -131,7 +131,7 @@ def put(self, file, token=None):
 
 			return request_object.json()
 
-	def delete(self, name, token):
+	def delete(self, token=None):
 		""" Delete file from storage.
 
 		| For more details:
@@ -145,17 +145,23 @@ def delete(self, name, token):
 			https://firebase.google.com/docs/storage/web/delete-files#delete_a_file
 
 
-		:type name: str
-		:param name: Cloud path to file.
-
 		:type token: str
-		:param token: Firebase Auth User ID Token
+		:param token: (Optional) Firebase Auth User ID Token, defaults 
+			to :data:`None`.
 		"""
 
+		# reset path
+		path = self.path
+		self.path = None
+
+		# remove leading backlash
+		if path.startswith('/'):
+			path = path[1:]
+
 		if self.credentials:
-			self.bucket.delete_blob(name)
+			self.bucket.delete_blob(path)
 		else:
-			request_ref = self.storage_bucket + "/o?name={0}".format(name)
+			request_ref = self.storage_bucket + "/o?name={0}".format(path)
 
 			if token:
 				headers = {"Authorization": "Firebase " + token}

tests/test_storage.py

@@ -23,7 +23,8 @@ def test_download(self, storage):
 		assert os.path.exists('tests/static/downloaded.txt')
 
 	def test_delete(self, storage):
-		assert storage.delete('firebase-test-001/uploaded-file.txt', None) is None
+		os.remove('tests/static/downloaded.txt')
+		assert storage.child('firebase-test-001/uploaded-file.txt').delete() is None
 
 	def test_list_of_files(self, storage):
 		assert storage.list_files()