Merge pull request #113 from tiagonnascimento/feature-avoidingStringA…

…llocation

fix: removing stdout and stderr from spawnPromise to avoid memory all…

src/modules/commands.ts

@@ -4,7 +4,7 @@ import { AuthParameters, Build } from './types.js';
 import BuildsUtils from './utils.js';
 
 export default class Commands {
-  public static async auth(authParms: AuthParameters): Promise<{ stdout: string; stderr: string }> {
+  public static async auth(authParms: AuthParameters): Promise<void> {
     console.log(' --- auth --- ');
     const authCommand = 'sf' as string;
     const authCommandArgs: string[] = [];
@@ -38,7 +38,7 @@ export default class Commands {
     return BuildsUtils.execCommand(authCommand, authCommandArgs);
   }
 
-  public static async disableTracking(username: string): Promise<{ stdout: string; stderr: string }> {
+  public static async disableTracking(username: string): Promise<void> {
     console.log(' --- disabling source tracking on target sandbox --- ');
     const configCommand = 'sf' as string;
     const configCommandArgs: string[] = [];
@@ -51,7 +51,7 @@ export default class Commands {
     return BuildsUtils.execCommand(configCommand, configCommandArgs);
   }
 
-  public static async deploy(build: Build, username: string): Promise<{ stdout: string; stderr: string }> {
+  public static async deploy(build: Build, username: string): Promise<void> {
     console.log(` --- build type: ${build.type} --- `);
 
     let buildCommand: string;

src/modules/utils.ts

@@ -13,30 +13,22 @@ export default class BuildsUtils {
    * @param options SpawnOptions for this request
    * @returns
    */
-  public static async spawnPromise(
-    command: string,
-    args: string[],
-    options: SpawnOptions
-  ): Promise<{ stdout: string; stderr: string }> {
+  public static async spawnPromise(command: string, args: string[], options: SpawnOptions): Promise<void> {
     return new Promise((resolve, reject) => {
       const process = spawn(command, args, options);
-      let stdout = '';
-      let stderr = '';
 
       process.stdout?.on('data', (data: Buffer) => {
         console.log(data.toString());
-        stdout += data.toString();
       });
 
       process.stderr?.on('data', (data: Buffer) => {
         console.error(data.toString());
-        stderr += data.toString();
       });
 
       process.on('close', (code) => {
         if (code === 0) {
           console.log('Command executed successfully');
-          resolve({ stdout, stderr });
+          resolve();
         } else {
           console.error(`Process exited with code ${code ?? 'null or undefined'}`);
           reject(new Error(`Process exited with code ${code ?? 'null or undefined'}`));
@@ -57,11 +49,7 @@ export default class BuildsUtils {
    * @param {*} args array with args
    * @param {*} workingFolder opcional
    */
-  public static async execCommand(
-    command: string,
-    args: string[],
-    workingFolder: string | null = null
-  ): Promise<{ stdout: string; stderr: string }> {
+  public static async execCommand(command: string, args: string[], workingFolder: string | null = null): Promise<void> {
     const options: SpawnOptions = {};
 
     if (workingFolder) {

test/commands/builds/deploy.test.ts

@@ -73,8 +73,7 @@ describe('BuildsDeploy', () => {
 
   const execSpawnSync = stub(BuildsUtils, 'spawnPromise').returns(
     new Promise((resolve, reject) => {
-      const cmdReturn: { stdout: string; stderr: string } = { stdout: 'stdout', stderr: 'stderr' };
-      resolve(cmdReturn);
+      resolve();
     })
   );
   const execReadFileSync = stub(BuildsUtils, 'execReadFileSync');