Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[FIXED JENKINS-28588] make the callable a static class.
Don't send over the CucumberTestResultArchiver just to find out what the
remote temporary directory is.
  • Loading branch information
jtnord committed May 27, 2015
1 parent 79c5fce commit b245954
Showing 1 changed file with 12 additions and 6 deletions.
Expand Up @@ -112,12 +112,7 @@ public boolean getIgnoreBadSteps(){
CucumberTestResult result = parser.parse(_testResults, build, launcher, listener);

// TODO - look at all of the Scenarios and see if there are any embedded items contained with in them
String remoteTempDir = launcher.getChannel().call(new Callable<String, InterruptedException >() {
@Override
public String call() throws InterruptedException {
return System.getProperty("java.io.tmpdir");
}
});
String remoteTempDir = launcher.getChannel().call(new TmpDirCallable());

// if so we need to copy them to the master.
for (FeatureResult f : result.getFeatures()) {
Expand Down Expand Up @@ -217,6 +212,17 @@ public DescriptorImpl getDescriptor() {
}


/**
* {@link Callable} that gets the temporary directory from the node.
*/
private final static class TmpDirCallable implements Callable<String, InterruptedException> {
@Override
public String call() throws InterruptedException {
return System.getProperty("java.io.tmpdir");
}
}



@Extension
public static class DescriptorImpl extends BuildStepDescriptor<Publisher> {
Expand Down

0 comments on commit b245954

Please sign in to comment.