Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[FIXED JENKINS-29153] Update to tap4j 4.1.1 and introduce new method …
…to ignore skipped tests (instead of considering them as failures)
  • Loading branch information
kinow committed Oct 1, 2015
1 parent 763784d commit 88720b6
Show file tree
Hide file tree
Showing 2 changed files with 61 additions and 46 deletions.
2 changes: 1 addition & 1 deletion pom.xml
Expand Up @@ -92,7 +92,7 @@
<dependency>
<groupId>org.tap4j</groupId>
<artifactId>tap4j</artifactId>
<version>4.0.8</version>
<version>4.1.1</version>
</dependency>
<dependency>
<groupId>org.jenkins-ci.plugins</groupId>
Expand Down
105 changes: 60 additions & 45 deletions src/main/java/org/tap4j/plugin/TapParser.java
Expand Up @@ -23,9 +23,6 @@
*/
package org.tap4j.plugin;

import hudson.FilePath;
import hudson.model.AbstractBuild;

import java.io.File;
import java.io.IOException;
import java.io.PrintStream;
Expand All @@ -34,11 +31,17 @@
import java.util.logging.Logger;

import org.apache.commons.io.FileUtils;
import org.tap4j.model.TestResult;
import org.tap4j.model.TestSet;
import org.tap4j.parser.ParserException;
import org.tap4j.parser.Tap13Parser;
import org.tap4j.plugin.model.ParseErrorTestSetMap;
import org.tap4j.plugin.model.TestSetMap;
import org.tap4j.util.DirectiveValues;
import org.tap4j.util.StatusValues;

import hudson.FilePath;
import hudson.model.AbstractBuild;

/**
* Executes remote TAP Stream retrieval and execution.
Expand All @@ -60,64 +63,74 @@ public class TapParser {
private final Boolean validateNumberOfTests;
private final Boolean planRequired;
private final Boolean verbose;

private boolean hasFailedTests;
private boolean parserErrors;

public TapParser(Boolean outputTapToConsole, Boolean enableSubtests, Boolean todoIsFailure,
Boolean includeCommentDiagnostics, Boolean validateNumberOfTests, Boolean planRequired,
Boolean verbose, PrintStream logger) {
this.outputTapToConsole = outputTapToConsole;
this.enableSubtests = enableSubtests;
this.todoIsFailure = todoIsFailure;
this.parserErrors = false;
this.includeCommentDiagnostics = includeCommentDiagnostics;
this.validateNumberOfTests = validateNumberOfTests;
this.planRequired = planRequired;
this.verbose = verbose;
this.logger = logger;
}

public Boolean hasParserErrors() {
Boolean includeCommentDiagnostics, Boolean validateNumberOfTests, Boolean planRequired, Boolean verbose,
PrintStream logger) {
this.outputTapToConsole = outputTapToConsole;
this.enableSubtests = enableSubtests;
this.todoIsFailure = todoIsFailure;
this.parserErrors = false;
this.includeCommentDiagnostics = includeCommentDiagnostics;
this.validateNumberOfTests = validateNumberOfTests;
this.planRequired = planRequired;
this.verbose = verbose;
this.logger = logger;
}

public Boolean hasParserErrors() {
return this.parserErrors;
}

public Boolean getOutputTapToConsole() {
return outputTapToConsole;
}
return outputTapToConsole;
}

public Boolean getTodoIsFailure() {
return todoIsFailure;
}
public Boolean getTodoIsFailure() {
return todoIsFailure;
}

public boolean getParserErrors() {
return parserErrors;
}
public boolean getParserErrors() {
return parserErrors;
}

public Boolean getIncludeCommentDiagnostics() {
return includeCommentDiagnostics;
}
public Boolean getIncludeCommentDiagnostics() {
return includeCommentDiagnostics;
}

public Boolean getValidateNumberOfTests() {
return validateNumberOfTests;
}
public Boolean getValidateNumberOfTests() {
return validateNumberOfTests;
}

public Boolean getPlanRequired() {
return planRequired;
}
public Boolean getPlanRequired() {
return planRequired;
}

public Boolean getEnableSubtests() {
return enableSubtests;
}
public Boolean getEnableSubtests() {
return enableSubtests;
}

public boolean hasFailedTests() {
public boolean hasFailedTests() {
return this.hasFailedTests;
}
public Boolean getVerbose() {

public Boolean getVerbose() {
return verbose;
}

private boolean containsNotOk(TestSet testSet) {
for (TestResult testResult : testSet.getTestResults()) {
if (testResult.getStatus().equals(StatusValues.NOT_OK) && !(testResult.getDirective() != null
&& DirectiveValues.SKIP == testResult.getDirective().getDirectiveValue())) {
return true;
}
}
return false;
}

public TapResult parse(FilePath[] results, AbstractBuild<?, ?> build) {
this.parserErrors = Boolean.FALSE;
this.hasFailedTests = Boolean.FALSE;
Expand All @@ -139,7 +152,7 @@ public TapResult parse(FilePath[] results, AbstractBuild<?, ?> build) {
final Tap13Parser parser = new Tap13Parser("UTF-8", enableSubtests, planRequired);
final TestSet testSet = parser.parseFile(tapFile);

if (testSet.containsNotOk() || testSet.containsBailOut()) {
if (containsNotOk(testSet) || testSet.containsBailOut()) {
this.hasFailedTests = Boolean.TRUE;
}

Expand All @@ -162,8 +175,10 @@ public TapResult parse(FilePath[] results, AbstractBuild<?, ?> build) {
}
}
}
//final TapResult testResult = new TapResult(UUID.randomUUID().toString(), build, testSets);
final TapResult testResult = new TapResult("TAP Test Results", build, testSets, this.todoIsFailure, this.includeCommentDiagnostics, this.validateNumberOfTests);
// final TapResult testResult = new
// TapResult(UUID.randomUUID().toString(), build, testSets);
final TapResult testResult = new TapResult("TAP Test Results", build, testSets, this.todoIsFailure,
this.includeCommentDiagnostics, this.validateNumberOfTests);
return testResult;
}

Expand Down

0 comments on commit 88720b6

Please sign in to comment.