captainhook-git / captainhook

tests/unit/Runner/Config/Setup/AdvancedTest.php

Indentation +10 added lines, -10 removed lines

@@ -29,11 +29,11 @@ 
         $config      = $this->createConfigMock();
         $config->expects($this->exactly(9))->method('getHookConfig')->willReturn($this->createHookConfigMock());
         $io->expects($invocations)
-           ->method('ask')
-           ->willReturnCallback(function ($parameters) use ($invocations) {
-               $results = ['y', 'y', 'echo \'foo\'', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n'];
-               return $results[$invocations->numberOfInvocations() - 1] ?? '';
-           });
+            ->method('ask')
+            ->willReturnCallback(function ($parameters) use ($invocations) {
+                $results = ['y', 'y', 'echo \'foo\'', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n'];
+                return $results[$invocations->numberOfInvocations() - 1] ?? '';
+            });
 
         $setup  = new Advanced($io);
         $setup->configureHooks($config);
@@ -46,11 +46,11 @@ 
         $config      = $this->createConfigMock();
         $config->method('getHookConfig')->willReturn($this->createHookConfigMock());
         $io->expects($invocations)
-           ->method('ask')
-           ->willReturnCallback(function ($parameters) use ($invocations) {
-               $results = ['y', 'y', '\\Foo\\Bar', 'y', 'foo:bar', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n'];
-               return $results[$invocations->numberOfInvocations() - 1] ?? '';
-           });
+            ->method('ask')
+            ->willReturnCallback(function ($parameters) use ($invocations) {
+                $results = ['y', 'y', '\\Foo\\Bar', 'y', 'foo:bar', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n'];
+                return $results[$invocations->numberOfInvocations() - 1] ?? '';
+            });
 
         $io->expects($this->once())->method('askAndValidate')->willReturn('foo:bar');
 

Spacing +4 added lines, -4 removed lines

@@ -30,12 +30,12 @@ 
         $config->expects($this->exactly(9))->method('getHookConfig')->willReturn($this->createHookConfigMock());
         $io->expects($invocations)
            ->method('ask')
-           ->willReturnCallback(function ($parameters) use ($invocations) {
+           ->willReturnCallback(function($parameters) use ($invocations) {
                $results = ['y', 'y', 'echo \'foo\'', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n'];
                return $results[$invocations->numberOfInvocations() - 1] ?? '';
            });
 
-        $setup  = new Advanced($io);
+        $setup = new Advanced($io);
         $setup->configureHooks($config);
     }
 
@@ -47,14 +47,14 @@ 
         $config->method('getHookConfig')->willReturn($this->createHookConfigMock());
         $io->expects($invocations)
            ->method('ask')
-           ->willReturnCallback(function ($parameters) use ($invocations) {
+           ->willReturnCallback(function($parameters) use ($invocations) {
                $results = ['y', 'y', '\\Foo\\Bar', 'y', 'foo:bar', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n'];
                return $results[$invocations->numberOfInvocations() - 1] ?? '';
            });
 
         $io->expects($this->once())->method('askAndValidate')->willReturn('foo:bar');
 
-        $setup  = new Advanced($io);
+        $setup = new Advanced($io);
         $setup->configureHooks($config);
     }
 }

tests/unit/Runner/Config/CreatorTest.php

Indentation +13 added lines, -13 removed lines

@@ -33,15 +33,15 @@ 
         $config      = $this->createConfigMock(true);
         $io          = $this->createIOMock();
         $io->expects($invocations)
-           ->method('ask')
-           ->willReturnCallback(function ($parameters) use ($invocations) {
-               $results = ['y', 'y', '\\Foo\\Bar', 'y', 'foo:bar', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n'];
-               return $results[$invocations->numberOfInvocations() - 1] ?? '';
-           });
+            ->method('ask')
+            ->willReturnCallback(function ($parameters) use ($invocations) {
+                $results = ['y', 'y', '\\Foo\\Bar', 'y', 'foo:bar', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n'];
+                return $results[$invocations->numberOfInvocations() - 1] ?? '';
+            });
 
         $runner = new Creator($io, $config);
         $runner->advanced(true)
-               ->run();
+                ->run();
     }
 
     /**
@@ -61,17 +61,17 @@ 
 
         $io = $this->createIOMock();
         $io->expects($invocations)
-           ->method('ask')
-           ->willReturnCallback(function ($parameters) use ($invocations) {
-               $results = ['y', 'y', '\\Foo\\Bar', 'y', 'foo:bar', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n'];
-               return $results[$invocations->numberOfInvocations() - 1] ?? '';
-           });
+            ->method('ask')
+            ->willReturnCallback(function ($parameters) use ($invocations) {
+                $results = ['y', 'y', '\\Foo\\Bar', 'y', 'foo:bar', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n'];
+                return $results[$invocations->numberOfInvocations() - 1] ?? '';
+            });
         $io->expects($this->once())->method('askAndValidate')->willReturn('foo:bar');
 
         $runner = new Creator($io, $config);
         $runner->extend(true)
-               ->advanced(true)
-               ->run();
+                ->advanced(true)
+                ->run();
 
         $configFileContentAfter = $configDir->getChild('captainhook.json')->getContent();
         $this->assertFileExists($configFile);

Spacing +2 added lines, -2 removed lines

@@ -34,7 +34,7 @@ 
         $io          = $this->createIOMock();
         $io->expects($invocations)
            ->method('ask')
-           ->willReturnCallback(function ($parameters) use ($invocations) {
+           ->willReturnCallback(function($parameters) use ($invocations) {
                $results = ['y', 'y', '\\Foo\\Bar', 'y', 'foo:bar', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n'];
                return $results[$invocations->numberOfInvocations() - 1] ?? '';
            });
@@ -62,7 +62,7 @@ 
         $io = $this->createIOMock();
         $io->expects($invocations)
            ->method('ask')
-           ->willReturnCallback(function ($parameters) use ($invocations) {
+           ->willReturnCallback(function($parameters) use ($invocations) {
                $results = ['y', 'y', '\\Foo\\Bar', 'y', 'foo:bar', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n'];
                return $results[$invocations->numberOfInvocations() - 1] ?? '';
            });

tests/unit/Runner/InstallerTest.php

Indentation +19 added lines, -19 removed lines

@@ -131,9 +131,9 @@ 
         $template = $this->createTemplateMock();
 
         $template->expects($this->once())
-                 ->method('getCode')
-                 ->with('pre-commit')
-                 ->willReturn('');
+                    ->method('getCode')
+                    ->with('pre-commit')
+                    ->willReturn('');
 
         $runner = new Installer($io, $config, $repo, $template);
         $runner->setHook('pre-commit');
@@ -168,8 +168,8 @@ 
         $template = $this->createTemplateMock();
 
         $file = $this->getMockBuilder(File::class)
-                     ->disableOriginalConstructor()
-                     ->getMock();
+                        ->disableOriginalConstructor()
+                        ->getMock();
 
         $runner = new FakeInstaller($io, $config, $repo, $template);
         $file->method('isLink')->willReturn(true);
@@ -189,8 +189,8 @@ 
         $template = $this->createTemplateMock();
 
         $file = $this->getMockBuilder(File::class)
-                     ->disableOriginalConstructor()
-                     ->getMock();
+                        ->disableOriginalConstructor()
+                        ->getMock();
 
         $runner = new FakeInstaller($io, $config, $repo, $template);
         $file->method('isLink')->willReturn(true);
@@ -260,14 +260,14 @@ 
         $repo     = new Repository($fakeRepo->getRoot());
 
         $template->expects($this->once())
-                 ->method('getCode')
-                 ->with('pre-commit')
-                 ->willReturn('');
+                    ->method('getCode')
+                    ->with('pre-commit')
+                    ->willReturn('');
 
         $runner = new Installer($io, $config, $repo, $template);
         $runner->setHook('pre-commit')
-               ->setMoveExistingTo('foo/bar/')
-               ->run();
+                ->setMoveExistingTo('foo/bar/')
+                ->run();
 
         $this->assertFileExists($fakeRepo->getHookDir() . '/pre-commit');
         $this->assertFileExists($fakeRepo->getRoot() . '/foo/bar/pre-commit');
@@ -295,14 +295,14 @@ 
         $repo     = new Repository($fakeRepo->getRoot());
 
         $template->expects($this->once())
-                 ->method('getCode')
-                 ->with('pre-commit')
-                 ->willReturn('');
+                    ->method('getCode')
+                    ->with('pre-commit')
+                    ->willReturn('');
 
         $runner = new Installer($io, $config, $repo, $template);
         $runner->setHook('pre-commit')
-               ->setMoveExistingTo('foo/bar/')
-               ->run();
+                ->setMoveExistingTo('foo/bar/')
+                ->run();
 
         $this->assertFileExists($fakeRepo->getHookDir() . '/pre-commit');
     }
@@ -333,8 +333,8 @@ 
 
         $runner = new Installer($io, $config, $repo, $template);
         $runner->setHook('pre-commit')
-               ->setMoveExistingTo('foo')
-               ->run();
+                ->setMoveExistingTo('foo')
+                ->run();
     }
 
     public function testSkipExisting(): void

tests/unit/Plugin/Hook/PreserveWorkingTreeTest.php

Spacing +3 added lines, -3 removed lines

@@ -201,7 +201,7 @@ 
         $this->statusOperator
             ->expects($this->once())
             ->method('restoreWorkingTree')
-            ->willReturnCallback(function (): bool {
+            ->willReturnCallback(function(): bool {
                 if (getenv(PreserveWorkingTree::SKIP_POST_CHECKOUT_VAR) != '1') {
                     $this->fail(PreserveWorkingTree::SKIP_POST_CHECKOUT_VAR . ' did not have the correct value');
                 }
@@ -211,7 +211,7 @@ 
         $this->diffOperator
             ->expects($this->once())
             ->method('applyPatches')
-            ->with($this->callback(function ($value) use ($patchFileConstraint): bool {
+            ->with($this->callback(function($value) use ($patchFileConstraint): bool {
                 $result = true;
                 foreach ($value as $item) {
                     $result = $result && $patchFileConstraint->evaluate($item, '', true);
@@ -266,7 +266,7 @@ 
         $this->diffOperator
             ->expects($this->exactly(3))
             ->method('applyPatches')
-            ->with($this->callback(function ($value) use ($patchFileConstraint): bool {
+            ->with($this->callback(function($value) use ($patchFileConstraint): bool {
                 $result = true;
                 foreach ($value as $item) {
                     $result = $result && $patchFileConstraint->evaluate($item, '', true);

tests/unit/ConfigTest.php

Spacing +1 added lines, -1 removed lines

@@ -177,7 +177,7 @@
             false,
             ['run-path' => '/usr/local/bin/captainhook', 'verbosity' => 'debug', 'custom' => ['foo' => 'bar']]
         );
-        $json   = $config->getJsonData();
+        $json = $config->getJsonData();
 
         $this->assertIsArray($json);
         $this->assertIsArray($json['config']);

src/Console/Command/Install.php

Indentation +63 added lines, -63 removed lines

@@ -42,64 +42,64 @@ 
     {
         parent::configure();
         $this->setName('install')
-             ->setDescription('Install hooks to your .git/hooks directory')
-             ->setHelp('Install git hooks to your .git/hooks directory')
-             ->addArgument(
-                 'hook',
-                 InputArgument::OPTIONAL,
-                 'Limit the hooks you want to install. ' .
-                 'You can specify multiple hooks with comma as delimiter. ' .
-                 'By default all hooks get installed'
-             )
-             ->addOption(
-                 'only-enabled',
-                 null,
-                 InputOption::VALUE_NONE,
-                 'Limit the hooks you want to install to those enabled in your conf. ' .
-                 'By default all hooks get installed'
-             )
-             ->addOption(
-                 'force',
-                 'f',
-                 InputOption::VALUE_NONE,
-                 'Force install without confirmation'
-             )
-             ->addOption(
-                 'skip-existing',
-                 's',
-                 InputOption::VALUE_NONE,
-                 'Do not overwrite existing hooks'
-             )
-             ->addOption(
-                 'move-existing-to',
-                 null,
-                 InputOption::VALUE_OPTIONAL,
-                 'Move existing hooks to given directory'
-             )
-             ->addOption(
-                 'bootstrap',
-                 'b',
-                 InputOption::VALUE_OPTIONAL,
-                 'Path to composers vendor/autoload.php'
-             )
-             ->addOption(
-                 'run-mode',
-                 'm',
-                 InputOption::VALUE_OPTIONAL,
-                 'Git hook run mode [php|shell|docker]'
-             )
-             ->addOption(
-                 'run-exec',
-                 'e',
-                 InputOption::VALUE_OPTIONAL,
-                 'The Docker command to start your container e.g. \'docker exec CONTAINER\''
-             )
-             ->addOption(
-                 'run-path',
-                 'p',
-                 InputOption::VALUE_OPTIONAL,
-                 'The path to the CaptainHook executable \'/usr/bin/captainhook\''
-             )
+                ->setDescription('Install hooks to your .git/hooks directory')
+                ->setHelp('Install git hooks to your .git/hooks directory')
+                ->addArgument(
+                    'hook',
+                    InputArgument::OPTIONAL,
+                    'Limit the hooks you want to install. ' .
+                    'You can specify multiple hooks with comma as delimiter. ' .
+                    'By default all hooks get installed'
+                )
+                ->addOption(
+                    'only-enabled',
+                    null,
+                    InputOption::VALUE_NONE,
+                    'Limit the hooks you want to install to those enabled in your conf. ' .
+                    'By default all hooks get installed'
+                )
+                ->addOption(
+                    'force',
+                    'f',
+                    InputOption::VALUE_NONE,
+                    'Force install without confirmation'
+                )
+                ->addOption(
+                    'skip-existing',
+                    's',
+                    InputOption::VALUE_NONE,
+                    'Do not overwrite existing hooks'
+                )
+                ->addOption(
+                    'move-existing-to',
+                    null,
+                    InputOption::VALUE_OPTIONAL,
+                    'Move existing hooks to given directory'
+                )
+                ->addOption(
+                    'bootstrap',
+                    'b',
+                    InputOption::VALUE_OPTIONAL,
+                    'Path to composers vendor/autoload.php'
+                )
+                ->addOption(
+                    'run-mode',
+                    'm',
+                    InputOption::VALUE_OPTIONAL,
+                    'Git hook run mode [php|shell|docker]'
+                )
+                ->addOption(
+                    'run-exec',
+                    'e',
+                    InputOption::VALUE_OPTIONAL,
+                    'The Docker command to start your container e.g. \'docker exec CONTAINER\''
+                )
+                ->addOption(
+                    'run-path',
+                    'p',
+                    InputOption::VALUE_OPTIONAL,
+                    'The path to the CaptainHook executable \'/usr/bin/captainhook\''
+                )
             ->addOption(
                 'run-git',
                 'd',
@@ -129,11 +129,11 @@ 
 
             $installer = new Installer($io, $config, $repo, $template);
             $installer->setHook(IOUtil::argToString($input->getArgument('hook')))
-                      ->setForce(IOUtil::argToBool($input->getOption('force')))
-                      ->setSkipExisting(IOUtil::argToBool($input->getOption('skip-existing')))
-                      ->setMoveExistingTo(IOUtil::argToString($input->getOption('move-existing-to')))
-                      ->setOnlyEnabled(IOUtil::argToBool($input->getOption('only-enabled')))
-                      ->run();
+                        ->setForce(IOUtil::argToBool($input->getOption('force')))
+                        ->setSkipExisting(IOUtil::argToBool($input->getOption('skip-existing')))
+                        ->setMoveExistingTo(IOUtil::argToString($input->getOption('move-existing-to')))
+                        ->setOnlyEnabled(IOUtil::argToBool($input->getOption('only-enabled')))
+                        ->run();
 
             return 0;
         } catch (Exception $e) {

src/Hook/Condition/File.php

Spacing +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
         if (empty($directories)) {
             return $files;
         }
-        return array_filter($files, function ($file) use ($directories) {
+        return array_filter($files, function($file) use ($directories) {
             foreach ($directories as $directory) {
                 if (str_starts_with($file, $directory)) {
                     return true;

src/Hook/Condition/File/OfType.php

Spacing +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
      * @param array<int, string>|string $types
      * @param array<int, string>|string $filter
      */
-    public function __construct(array|string $types, array|string $filter = [])
+    public function __construct(array | string $types, array | string $filter = [])
     {
         $this->suffixes   = is_array($types) ? $types : explode(',', $types);
         $this->diffFilter = FilterUtil::filterFromConfigValue($filter);

src/Hook/Condition/File/InDirectory.php

Spacing +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
      * @param array<int, string>|string $directories
      * @param array<int, string>|string $diffFilter
      */
-    public function __construct(array|string $directories, array|string $diffFilter = [])
+    public function __construct(array | string $directories, array | string $diffFilter = [])
     {
         $this->directories = is_array($directories) ? $directories : [$directories];
         $this->diffFilter  = FilterUtil::filterFromConfigValue($diffFilter);