[Erp5-report] r28977 - /erp5/trunk/products/ERP5OOo/tests/testDms.py

nobody at svn.erp5.org nobody at svn.erp5.org
Fri Sep 11 16:29:54 CEST 2009


Author: nicolas
Date: Fri Sep 11 16:29:53 2009
New Revision: 28977

URL: http://svn.erp5.org?rev=28977&view=rev
Log:
Here we use edit instead of setFile, because only edit method set filename as source_reference.
This is a feature related to portal_contribution, sometimes filename is replaced by value provided by
rewriteIngestionData type based method.
setFile should keep value provided by portal_contribution instead of reading it from file itself.
See special parameter set_filename__ in EPR5.Tool.ContributionTool


Modified:
    erp5/trunk/products/ERP5OOo/tests/testDms.py

Modified: erp5/trunk/products/ERP5OOo/tests/testDms.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5OOo/tests/testDms.py?rev=28977&r1=28976&r2=28977&view=diff
==============================================================================
--- erp5/trunk/products/ERP5OOo/tests/testDms.py [utf8] (original)
+++ erp5/trunk/products/ERP5OOo/tests/testDms.py [utf8] Fri Sep 11 16:29:53 2009
@@ -790,8 +790,16 @@
 
   def test_PDF_content_content_type(self):
     upload_file = makeFileUpload('REF-en-001.pdf')
-    document = self.portal.document_module.newContent(portal_type='PDF',
-                                                      file=upload_file)
+    document = self.portal.document_module.newContent(portal_type='PDF')
+    # Here we use edit instead of setFile,
+    # because only edit method set filename as source_reference.
+    # This is a feature related to portal_contribution,
+    # sometimes filename is replaced by value provided by
+    # rewriteIngestionData type based method.
+    # setFile should keep value provided by portal_contribution
+    # instead of reading it from file itself.
+    # See special parameter set_filename__ in EPR5.Tool.ContributionTool
+    document.edit(file=upload_file)
     self.assertEquals('application/pdf', document.getContentType())
 
 class TestDocumentWithSecurity(ERP5TypeTestCase):




More information about the Erp5-report mailing list