Fixed #26495 -- Added name arg to Storage.save()'s File wrapping.

1 files changed, 23 insertions, 0 deletions

diff --git a/tests/file_storage/tests.py b/tests/file_storage/tests.py
index d1ea0021c5..42699dddb8 100644
--- a/tests/file_storage/tests.py
+++ b/tests/file_storage/tests.py
@@ -644,6 +644,29 @@ class CustomStorageLegacyDatetimeHandlingTests(FileStorageTests):
     storage_class = CustomStorageLegacyDatetimeHandling
 
 
+class DiscardingFalseContentStorage(FileSystemStorage):
+    def _save(self, name, content):
+        if content:
+            return super(DiscardingFalseContentStorage, self)._save(name, content)
+        return ''
+
+
+class DiscardingFalseContentStorageTests(FileStorageTests):
+    storage_class = DiscardingFalseContentStorage
+
+    def test_custom_storage_discarding_empty_content(self):
+        """
+        When Storage.save() wraps a file-like object in File, it should include
+        the name argument so that bool(file) evaluates to True (#26495).
+        """
+        output = six.StringIO('content')
+        self.storage.save('tests/stringio', output)
+        self.assertTrue(self.storage.exists('tests/stringio'))
+
+        with self.storage.open('tests/stringio') as f:
+            self.assertEqual(f.read(), b'content')
+
+
 class FileFieldStorageTests(TestCase):
     def tearDown(self):
         shutil.rmtree(temp_storage_location)