Remove DEBUG check to fallback to console.error

uphold · Jan 2, 2025 · 2d7f1c3 · 2d7f1c3
1 parent 0dd3e29
commit 2d7f1c3
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 27 deletions.
diff --git a/src/index.js b/src/index.js
@@ -67,13 +67,6 @@ class ProcessManager {
       this.log.error(`Exiting with ${this.errors.length} error${this.errors.length > 1 ? 's' : ''}`);
       this.errors.forEach(err => this.log.error(err));
 
-      // Output console to error in case no `DEBUG` namespace has been set.
-      // This mimicks the default node behaviour of not silencing errors.
-      if (!process.env.DEBUG) {
-        // eslint-disable-next-line no-console
-        console.error(...this.errors);
-      }
-
       return process.exit(1);
     }
 

diff --git a/test/index.test.js b/test/index.test.js
@@ -151,26 +151,6 @@ describe('ProcessManager', () => {
       expect(process.exit).toHaveBeenCalled();
       expect(process.exit).toHaveBeenCalledWith(1);
     });
-
-    test('calls `console.error` if `DEBUG` is not set', () => {
-      processManager.errors = [new Error()];
-
-      processManager.exit();
-
-      expect(console.error).toHaveBeenCalled();
-      expect(console.error).toHaveBeenCalledWith(...processManager.errors);
-    });
-
-    test('does not call `console.error` if `DEBUG` is set', () => {
-      process.env.DEBUG = 'foo';
-      processManager.errors = [new Error()];
-
-      processManager.exit();
-
-      expect(console.error).not.toHaveBeenCalled();
-
-      delete process.env.DEBUG;
-    });
   });
 
   describe('hook()', () => {