Merge pull request #193 from Financial-Times/comply-standards

FTDCS-196 - Comply with the standards
Financial-Times · Jun 30, 2022 · f761c46 · f761c46
2 parents dbcb4ee + e047251
commit f761c46
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/README.md b/README.md
@@ -1,6 +1,7 @@
 # n-health [![CircleCI](https://circleci.com/gh/Financial-Times/n-health.svg?style=svg)](https://circleci.com/gh/Financial-Times/n-health)
 
-Collection of healthcheck classes to use in your nodejs application
+Collection of healthcheck classes to use in your nodejs application.   
+To create a new health check, please follow the [current standard](https://docs.google.com/document/edit?id=1ftlkDj1SUXvKvKJGvoMoF1GnSUInCNPnNGomqTpJaFk)
 
 ## Usage
 

diff --git a/src/checks/fastlyKeyExpiration.check.js b/src/checks/fastlyKeyExpiration.check.js
@@ -52,7 +52,7 @@ class FastlyKeyExpirationCheck extends Check {
 			},
 			PASSED: {
 				status: status.PASSED,
-				checkOutput: 'Fastly key expiration date is ok',
+				checkOutput: `Fastly key ${this.name} is configured correctly`,
 				severity: this.severity
 			}
 		});

diff --git a/test/fastlyKeyExpiration.check.spec.js b/test/fastlyKeyExpiration.check.spec.js
@@ -49,7 +49,7 @@ describe('Fastly Key Expiration Check', () => {
 			},
 			PASSED: {
 				status: status.PASSED,
-				checkOutput: 'Fastly key expiration date is ok',
+				checkOutput: `Fastly key ${defaultOptions.name} is configured correctly`,
 				severity: 2
 			}
 		};