Closed (fixed)
Project:
Rules
Version:
8.x-3.x-dev
Component:
Rules Core
Priority:
Minor
Category:
Bug report
Assigned:
Reporter:
Created:
18 Nov 2019 at 08:02 UTC
Updated:
2 Dec 2019 at 18:44 UTC
Jump to comment: Most recent
The changes in #2572119: Update actions & conditions to use doExecute / doEvaluate were never applied to actions defined by test modules.
This was noted in #2572119-8: Update actions & conditions to use doExecute / doEvaluate at the time, but dismissed by #2572119-11: Update actions & conditions to use doExecute / doEvaluate. I do not agree with the objections in that latter post. In order for tests to be effective, we must program test actions the same way we would program real actions, otherwise we don't end up testing the way real code works.
This patch applies the changes to the three actions defined in our test modules.
| Comment | File | Size | Author |
|---|---|---|---|
| do-execute-in-test-actions.patch | 2.47 KB | tr |
Comments
Comment #3
tr commentedCommitted.