removed code that filters out results with no stats

jenkinsci · Dec 12, 2018 · 8b57ed1 · 8b57ed1
1 parent ae4f65f
commit 8b57ed1
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 5 deletions.
diff --git a/.gitignore b/.gitignore
@@ -4,4 +4,5 @@
 .settings
 .classpath
 .project
-*.iml
+*.iml
+/bin/
diff --git a/src/main/java/com/overops/plugins/jenkins/query/OverOpsBuildAction.java b/src/main/java/com/overops/plugins/jenkins/query/OverOpsBuildAction.java
@@ -122,8 +122,8 @@ public String getSummary() {
 	}
 
 	private static void appendSummaryValue(StringBuilder builder, String name, int value, boolean appendPostfix) {
-
-		if (value > 0) {
+//		Eric commented out 
+//		if (value > 0) {
 			if (builder.length() > 0) {
 				builder.append(", ");
 			}
@@ -140,7 +140,7 @@ private static void appendSummaryValue(StringBuilder builder, String name, int v
 			if (value > 1) {
 				builder.append("s");
 			}
-		}
+//		}
 	}
 
 	public List<OOReportRegressedEvent> getRegressedEvents() {

diff --git a/src/main/java/com/overops/plugins/jenkins/query/RegressionReportBuilder.java b/src/main/java/com/overops/plugins/jenkins/query/RegressionReportBuilder.java
@@ -299,7 +299,9 @@ private static ReportVolume getReportVolume(ApiClient apiClient,
 			if (event.stats != null) {
 				result.volume += event.stats.hits;
 
-				if ((event.stats .hits > 0) && (result.topEvents.size() < limit)) {
+				//commented out for now until stats collection works every time
+//				if ((event.stats .hits > 0) && (result.topEvents.size() < limit)) {
+				if (result.topEvents.size() < limit) {
 					String arcLink = getArcLink(apiClient, event.id, input, rateRegression);
 					result.topEvents.add(new OOReportEvent(event, arcLink));
 				}