captainhook-git / captainhook

tests/unit/Console/IOUtilTest.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
 
     public function testFormatHeadlineShort(): void
     {
-        $text     = str_repeat('x', 70) ;
+        $text     = str_repeat('x', 70);
         $expected = '==== ' . $text . ' ====';
         $headline = IOUtil::formatHeadline($text, 80);
 

tests/unit/Runner/Config/EditorTest.php

Indentation +17 added lines, -17 removed lines

@@ -33,8 +33,8 @@ 
 
         $runner = new Editor($io, $config);
         $runner->setHook('foo')
-               ->setChange('EnableHook')
-               ->run();
+                ->setChange('EnableHook')
+                ->run();
     }
 
     public function testNoHook(): void
@@ -47,7 +47,7 @@ 
 
         $runner = new Editor($io, $config);
         $runner->setChange('EnableHook')
-               ->run();
+                ->run();
     }
 
     public function testNoChange(): void
@@ -60,7 +60,7 @@ 
 
         $runner = new Editor($io, $config);
         $runner->setHook('pre-commit')
-               ->run();
+                ->run();
     }
 
     public function testInvalidChange(): void
@@ -73,8 +73,8 @@ 
 
         $runner = new Editor($io, $config);
         $runner->setChange('InvalidChange')
-               ->setHook('pre-commit')
-               ->run();
+                ->setHook('pre-commit')
+                ->run();
     }
 
     public function testMissingHook(): void
@@ -87,7 +87,7 @@ 
 
         $runner = new Editor($io, $config);
         $runner->setChange('EnableHook')
-               ->run();
+                ->run();
     }
 
     public function testMissingChange(): void
@@ -100,7 +100,7 @@ 
 
         $runner = new Editor($io, $config);
         $runner->setHook('pre-commit')
-               ->run();
+                ->run();
     }
 
     public function testNoConfiguration()
@@ -113,8 +113,8 @@ 
 
         $runner = new Editor($io, $config);
         $runner->setChange('AddAction')
-               ->setHook('pre-commit')
-               ->run();
+                ->setHook('pre-commit')
+                ->run();
     }
 
     public function testConfigureFileExtend()
@@ -125,17 +125,17 @@ 
         $config      = $this->createConfigMock(true, $configDir->url() . '/captainhook.json');
         $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');
 
         $runner = new Creator($io, $config);
         $runner->extend(true)
-               ->advanced(true)
-               ->run();
+                ->advanced(true)
+                ->run();
 
         $this->assertFileExists($configDir->url() . '/captainhook.json');
     }

Spacing +1 added lines, -1 removed lines

@@ -126,7 +126,7 @@
         $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] ?? '';
            });

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

Indentation +5 added lines, -5 removed lines

@@ -29,11 +29,11 @@
         $config       = $this->createConfigMock();
         $config->expects($this->exactly(2))->method('getHookConfig')->willReturn($this->createHookConfigMock());
         $io->expects($invokedCount)
-           ->method('ask')
-           ->willReturnCallback(function ($parameters) use ($invokedCount) {
-               $results = ['y', 'y', 'y', 'phpunit', 'y', 'phpcs'];
-               return $results[$invokedCount->numberOfInvocations() - 1] ?? '';
-           });
+            ->method('ask')
+            ->willReturnCallback(function ($parameters) use ($invokedCount) {
+                $results = ['y', 'y', 'y', 'phpunit', 'y', 'phpcs'];
+                return $results[$invokedCount->numberOfInvocations() - 1] ?? '';
+            });
 
         $setup  = new Express($io);
         $setup->configureHooks($config);

Spacing +2 added lines, -2 removed lines

@@ -30,12 +30,12 @@
         $config->expects($this->exactly(2))->method('getHookConfig')->willReturn($this->createHookConfigMock());
         $io->expects($invokedCount)
            ->method('ask')
-           ->willReturnCallback(function ($parameters) use ($invokedCount) {
+           ->willReturnCallback(function($parameters) use ($invokedCount) {
                $results = ['y', 'y', 'y', 'phpunit', 'y', 'phpcs'];
                return $results[$invokedCount->numberOfInvocations() - 1] ?? '';
            });
 
-        $setup  = new Express($io);
+        $setup = new Express($io);
         $setup->configureHooks($config);
     }
 }

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);