feat: increase test coverage for failing status in pre-receive hook

Author: fabiovincenzi

src/proxy/processors/push-action/preReceive.js

@@ -31,10 +31,13 @@ const exec = async (req, action, hookFilePath = './hooks/pre-receive.sh') => {
 
     const { stdout, stderr, status } = hookProcess;
 
+    const stderrTrimmed = stderr ? stderr.trim() : '';
+    const stdoutTrimmed = stdout ? stdout.trim() : '';
+
     if (status !== 0) {
       step.error = true;
-      step.log(`Hook stderr: ${stderr.trim()}`);
-      step.setError(stdout.trim());
+      step.log(`Hook stderr: ${stderrTrimmed}`);
+      step.setError(stdoutTrimmed);
       action.addStep(step);
       return action;
     }

test/preReceive/preReceive.test.js

@@ -56,7 +56,14 @@ describe('Pre-Receive Hook Execution', function () {
     const result = await exec(req, action, scriptPath);
 
     expect(result.steps).to.have.lengthOf(1);
-    expect(result.steps[0].error).to.be.true;
-    expect(result.steps[0].logs.some((log) => log.includes('Hook execution error:'))).to.be.true;
+
+    const step = result.steps[0];
+
+    expect(step.error).to.be.true;
+    expect(step.logs.some((log) => log.includes('Hook stderr:'))).to.be.true;
+
+    expect(step.errorMessage).to.exist;
+
+    expect(action.steps).to.deep.include(step);
   });
 });