Show spike data in error message (#117)

Show spike data in error message
Financial-Times · Jun 4, 2019 · 4e29ce7 · 4e29ce7
2 parents ce34059 + 669bf58
commit 4e29ce7
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/src/checks/graphiteSpike.check.js b/src/checks/graphiteSpike.check.js
@@ -85,8 +85,14 @@ class GraphiteSpikeCheck extends Check {
 				? data.sample / data.baseline < this.threshold
 				: data.sample / data.baseline > 1 / this.threshold;
 
-			this.status = ok ? status.PASSED : status.FAILED;
-			this.checkOutput = ok ? 'No spike detected in graphite data' : 'Spike detected in graphite data';
+			const details = `Direction: ${this.direction} Sample: ${data.sample} Baseline: ${data.baseline} Threshold: ${this.threshold}`
+			if (ok) {
+				this.status = status.PASSED;
+			 	this.checkOutput = `No spike detected in graphite data. ${details}`;
+			} else {
+				this.status = status.FAILED;
+				this.checkOutput = `Spike detected in graphite data. ${details}`;
+			}
 
 		} catch(err) {
 			logger.error({ event: `${logEventPrefix}_ERROR`, url: this.sampleUrl }, err);