Merge lp:~rvb/maas/empty-files-1.3 into lp:maas/1.3

[Raphaël Badin]

Backport revision 1540: accept empty files.

[Raphaël Badin (rvb)]

review: Approve

=== modified file 'src/maasserver/models/filestorage.py'
--- src/maasserver/models/filestorage.py	2013-02-22 12:16:44 +0000
+++ src/maasserver/models/filestorage.py	2013-07-26 09:39:19 +0000
@@ -83,7 +83,7 @@
         unique_together = ('filename', 'owner')
 
     filename = CharField(max_length=255, unique=False, editable=False)
-    content = BinaryField(null=False)
+    content = BinaryField(null=False, blank=True)
     # owner can be None: this is to support upgrading existing
     # installations where the files were not linked to users yet.
     owner = ForeignKey(

=== modified file 'src/maasserver/tests/test_api.py'
--- src/maasserver/tests/test_api.py	2013-03-12 15:25:43 +0000
+++ src/maasserver/tests/test_api.py	2013-07-26 09:39:19 +0000
@@ -2719,6 +2719,17 @@
         self.assertIn('text/plain', response['Content-Type'])
         self.assertEqual("Filename not supplied", response.content)
 
+    def test_add_empty_file(self):
+        filename = "filename"
+        response = self.make_API_POST_request(
+            "add", filename=filename,
+            fileObj=factory.make_file_upload(content=b''))
+        self.assertEqual(httplib.CREATED, response.status_code)
+        self.assertItemsEqual(
+            [filename],
+            FileStorage.objects.filter(
+                filename=filename).values_list('filename', flat=True))
+
     def test_add_file_fails_with_no_file_attached(self):
         response = self.make_API_POST_request("add", "foo")