@@ -56,16 +56,16 @@ |
||
56 | 56 | ); |
57 | 57 | |
58 | 58 | $io = $this->getMockBuilder(DefaultIO::class) |
59 | - ->disableOriginalConstructor() |
|
60 | - ->getMock(); |
|
59 | + ->disableOriginalConstructor() |
|
60 | + ->getMock(); |
|
61 | 61 | |
62 | 62 | $invokedCount = $this->atLeast(2); |
63 | 63 | $io->expects($invokedCount) |
64 | - ->method('ask') |
|
65 | - ->willReturnCallback(function ($parameters) use ($invokedCount) { |
|
66 | - $results = ['\\Foo\\Bar', 'n']; |
|
67 | - return $results[$invokedCount->numberOfInvocations() - 1] ?? ''; |
|
68 | - }); |
|
64 | + ->method('ask') |
|
65 | + ->willReturnCallback(function ($parameters) use ($invokedCount) { |
|
66 | + $results = ['\\Foo\\Bar', 'n']; |
|
67 | + return $results[$invokedCount->numberOfInvocations() - 1] ?? ''; |
|
68 | + }); |
|
69 | 69 | |
70 | 70 | $io->expects($this->once())->method('write'); |
71 | 71 |
@@ -42,8 +42,8 @@ |
||
42 | 42 | $config = '/foo/bar/fiz/baz/config.json'; |
43 | 43 | $input = new ArrayInput(['--configuration' => $config]); |
44 | 44 | $output = $this->getMockBuilder(OutputInterface::class) |
45 | - ->disableOriginalConstructor() |
|
46 | - ->getMock(); |
|
45 | + ->disableOriginalConstructor() |
|
46 | + ->getMock(); |
|
47 | 47 | $output->method('isVerbose')->willReturn(true); |
48 | 48 | |
49 | 49 | $configure = new Configuration($resolver); |
@@ -176,8 +176,8 @@ discard block |
||
176 | 176 | |
177 | 177 | $runner = new Uninstaller($io, $config, $repo); |
178 | 178 | $runner->setHook('post-merge') |
179 | - ->setMoveExistingTo('foo/bar/') |
|
180 | - ->run(); |
|
179 | + ->setMoveExistingTo('foo/bar/') |
|
180 | + ->run(); |
|
181 | 181 | |
182 | 182 | $this->assertFileDoesNotExist($fakeRepo->getHookDir() . '/post-merge'); |
183 | 183 | $this->assertFileExists($fakeRepo->getRoot() . '/foo/bar/post-merge'); |
@@ -208,8 +208,8 @@ discard block |
||
208 | 208 | |
209 | 209 | $runner = new Uninstaller($io, $config, $repo); |
210 | 210 | $runner->setHook('pre-commit') |
211 | - ->setMoveExistingTo('foo') |
|
212 | - ->run(); |
|
211 | + ->setMoveExistingTo('foo') |
|
212 | + ->run(); |
|
213 | 213 | } |
214 | 214 | |
215 | 215 | public function testMoveExistingHookWhenMoveExistingIsAnAbsolutePath(): void |
@@ -237,8 +237,8 @@ discard block |
||
237 | 237 | |
238 | 238 | $runner = new Uninstaller($io, $config, $repo); |
239 | 239 | $runner->setHook('pre-commit') |
240 | - ->setMoveExistingTo($virtualFs->url() . '/foo/bar') |
|
241 | - ->run(); |
|
240 | + ->setMoveExistingTo($virtualFs->url() . '/foo/bar') |
|
241 | + ->run(); |
|
242 | 242 | |
243 | 243 | $this->assertFileDoesNotExist($fakeRepo->getHookDir() . '/pre-commit'); |
244 | 244 | $this->assertFileExists($fakeRepo->getRoot() . '/foo/bar/pre-commit'); |
@@ -33,8 +33,8 @@ discard block |
||
33 | 33 | |
34 | 34 | $runner = new Editor($io, $config); |
35 | 35 | $runner->setHook('foo') |
36 | - ->setChange('EnableHook') |
|
37 | - ->run(); |
|
36 | + ->setChange('EnableHook') |
|
37 | + ->run(); |
|
38 | 38 | } |
39 | 39 | |
40 | 40 | public function testNoHook(): void |
@@ -47,7 +47,7 @@ discard block |
||
47 | 47 | |
48 | 48 | $runner = new Editor($io, $config); |
49 | 49 | $runner->setChange('EnableHook') |
50 | - ->run(); |
|
50 | + ->run(); |
|
51 | 51 | } |
52 | 52 | |
53 | 53 | public function testNoChange(): void |
@@ -60,7 +60,7 @@ discard block |
||
60 | 60 | |
61 | 61 | $runner = new Editor($io, $config); |
62 | 62 | $runner->setHook('pre-commit') |
63 | - ->run(); |
|
63 | + ->run(); |
|
64 | 64 | } |
65 | 65 | |
66 | 66 | public function testInvalidChange(): void |
@@ -73,8 +73,8 @@ discard block |
||
73 | 73 | |
74 | 74 | $runner = new Editor($io, $config); |
75 | 75 | $runner->setChange('InvalidChange') |
76 | - ->setHook('pre-commit') |
|
77 | - ->run(); |
|
76 | + ->setHook('pre-commit') |
|
77 | + ->run(); |
|
78 | 78 | } |
79 | 79 | |
80 | 80 | public function testMissingHook(): void |
@@ -87,7 +87,7 @@ discard block |
||
87 | 87 | |
88 | 88 | $runner = new Editor($io, $config); |
89 | 89 | $runner->setChange('EnableHook') |
90 | - ->run(); |
|
90 | + ->run(); |
|
91 | 91 | } |
92 | 92 | |
93 | 93 | public function testMissingChange(): void |
@@ -100,7 +100,7 @@ discard block |
||
100 | 100 | |
101 | 101 | $runner = new Editor($io, $config); |
102 | 102 | $runner->setHook('pre-commit') |
103 | - ->run(); |
|
103 | + ->run(); |
|
104 | 104 | } |
105 | 105 | |
106 | 106 | public function testNoConfiguration() |
@@ -113,8 +113,8 @@ discard block |
||
113 | 113 | |
114 | 114 | $runner = new Editor($io, $config); |
115 | 115 | $runner->setChange('AddAction') |
116 | - ->setHook('pre-commit') |
|
117 | - ->run(); |
|
116 | + ->setHook('pre-commit') |
|
117 | + ->run(); |
|
118 | 118 | } |
119 | 119 | |
120 | 120 | public function testConfigureFileExtend() |
@@ -125,17 +125,17 @@ discard block |
||
125 | 125 | $config = $this->createConfigMock(true, $configDir->url() . '/captainhook.json'); |
126 | 126 | $config->method('getHookConfig')->willReturn($this->createHookConfigMock()); |
127 | 127 | $io->expects($invocations) |
128 | - ->method('ask') |
|
129 | - ->willReturnCallback(function ($parameters) use ($invocations) { |
|
130 | - $results = ['y', 'y', '\\Foo\\Bar', 'y', 'foo:bar', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n']; |
|
131 | - return $results[$invocations->numberOfInvocations() - 1] ?? ''; |
|
132 | - }); |
|
128 | + ->method('ask') |
|
129 | + ->willReturnCallback(function ($parameters) use ($invocations) { |
|
130 | + $results = ['y', 'y', '\\Foo\\Bar', 'y', 'foo:bar', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n']; |
|
131 | + return $results[$invocations->numberOfInvocations() - 1] ?? ''; |
|
132 | + }); |
|
133 | 133 | $io->expects($this->once())->method('askAndValidate')->willReturn('foo:bar'); |
134 | 134 | |
135 | 135 | $runner = new Creator($io, $config); |
136 | 136 | $runner->extend(true) |
137 | - ->advanced(true) |
|
138 | - ->run(); |
|
137 | + ->advanced(true) |
|
138 | + ->run(); |
|
139 | 139 | |
140 | 140 | $this->assertFileExists($configDir->url() . '/captainhook.json'); |
141 | 141 | } |
@@ -29,11 +29,11 @@ |
||
29 | 29 | $config = $this->createConfigMock(); |
30 | 30 | $config->expects($this->exactly(2))->method('getHookConfig')->willReturn($this->createHookConfigMock()); |
31 | 31 | $io->expects($invokedCount) |
32 | - ->method('ask') |
|
33 | - ->willReturnCallback(function ($parameters) use ($invokedCount) { |
|
34 | - $results = ['y', 'y', 'y', 'phpunit', 'y', 'phpcs']; |
|
35 | - return $results[$invokedCount->numberOfInvocations() - 1] ?? ''; |
|
36 | - }); |
|
32 | + ->method('ask') |
|
33 | + ->willReturnCallback(function ($parameters) use ($invokedCount) { |
|
34 | + $results = ['y', 'y', 'y', 'phpunit', 'y', 'phpcs']; |
|
35 | + return $results[$invokedCount->numberOfInvocations() - 1] ?? ''; |
|
36 | + }); |
|
37 | 37 | |
38 | 38 | $setup = new Express($io); |
39 | 39 | $setup->configureHooks($config); |
@@ -29,11 +29,11 @@ discard block |
||
29 | 29 | $config = $this->createConfigMock(); |
30 | 30 | $config->expects($this->exactly(9))->method('getHookConfig')->willReturn($this->createHookConfigMock()); |
31 | 31 | $io->expects($invocations) |
32 | - ->method('ask') |
|
33 | - ->willReturnCallback(function ($parameters) use ($invocations) { |
|
34 | - $results = ['y', 'y', 'echo \'foo\'', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n']; |
|
35 | - return $results[$invocations->numberOfInvocations() - 1] ?? ''; |
|
36 | - }); |
|
32 | + ->method('ask') |
|
33 | + ->willReturnCallback(function ($parameters) use ($invocations) { |
|
34 | + $results = ['y', 'y', 'echo \'foo\'', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n']; |
|
35 | + return $results[$invocations->numberOfInvocations() - 1] ?? ''; |
|
36 | + }); |
|
37 | 37 | |
38 | 38 | $setup = new Advanced($io); |
39 | 39 | $setup->configureHooks($config); |
@@ -46,11 +46,11 @@ discard block |
||
46 | 46 | $config = $this->createConfigMock(); |
47 | 47 | $config->method('getHookConfig')->willReturn($this->createHookConfigMock()); |
48 | 48 | $io->expects($invocations) |
49 | - ->method('ask') |
|
50 | - ->willReturnCallback(function ($parameters) use ($invocations) { |
|
51 | - $results = ['y', 'y', '\\Foo\\Bar', 'y', 'foo:bar', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n']; |
|
52 | - return $results[$invocations->numberOfInvocations() - 1] ?? ''; |
|
53 | - }); |
|
49 | + ->method('ask') |
|
50 | + ->willReturnCallback(function ($parameters) use ($invocations) { |
|
51 | + $results = ['y', 'y', '\\Foo\\Bar', 'y', 'foo:bar', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n']; |
|
52 | + return $results[$invocations->numberOfInvocations() - 1] ?? ''; |
|
53 | + }); |
|
54 | 54 | |
55 | 55 | $io->expects($this->once())->method('askAndValidate')->willReturn('foo:bar'); |
56 | 56 |
@@ -33,15 +33,15 @@ discard block |
||
33 | 33 | $config = $this->createConfigMock(true); |
34 | 34 | $io = $this->createIOMock(); |
35 | 35 | $io->expects($invocations) |
36 | - ->method('ask') |
|
37 | - ->willReturnCallback(function ($parameters) use ($invocations) { |
|
38 | - $results = ['y', 'y', '\\Foo\\Bar', 'y', 'foo:bar', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n']; |
|
39 | - return $results[$invocations->numberOfInvocations() - 1] ?? ''; |
|
40 | - }); |
|
36 | + ->method('ask') |
|
37 | + ->willReturnCallback(function ($parameters) use ($invocations) { |
|
38 | + $results = ['y', 'y', '\\Foo\\Bar', 'y', 'foo:bar', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n']; |
|
39 | + return $results[$invocations->numberOfInvocations() - 1] ?? ''; |
|
40 | + }); |
|
41 | 41 | |
42 | 42 | $runner = new Creator($io, $config); |
43 | 43 | $runner->advanced(true) |
44 | - ->run(); |
|
44 | + ->run(); |
|
45 | 45 | } |
46 | 46 | |
47 | 47 | /** |
@@ -61,17 +61,17 @@ discard block |
||
61 | 61 | |
62 | 62 | $io = $this->createIOMock(); |
63 | 63 | $io->expects($invocations) |
64 | - ->method('ask') |
|
65 | - ->willReturnCallback(function ($parameters) use ($invocations) { |
|
66 | - $results = ['y', 'y', '\\Foo\\Bar', 'y', 'foo:bar', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n']; |
|
67 | - return $results[$invocations->numberOfInvocations() - 1] ?? ''; |
|
68 | - }); |
|
64 | + ->method('ask') |
|
65 | + ->willReturnCallback(function ($parameters) use ($invocations) { |
|
66 | + $results = ['y', 'y', '\\Foo\\Bar', 'y', 'foo:bar', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n', 'n']; |
|
67 | + return $results[$invocations->numberOfInvocations() - 1] ?? ''; |
|
68 | + }); |
|
69 | 69 | $io->expects($this->once())->method('askAndValidate')->willReturn('foo:bar'); |
70 | 70 | |
71 | 71 | $runner = new Creator($io, $config); |
72 | 72 | $runner->extend(true) |
73 | - ->advanced(true) |
|
74 | - ->run(); |
|
73 | + ->advanced(true) |
|
74 | + ->run(); |
|
75 | 75 | |
76 | 76 | $configFileContentAfter = $configDir->getChild('captainhook.json')->getContent(); |
77 | 77 | $this->assertFileExists($configFile); |
@@ -131,9 +131,9 @@ discard block |
||
131 | 131 | $template = $this->createTemplateMock(); |
132 | 132 | |
133 | 133 | $template->expects($this->once()) |
134 | - ->method('getCode') |
|
135 | - ->with('pre-commit') |
|
136 | - ->willReturn(''); |
|
134 | + ->method('getCode') |
|
135 | + ->with('pre-commit') |
|
136 | + ->willReturn(''); |
|
137 | 137 | |
138 | 138 | $runner = new Installer($io, $config, $repo, $template); |
139 | 139 | $runner->setHook('pre-commit'); |
@@ -168,8 +168,8 @@ discard block |
||
168 | 168 | $template = $this->createTemplateMock(); |
169 | 169 | |
170 | 170 | $file = $this->getMockBuilder(File::class) |
171 | - ->disableOriginalConstructor() |
|
172 | - ->getMock(); |
|
171 | + ->disableOriginalConstructor() |
|
172 | + ->getMock(); |
|
173 | 173 | |
174 | 174 | $runner = new FakeInstaller($io, $config, $repo, $template); |
175 | 175 | $file->method('isLink')->willReturn(true); |
@@ -189,8 +189,8 @@ discard block |
||
189 | 189 | $template = $this->createTemplateMock(); |
190 | 190 | |
191 | 191 | $file = $this->getMockBuilder(File::class) |
192 | - ->disableOriginalConstructor() |
|
193 | - ->getMock(); |
|
192 | + ->disableOriginalConstructor() |
|
193 | + ->getMock(); |
|
194 | 194 | |
195 | 195 | $runner = new FakeInstaller($io, $config, $repo, $template); |
196 | 196 | $file->method('isLink')->willReturn(true); |
@@ -260,14 +260,14 @@ discard block |
||
260 | 260 | $repo = new Repository($fakeRepo->getRoot()); |
261 | 261 | |
262 | 262 | $template->expects($this->once()) |
263 | - ->method('getCode') |
|
264 | - ->with('pre-commit') |
|
265 | - ->willReturn(''); |
|
263 | + ->method('getCode') |
|
264 | + ->with('pre-commit') |
|
265 | + ->willReturn(''); |
|
266 | 266 | |
267 | 267 | $runner = new Installer($io, $config, $repo, $template); |
268 | 268 | $runner->setHook('pre-commit') |
269 | - ->setMoveExistingTo('foo/bar/') |
|
270 | - ->run(); |
|
269 | + ->setMoveExistingTo('foo/bar/') |
|
270 | + ->run(); |
|
271 | 271 | |
272 | 272 | $this->assertFileExists($fakeRepo->getHookDir() . '/pre-commit'); |
273 | 273 | $this->assertFileExists($fakeRepo->getRoot() . '/foo/bar/pre-commit'); |
@@ -295,14 +295,14 @@ discard block |
||
295 | 295 | $repo = new Repository($fakeRepo->getRoot()); |
296 | 296 | |
297 | 297 | $template->expects($this->once()) |
298 | - ->method('getCode') |
|
299 | - ->with('pre-commit') |
|
300 | - ->willReturn(''); |
|
298 | + ->method('getCode') |
|
299 | + ->with('pre-commit') |
|
300 | + ->willReturn(''); |
|
301 | 301 | |
302 | 302 | $runner = new Installer($io, $config, $repo, $template); |
303 | 303 | $runner->setHook('pre-commit') |
304 | - ->setMoveExistingTo('foo/bar/') |
|
305 | - ->run(); |
|
304 | + ->setMoveExistingTo('foo/bar/') |
|
305 | + ->run(); |
|
306 | 306 | |
307 | 307 | $this->assertFileExists($fakeRepo->getHookDir() . '/pre-commit'); |
308 | 308 | } |
@@ -333,8 +333,8 @@ discard block |
||
333 | 333 | |
334 | 334 | $runner = new Installer($io, $config, $repo, $template); |
335 | 335 | $runner->setHook('pre-commit') |
336 | - ->setMoveExistingTo('foo') |
|
337 | - ->run(); |
|
336 | + ->setMoveExistingTo('foo') |
|
337 | + ->run(); |
|
338 | 338 | } |
339 | 339 | |
340 | 340 | public function testSkipExisting(): void |