Skip to content
Permalink
Browse files
[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 35fab78 commit f076b53967609a5a8e500b776c83c76aa6679f53
Showing 1 changed file with 12 additions and 6 deletions.
@@ -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()) {
@@ -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> {

0 comments on commit f076b53

Please sign in to comment.