Added tests for the new multiple hooks specifying mechanism

Author: alexandrmazur96

tests/unit/Console/Command/InstallTest.php

@@ -114,4 +114,99 @@ public function testInstallPreCommitHook(): void
 
         $this->assertTrue($repo->hookExists('pre-commit'));
     }
+
+	/**
+	 * Tests Install::run
+	 *
+	 * @throws \Exception
+	 */
+	public function testInstallMultipleHooks(): void
+	{
+		$repo   = new DummyRepo();
+		$output = new NullOutput();
+		$input  = new ArrayInput(
+			[
+				'hook'            => 'pre-commit,pre-push,post-checkout',
+				'--configuration' => CH_PATH_FILES . '/template/captainhook.json',
+				'--git-directory' => $repo->getGitDir()
+			]
+		);
+
+		$install = new Install(new Resolver(CH_PATH_FILES . '/bin/captainhook'));
+		$install->run($input, $output);
+
+		$this->assertTrue($repo->hookExists('pre-commit'));
+		$this->assertTrue($repo->hookExists('pre-push'));
+		$this->assertTrue($repo->hookExists('post-checkout'));
+	}
+
+	/**
+	 * Tests Install::run
+	 *
+	 * @throws \Exception
+	 */
+	public function testInstallMultipleHooksWithSpacesAfterAndBetweenSeparator(): void
+	{
+		$repo   = new DummyRepo();
+		$output = new NullOutput();
+		$input  = new ArrayInput(
+			[
+				'hook'            => ' pre-commit , pre-push , post-checkout, post-commit',
+				'--configuration' => CH_PATH_FILES . '/template/captainhook.json',
+				'--git-directory' => $repo->getGitDir()
+			]
+		);
+
+		$install = new Install(new Resolver(CH_PATH_FILES . '/bin/captainhook'));
+		$install->run($input, $output);
+
+		$this->assertTrue($repo->hookExists('pre-commit'));
+		$this->assertTrue($repo->hookExists('pre-push'));
+		$this->assertTrue($repo->hookExists('post-checkout'));
+	}
+
+	/**
+	 * Tests Install::run
+	 *
+	 * @throws \Exception
+	 */
+	public function testInstallMultipleHooksWithOneWrong(): void
+	{
+		$this->expectException(\CaptainHook\App\Exception\InvalidHookName::class);
+		$repo   = new DummyRepo();
+		$output = new NullOutput();
+		$input  = new ArrayInput(
+			[
+				'hook'            => 'pre-commit,pre-push,post-checkout,something-wrong',
+				'--configuration' => CH_PATH_FILES . '/template/captainhook.json',
+				'--git-directory' => $repo->getGitDir()
+			]
+		);
+
+		$install = new Install(new Resolver(CH_PATH_FILES . '/bin/captainhook'));
+		$install->run($input, $output);
+	}
+
+
+	/**
+	 * Tests Install::run
+	 *
+	 * @throws \Exception
+	 */
+	public function testInstallMultipleHooksWithMultipleWrong(): void
+	{
+		$this->expectException(\CaptainHook\App\Exception\InvalidHookName::class);
+		$repo   = new DummyRepo();
+		$output = new NullOutput();
+		$input  = new ArrayInput(
+			[
+				'hook'            => 'pre-commit,pre-push,post-checkout,something-wrong1,something-wrong2',
+				'--configuration' => CH_PATH_FILES . '/template/captainhook.json',
+				'--git-directory' => $repo->getGitDir()
+			]
+		);
+
+		$install = new Install(new Resolver(CH_PATH_FILES . '/bin/captainhook'));
+		$install->run($input, $output);
+	}
 }

tests/unit/Runner/InstallerTest.php

@@ -47,6 +47,24 @@ public function testSetInvalidHook(): void
         $runner->setHook('itDoNotExist');
     }
 
+    /**
+     * Tests Installer::setHook
+     *
+     * @throws \CaptainHook\App\Exception\InvalidHookName
+     */
+    public function testSetMultipleInvalidHooks(): void
+    {
+        $this->expectException(InvalidHookName::class);
+
+        $io       = $this->createIOMock();
+        $config   = $this->createConfigMock();
+        $repo     = $this->createRepositoryMock();
+        $template = $this->createTemplateMock();
+
+        $runner = new Installer($io, $config, $repo, $template);
+        $runner->setHook('itDoNotExist1,itDoNotExist2,itDontExist3');
+    }
+
     /**
      * Tests Installer::setHook
      *
@@ -132,6 +150,27 @@ public function testWriteHook(): void
         $this->assertFileExists($fakeRepo->getHookDir() . '/pre-commit');
     }
 
+    /**
+     * Tests Installer::run
+     */
+    public function testWriteMultipleHooks(): void
+    {
+        $fakeRepo = new DummyRepo();
+
+        $io       = $this->createIOMock();
+        $config   = $this->createConfigMock();
+        $repo     = $this->createRepositoryMock($fakeRepo->getRoot());
+        $template = $this->createTemplateMock();
+
+        $runner = new Installer($io, $config, $repo, $template);
+        $runner->setHook('pre-commit,pre-push,post-checkout');
+        $runner->run();
+
+        $this->assertFileExists($fakeRepo->getHookDir() . '/pre-commit');
+        $this->assertFileExists($fakeRepo->getHookDir() . '/pre-push');
+        $this->assertFileExists($fakeRepo->getHookDir() . '/post-checkout');
+    }
+
     /**
      * Tests Installer::run
      */