Fix unit test

mekomsolutions · Oct 31, 2024 · b0afd6e · b0afd6e
1 parent b2f458b
commit b0afd6e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/lib/container-to-consul.js b/lib/container-to-consul.js
@@ -44,7 +44,7 @@ class ContainertoConsul {
       // deregister containers that do not run
       .then(() => this.consul.kv.keys('docker/service-ids/')
         .catch(error => {
-          if (error.response.status && error.response.status === 404) {
+          if (error.response && error.response.status && error.response.status === 404) {
             return [];
           }
           throw error;
@@ -74,6 +74,7 @@ class ContainertoConsul {
       .then(containers => this.registerContainers(containers))
       .catch(error => {
         this.log.error(error);
+        throw error;
       });
   }
 

diff --git a/test/lib/container-to-consul.test.js b/test/lib/container-to-consul.test.js
@@ -236,7 +236,7 @@ describe('container-to-consul', () => {
 
     it('should not fail if no docker/service-ids key is stored in consul', () => {
       var error = new Error('test');
-      error.statusCode = 404;
+      error.response = {status: 404};
 
       containertoConsul.consul.kv.keys.rejects(error);