fjtirado commented on code in PR #3891:
URL: 
https://github.com/apache/incubator-kie-kogito-runtimes/pull/3891#discussion_r2037991887


##########
jbpm/jbpm-flow/src/main/java/org/kie/kogito/process/impl/AbstractProcessInstance.java:
##########
@@ -248,6 +248,10 @@ public void 
internalRemoveProcessInstance(Consumer<AbstractProcessInstance<?>> r
         processInstance = null;
     }
 
+    public boolean hasHeader(String headerName) {

Review Comment:
   I added this one (in the implementation class, not the interface) because I 
want to unit test that the header is there once the ProcessInstance is 
persisted and read from DB, but I do not want to add a public API that retrieve 
the headers (because they hold sensitive information) so moving 
processInstance() from protected to public is not a good idea. 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to