HookFailException tests

Author: igorsgm

src/Traits/ProcessHelper.php

@@ -5,11 +5,15 @@
 use RuntimeException;
 use Symfony\Component\Process\Process;
 
+/**
+ * @codeCoverageIgnore
+ */
 trait ProcessHelper
 {
     /**
      * Run the given commands.
      *
+     *
      * @param  array|string  $commands
      * @param  array  $params
      * @return \Symfony\Component\Process\Process

tests/Features/Commands/PostCommitTest.php

@@ -1,6 +1,7 @@
 <?php
 
 use Igorsgm\GitHooks\Contracts\PostCommitHook;
+use Igorsgm\GitHooks\Exceptions\HookFailException;
 use Igorsgm\GitHooks\Facades\GitHooks;
 use Igorsgm\GitHooks\Git\Log;
 
@@ -19,3 +20,18 @@
 
     $this->artisan('git-hooks:post-commit')->assertSuccessful();
 })->with('lastCommitLogText');
+
+it('Returns 1 on HookFailException', function ($logText) {
+    $postCommitHook1 = mock(PostCommitHook::class)->expect(
+        handle: function (Log $log, Closure $closure) {
+            throw new HookFailException();
+        }
+    );
+
+    $this->config->set('git-hooks.post-commit', [
+        $postCommitHook1,
+    ]);
+
+    GitHooks::shouldReceive('getLastCommitFromLog')->andReturn($logText);
+    $this->artisan('git-hooks:post-commit')->assertExitCode(1);
+})->with('lastCommitLogText');

tests/Features/Commands/PrepareCommitMessageTest.php

@@ -1,6 +1,9 @@
 <?php
 
+use Igorsgm\GitHooks\Contracts\MessageHook;
+use Igorsgm\GitHooks\Exceptions\HookFailException;
 use Igorsgm\GitHooks\Facades\GitHooks;
+use Igorsgm\GitHooks\Git\CommitMessage;
 use Igorsgm\GitHooks\Tests\Fixtures\PrepareCommitMessageFixtureHook1;
 use Igorsgm\GitHooks\Tests\Fixtures\PrepareCommitMessageFixtureHook2;
 
@@ -30,3 +33,28 @@
         $command->expectsOutputToContain(sprintf('Hook: %s...', resolve($hook)->getName()));
     }
 })->with('listOfChangedFiles');
+
+it('Returns 1 on HookFailException', function ($listOfChangedFiles) {
+    $postCommitHook1 = mock(MessageHook::class)->expect(
+        handle: function (CommitMessage $commitMessage, Closure $closure) {
+            throw new HookFailException();
+        }
+    );
+
+    $this->config->set('git-hooks.prepare-commit-msg', [
+        $postCommitHook1,
+    ]);
+
+    $file = 'tmp/COMMIT_MESSAGE';
+
+    GitHooks::shouldReceive('getCommitMessageContentFromFile')
+        ->andReturn('Test commit');
+
+    GitHooks::shouldReceive('getListOfChangedFiles')
+        ->andReturn($listOfChangedFiles);
+
+    GitHooks::shouldReceive('updateCommitMessageContentInFile')
+        ->with(base_path($file), 'Test commit hook1 hook2');
+
+    $this->artisan('git-hooks:prepare-commit-msg', ['file' => $file])->assertExitCode(1);
+})->with('listOfChangedFiles');