fix(packagers): add logs for SLS_DEBUG

Author: kennyhyun

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);

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);

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');

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');
  });