- 14 Oct, 2016 5 commits
-
-
Jérome Perrin authored
This is only meaningful if reference is protected. I don't know if this is really a use case ... testGUIwithSecurity does this, but this is not a realistic situation. Maybe it should test another permission.
-
Jérome Perrin authored
-
Jérome Perrin authored
-
Jérome Perrin authored
maybe a little to strict. I don't know how we can check that fields does not display value without checking the markup in a so low level way
-
Jérome Perrin authored
-
- 11 Oct, 2016 1 commit
-
-
Jérome Perrin authored
If we don't propagate movement_list_mode to catalog, this prevents catalog from raising an error about this unknown parameter.
-
- 07 Oct, 2016 2 commits
-
-
Ayush Tiwari authored
It's better to create new objects using same script rather than keeping 2 scripts that try to do the same thing: Folder_create (script used to add in case we create new object using the icon next to actions bar) did not support the getRedirectParameter API. We now do all the work related to adding new object in Folder_create itself. Test Result : https://nexedi.erp5.net/test_result_module/20161006-39C681E1 Reviewed by: @jm /reviewed-on nexedi/erp5!177
-
Jérome Perrin authored
This is for backward compatibility. Also add some explicit tests for check_consistency, it was missing in the first place.
-
- 06 Oct, 2016 2 commits
-
-
Cédric Le Ninivin authored
by adding timeout in order to prevent an error from changing page before finishing loading the iframed setting gadget
-
Jérome Perrin authored
When pusing to for_testrunner_1 branch, we often have this failure on testFormPrintoutAsODT: ``` ====================================================================== FAIL: test_field_replacement_with_variable (testFormPrintoutAsODT.TestFormPrintoutAsODT) test variables replacement in ODT documents. ---------------------------------------------------------------------- Traceback (most recent call last): File "/srv/slapgrid/slappart9/srv/testnode/zl/soft/4a13de8a0fdf926ddebab0387de0dc5e/parts/erp5/Products/ERP5OOo/tests/testFormPrintoutAsODT.py", line 1252, in test_field_replacement_with_variable self.assertEqual(node.text, value.strftime('%d/%m/%Y %H:%M:%S')) AssertionError: '06/12/2010 11:33:15' != '06/12/2010 11:24:15' ``` After some investigation, turned out that: * tests where always failing when running by `COMP-2544-3Nodes-ERP5PROJECT2` testnode * This machine uses `SystemV/CST6CDT` as timezone * the test fails in the same way when running locally with `TZ=SystemV/CST6CDT` I belive this fixes, I tried running under `SystemV/CST6CDT`, `UTC` and `Europe/Paris` TZ and it passes for all cases. /reviewed-on nexedi/erp5!174
-
- 05 Oct, 2016 19 commits
-
-
Julien Muchembled authored
-
Sven Franck authored
-
Sven Franck authored
-
Sven Franck authored
-
Hardik Juneja authored
/reviewed-on nexedi/erp5!175
-
Cédric Le Ninivin authored
Use of fail is now a coding crime
-
Jérome Perrin authored
Since ODF dates are "timezone naïve", just calculate the timestamp with the input date timezone as reference timezone. I don't think the "XXX Works only if the timezone is the same in OpenOffice" comment was true, so I just remove it.
-
Jérome Perrin authored
It's better to explicitly specify the expected value, this test should not depend on DateTime.strftime implementation, especially since this implementation does some timezone conversion depending on time.tzname
-
Jérome Perrin authored
-
Jérome Perrin authored
this way after a zelenium tests ERP5Site_setupDummyMailHost this will also reset the mails that could have been sent in previous tests.
-
Jérome Perrin authored
-
Jérome Perrin authored
This sends too many emails.
-
Jérome Perrin authored
Some parts were outdated
-
Jérome Perrin authored
The main purpose of these changes is to activate call to portal_sms.send; This way it is isolated it in a transaction that is less likely to fail. Activate with max_retry=0 and conflict_retry=False not to send activity twice in case of error. There are other cleanups, see individuals commits. /cc @seb @kazuhiko @gabriel /reviewed-on nexedi/erp5!49
-
Jérome Perrin authored
-
Jérome Perrin authored
-
Jérome Perrin authored
-
Jérome Perrin authored
-
Jérome Perrin authored
Since sending email will not work without destination email, we can globally set this constaint
-
- 04 Oct, 2016 4 commits
-
-
Julien Muchembled authored
-
Jérome Perrin authored
-
Jérome Perrin authored
This way it is isolated it in a transaction that is less likely to fail. Activate with max_retry=0 and conflict_retry=False not to send activity twice in case of error. As with email, if there was an error in activity processing, it is advised to investigate the transport logs to see if message was send or not.
-
Jérome Perrin authored
event_simulation_workflow already call `send` type based method upon start transitions. This workflow was not chained to portal_type, but if it would have been then message would be sent twice.
-
- 03 Oct, 2016 7 commits
-
-
Cédric Le Ninivin authored
It allows to not cancel the loading of the iframe and to prepare the appcache. The former behavior removed the iframe thus interrupting the preparation of the appcache.
-
Romain Courteaud authored
Putting field in center increases the string field size for ERP5JS
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Sebastien Robin authored
-
Jérome Perrin authored
re-apply 25cd1201 that was mistakenly reverted by b847b45e
-
Jérome Perrin authored
Hey @Nicolas can you please take a look at all this ? All this started when I realised we probably do not want to allow grouping reference to be set when we have transactions from different ledgers ( edda8b58 ) and then realized it would be required to allow choosing ledger in grouping fast input instead of just showing all ledgers together ( d1ecbe29 ). The rest are small changes to move out complex logic from TALES to a script and small fixups. Thanks /reviewed-on nexedi/erp5!170
-