Merge pull request #1803 from kennyhyun/fix/utils-error-logging

Fix/utils error logging
serverless-heaven · Jun 18, 2024 · 984d359 · 984d359
2 parents 778338e + 8539948
commit 984d359
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 0 deletions.
diff --git a/lib/packagers/npm.js b/lib/packagers/npm.js
@@ -90,6 +90,9 @@ class NPM {
           if (!failed && !_.isEmpty(err.stdout)) {
             return BbPromise.resolve({ stdout: err.stdout });
           }
+          if (process.env.SLS_DEBUG) {
+            console.error(`DEBUG: ${err.stdout}\nSTDERR: ${err.stderr}`);
+          }
         }
 
         return BbPromise.reject(err);

diff --git a/lib/packagers/yarn.js b/lib/packagers/yarn.js
@@ -79,6 +79,9 @@ class Yarn {
           if (!failed && !_.isEmpty(err.stdout)) {
             return BbPromise.resolve({ stdout: err.stdout });
           }
+          if (process.env.SLS_DEBUG) {
+            console.error(`DEBUG: ${err.stdout}\nSTDERR: ${err.stderr}`);
+          }
         }
 
         return BbPromise.reject(err);

diff --git a/tests/packagers/npm.test.js b/tests/packagers/npm.test.js
@@ -22,9 +22,17 @@ jest.mock('../../lib/utils', () => {
 });
 
 describe('npm', () => {
+  const ENV = process.env;
+
+  beforeAll(() => {
+    process.env = { ...ENV, SLS_DEBUG: '*' };
+  });
   beforeEach(() => {
     fsMock.readFileSync.mockReturnValue(false);
   });
+  afterAll(() => {
+    process.env = ENV;
+  });
 
   it('should return "package-lock.json" as lockfile name', () => {
     expect(npmModule.lockfileName).toEqual('package-lock.json');

diff --git a/tests/packagers/yarn.test.js b/tests/packagers/yarn.test.js
@@ -16,6 +16,15 @@ jest.mock('../../lib/utils', () => {
 });
 
 describe('yarn', () => {
+  const ENV = process.env;
+
+  beforeAll(() => {
+    process.env = { ...ENV, SLS_DEBUG: '*' };
+  });
+  afterAll(() => {
+    process.env = ENV;
+  });
+
   it('should return "yarn.lock" as lockfile name', () => {
     expect(yarnModule.lockfileName).toEqual('yarn.lock');
   });