- PostCommitTest implemented

- CommitMessageTest fixed
- orchestra/testbench upgraded

Author: igorsgm

composer.json

@@ -31,8 +31,8 @@
     },
     "require-dev": {
         "laravel/pint": "^1.2",
-        "mockery/mockery": "^1.4.4",
-        "orchestra/testbench": "^6.0",
+        "mockery/mockery": "^1.5.1",
+        "orchestra/testbench": "^7.0",
         "pestphp/pest": "^1.22",
         "pestphp/pest-plugin-mock": "^1.0",
         "phpunit/phpunit": "^9.0"

tests/Feature/CommitMessageTest.php

@@ -4,7 +4,6 @@
 use Igorsgm\GitHooks\Tests\Fixtures\CommitMessageTestHook1;
 use Igorsgm\GitHooks\Tests\Fixtures\CommitMessageTestHook2;
 use Igorsgm\GitHooks\Tests\Fixtures\CommitMessageTestHook4;
-use Illuminate\Support\Facades\Artisan;
 
 test('Commit Message is sent through HookPipes', function () {
     $commitMessageHooks = [
@@ -25,15 +24,11 @@
     GitHooks::shouldReceive('updateCommitMessageContentInFile')
         ->with(base_path($file), 'Test commit hook1 hook2');
 
-    $this->artisan('git-hooks:commit-msg', ['file' => $file]);
-
-    $output = Artisan::output();
+    $command = $this->artisan('git-hooks:commit-msg', ['file' => $file])
+        ->assertExitCode(0);
 
     foreach ($commitMessageHooks as $hook) {
-        $this->assertStringContainsString(
-            sprintf('Hook: %s...', resolve($hook)->getName()),
-            $output
-        );
+        $command->expectsOutputToContain(sprintf('Hook: %s...', resolve($hook)->getName()));
     }
 });
 
@@ -58,16 +53,11 @@
     GitHooks::shouldReceive('updateCommitMessageContentInFile')
         ->with(base_path($file), 'Test commit hello world');
 
-    $this->artisan('git-hooks:commit-msg', ['file' => $file]);
-
-    $output = Artisan::output();
+    $command = $this->artisan('git-hooks:commit-msg', ['file' => $file])
+        ->assertExitCode(0);
 
     foreach ($commitMessageHooks as $hook => $parameters) {
         $hook = resolve($hook, compact('parameters'));
-
-        $this->assertStringContainsString(
-            sprintf('Hook: %s...', $hook->getName()),
-            $output
-        );
+        $command->expectsOutputToContain(sprintf('Hook: %s...', $hook->getName()));
     }
 });

tests/Feature/PostCommitTest.php

@@ -0,0 +1,29 @@
+<?php
+
+use Igorsgm\GitHooks\Contracts\PostCommitHook;
+use Igorsgm\GitHooks\Facades\GitHooks;
+use Igorsgm\GitHooks\Git\Log;
+
+test('Git Log is sent through HookPipes', function () {
+    $commitHash = 'b636c88159e121d0c8276c417576d57ebb380dc3';
+
+    $postCommitHook1 = mock(PostCommitHook::class)->expect(
+        handle: fn(Log $log, Closure $closure) => expect($log->getHash())->toBe($commitHash)
+    );
+    $postCommitHook2 = clone $postCommitHook1;
+
+    $this->config->set('git-hooks.post-commit', [
+        $postCommitHook1,
+        $postCommitHook2,
+    ]);
+
+    GitHooks::shouldReceive('getLastCommitFromLog')
+        ->andReturn("commit $commitHash
+Author: Igor Moraes <[email protected]>
+Date:   Wed Nov 9 04:50:40 2022 -0800
+
+    wip
+");
+
+    $this->artisan('git-hooks:post-commit')->assertSuccessful();
+});

tests/TestCase.php

@@ -12,12 +12,6 @@ class TestCase extends \Orchestra\Testbench\TestCase
      */
     public $config;
 
-    protected function setUp(): void
-    {
-        parent::setUp();
-        $this->withoutMockingConsoleOutput();
-    }
-
     /**
      * Define environment setup.
      *