-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[JBPM-9900] RequestInfo.owner is null even if org.kie.executor.id is defined #2026
Conversation
1f1b1d3
to
2057d8e
Compare
Jenkins retest this please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, way to go @elguardian !
Some minor comments and a couple of suggestions for additional tests (regressions)
jbpm-services/jbpm-executor/src/test/java/org/jbpm/executor/ExtendedExecutorTest.java
Outdated
Show resolved
Hide resolved
jbpm-services/jbpm-executor/src/test/java/org/jbpm/executor/ExtendedExecutorTest.java
Outdated
Show resolved
Hide resolved
jbpm-services/jbpm-executor/src/test/java/org/jbpm/executor/ExtendedExecutorTest.java
Outdated
Show resolved
Hide resolved
jbpm-services/jbpm-executor/src/test/java/org/jbpm/executor/ExtendedExecutorTest.java
Outdated
Show resolved
Hide resolved
jbpm-services/jbpm-executor/src/test/java/org/jbpm/executor/ExtendedExecutorTest.java
Show resolved
Hide resolved
jenkins retest this |
1 similar comment
jenkins retest this |
2057d8e
to
49af0ae
Compare
Jenkins retest this please |
1 similar comment
Jenkins retest this please |
@gmunozfe I had to change one thing to make the test work in this environment. |
Jenkins retest this please |
Kudos, SonarCloud Quality Gate passed! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, well done @elguardian
Jira: https://issues.redhat.com/browse/JBPM-9900
depends on kiegroup/droolsjbpm-knowledge#550