• Type: Bug Report
• Resolution: Unresolved
• Priority: L3 - Default
• Fix Version/s: None
• Affects Version/s: 7.17.6
• Component/s: assert
• Labels:

  None

This is the controller panel for Smart Panels app

[CAM-14929] org.camunda.bpm / complete do not allow to pass local variables

dakol created issue - 07/Oct/22 11:52 AM

dakol made changes - 07/Oct/22 12:12 PM

Description

Original: h3. Environment (Required on creation): h3. Description (Required on creation; please attach any relevant screenshots, stacktraces, log files, etc. to the ticket): when testing BPMN which expected ExternalTask to send local variables, there is no _complet_ method allowing to pass {_}local variable{_}the complet() function do not allow   h3. Steps to reproduce (Required on creation): h3. Observed Behavior (Required on creation): h3. Expected behavior (Required on creation): h3. Root Cause (Required on prioritization): h3. Solution Ideas (Optional): update test-utils/assert/core/src/main/java/org/camunda/bpm/engine/test/assertions/bpmn/BpmnAwareTests.java   along with: {code:java} public static void complete(ExternalTask externalTask, Map<String, Object> variables)    ...    complete(lockedTasks.get(0), variables); {code} add: {code:java} public static void complete(ExternalTask externalTask, Map<String, Object> variables), Map<String, Object> localVariables){     ...     complete(lockedTasks.get(0), variables, localVariables);   {code} And the same for {code:java} public static void complete(LockedExternalTask lockedExternalTask, Map<String, Object> variables, {code}     Hints (optional):

New: h3. Environment (Required on creation): h3. Description (Required on creation; please attach any relevant screenshots, stacktraces, log files, etc. to the ticket): when testing BPMN which expected ExternalTask to send local variables, there is no _complete_ method allowing to pass {_}local variable{_}the complete() function do not allow   h3. Steps to reproduce (Required on creation): h3. Observed Behavior (Required on creation): h3. Expected behavior (Required on creation): h3. Root Cause (Required on prioritization): h3. Solution Ideas (Optional): update test-utils/assert/core/src/main/java/org/camunda/bpm/engine/test/assertions/bpmn/BpmnAwareTests.java   along with: {code:java} public static void complete(ExternalTask externalTask, Map<String, Object> variables)    ...    complete(lockedTasks.get(0), variables); {code} add: {code:java} public static void complete(ExternalTask externalTask, Map<String, Object> variables), Map<String, Object> localVariables){     ...     complete(lockedTasks.get(0), variables, localVariables);   {code} And the same for {code:java} public static void complete(LockedExternalTask lockedExternalTask, Map<String, Object> variables, {code}     Hints (optional):

Tassilo Weidner made changes - 07/Oct/22 5:01 PM

Assignee

New: Tassilo Weidner [ tassilo.weidner ]

Collapse comment: Tassilo Weidner added a comment - 07/Oct/22 5:02 PM

Tassilo Weidner added a comment - 07/Oct/22 5:02 PM

Hi dakol,

Thank you for your contribution.

I review contributions every fortnight.

Stay tuned!

Best,
Tassilo

Expand comment: Tassilo Weidner added a comment - 07/Oct/22 5:02 PM

Tassilo Weidner added a comment - 07/Oct/22 5:02 PM Hi dakol , Thank you for your contribution. I review contributions every fortnight. Stay tuned! Best, Tassilo

Collapse comment: dakol added a comment - 10/Oct/22 9:41 AM

dakol added a comment - 10/Oct/22 9:41 AM

https://github.com/camunda/camunda-bpm-platform/pull/2034

Expand comment: dakol added a comment - 10/Oct/22 9:41 AM

dakol added a comment - 10/Oct/22 9:41 AM https://github.com/camunda/camunda-bpm-platform/pull/2034

Collapse comment: Thorben Lindhauer added a comment - 12/Oct/22 12:11 PM

Thorben Lindhauer added a comment - 12/Oct/22 12:11 PM

This ticket was migrated to github: https://github.com/camunda/camunda-bpm-platform/issues/2815. Please use this link for any future references and continue any discussion there.

Expand comment: Thorben Lindhauer added a comment - 12/Oct/22 12:11 PM

Thorben Lindhauer added a comment - 12/Oct/22 12:11 PM This ticket was migrated to github: https://github.com/camunda/camunda-bpm-platform/issues/2815 . Please use this link for any future references and continue any discussion there.