btrz-health-check - npm Package Compare versions

Comparing version 1.4.0 to 1.5.0

package.json

 {
   "name": "btrz-health-check",
-  "version": "1.4.0",
+  "version": "1.5.0",
   "description": "A series of classes that will ping different service types to verify accessibility.",
@@ -41,3 +41,7 @@ "main": "index.js",
     "publish": "^0.5.0"
+  },
+  "dependencies": {
+    "ec2-metadata": "^1.0.1",
+    "git-rev": "^0.2.1"
   }
 }

src/health-checkers.js

 "use strict";
 
+let EnvironmentInfo = require("./environment-info").EnvironmentInfo;
+
 function* getPromises(serviceCheckers) {
@@ -9,11 +11,24 @@ for (var i=0; i < serviceCheckers.length; i++) {
 
+function services(serviceCheckers) {
+  let promises = getPromises(serviceCheckers);
+  return Promise.all(promises)
+    .then(function (services) {
+      return {status: 200, services: services};
+    })
+    .catch(function (failingService) {
+      return {status: failingService.status, services: [failingService]};
+    });
+}
+
 class HealthCheckers {
+  constructor() {
+    this.env = new EnvironmentInfo();
+  }
   checkStatus(serviceCheckers) {
-    let promises = getPromises(serviceCheckers);
-    return Promise.all(promises)
-      .then(function (services) {
-        return {status: 200, services: services};
-      })
-      .catch(function (failingService) {
-        return {status: failingService.status, services: [failingService]};
+    let self = this;
+    return Promise.all([services(serviceCheckers), self.env.values()])
+      .then(function (results) {
+        results[0].commit = results[1].commit;
+        results[0].instanceId = results[1].instanceId;
+        return results[0];
       });
@@ -20,0 +35,0 @@ }

test/health-checkers_tests.js

@@ -7,2 +7,10 @@ "use strict";
 
+  it("should include the ec2 instance Id and the git revision", function (done) {
+    var checker = new HealthCheckers();
+    checker.checkStatus([]).then(function (result) {
+      expect(result.instanceId).to.not.be.undefined;
+      expect(result.commit).to.not.be.undefined;
+      done();
+    });
+  });
 
@@ -9,0 +17,0 @@ it("should return 200 if all services are ok", function (done) {

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

24610

increased by11.08%

Number of package files

20

increased by11.11%

Lines of code

534

increased by18.4%

Worsened metrics

Dependency count

2

increased byInfinity%

Dependency changes

• + Addedec2-metadata@^1.0.1

• + Addedgit-rev@^0.2.1

• + Addedcookiejar@1.3.0(transitive)

• + Addeddebug@0.7.4(transitive)

• + Addedec2-metadata@1.0.1(transitive)

• + Addedemitter-component@1.0.0(transitive)

• + Addedextend@1.2.1(transitive)

• + Addedformidable@1.0.14(transitive)

• + Addedgit-rev@0.2.1(transitive)

• + Addedmethods@0.0.1(transitive)

• + Addedmime@1.2.5(transitive)

• + Addedqs@0.6.5(transitive)

• + Addedreduce-component@1.0.1(transitive)

• + Addedsuperagent@0.17.0(transitive)