Skip to content
Browse files

Seems to also allow logging from steps which run a block first, thoug…

…h the logging does come out of logical order (which JENKINS-38381 would address).
  • Loading branch information...
jglick committed Aug 3, 2017
1 parent cea93a6 commit f7777331bfda5ad7694451f748a3d9e80859c2c2
Showing with 21 additions and 17 deletions.
  1. +1 −1 pom.xml
  2. +20 −16 src/test/java/org/jenkinsci/plugins/workflow/support/actions/
@@ -77,7 +77,7 @@
<version>2.20-20170731.163505-3</version> <!-- TODO -->
<version>2.20-20170803.185040-4</version> <!-- TODO -->
@@ -24,18 +24,19 @@


import hudson.EnvVars;
import hudson.model.TaskListener;
import java.util.Collections;
import java.util.LinkedList;
import java.util.Set;
import org.jenkinsci.plugins.workflow.cps.CpsFlowDefinition;
import org.jenkinsci.plugins.workflow.job.WorkflowJob;
import org.jenkinsci.plugins.workflow.job.WorkflowRun;
import org.jenkinsci.plugins.workflow.steps.AbstractStepDescriptorImpl;
import org.jenkinsci.plugins.workflow.steps.AbstractStepExecutionImpl;
import org.jenkinsci.plugins.workflow.steps.AbstractStepImpl;
import org.jenkinsci.plugins.workflow.steps.BodyExecutionCallback;
import org.jenkinsci.plugins.workflow.steps.Step;
import org.jenkinsci.plugins.workflow.steps.StepContext;
import org.jenkinsci.plugins.workflow.steps.StepDescriptor;
import org.jenkinsci.plugins.workflow.steps.StepExecution;
import org.junit.ClassRule;
import org.junit.Test;
import org.junit.Rule;
@@ -58,27 +59,30 @@
r.assertLogContains("atom step in A with 1 commands to go", b);
r.assertLogContains("logging from LBBL with 0 commands to go", b);
r.assertLogContains("atom step in B with 1 commands to go", b);
/* TODO misuse of FlowNode.isRunning in WorkflowRun.copyLogs prevents this from appearing unless we access TaskListener from start(); should be fixed by JENKINS-38381, or by switching to isActive:
r.assertLogContains("logging from BL with 0 commands to go", b);
public static class ChattyStep extends AbstractStepImpl {
public static class ChattyStep extends Step {
public final String pattern;
@DataBoundConstructor public ChattyStep(String pattern) {this.pattern = pattern;}
@TestExtension("logsAndBlocks") public static class DescriptorImpl extends AbstractStepDescriptorImpl {
public DescriptorImpl() {
@Override public StepExecution start(StepContext context) throws Exception {
return new Execution(context, pattern);
@TestExtension("logsAndBlocks") public static class DescriptorImpl extends StepDescriptor {
@Override public String getFunctionName() {return "chatty";}
@Override public boolean takesImplicitBlockArgument() {return true;}
@Override public Set<? extends Class<?>> getRequiredContext() {
return Collections.singleton(TaskListener.class);
public static class Execution extends AbstractStepExecutionImpl {
@Inject(optional=true) transient ChattyStep step;
String pattern;
private static class Execution extends StepExecution {
Execution(StepContext context, String pattern) {
this.pattern = pattern;
final String pattern;
LinkedList<Boolean> commands; // L ~ false to log, B ~ true to run block
@Override public boolean start() throws Exception {
commands = new LinkedList<>();
pattern = step.pattern;
for (char c : pattern.toCharArray()) {
if (c == 'L') {
@@ -102,7 +106,7 @@ private void run() throws Exception {
@Override public void stop(Throwable cause) throws Exception {}
private final class Callback extends BodyExecutionCallback {
private final class Callback extends BodyExecutionCallback { // not using TailCall since run() sometimes calls onSuccess itself
@Override public void onSuccess(StepContext context, Object result) {
try {

0 comments on commit f777733

Please sign in to comment.
You can’t perform that action at this time.