Changeset 12444 for main/waeup.ikoba/trunk/src/waeup/ikoba/documents/tests

Timestamp:

11 Jan 2015, 22:43:35 (10 years ago)

Author:

Henrik Bettermann

Message:

We need to provide unique filenames. Let's use the document id for the filename and not the download_name of the respective viewlet.

File:

• main/waeup.ikoba/trunk/src/waeup/ikoba/documents/tests/test_browser.py (modified) (4 diffs)

main/waeup.ikoba/trunk/src/waeup/ikoba/documents/tests/test_browser.py

@@ -148 +148 @@
             name='upload_pdfscanmanageupload').click()
         self.assertTrue(
-            'href="http://localhost/app/documents/DOC1/sample.pdf">PDF File</a>'
+            'href="http://localhost/app/documents/DOC1/file.pdf">DOC1.pdf</a>'
             in self.browser.contents)
         # The file can be found in the file system
         file = getUtility(IExtFileStore).getFileByContext(
-            document, attr='sample.pdf')
+            document, attr='file.pdf')
         file_content = file.read()
         pdf.seek(0)
@@ -159 +159 @@
         # Browsing the link shows a real pdf only if the document
         # has been published
-        self.browser.getLink("PDF File").click()
+        self.browser.getLink("DOC1.pdf").click()
         self.assertTrue(
             'The document requested has not yet been published'
             in self.browser.contents)
         IWorkflowState(document).setState(PUBLISHED)
-        self.browser.open(self.container_path + '/DOC1/sample.pdf')
+        self.browser.open(self.container_path + '/DOC1/file.pdf')
         self.assertEqual(
             self.browser.headers['content-type'], 'application/pdf')
+        # The name of the downloaded file will be different
+        self.assertEqual(
+            self.browser.headers['Content-Disposition'],
+            'attachment; filename="DOC1.pdf')
 
         # Transitions can be performed
@@ -186 +190 @@
         # File has been removed too
         file = getUtility(IExtFileStore).getFileByContext(
-            document, attr='sample.pdf')
+            document, attr='file.pdf')
         self.assertTrue(file is None)
 
@@ -204 +208 @@
             % document.document_id in logcontent)
         self.assertTrue(
-            'INFO - zope.mgr - documents.browser.DocumentManageFormPage - %s - uploaded: sample.pdf (my_sample_scan.pdf)'
+            'INFO - zope.mgr - documents.browser.DocumentManageFormPage - %s - uploaded: file.pdf (my_sample_scan.pdf)'
             % document.document_id in logcontent)
         self.assertTrue(