- PrepareCommitMessageTest implemented

Author: igorsgm

tests/Feature/CommitMessageTest.php

@@ -1,14 +1,14 @@
 <?php
 
 use Igorsgm\GitHooks\Facades\GitHooks;
-use Igorsgm\GitHooks\Tests\Fixtures\CommitMessageTestHook1;
-use Igorsgm\GitHooks\Tests\Fixtures\CommitMessageTestHook2;
-use Igorsgm\GitHooks\Tests\Fixtures\CommitMessageTestHook4;
+use Igorsgm\GitHooks\Tests\Fixtures\CommitMessageFixtureHook1;
+use Igorsgm\GitHooks\Tests\Fixtures\CommitMessageFixtureHook2;
+use Igorsgm\GitHooks\Tests\Fixtures\CommitMessageFixtureHook4;
 
 test('Commit Message is sent through HookPipes', function () {
     $commitMessageHooks = [
-        CommitMessageTestHook1::class,
-        CommitMessageTestHook2::class,
+        CommitMessageFixtureHook1::class,
+        CommitMessageFixtureHook2::class,
     ];
 
     $this->config->set('git-hooks.commit-msg', $commitMessageHooks);
@@ -34,7 +34,7 @@
 
 test('Pass parameters into Commit Hook class', function () {
     $commitMessageHooks = [
-        CommitMessageTestHook4::class => [
+        CommitMessageFixtureHook4::class => [
             'param1' => 'hello',
             'param2' => 'world',
         ],

tests/Feature/PreCommitTest.php

@@ -27,8 +27,6 @@
 });
 
 it('returns 1 on HookFailException', function () {
-    $changedFilesString = 'AM src/ChangedFiles.php';
-
     $preCommitHook1 = mock(PreCommitHook::class)->expect(
         handle: function (ChangedFiles $files, Closure $closure) {
             throw new HookFailException();
@@ -39,7 +37,5 @@
         $preCommitHook1,
     ]);
 
-    GitHooks::shouldReceive('getListOfChangedFiles')->andReturn($changedFilesString);
-
     $this->artisan('git-hooks:pre-commit')->assertExitCode(1);
 });

tests/Feature/PrepareCommitMessageTest.php

@@ -0,0 +1,32 @@
+<?php
+
+use Igorsgm\GitHooks\Facades\GitHooks;
+use Igorsgm\GitHooks\Tests\Fixtures\PrepareCommitMessageFixtureHook1;
+use Igorsgm\GitHooks\Tests\Fixtures\PrepareCommitMessageFixtureHook2;
+
+test('Commit Message is sent through HookPipes', function () {
+    $prepareCommitMessageHooks = [
+        PrepareCommitMessageFixtureHook1::class,
+        PrepareCommitMessageFixtureHook2::class,
+    ];
+
+    $this->config->set('git-hooks.prepare-commit-msg', $prepareCommitMessageHooks);
+
+    $file = 'tmp/COMMIT_MESSAGE';
+
+    GitHooks::shouldReceive('getCommitMessageContentFromFile')
+        ->andReturn('Test commit');
+
+    GitHooks::shouldReceive('getListOfChangedFiles')
+        ->andReturn('AM src/ChangedFiles.php');
+
+    GitHooks::shouldReceive('updateCommitMessageContentInFile')
+        ->with(base_path($file), 'Test commit hook1 hook2');
+
+    $command = $this->artisan('git-hooks:prepare-commit-msg', ['file' => $file])
+        ->assertExitCode(0);
+
+    foreach ($prepareCommitMessageHooks as $hook) {
+        $command->expectsOutputToContain(sprintf('Hook: %s...', resolve($hook)->getName()));
+    }
+});

tests/Fixtures/CommitMessageTestHook1.php renamed to tests/Fixtures/CommitMessageFixtureHook1.php

@@ -5,7 +5,7 @@
 use Closure;
 use Igorsgm\GitHooks\Contracts\MessageHook;
 
-class CommitMessageTestHook1 implements MessageHook
+class CommitMessageFixtureHook1 implements MessageHook
 {
     /**
      * {@inheritDoc}

tests/Fixtures/CommitMessageTestHook2.php renamed to tests/Fixtures/CommitMessageFixtureHook2.php

@@ -5,7 +5,7 @@
 use Closure;
 use Igorsgm\GitHooks\Contracts\MessageHook;
 
-class CommitMessageTestHook2 implements MessageHook
+class CommitMessageFixtureHook2 implements MessageHook
 {
     /**
      * {@inheritDoc}

tests/Fixtures/CommitMessageTestHook4.php renamed to tests/Fixtures/CommitMessageFixtureHook4.php

@@ -5,7 +5,7 @@
 use Closure;
 use Igorsgm\GitHooks\Contracts\MessageHook;
 
-class CommitMessageTestHook4 implements MessageHook
+class CommitMessageFixtureHook4 implements MessageHook
 {
     /**
      * @var array

tests/Fixtures/PrepareCommitMessageFixtureHook1.php

@@ -0,0 +1,28 @@
+<?php
+
+namespace Igorsgm\GitHooks\Tests\Fixtures;
+
+use Closure;
+use Igorsgm\GitHooks\Contracts\MessageHook;
+use Igorsgm\GitHooks\Git\CommitMessage;
+
+class PrepareCommitMessageFixtureHook1 implements MessageHook
+{
+    /**
+     * {@inheritDoc}
+     */
+    public function handle(CommitMessage $message, Closure $next)
+    {
+        $message->setMessage($message->getMessage().' hook1');
+
+        return $next($message);
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function getName(): string
+    {
+        return 'Prepare Commit Message Hook 1';
+    }
+}

tests/Fixtures/PrepareCommitMessageFixtureHook2.php

@@ -0,0 +1,28 @@
+<?php
+
+namespace Igorsgm\GitHooks\Tests\Fixtures;
+
+use Closure;
+use Igorsgm\GitHooks\Contracts\MessageHook;
+use Igorsgm\GitHooks\Git\CommitMessage;
+
+class PrepareCommitMessageFixtureHook2 implements MessageHook
+{
+    /**
+     * {@inheritDoc}
+     */
+    public function handle(CommitMessage $message, Closure $next)
+    {
+        $message->setMessage($message->getMessage().' hook2');
+
+        return $next($message);
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function getName(): string
+    {
+        return 'Prepare Commit Message Hook 2';
+    }
+}