- 17 Jan, 2022 1 commit
-
-
Jérome Perrin authored
This is a step to be able to run ERP5 on zope4, where servers are no longer started by ZConfig. This will be used by nexedi/slapos!1114 See merge request nexedi/erp5!1529
-
- 14 Jan, 2022 1 commit
-
-
Yusei Tahara authored
-
- 13 Jan, 2022 9 commits
-
-
Jérome Perrin authored
no need to create temp objects because anyway the transaction is aborted at the end of each test. This solve a problem that some APIs behave differently on temp objects and we don't really have use cases where we use actions on temp objects. For reference, the case which fail is an action with a TALES condition using a related accessor, the exact condition was: python: object.getAggregateRelated(portal_type="Implicit Item Movement") is not None
-
Jérome Perrin authored
This was not patching the right place (I should have used mock ...)
-
Jérome Perrin authored
See merge request nexedi/erp5!1531
-
Jérome Perrin authored
-
Jérome Perrin authored
The portal types using this form are all PresencePeriod or subclasses, so use PresencePeriod as a prefix.
-
Jérome Perrin authored
TaskMovement_getSimulationStateColorText was same content and calendar depends on project
-
Jérome Perrin authored
Rename Orgnaisation_generatePlanningDomain to Organisation_generatePlanningDomain
-
Jérome Perrin authored
also rewrite a sorted to use key instead of cmp
-
Jérome Perrin authored
by calling proxy_field.checkConsistency(fixit=True) on them
-
- 12 Jan, 2022 2 commits
-
-
Vincent Pelletier authored
This tic call has a lot of activities to process. Checking test result history over the last 2 months, the whole test runtime is mostly under 800s, but with a handful of spikes around 1600s, with one at 2300s. So pick just slightly more than the worst over 2 months.
-
Vincent Pelletier authored
-
- 11 Jan, 2022 1 commit
-
-
Jérome Perrin authored
-
- 07 Jan, 2022 9 commits
-
-
Xiaowu Zhang authored
-
Vincent Pelletier authored
Also revert related fixup commits: "CMFActivity: fixup do not loop on tic if the node is the distribution node" "Products.ERP5Type.tests: Follow-up on ActivityTool.tic signature change." While the original commit did improve the specific workload it was designed to improve, it turned out to degrade too much intensive activity workloads, like initial ERP5 site creation and tests (which, for the purposes of this change, are the same as a single-zope instance). Given how easy it is to get a multi-Zope instance, which would solve the original issue and also provide the better performance necessary anyway for an instance managing a non-trivial amount of documents, I choose to revert this change. I am not reverting several loosely-related changes I applied, which rather fix real bugs uncovered by the different activity execution scheme this change provided, especially by letting tests' "stop_condition" callback being executed a lot more often between activities, uncovering missing dependencies and unrealistic test expectations, whose fixes should be beneficial independently from the reverted code. This reverts commit 4dfafbc9. This reverts commit 4eb26017. This reverts commit 041642d0.
-
Vincent Pelletier authored
Products.ERP5Security.test.testERP5Security: Fix TestMigration.test_DuplicateUserIdPreventionDuringMigration scope.
-
Vincent Pelletier authored
-
Vincent Pelletier authored
For realism: the goal of this exception is to prevent duplicate, and it relies on the exception triggering a rollback in the transaction manager (like any transaction error). Also, use "any()" rather than building entire lists just to check for non- emptiness.
-
Vincent Pelletier authored
Person and ERP5 Login documents must be indexed before ERP5Site_disableERP5UserManager may run, so set the migration tag on these indexation activities.
-
Vincent Pelletier authored
Several tests are failing on test nodes with the chosen 100s timeout. Increase it to 10 minutes. Longer, more exceptional, calls may benefit from exposing this delay as a tic argument.
-
Vincent Pelletier authored
Consistently with the Selenium timeout increase.
-
Vincent Pelletier authored
Products.ERP5Type.tests.ProcessingNodeTestCase: Make tic independent from the number of activities processed per tic. That number was vastly decreased, causing tests to bail with "tic is looping forever" for no good reason. Convert such logic into a deadline one.
-
- 06 Jan, 2022 5 commits
-
-
Jérome Perrin authored
When running with ZServer, TimerServer will be started by ZConfig machinery, because ZServer comes with some ZConfig data types for server factory which start servers. One Zope 2, ZServer was imported from zopeschema: https://github.com/zopefoundation/Zope/blob/2.13.30/src/Zope2/Startup/zopeschema.xml#L10 and ZConfig used the TimerServerFactory to create a TimerService. On Zope 4, ZServer is not imported anymore in the new wsgischema: https://github.com/zopefoundation/Zope/blob/4.6.3/src/Zope2/Startup/wsgischema.xml so the ZConfig data type can not be used to create the server. Instead of relying on ZServer's ZConfig, we create the server directly from the startup script, the logic to attach to waitress is already here and startup and factory just called create, so the only difference is that we no longer use zope.conf to configure TimerService, but configure from the script. SlapOS profile also needs to be modified in order to not include ZConfig bits for timerserver and to pass the timerserver interval in the wrapper to this script.
-
Jérome Perrin authored
Since https://github.com/zopefoundation/Zope/commit/2da2c2c28a8b21e78e8ef93aa17da926e3102c26 SERVER_PORT is assumed to be convertible to int. The value of this port is not really supposed to be used because TimerServer requests does not have a server (for example in CMFActivity we recreate the initial request), so put a value of -1 to prevent accidental use.
-
Jérome Perrin authored
Only catch exceptions, we don't need a bare except here. In case of exception, log the traceback instead of being silent.
-
Vincent Pelletier authored
Ownership and Owner role are distinct: catalog's "viewable_owner" is role-based, but "owner" is ownership based. As permissions are applied by catalog in addition to any given parameter, this should not change the result of this query. And as any owner which does not have the Owner role would cause this script to fail on the assertion, this should not change the result of this script in any way besides avoiding AssertionErrors.
-
Vincent Pelletier authored
Also, reindex modified documents, as ownership and Owner role are present in catalog.
-
- 04 Jan, 2022 1 commit
-
-
Romain Courteaud authored
-
- 03 Jan, 2022 4 commits
-
-
Romain Courteaud authored
-
Xiaowu Zhang authored
-
Xiaowu Zhang authored
-
Xiaowu Zhang authored
-
- 31 Dec, 2021 4 commits
-
-
Roque authored
See merge request nexedi/erp5!1528
-
Roque authored
-
Roque authored
- image editor - web table editor
-
Roque authored
- text editor - slide editor - svg editor - javascript editor - pdf viewer
-
- 28 Dec, 2021 3 commits
-
-
Vincent Pelletier authored
erp5_ui_test_core: Make waitForActivities independent from the number of activities processed per tic. That number was vastly decreased, causing UI tests to bail with "tic is looping forever" for no good reason. Convert such logic into a deadline one.
-
Jérome Perrin authored
This is intended to replace portal_workflow.doActionFor with edit_action, which was only available for users with modify portal content permission on the document and also is a bit low level. It is a common, especially in project specific code that we use edit_action to leave notes in history and provide better traceability and we sometimes do this on documents where user does not have the modify portal content permission.
-
Gabriel Monnerat authored
We don't need Permissions.ModifyPortalContent, Permissions.AccessContentsInformation is enough. This is consistent with the security declarations in Base and Folder. https://lab.nexedi.com/nexedi/erp5/blob/08be2f8cc936fbdc3d5bafb4355c3a475198abc1/product/ERP5Type/Base.py#L2632 https://lab.nexedi.com/nexedi/erp5/blob/08be2f8cc936fbdc3d5bafb4355c3a475198abc1/product/ERP5Type/Core/Folder.py#L1434
-