[FIX] document_ftp: tests: smash those tests with lots of commits.
authorVo Minh Thu <vmt@openerp.com>
Fri, 7 Oct 2011 09:17:37 +0000 (11:17 +0200)
committerVo Minh Thu <vmt@openerp.com>
Fri, 7 Oct 2011 09:17:37 +0000 (11:17 +0200)
(Otherwise changes happening on one cursor are not visible to the other one.)
Normally, as those yaml files commit(), they sould be inside demo data
instead of tests. But really we want to test a completely initialized
database, not being executed while the database is being initialized
(as the demo data are). This is just a matter of convention as the
tests are only executed when you also install demo data.

bzr revid: vmt@openerp.com-20111007091737-2dzocv2rgm2gfbi2

addons/document_ftp/test/document_ftp_test2.yml
addons/document_ftp/test/document_ftp_test4.yml

index 872456e..33ce141 100644 (file)
         fdata = StringIO('abcd')
         ftp.storbinary('STOR test2.txt', fdata)
         ftp.close()
+        cr.commit()
 -
     I look for the "test2.txt" file at the server
 -
     !python {model: ir.attachment }: |
         ids = self.search(cr, uid, [('name', '=', 'test2.txt')])
-        assert ids
+        assert ids, "No test2.txt file found."
 -
     I delete the "test2.txt" file using FTP.
 -
         ftp = te.get_ftp_folder(cr, uid, self, 'Documents')
         ftp.delete('test2.txt')
         ftp.close()
+        cr.commit()
 -
     I check at the server that test2.txt is deleted
 -
     !python {model: ir.attachment }: |
         ids = self.search(cr, uid, [('name', '=', 'test2.txt')])
-        assert not ids
+        assert not ids, "test2.txt file can still be found."
 -
     I create a test2.txt file again.
 -
         fdata = StringIO('abcd')
         ftp.storbinary('STOR test2.txt', fdata)
         ftp.close()
+        cr.commit()
 -
     I delete the test2.txt from the server (RPC).
 -
index f05fac3..0a3ff33 100644 (file)
@@ -4,6 +4,7 @@
     !python {model: ir.attachment}: |
         from document_ftp import test_easyftp as te
         ftp = te.get_plain_ftp(timeout=1.0)
+        cr.commit()
 - |
     I create two partners 'Partner1' and 'Partner2'.
     I create three partner categories: 'none', 'pat1' and 'all'
@@ -53,6 +54,7 @@
         dirs = ftp.nlst()
         for dir in [ 'All Partner1+2', 'No partners', 'Pat 1' ]:
             assert dir in dirs, "Dir %s not in folder" % dir
+        cr.commit()
 -
     I create a 'partners' folder by the first resource one.
 -
@@ -79,6 +81,7 @@
         for dir in correct:
             res = ftp.nlst(dir+'/Partners of Test')
             assert res == correct[dir], "Dir %s falsely contains %s" %(dir, res)
+        cr.commit()
 -
     I create an ir.attachment, attached (not related) to Partner1
 -