Fix broken tests by updating mock implementations

- Update PostCommitTest.php, PreCommitTest.php, PrepareCommitMessageTest.php, and PrePushTest.php to use alternative mock implementation due to issues with the current version of Mockery.
- Add @todo comments to revisit these tests once Pest or Mockery versions are updated.

Author: igorsgm

tests/Features/Commands/PostCommitTest.php

@@ -6,9 +6,16 @@
 use Igorsgm\GitHooks\Git\Log;
 
 test('Git Log is sent through HookPipes', function (string $logText) {
-    $postCommitHook1 = mock(PostCommitHook::class)->expect(
-        handle: fn (Log $log, Closure $closure) => expect($log->getHash())->toBe(mockCommitHash())
-    );
+// This approach is broken in the current version of Mockery
+// @TODO: Update this test once Pest or Mockery versions are updated
+//    $postCommitHook1 = mock(PostCommitHook::class)->expect(
+//        handle: fn (Log $log, Closure $closure) => expect($log->getHash())->toBe(mockCommitHash())
+//    );
+
+    $postCommitHook1 = mock(PostCommitHook::class);
+    $postCommitHook1->expects('handle')
+        ->withArgs(fn($log, $closure) => $log->getHash() === mockCommitHash());
+
     $postCommitHook2 = clone $postCommitHook1;
 
     $this->config->set('git-hooks.post-commit', [
@@ -22,11 +29,19 @@
 })->with('lastCommitLogText');
 
 it('Returns 1 on HookFailException', function ($logText) {
-    $postCommitHook1 = mock(PostCommitHook::class)->expect(
-        handle: function (Log $log, Closure $closure) {
+// This approach is broken in the current version of Mockery
+// @TODO: Update this test once Pest or Mockery versions are updated
+//    $postCommitHook1 = mock(PostCommitHook::class)->expect(
+//        handle: function (Log $log, Closure $closure) {
+//            throw new HookFailException();
+//        }
+//    );
+
+    $postCommitHook1 = mock(PostCommitHook::class);
+    $postCommitHook1->expects('handle')
+        ->andReturnUsing(function (Log $log, Closure $closure) {
             throw new HookFailException();
-        }
-    );
+        });
 
     $this->config->set('git-hooks.post-commit', [
         $postCommitHook1,

tests/Features/Commands/PreCommitTest.php

@@ -6,12 +6,21 @@
 use Igorsgm\GitHooks\Git\ChangedFiles;
 
 test('Sends ChangedFiles through HookPipes', function (string $listOfChangedFiles) {
-    $preCommitHook1 = mock(PreCommitHook::class)->expect(
-        handle: function (ChangedFiles $files, Closure $closure) use ($listOfChangedFiles) {
+// This approach is broken in the current version of Mockery
+// @TODO: Update this test once Pest or Mockery versions are updated
+//    $preCommitHook1 = mock(PreCommitHook::class)->expect(
+//        handle: function (ChangedFiles $files, Closure $closure) use ($listOfChangedFiles) {
+//            $firstChangedFile = (string) $files->getFiles()->first();
+//            expect($firstChangedFile)->toBe($listOfChangedFiles);
+//        }
+//    );
+    $preCommitHook1 = mock(PreCommitHook::class);
+    $preCommitHook1->expects('handle')
+        ->andReturnUsing(function (ChangedFiles $files, Closure $closure) use ($listOfChangedFiles) {
             $firstChangedFile = (string) $files->getFiles()->first();
             expect($firstChangedFile)->toBe($listOfChangedFiles);
-        }
-    );
+        });
+
     $preCommitHook2 = clone $preCommitHook1;
 
     $this->config->set('git-hooks.pre-commit', [
@@ -25,11 +34,19 @@
 })->with('listOfChangedFiles');
 
 it('Returns 1 on HookFailException', function ($listOfChangedFiles) {
-    $preCommitHook1 = mock(PreCommitHook::class)->expect(
-        handle: function (ChangedFiles $files, Closure $closure) {
+// This approach is broken in the current version of Mockery
+// @TODO: Update this test once Pest or Mockery versions are updated
+//    $preCommitHook1 = mock(PreCommitHook::class)->expect(
+//        handle: function (ChangedFiles $files, Closure $closure) {
+//            throw new HookFailException();
+//        }
+//    );
+    $preCommitHook1 = mock(PreCommitHook::class);
+    $preCommitHook1->expects('handle')
+        ->andReturnUsing(function (ChangedFiles $files, Closure $closure) {
             throw new HookFailException();
-        }
-    );
+        });
+
 
     $this->config->set('git-hooks.pre-commit', [
         $preCommitHook1,

tests/Features/Commands/PrePushTest.php

@@ -5,9 +5,16 @@
 use Igorsgm\GitHooks\Git\Log;
 
 test('Git Log is sent through HookPipes', function (string $logText) {
-    $prePushHook1 = mock(PostCommitHook::class)->expect(
-        handle: fn (Log $log, Closure $closure) => expect($log->getHash())->toBe(mockCommitHash())
-    );
+// This approach is broken in the current version of Mockery
+// @TODO: Update this test once Pest or Mockery versions are updated
+//    $prePushHook1 = mock(PostCommitHook::class)->expect(
+//        handle: fn (Log $log, Closure $closure) => expect($log->getHash())->toBe(mockCommitHash())
+//    );
+    $prePushHook1 = mock(PostCommitHook::class);
+    $prePushHook1->expects('handle')
+        ->withArgs(fn(Log $log, Closure $closure) => $log->getHash() === mockCommitHash())
+        ->once();
+
     $prePushHook2 = clone $prePushHook1;
 
     $this->config->set('git-hooks.pre-push', [

tests/Features/Commands/PrepareCommitMessageTest.php

@@ -35,11 +35,19 @@
 })->with('listOfChangedFiles');
 
 it('Returns 1 on HookFailException', function ($listOfChangedFiles) {
-    $postCommitHook1 = mock(MessageHook::class)->expect(
-        handle: function (CommitMessage $commitMessage, Closure $closure) {
+// This approach is broken in the current version of Mockery
+// @TODO: Update this test once Pest or Mockery versions are updated
+//    $postCommitHook1 = mock(MessageHook::class)->expect(
+//        handle: function (CommitMessage $commitMessage, Closure $closure) {
+//            throw new HookFailException();
+//        }
+//    );
+    $postCommitHook1 = mock(MessageHook::class);
+    $postCommitHook1->expects('handle')
+        ->andReturnUsing(function (CommitMessage $commitMessage, Closure $closure) {
             throw new HookFailException();
-        }
-    );
+        });
+
 
     $this->config->set('git-hooks.prepare-commit-msg', [
         $postCommitHook1,