chore: add env to test git command

Author: technote-space

__tests__/git-helper.test.ts

@@ -902,13 +902,13 @@ describe('GitHelper without params', () => {
   });
 });
 
-describe('ACTIONS_STEP_DEBUG test', () => {
+describe('Debug', () => {
   testEnv();
   testChildProcess();
 
   const helper = new GitHelper(new Logger(), {token: 'token1'});
 
-  it('should add suppress error command', async() => {
+  it('should add command to suppress error and output', async() => {
     const mockExec   = spyOnSpawn();
     const mockStdout = spyOnStdout();
     mockExec.mockImplementation(() => {
@@ -929,7 +929,29 @@ describe('ACTIONS_STEP_DEBUG test', () => {
     ]);
   });
 
-  it('should not add suppress error command', async() => {
+  it('should not add command to suppress error', async() => {
+    process.env.ACTIONS_UTILS_DEBUG = 'true';
+    const mockExec                  = spyOnSpawn();
+    const mockStdout                = spyOnStdout();
+    mockExec.mockImplementation(() => {
+      const error: ExecException = new Error('test error');
+      error.code                 = 123;
+      throw error;
+    });
+
+    await expect(helper.addOrigin(workDir, context())).rejects.toThrow('command [git remote add origin] exited with code 123.');
+
+    execCalledWith(mockExec, [
+      'git remote add origin \'https://octocat:[email protected]/hello/world.git\' > /dev/null 2>&1',
+    ]);
+    stdoutCalledWith(mockStdout, [
+      '[command]git remote add origin',
+      'undefined',
+      '{}',
+    ]);
+  });
+
+  it('should not add command to suppress error output', async() => {
     process.env.ACTIONS_STEP_DEBUG = 'true';
     const mockExec                 = spyOnSpawn();
     const mockStdout               = spyOnStdout();
@@ -942,7 +964,7 @@ describe('ACTIONS_STEP_DEBUG test', () => {
     await expect(helper.addOrigin(workDir, context())).rejects.toThrow('command [git remote add origin] exited with code 123.');
 
     execCalledWith(mockExec, [
-      'git remote add origin \'https://octocat:[email protected]/hello/world.git\'',
+      'git remote add origin \'https://octocat:[email protected]/hello/world.git\' || :',
     ]);
     stdoutCalledWith(mockStdout, [
       '[command]git remote add origin',

__tests__/utils2.test.ts

@@ -2,8 +2,8 @@
 import {testEnv} from '@technote-space/github-action-test-helper';
 import {Utils} from '../src';
 
-const {generateNewPatchVersion, generateNewMinorVersion, generateNewMajorVersion, arrayChunk, versionCompare, mask, isActionsStepDebug} = Utils;
-const {isBranch, isTagRef, normalizeRef, trimRef, getTag, getRefspec, getRemoteRefspec, getLocalRefspec, getOctokit, replaceVariables}  = Utils;
+const {generateNewPatchVersion, generateNewMinorVersion, generateNewMajorVersion, arrayChunk, versionCompare, mask, isActionsStepDebug}         = Utils;
+const {isBranch, isTagRef, normalizeRef, trimRef, getTag, getRefspec, getRemoteRefspec, getLocalRefspec, getOctokit, replaceVariables, isDebug} = Utils;
 
 jest.useFakeTimers();
 
@@ -264,6 +264,34 @@ describe('replaceVariables', () => {
   });
 });
 
+describe('isDebug', () => {
+  testEnv();
+
+  it('should return true', () => {
+    process.env.ACTIONS_UTILS_DEBUG = 'true';
+    expect(isDebug()).toBe(true);
+  });
+
+  it('should return false 1', () => {
+    expect(isDebug()).toBe(false);
+  });
+
+  it('should return false 2', () => {
+    process.env.ACTIONS_UTILS_DEBUG = '';
+    expect(isDebug()).toBe(false);
+  });
+
+  it('should return false 3', () => {
+    process.env.ACTIONS_UTILS_DEBUG = '1';
+    expect(isDebug()).toBe(false);
+  });
+
+  it('should return false 4', () => {
+    process.env.ACTIONS_UTILS_DEBUG = 'abc';
+    expect(isDebug()).toBe(false);
+  });
+});
+
 describe('isActionsStepDebug', () => {
   testEnv();
 

src/git-helper.ts

@@ -15,6 +15,7 @@ import {
   getAccessToken,
   generateNewMinorVersion,
   generateNewMajorVersion,
+  isDebug,
   isActionsStepDebug,
 } from './utils';
 import {getGitUrlWithToken} from './context-helper';
@@ -67,12 +68,12 @@ export default class GitHelper {
   /**
    * @return {boolean} should suppress error
    */
-  private shouldSuppressError = (): boolean => !isActionsStepDebug();
+  private shouldSuppressError = (): boolean => !isDebug();
 
   /**
    * @return {boolean} is quiet?
    */
-  private isQuiet = (): boolean => this.shouldSuppressError() && (!this.origin || this.quietIfNotOrigin);
+  private isQuiet = (): boolean => !isActionsStepDebug() && (!this.origin || this.quietIfNotOrigin);
 
   /**
    * @param {string} workDir work dir

src/utils.ts

@@ -253,4 +253,6 @@ export const replaceVariables = async(string: string, variables: { key: string;
   return replaced;
 };
 
+export const isDebug = (): boolean => process.env.ACTIONS_UTILS_DEBUG === 'true';
+
 export const isActionsStepDebug = (): boolean => process.env.ACTIONS_STEP_DEBUG === 'true';