@@ -23,13 +23,13 @@ discard block |
||
| 23 | 23 | class PrefixerTest extends TestCase |
| 24 | 24 | { |
| 25 | 25 | |
| 26 | - protected StraussConfig $config; |
|
| 26 | + protected StraussConfig $config; |
|
| 27 | 27 | |
| 28 | - protected function aaasetUp(): void |
|
| 29 | - { |
|
| 30 | - parent::setUp(); |
|
| 28 | + protected function aaasetUp(): void |
|
| 29 | + { |
|
| 30 | + parent::setUp(); |
|
| 31 | 31 | |
| 32 | - $composerJson = <<<'EOD' |
|
| 32 | + $composerJson = <<<'EOD' |
|
| 33 | 33 | { |
| 34 | 34 | "name": "brianhenryie/strauss-replacer-test", |
| 35 | 35 | "extra": { |
@@ -38,18 +38,18 @@ discard block |
||
| 38 | 38 | } |
| 39 | 39 | EOD; |
| 40 | 40 | |
| 41 | - $composerConfig = new Config(false); |
|
| 42 | - $composerConfig->merge(json_decode($composerJson, true)); |
|
| 43 | - $composer = new Composer(); |
|
| 44 | - $composer->setConfig($composerConfig); |
|
| 41 | + $composerConfig = new Config(false); |
|
| 42 | + $composerConfig->merge(json_decode($composerJson, true)); |
|
| 43 | + $composer = new Composer(); |
|
| 44 | + $composer->setConfig($composerConfig); |
|
| 45 | 45 | |
| 46 | - $this->config = new StraussConfig($composer); |
|
| 47 | - } |
|
| 46 | + $this->config = new StraussConfig($composer); |
|
| 47 | + } |
|
| 48 | 48 | |
| 49 | - public function testNamespaceReplacer() |
|
| 50 | - { |
|
| 49 | + public function testNamespaceReplacer() |
|
| 50 | + { |
|
| 51 | 51 | |
| 52 | - $contents = <<<'EOD' |
|
| 52 | + $contents = <<<'EOD' |
|
| 53 | 53 | <?php |
| 54 | 54 | /* |
| 55 | 55 | * Copyright 2010 Google Inc. |
@@ -122,25 +122,25 @@ discard block |
||
| 122 | 122 | } |
| 123 | 123 | } |
| 124 | 124 | EOD; |
| 125 | - $config = $this->createMock(StraussConfig::class); |
|
| 125 | + $config = $this->createMock(StraussConfig::class); |
|
| 126 | 126 | |
| 127 | - $replacer = new Prefixer($config, __DIR__); |
|
| 127 | + $replacer = new Prefixer($config, __DIR__); |
|
| 128 | 128 | |
| 129 | - $original = "Google\\Http\\Batch"; |
|
| 130 | - $replacement = "BrianHenryIE\\Strauss\\Google\\Http\\Batch"; |
|
| 129 | + $original = "Google\\Http\\Batch"; |
|
| 130 | + $replacement = "BrianHenryIE\\Strauss\\Google\\Http\\Batch"; |
|
| 131 | 131 | |
| 132 | - $result = $replacer->replaceNamespace($contents, $original, $replacement); |
|
| 132 | + $result = $replacer->replaceNamespace($contents, $original, $replacement); |
|
| 133 | 133 | |
| 134 | - $expected = "use BrianHenryIE\\Strauss\\Google\\Http\\Batch;"; |
|
| 134 | + $expected = "use BrianHenryIE\\Strauss\\Google\\Http\\Batch;"; |
|
| 135 | 135 | |
| 136 | - $this->assertStringContainsString($expected, $result); |
|
| 137 | - } |
|
| 136 | + $this->assertStringContainsString($expected, $result); |
|
| 137 | + } |
|
| 138 | 138 | |
| 139 | 139 | |
| 140 | - public function testClassnameReplacer() |
|
| 141 | - { |
|
| 140 | + public function testClassnameReplacer() |
|
| 141 | + { |
|
| 142 | 142 | |
| 143 | - $contents = <<<'EOD' |
|
| 143 | + $contents = <<<'EOD' |
|
| 144 | 144 | <?php |
| 145 | 145 | /******************************************************************************* |
| 146 | 146 | * FPDF * |
@@ -161,263 +161,263 @@ discard block |
||
| 161 | 161 | } |
| 162 | 162 | EOD; |
| 163 | 163 | |
| 164 | - $config = $this->createMock(StraussConfig::class); |
|
| 164 | + $config = $this->createMock(StraussConfig::class); |
|
| 165 | 165 | |
| 166 | - $replacer = new Prefixer($config, __DIR__); |
|
| 166 | + $replacer = new Prefixer($config, __DIR__); |
|
| 167 | 167 | |
| 168 | - $original = "FPDF"; |
|
| 169 | - $classnamePrefix = "BrianHenryIE_Strauss_"; |
|
| 168 | + $original = "FPDF"; |
|
| 169 | + $classnamePrefix = "BrianHenryIE_Strauss_"; |
|
| 170 | 170 | |
| 171 | - $result = $replacer->replaceClassname($contents, $original, $classnamePrefix); |
|
| 171 | + $result = $replacer->replaceClassname($contents, $original, $classnamePrefix); |
|
| 172 | 172 | |
| 173 | - $expected = "class BrianHenryIE_Strauss_FPDF"; |
|
| 173 | + $expected = "class BrianHenryIE_Strauss_FPDF"; |
|
| 174 | 174 | |
| 175 | - $this->assertStringContainsString($expected, $result); |
|
| 176 | - } |
|
| 175 | + $this->assertStringContainsString($expected, $result); |
|
| 176 | + } |
|
| 177 | 177 | |
| 178 | - /** |
|
| 179 | - * PHP 7.4 typed parameters were being prefixed. |
|
| 180 | - */ |
|
| 181 | - public function testTypeFunctionParameter() |
|
| 182 | - { |
|
| 183 | - $this->markTestIncomplete(); |
|
| 184 | - } |
|
| 178 | + /** |
|
| 179 | + * PHP 7.4 typed parameters were being prefixed. |
|
| 180 | + */ |
|
| 181 | + public function testTypeFunctionParameter() |
|
| 182 | + { |
|
| 183 | + $this->markTestIncomplete(); |
|
| 184 | + } |
|
| 185 | 185 | |
| 186 | - /** |
|
| 187 | - * @author CoenJacobs |
|
| 188 | - */ |
|
| 189 | - public function test_it_replaces_class_declarations(): void |
|
| 190 | - { |
|
| 191 | - $contents = 'class Hello_World {'; |
|
| 192 | - $originalClassname = 'Hello_World'; |
|
| 193 | - $classnamePrefix = 'Mozart_'; |
|
| 186 | + /** |
|
| 187 | + * @author CoenJacobs |
|
| 188 | + */ |
|
| 189 | + public function test_it_replaces_class_declarations(): void |
|
| 190 | + { |
|
| 191 | + $contents = 'class Hello_World {'; |
|
| 192 | + $originalClassname = 'Hello_World'; |
|
| 193 | + $classnamePrefix = 'Mozart_'; |
|
| 194 | 194 | |
| 195 | - $config = $this->createMock(StraussConfig::class); |
|
| 195 | + $config = $this->createMock(StraussConfig::class); |
|
| 196 | 196 | |
| 197 | - $replacer = new Prefixer($config, __DIR__); |
|
| 197 | + $replacer = new Prefixer($config, __DIR__); |
|
| 198 | 198 | |
| 199 | - $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
| 199 | + $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
| 200 | 200 | |
| 201 | - $this->assertEquals('class Mozart_Hello_World {', $result); |
|
| 202 | - } |
|
| 201 | + $this->assertEquals('class Mozart_Hello_World {', $result); |
|
| 202 | + } |
|
| 203 | 203 | |
| 204 | - /** |
|
| 205 | - * @author CoenJacobs |
|
| 206 | - */ |
|
| 207 | - public function test_it_replaces_abstract_class_declarations(): void |
|
| 208 | - { |
|
| 209 | - $contents = 'abstract class Hello_World {'; |
|
| 204 | + /** |
|
| 205 | + * @author CoenJacobs |
|
| 206 | + */ |
|
| 207 | + public function test_it_replaces_abstract_class_declarations(): void |
|
| 208 | + { |
|
| 209 | + $contents = 'abstract class Hello_World {'; |
|
| 210 | 210 | |
| 211 | - $originalClassname = 'Hello_World'; |
|
| 212 | - $classnamePrefix = 'Mozart_'; |
|
| 211 | + $originalClassname = 'Hello_World'; |
|
| 212 | + $classnamePrefix = 'Mozart_'; |
|
| 213 | 213 | |
| 214 | - $config = $this->createMock(StraussConfig::class); |
|
| 214 | + $config = $this->createMock(StraussConfig::class); |
|
| 215 | 215 | |
| 216 | - $replacer = new Prefixer($config, __DIR__); |
|
| 216 | + $replacer = new Prefixer($config, __DIR__); |
|
| 217 | 217 | |
| 218 | - $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
| 218 | + $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
| 219 | 219 | |
| 220 | - $this->assertEquals('abstract class Mozart_Hello_World {', $result); |
|
| 221 | - } |
|
| 220 | + $this->assertEquals('abstract class Mozart_Hello_World {', $result); |
|
| 221 | + } |
|
| 222 | 222 | |
| 223 | - /** |
|
| 224 | - * @author CoenJacobs |
|
| 225 | - */ |
|
| 226 | - public function test_it_replaces_interface_class_declarations(): void |
|
| 227 | - { |
|
| 228 | - $contents = 'interface Hello_World {'; |
|
| 223 | + /** |
|
| 224 | + * @author CoenJacobs |
|
| 225 | + */ |
|
| 226 | + public function test_it_replaces_interface_class_declarations(): void |
|
| 227 | + { |
|
| 228 | + $contents = 'interface Hello_World {'; |
|
| 229 | 229 | |
| 230 | - $originalClassname = 'Hello_World'; |
|
| 231 | - $classnamePrefix = 'Mozart_'; |
|
| 230 | + $originalClassname = 'Hello_World'; |
|
| 231 | + $classnamePrefix = 'Mozart_'; |
|
| 232 | 232 | |
| 233 | - $config = $this->createMock(StraussConfig::class); |
|
| 233 | + $config = $this->createMock(StraussConfig::class); |
|
| 234 | 234 | |
| 235 | - $replacer = new Prefixer($config, __DIR__); |
|
| 235 | + $replacer = new Prefixer($config, __DIR__); |
|
| 236 | 236 | |
| 237 | - $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
| 237 | + $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
| 238 | 238 | |
| 239 | - $this->assertEquals('interface Mozart_Hello_World {', $result); |
|
| 240 | - } |
|
| 239 | + $this->assertEquals('interface Mozart_Hello_World {', $result); |
|
| 240 | + } |
|
| 241 | 241 | |
| 242 | - /** |
|
| 243 | - * @author CoenJacobs |
|
| 244 | - */ |
|
| 245 | - public function test_it_replaces_class_declarations_that_extend_other_classes(): void |
|
| 246 | - { |
|
| 247 | - $contents = 'class Hello_World extends Bye_World {'; |
|
| 242 | + /** |
|
| 243 | + * @author CoenJacobs |
|
| 244 | + */ |
|
| 245 | + public function test_it_replaces_class_declarations_that_extend_other_classes(): void |
|
| 246 | + { |
|
| 247 | + $contents = 'class Hello_World extends Bye_World {'; |
|
| 248 | 248 | |
| 249 | - $originalClassname = 'Hello_World'; |
|
| 250 | - $classnamePrefix = 'Mozart_'; |
|
| 249 | + $originalClassname = 'Hello_World'; |
|
| 250 | + $classnamePrefix = 'Mozart_'; |
|
| 251 | 251 | |
| 252 | - $config = $this->createMock(StraussConfig::class); |
|
| 252 | + $config = $this->createMock(StraussConfig::class); |
|
| 253 | 253 | |
| 254 | - $replacer = new Prefixer($config, __DIR__); |
|
| 254 | + $replacer = new Prefixer($config, __DIR__); |
|
| 255 | 255 | |
| 256 | - $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
| 256 | + $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
| 257 | 257 | |
| 258 | - $this->assertEquals('class Mozart_Hello_World extends Bye_World {', $result); |
|
| 259 | - } |
|
| 258 | + $this->assertEquals('class Mozart_Hello_World extends Bye_World {', $result); |
|
| 259 | + } |
|
| 260 | 260 | |
| 261 | - /** |
|
| 262 | - * @author CoenJacobs |
|
| 263 | - */ |
|
| 264 | - public function test_it_replaces_class_declarations_that_implement_interfaces(): void |
|
| 265 | - { |
|
| 266 | - $contents = 'class Hello_World implements Bye_World {'; |
|
| 261 | + /** |
|
| 262 | + * @author CoenJacobs |
|
| 263 | + */ |
|
| 264 | + public function test_it_replaces_class_declarations_that_implement_interfaces(): void |
|
| 265 | + { |
|
| 266 | + $contents = 'class Hello_World implements Bye_World {'; |
|
| 267 | 267 | |
| 268 | - $originalClassname = 'Hello_World'; |
|
| 269 | - $classnamePrefix = 'Mozart_'; |
|
| 268 | + $originalClassname = 'Hello_World'; |
|
| 269 | + $classnamePrefix = 'Mozart_'; |
|
| 270 | 270 | |
| 271 | - $config = $this->createMock(StraussConfig::class); |
|
| 271 | + $config = $this->createMock(StraussConfig::class); |
|
| 272 | 272 | |
| 273 | - $replacer = new Prefixer($config, __DIR__); |
|
| 273 | + $replacer = new Prefixer($config, __DIR__); |
|
| 274 | 274 | |
| 275 | - $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
| 275 | + $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
| 276 | 276 | |
| 277 | - $this->assertEquals('class Mozart_Hello_World implements Bye_World {', $result); |
|
| 278 | - } |
|
| 277 | + $this->assertEquals('class Mozart_Hello_World implements Bye_World {', $result); |
|
| 278 | + } |
|
| 279 | 279 | |
| 280 | 280 | |
| 281 | - /** |
|
| 282 | - * @author BrianHenryIE |
|
| 283 | - */ |
|
| 284 | - public function testItReplacesNamespacesInInterface(): void |
|
| 285 | - { |
|
| 286 | - $contents = 'class Hello_World implements \Strauss\Bye_World {'; |
|
| 281 | + /** |
|
| 282 | + * @author BrianHenryIE |
|
| 283 | + */ |
|
| 284 | + public function testItReplacesNamespacesInInterface(): void |
|
| 285 | + { |
|
| 286 | + $contents = 'class Hello_World implements \Strauss\Bye_World {'; |
|
| 287 | 287 | |
| 288 | - $originalNamespace = 'Strauss'; |
|
| 289 | - $replacement = 'Prefix\Strauss'; |
|
| 288 | + $originalNamespace = 'Strauss'; |
|
| 289 | + $replacement = 'Prefix\Strauss'; |
|
| 290 | 290 | |
| 291 | - $config = $this->createMock(StraussConfig::class); |
|
| 291 | + $config = $this->createMock(StraussConfig::class); |
|
| 292 | 292 | |
| 293 | - $replacer = new Prefixer($config, __DIR__); |
|
| 293 | + $replacer = new Prefixer($config, __DIR__); |
|
| 294 | 294 | |
| 295 | - $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
| 295 | + $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
| 296 | 296 | |
| 297 | - $this->assertEquals('class Hello_World implements \Prefix\Strauss\Bye_World {', $result); |
|
| 298 | - } |
|
| 297 | + $this->assertEquals('class Hello_World implements \Prefix\Strauss\Bye_World {', $result); |
|
| 298 | + } |
|
| 299 | 299 | |
| 300 | - /** |
|
| 301 | - * @author CoenJacobs |
|
| 302 | - */ |
|
| 303 | - public function test_it_stores_replaced_class_names(): void |
|
| 304 | - { |
|
| 305 | - $this->markTestIncomplete('TODO Delete/move'); |
|
| 306 | - |
|
| 307 | - $contents = 'class Hello_World {'; |
|
| 308 | - $replacer = new Prefixer($config, __DIR__); |
|
| 309 | - $replacer->setClassmapPrefix('Mozart_'); |
|
| 310 | - $replacer->replace($contents); |
|
| 311 | - $this->assertArrayHasKey('Hello_World', $replacer->getReplacedClasses()); |
|
| 312 | - } |
|
| 300 | + /** |
|
| 301 | + * @author CoenJacobs |
|
| 302 | + */ |
|
| 303 | + public function test_it_stores_replaced_class_names(): void |
|
| 304 | + { |
|
| 305 | + $this->markTestIncomplete('TODO Delete/move'); |
|
| 313 | 306 | |
| 314 | - /** |
|
| 315 | - * @author https://github.com/stephenharris |
|
| 316 | - * @see https://github.com/coenjacobs/mozart/commit/fd7906943396c9a17110d1bfaf9d778f3b1f322a#diff-87828794e62b55ce8d7263e3ab1a918d1370e283ac750cd44e3ac61db5daee54 |
|
| 317 | - */ |
|
| 318 | - public function test_it_replaces_class_declarations_psr2(): void |
|
| 319 | - { |
|
| 320 | - $contents = "class Hello_World\n{"; |
|
| 307 | + $contents = 'class Hello_World {'; |
|
| 308 | + $replacer = new Prefixer($config, __DIR__); |
|
| 309 | + $replacer->setClassmapPrefix('Mozart_'); |
|
| 310 | + $replacer->replace($contents); |
|
| 311 | + $this->assertArrayHasKey('Hello_World', $replacer->getReplacedClasses()); |
|
| 312 | + } |
|
| 321 | 313 | |
| 322 | - $originalClassname = 'Hello_World'; |
|
| 323 | - $classnamePrefix = 'Mozart_'; |
|
| 314 | + /** |
|
| 315 | + * @author https://github.com/stephenharris |
|
| 316 | + * @see https://github.com/coenjacobs/mozart/commit/fd7906943396c9a17110d1bfaf9d778f3b1f322a#diff-87828794e62b55ce8d7263e3ab1a918d1370e283ac750cd44e3ac61db5daee54 |
|
| 317 | + */ |
|
| 318 | + public function test_it_replaces_class_declarations_psr2(): void |
|
| 319 | + { |
|
| 320 | + $contents = "class Hello_World\n{"; |
|
| 324 | 321 | |
| 325 | - $config = $this->createMock(StraussConfig::class); |
|
| 322 | + $originalClassname = 'Hello_World'; |
|
| 323 | + $classnamePrefix = 'Mozart_'; |
|
| 326 | 324 | |
| 327 | - $replacer = new Prefixer($config, __DIR__); |
|
| 325 | + $config = $this->createMock(StraussConfig::class); |
|
| 328 | 326 | |
| 329 | - $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
| 327 | + $replacer = new Prefixer($config, __DIR__); |
|
| 330 | 328 | |
| 331 | - $this->assertEquals("class Mozart_Hello_World\n{", $result); |
|
| 332 | - } |
|
| 329 | + $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
| 333 | 330 | |
| 334 | - /** |
|
| 335 | - * @see https://github.com/coenjacobs/mozart/issues/81 |
|
| 336 | - * @author BrianHenryIE |
|
| 337 | - * |
|
| 338 | - */ |
|
| 339 | - public function test_it_replaces_class(): void |
|
| 340 | - { |
|
| 341 | - $contents = "class Hello_World {"; |
|
| 331 | + $this->assertEquals("class Mozart_Hello_World\n{", $result); |
|
| 332 | + } |
|
| 342 | 333 | |
| 343 | - $originalClassname = 'Hello_World'; |
|
| 344 | - $classnamePrefix = 'Mozart_'; |
|
| 334 | + /** |
|
| 335 | + * @see https://github.com/coenjacobs/mozart/issues/81 |
|
| 336 | + * @author BrianHenryIE |
|
| 337 | + * |
|
| 338 | + */ |
|
| 339 | + public function test_it_replaces_class(): void |
|
| 340 | + { |
|
| 341 | + $contents = "class Hello_World {"; |
|
| 345 | 342 | |
| 346 | - $config = $this->createMock(StraussConfig::class); |
|
| 343 | + $originalClassname = 'Hello_World'; |
|
| 344 | + $classnamePrefix = 'Mozart_'; |
|
| 347 | 345 | |
| 348 | - $replacer = new Prefixer($config, __DIR__); |
|
| 346 | + $config = $this->createMock(StraussConfig::class); |
|
| 349 | 347 | |
| 350 | - $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
| 348 | + $replacer = new Prefixer($config, __DIR__); |
|
| 351 | 349 | |
| 352 | - $this->assertEquals("class Mozart_Hello_World {", $result); |
|
| 353 | - } |
|
| 350 | + $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
| 354 | 351 | |
| 352 | + $this->assertEquals("class Mozart_Hello_World {", $result); |
|
| 353 | + } |
|
| 355 | 354 | |
| 356 | - /** |
|
| 357 | - * @see ClassmapReplacerIntegrationTest::test_it_does_not_make_classname_replacement_inside_namespaced_file() |
|
| 358 | - * @see https://github.com/coenjacobs/mozart/issues/93 |
|
| 359 | - * |
|
| 360 | - * @author BrianHenryIE |
|
| 361 | - * |
|
| 362 | - * @test |
|
| 363 | - */ |
|
| 364 | - public function it_does_not_replace_inside_namespace_multiline(): void |
|
| 365 | - { |
|
| 366 | - $contents = " |
|
| 355 | + |
|
| 356 | + /** |
|
| 357 | + * @see ClassmapReplacerIntegrationTest::test_it_does_not_make_classname_replacement_inside_namespaced_file() |
|
| 358 | + * @see https://github.com/coenjacobs/mozart/issues/93 |
|
| 359 | + * |
|
| 360 | + * @author BrianHenryIE |
|
| 361 | + * |
|
| 362 | + * @test |
|
| 363 | + */ |
|
| 364 | + public function it_does_not_replace_inside_namespace_multiline(): void |
|
| 365 | + { |
|
| 366 | + $contents = " |
|
| 367 | 367 | namespace Mozart; |
| 368 | 368 | class Hello_World |
| 369 | 369 | "; |
| 370 | 370 | |
| 371 | - $originalClassname = 'Hello_World'; |
|
| 372 | - $classnamePrefix = 'Mozart_'; |
|
| 371 | + $originalClassname = 'Hello_World'; |
|
| 372 | + $classnamePrefix = 'Mozart_'; |
|
| 373 | 373 | |
| 374 | - $config = $this->createMock(StraussConfig::class); |
|
| 375 | - $config->method("getClassmapPrefix")->willReturn($classnamePrefix); |
|
| 374 | + $config = $this->createMock(StraussConfig::class); |
|
| 375 | + $config->method("getClassmapPrefix")->willReturn($classnamePrefix); |
|
| 376 | 376 | |
| 377 | - $replacer = new Prefixer($config, __DIR__); |
|
| 377 | + $replacer = new Prefixer($config, __DIR__); |
|
| 378 | 378 | |
| 379 | - $result = $replacer->replaceInString([$originalClassname], [], $contents); |
|
| 379 | + $result = $replacer->replaceInString([$originalClassname], [], $contents); |
|
| 380 | 380 | |
| 381 | - $this->assertEquals($contents, $result); |
|
| 382 | - } |
|
| 381 | + $this->assertEquals($contents, $result); |
|
| 382 | + } |
|
| 383 | 383 | |
| 384 | - /** |
|
| 385 | - * @see ClassmapReplacerIntegrationTest::test_it_does_not_make_classname_replacement_inside_namespaced_file() |
|
| 386 | - * @see https://github.com/coenjacobs/mozart/issues/93 |
|
| 387 | - * |
|
| 388 | - * @author BrianHenryIE |
|
| 389 | - */ |
|
| 390 | - public function test_it_does_not_replace_inside_namespace_singleline(): void |
|
| 391 | - { |
|
| 392 | - $contents = "namespace Mozart; class Hello_World"; |
|
| 384 | + /** |
|
| 385 | + * @see ClassmapReplacerIntegrationTest::test_it_does_not_make_classname_replacement_inside_namespaced_file() |
|
| 386 | + * @see https://github.com/coenjacobs/mozart/issues/93 |
|
| 387 | + * |
|
| 388 | + * @author BrianHenryIE |
|
| 389 | + */ |
|
| 390 | + public function test_it_does_not_replace_inside_namespace_singleline(): void |
|
| 391 | + { |
|
| 392 | + $contents = "namespace Mozart; class Hello_World"; |
|
| 393 | 393 | |
| 394 | - $originalClassname = 'Hello_World'; |
|
| 395 | - $classnamePrefix = 'Mozart_'; |
|
| 394 | + $originalClassname = 'Hello_World'; |
|
| 395 | + $classnamePrefix = 'Mozart_'; |
|
| 396 | 396 | |
| 397 | - $config = $this->createMock(StraussConfig::class); |
|
| 397 | + $config = $this->createMock(StraussConfig::class); |
|
| 398 | 398 | |
| 399 | - $replacer = new Prefixer($config, __DIR__); |
|
| 399 | + $replacer = new Prefixer($config, __DIR__); |
|
| 400 | 400 | |
| 401 | - $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
| 401 | + $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
| 402 | 402 | |
| 403 | - $this->assertEquals($contents, $result); |
|
| 404 | - } |
|
| 403 | + $this->assertEquals($contents, $result); |
|
| 404 | + } |
|
| 405 | 405 | |
| 406 | 406 | |
| 407 | 407 | |
| 408 | - /** |
|
| 409 | - * It's possible to have multiple namespaces inside one file. |
|
| 410 | - * |
|
| 411 | - * To have two classes in one file, one in a namespace and the other not, the global namespace needs to be explicit. |
|
| 412 | - * |
|
| 413 | - * @author BrianHenryIE |
|
| 414 | - * |
|
| 415 | - * @test |
|
| 416 | - */ |
|
| 417 | - public function it_does_not_replace_inside_named_namespace_but_does_inside_explicit_global_namespace_b(): void |
|
| 418 | - { |
|
| 408 | + /** |
|
| 409 | + * It's possible to have multiple namespaces inside one file. |
|
| 410 | + * |
|
| 411 | + * To have two classes in one file, one in a namespace and the other not, the global namespace needs to be explicit. |
|
| 412 | + * |
|
| 413 | + * @author BrianHenryIE |
|
| 414 | + * |
|
| 415 | + * @test |
|
| 416 | + */ |
|
| 417 | + public function it_does_not_replace_inside_named_namespace_but_does_inside_explicit_global_namespace_b(): void |
|
| 418 | + { |
|
| 419 | 419 | |
| 420 | - $contents = " |
|
| 420 | + $contents = " |
|
| 421 | 421 | namespace My_Project { |
| 422 | 422 | class A_Class { } |
| 423 | 423 | } |
@@ -426,389 +426,389 @@ discard block |
||
| 426 | 426 | } |
| 427 | 427 | "; |
| 428 | 428 | |
| 429 | - $classnamePrefix = 'Mozart_'; |
|
| 429 | + $classnamePrefix = 'Mozart_'; |
|
| 430 | 430 | |
| 431 | - $config = $this->createMock(StraussConfig::class); |
|
| 431 | + $config = $this->createMock(StraussConfig::class); |
|
| 432 | 432 | |
| 433 | - $replacer = new Prefixer($config, __DIR__); |
|
| 433 | + $replacer = new Prefixer($config, __DIR__); |
|
| 434 | 434 | |
| 435 | - $result = $replacer->replaceClassname($contents, 'B_Class', $classnamePrefix); |
|
| 435 | + $result = $replacer->replaceClassname($contents, 'B_Class', $classnamePrefix); |
|
| 436 | 436 | |
| 437 | - $this->assertStringContainsString('Mozart_B_Class', $result); |
|
| 438 | - } |
|
| 437 | + $this->assertStringContainsString('Mozart_B_Class', $result); |
|
| 438 | + } |
|
| 439 | 439 | |
| 440 | - /** @test */ |
|
| 441 | - public function it_replaces_namespace_declarations(): void |
|
| 442 | - { |
|
| 443 | - $contents = 'namespace Test\\Test;'; |
|
| 440 | + /** @test */ |
|
| 441 | + public function it_replaces_namespace_declarations(): void |
|
| 442 | + { |
|
| 443 | + $contents = 'namespace Test\\Test;'; |
|
| 444 | 444 | |
| 445 | - $namespace = "Test\\Test"; |
|
| 446 | - $replacement = "My\\Mozart\\Prefix\\Test\\Test"; |
|
| 445 | + $namespace = "Test\\Test"; |
|
| 446 | + $replacement = "My\\Mozart\\Prefix\\Test\\Test"; |
|
| 447 | 447 | |
| 448 | - $config = $this->createMock(StraussConfig::class); |
|
| 448 | + $config = $this->createMock(StraussConfig::class); |
|
| 449 | 449 | |
| 450 | - $replacer = new Prefixer($config, __DIR__); |
|
| 450 | + $replacer = new Prefixer($config, __DIR__); |
|
| 451 | 451 | |
| 452 | - $result = $replacer->replaceNamespace($contents, $namespace, $replacement); |
|
| 452 | + $result = $replacer->replaceNamespace($contents, $namespace, $replacement); |
|
| 453 | 453 | |
| 454 | - $this->assertEquals('namespace My\\Mozart\\Prefix\\Test\\Test;', $result); |
|
| 455 | - } |
|
| 454 | + $this->assertEquals('namespace My\\Mozart\\Prefix\\Test\\Test;', $result); |
|
| 455 | + } |
|
| 456 | 456 | |
| 457 | 457 | |
| 458 | - /** |
|
| 459 | - * This test doesn't seem to match its name. |
|
| 460 | - */ |
|
| 461 | - public function test_it_doesnt_replaces_namespace_inside_namespace(): void |
|
| 462 | - { |
|
| 463 | - $contents = "namespace Test\\Something;\n\nuse Test\\Test;"; |
|
| 458 | + /** |
|
| 459 | + * This test doesn't seem to match its name. |
|
| 460 | + */ |
|
| 461 | + public function test_it_doesnt_replaces_namespace_inside_namespace(): void |
|
| 462 | + { |
|
| 463 | + $contents = "namespace Test\\Something;\n\nuse Test\\Test;"; |
|
| 464 | 464 | |
| 465 | - $config = $this->createMock(StraussConfig::class); |
|
| 465 | + $config = $this->createMock(StraussConfig::class); |
|
| 466 | 466 | |
| 467 | - $replacer = new Prefixer($config, __DIR__); |
|
| 468 | - $result = $replacer->replaceNamespace($contents, "Test\\Something", "My\\Mozart\\Prefix\\Test\\Something"); |
|
| 469 | - $result = $replacer->replaceNamespace($result, "Test\\Test", "My\\Mozart\\Prefix\\Test\\Test"); |
|
| 467 | + $replacer = new Prefixer($config, __DIR__); |
|
| 468 | + $result = $replacer->replaceNamespace($contents, "Test\\Something", "My\\Mozart\\Prefix\\Test\\Something"); |
|
| 469 | + $result = $replacer->replaceNamespace($result, "Test\\Test", "My\\Mozart\\Prefix\\Test\\Test"); |
|
| 470 | 470 | |
| 471 | - $this->assertEquals("namespace My\\Mozart\\Prefix\\Test\\Something;\n\nuse My\\Mozart\\Prefix\\Test\\Test;", $result); |
|
| 472 | - } |
|
| 471 | + $this->assertEquals("namespace My\\Mozart\\Prefix\\Test\\Something;\n\nuse My\\Mozart\\Prefix\\Test\\Test;", $result); |
|
| 472 | + } |
|
| 473 | 473 | |
| 474 | - /** |
|
| 475 | - * |
|
| 476 | - */ |
|
| 477 | - public function test_it_does_notreplaces_partial_namespace_declarations(): void |
|
| 478 | - { |
|
| 479 | - $contents = 'namespace Test\\Test\\Another;'; |
|
| 474 | + /** |
|
| 475 | + * |
|
| 476 | + */ |
|
| 477 | + public function test_it_does_notreplaces_partial_namespace_declarations(): void |
|
| 478 | + { |
|
| 479 | + $contents = 'namespace Test\\Test\\Another;'; |
|
| 480 | 480 | |
| 481 | - $namespace = "Test\\Another"; |
|
| 482 | - $prefix = "My\\Mozart\\Prefix"; |
|
| 481 | + $namespace = "Test\\Another"; |
|
| 482 | + $prefix = "My\\Mozart\\Prefix"; |
|
| 483 | 483 | |
| 484 | - $config = $this->createMock(StraussConfig::class); |
|
| 484 | + $config = $this->createMock(StraussConfig::class); |
|
| 485 | 485 | |
| 486 | - $replacer = new Prefixer($config, __DIR__); |
|
| 487 | - $result = $replacer->replaceNamespace($contents, $namespace, $prefix); |
|
| 486 | + $replacer = new Prefixer($config, __DIR__); |
|
| 487 | + $result = $replacer->replaceNamespace($contents, $namespace, $prefix); |
|
| 488 | 488 | |
| 489 | - $this->assertEquals('namespace Test\\Test\\Another;', $result); |
|
| 490 | - } |
|
| 489 | + $this->assertEquals('namespace Test\\Test\\Another;', $result); |
|
| 490 | + } |
|
| 491 | 491 | |
| 492 | 492 | |
| 493 | - public function test_it_doesnt_prefix_already_prefixed_namespace(): void |
|
| 494 | - { |
|
| 493 | + public function test_it_doesnt_prefix_already_prefixed_namespace(): void |
|
| 494 | + { |
|
| 495 | 495 | |
| 496 | - $contents = 'namespace My\\Mozart\\Prefix\\Test\\Another;'; |
|
| 496 | + $contents = 'namespace My\\Mozart\\Prefix\\Test\\Another;'; |
|
| 497 | 497 | |
| 498 | - $namespace = "Test\\Another"; |
|
| 499 | - $prefix = "My\\Mozart\\Prefix"; |
|
| 498 | + $namespace = "Test\\Another"; |
|
| 499 | + $prefix = "My\\Mozart\\Prefix"; |
|
| 500 | 500 | |
| 501 | - $config = $this->createMock(StraussConfig::class); |
|
| 501 | + $config = $this->createMock(StraussConfig::class); |
|
| 502 | 502 | |
| 503 | - $replacer = new Prefixer($config, __DIR__); |
|
| 504 | - $result = $replacer->replaceNamespace($contents, $namespace, $prefix); |
|
| 503 | + $replacer = new Prefixer($config, __DIR__); |
|
| 504 | + $result = $replacer->replaceNamespace($contents, $namespace, $prefix); |
|
| 505 | 505 | |
| 506 | - $this->assertEquals('namespace My\\Mozart\\Prefix\\Test\\Another;', $result); |
|
| 507 | - } |
|
| 506 | + $this->assertEquals('namespace My\\Mozart\\Prefix\\Test\\Another;', $result); |
|
| 507 | + } |
|
| 508 | 508 | |
| 509 | - /** |
|
| 510 | - * @author markjaquith |
|
| 511 | - */ |
|
| 512 | - public function test_it_doesnt_double_replace_namespaces_that_also_exist_inside_another_namespace(): void |
|
| 513 | - { |
|
| 509 | + /** |
|
| 510 | + * @author markjaquith |
|
| 511 | + */ |
|
| 512 | + public function test_it_doesnt_double_replace_namespaces_that_also_exist_inside_another_namespace(): void |
|
| 513 | + { |
|
| 514 | 514 | |
| 515 | - // This is a tricky situation. We are referencing Chicken\Egg, |
|
| 516 | - // but Egg *also* exists as a separate top level class. |
|
| 517 | - $contents = 'use Chicken\\Egg;'; |
|
| 518 | - $expected = 'use My\\Mozart\\Prefix\\Chicken\\Egg;'; |
|
| 515 | + // This is a tricky situation. We are referencing Chicken\Egg, |
|
| 516 | + // but Egg *also* exists as a separate top level class. |
|
| 517 | + $contents = 'use Chicken\\Egg;'; |
|
| 518 | + $expected = 'use My\\Mozart\\Prefix\\Chicken\\Egg;'; |
|
| 519 | 519 | |
| 520 | - $config = $this->createMock(StraussConfig::class); |
|
| 520 | + $config = $this->createMock(StraussConfig::class); |
|
| 521 | 521 | |
| 522 | - $replacer = new Prefixer($config, __DIR__); |
|
| 522 | + $replacer = new Prefixer($config, __DIR__); |
|
| 523 | 523 | |
| 524 | - $result = $replacer->replaceNamespace($contents, 'Chicken', 'My\\Mozart\\Prefix\\Chicken'); |
|
| 525 | - $result = $replacer->replaceNamespace($result, 'Egg', 'My\\Mozart\\Prefix\\Egg'); |
|
| 524 | + $result = $replacer->replaceNamespace($contents, 'Chicken', 'My\\Mozart\\Prefix\\Chicken'); |
|
| 525 | + $result = $replacer->replaceNamespace($result, 'Egg', 'My\\Mozart\\Prefix\\Egg'); |
|
| 526 | 526 | |
| 527 | - $this->assertEquals($expected, $result); |
|
| 528 | - } |
|
| 527 | + $this->assertEquals($expected, $result); |
|
| 528 | + } |
|
| 529 | 529 | |
| 530 | - /** |
|
| 531 | - * @see https://github.com/coenjacobs/mozart/issues/75 |
|
| 532 | - * |
|
| 533 | - * @test |
|
| 534 | - */ |
|
| 535 | - public function it_replaces_namespace_use_as_declarations(): void |
|
| 536 | - { |
|
| 537 | - $namespace = 'Symfony\Polyfill\Mbstring'; |
|
| 538 | - $replacement = "MBViews\Dependencies\Symfony\Polyfill\Mbstring"; |
|
| 530 | + /** |
|
| 531 | + * @see https://github.com/coenjacobs/mozart/issues/75 |
|
| 532 | + * |
|
| 533 | + * @test |
|
| 534 | + */ |
|
| 535 | + public function it_replaces_namespace_use_as_declarations(): void |
|
| 536 | + { |
|
| 537 | + $namespace = 'Symfony\Polyfill\Mbstring'; |
|
| 538 | + $replacement = "MBViews\Dependencies\Symfony\Polyfill\Mbstring"; |
|
| 539 | 539 | |
| 540 | - $contents = "use Symfony\Polyfill\Mbstring as p;"; |
|
| 540 | + $contents = "use Symfony\Polyfill\Mbstring as p;"; |
|
| 541 | 541 | |
| 542 | - $config = $this->createMock(StraussConfig::class); |
|
| 542 | + $config = $this->createMock(StraussConfig::class); |
|
| 543 | 543 | |
| 544 | - $replacer = new Prefixer($config, __DIR__); |
|
| 545 | - $result = $replacer->replaceNamespace($contents, $namespace, $replacement); |
|
| 544 | + $replacer = new Prefixer($config, __DIR__); |
|
| 545 | + $result = $replacer->replaceNamespace($contents, $namespace, $replacement); |
|
| 546 | 546 | |
| 547 | - $expected = "use MBViews\Dependencies\Symfony\Polyfill\Mbstring as p;"; |
|
| 547 | + $expected = "use MBViews\Dependencies\Symfony\Polyfill\Mbstring as p;"; |
|
| 548 | 548 | |
| 549 | - $this->assertEquals($expected, $result); |
|
| 550 | - } |
|
| 549 | + $this->assertEquals($expected, $result); |
|
| 550 | + } |
|
| 551 | 551 | |
| 552 | - /** |
|
| 553 | - * @author BrianHenryIE |
|
| 554 | - */ |
|
| 555 | - public function test_it_doesnt_prefix_function_types_that_happen_to_match_the_namespace() |
|
| 556 | - { |
|
| 557 | - $namespace = 'Mpdf'; |
|
| 558 | - $prefix = "Mozart"; |
|
| 559 | - $contents = 'public function getServices( Mpdf $mpdf, LoggerInterface $logger, $config, )'; |
|
| 552 | + /** |
|
| 553 | + * @author BrianHenryIE |
|
| 554 | + */ |
|
| 555 | + public function test_it_doesnt_prefix_function_types_that_happen_to_match_the_namespace() |
|
| 556 | + { |
|
| 557 | + $namespace = 'Mpdf'; |
|
| 558 | + $prefix = "Mozart"; |
|
| 559 | + $contents = 'public function getServices( Mpdf $mpdf, LoggerInterface $logger, $config, )'; |
|
| 560 | 560 | |
| 561 | - $config = $this->createMock(StraussConfig::class); |
|
| 561 | + $config = $this->createMock(StraussConfig::class); |
|
| 562 | 562 | |
| 563 | - $replacer = new Prefixer($config, __DIR__); |
|
| 564 | - $result = $replacer->replaceNamespace($contents, $namespace, $prefix); |
|
| 563 | + $replacer = new Prefixer($config, __DIR__); |
|
| 564 | + $result = $replacer->replaceNamespace($contents, $namespace, $prefix); |
|
| 565 | 565 | |
| 566 | - $expected = 'public function getServices( Mpdf $mpdf, LoggerInterface $logger, $config, )'; |
|
| 566 | + $expected = 'public function getServices( Mpdf $mpdf, LoggerInterface $logger, $config, )'; |
|
| 567 | 567 | |
| 568 | - $this->assertEquals($expected, $result); |
|
| 569 | - } |
|
| 568 | + $this->assertEquals($expected, $result); |
|
| 569 | + } |
|
| 570 | 570 | |
| 571 | - public function testLeadingSlashInString() |
|
| 572 | - { |
|
| 573 | - $originalNamespace = "Strauss\\Test"; |
|
| 574 | - $replacement = "Prefix\\Strauss\\Test"; |
|
| 575 | - $contents = '$mentionedClass = "\\Strauss\\Test\\Classname";'; |
|
| 571 | + public function testLeadingSlashInString() |
|
| 572 | + { |
|
| 573 | + $originalNamespace = "Strauss\\Test"; |
|
| 574 | + $replacement = "Prefix\\Strauss\\Test"; |
|
| 575 | + $contents = '$mentionedClass = "\\Strauss\\Test\\Classname";'; |
|
| 576 | 576 | |
| 577 | - $config = $this->createMock(StraussConfig::class); |
|
| 577 | + $config = $this->createMock(StraussConfig::class); |
|
| 578 | 578 | |
| 579 | - $replacer = new Prefixer($config, __DIR__); |
|
| 580 | - $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
| 579 | + $replacer = new Prefixer($config, __DIR__); |
|
| 580 | + $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
| 581 | 581 | |
| 582 | - $expected = '$mentionedClass = "\\Prefix\\Strauss\\Test\\Classname";'; |
|
| 582 | + $expected = '$mentionedClass = "\\Prefix\\Strauss\\Test\\Classname";'; |
|
| 583 | 583 | |
| 584 | - $this->assertEquals($expected, $result); |
|
| 585 | - } |
|
| 584 | + $this->assertEquals($expected, $result); |
|
| 585 | + } |
|
| 586 | 586 | |
| 587 | - public function testDoubleLeadingSlashInString() |
|
| 588 | - { |
|
| 589 | - $originalNamespace = "Strauss\\Test"; |
|
| 590 | - $replacement = "Prefix\\Strauss\\Test"; |
|
| 591 | - $contents = '$mentionedClass = "\\\\Strauss\\\\Test\\\\Classname";'; |
|
| 587 | + public function testDoubleLeadingSlashInString() |
|
| 588 | + { |
|
| 589 | + $originalNamespace = "Strauss\\Test"; |
|
| 590 | + $replacement = "Prefix\\Strauss\\Test"; |
|
| 591 | + $contents = '$mentionedClass = "\\\\Strauss\\\\Test\\\\Classname";'; |
|
| 592 | 592 | |
| 593 | - $config = $this->createMock(StraussConfig::class); |
|
| 593 | + $config = $this->createMock(StraussConfig::class); |
|
| 594 | 594 | |
| 595 | - $replacer = new Prefixer($config, __DIR__); |
|
| 596 | - $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
| 595 | + $replacer = new Prefixer($config, __DIR__); |
|
| 596 | + $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
| 597 | 597 | |
| 598 | - $expected = '$mentionedClass = "\\\\Prefix\\\\Strauss\\\\Test\\\\Classname";'; |
|
| 598 | + $expected = '$mentionedClass = "\\\\Prefix\\\\Strauss\\\\Test\\\\Classname";'; |
|
| 599 | 599 | |
| 600 | - $this->assertEquals($expected, $result); |
|
| 601 | - } |
|
| 600 | + $this->assertEquals($expected, $result); |
|
| 601 | + } |
|
| 602 | 602 | |
| 603 | - public function testItReplacesSlashedNamespaceInFunctionParameter() |
|
| 604 | - { |
|
| 603 | + public function testItReplacesSlashedNamespaceInFunctionParameter() |
|
| 604 | + { |
|
| 605 | 605 | |
| 606 | - $originalNamespace = "net\\authorize\\api\\contract\\v1"; |
|
| 607 | - $replacement = "Prefix\\net\\authorize\\api\\contract\\v1"; |
|
| 608 | - $contents = "public function __construct(\\net\\authorize\\api\\contract\\v1\\AnetApiRequestType \$request, \$responseType)"; |
|
| 606 | + $originalNamespace = "net\\authorize\\api\\contract\\v1"; |
|
| 607 | + $replacement = "Prefix\\net\\authorize\\api\\contract\\v1"; |
|
| 608 | + $contents = "public function __construct(\\net\\authorize\\api\\contract\\v1\\AnetApiRequestType \$request, \$responseType)"; |
|
| 609 | 609 | |
| 610 | - $config = $this->createMock(StraussConfig::class); |
|
| 610 | + $config = $this->createMock(StraussConfig::class); |
|
| 611 | 611 | |
| 612 | - $replacer = new Prefixer($config, __DIR__); |
|
| 613 | - $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
| 612 | + $replacer = new Prefixer($config, __DIR__); |
|
| 613 | + $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
| 614 | 614 | |
| 615 | - $expected = "public function __construct(\\Prefix\\net\\authorize\\api\\contract\\v1\\AnetApiRequestType \$request, \$responseType)"; |
|
| 615 | + $expected = "public function __construct(\\Prefix\\net\\authorize\\api\\contract\\v1\\AnetApiRequestType \$request, \$responseType)"; |
|
| 616 | 616 | |
| 617 | - $this->assertEquals($expected, $result); |
|
| 618 | - } |
|
| 617 | + $this->assertEquals($expected, $result); |
|
| 618 | + } |
|
| 619 | 619 | |
| 620 | 620 | |
| 621 | - public function testItReplacesNamespaceInFunctionParameterDefaultAgumentValue() |
|
| 622 | - { |
|
| 621 | + public function testItReplacesNamespaceInFunctionParameterDefaultAgumentValue() |
|
| 622 | + { |
|
| 623 | 623 | |
| 624 | - $originalNamespace = "net\\authorize\\api\constants"; |
|
| 625 | - $replacement = "Prefix\\net\\authorize\\api\constants"; |
|
| 626 | - $contents = "public function executeWithApiResponse(\$endPoint = \\net\\authorize\\api\\constants\\ANetEnvironment::CUSTOM)"; |
|
| 624 | + $originalNamespace = "net\\authorize\\api\constants"; |
|
| 625 | + $replacement = "Prefix\\net\\authorize\\api\constants"; |
|
| 626 | + $contents = "public function executeWithApiResponse(\$endPoint = \\net\\authorize\\api\\constants\\ANetEnvironment::CUSTOM)"; |
|
| 627 | 627 | |
| 628 | - $config = $this->createMock(StraussConfig::class); |
|
| 628 | + $config = $this->createMock(StraussConfig::class); |
|
| 629 | 629 | |
| 630 | - $replacer = new Prefixer($config, __DIR__); |
|
| 631 | - $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
| 630 | + $replacer = new Prefixer($config, __DIR__); |
|
| 631 | + $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
| 632 | 632 | |
| 633 | - $expected = "public function executeWithApiResponse(\$endPoint = \\Prefix\\net\\authorize\\api\\constants\\ANetEnvironment::CUSTOM)"; |
|
| 633 | + $expected = "public function executeWithApiResponse(\$endPoint = \\Prefix\\net\\authorize\\api\\constants\\ANetEnvironment::CUSTOM)"; |
|
| 634 | 634 | |
| 635 | - $this->assertEquals($expected, $result); |
|
| 636 | - } |
|
| 635 | + $this->assertEquals($expected, $result); |
|
| 636 | + } |
|
| 637 | 637 | |
| 638 | 638 | |
| 639 | - public function testItReplacesNamespaceConcatenatedStringConst() |
|
| 640 | - { |
|
| 639 | + public function testItReplacesNamespaceConcatenatedStringConst() |
|
| 640 | + { |
|
| 641 | 641 | |
| 642 | - $originalNamespace = "net\\authorize\\api\\constants"; |
|
| 643 | - $replacement = "Prefix\\net\\authorize\\api\\constants"; |
|
| 644 | - $contents = "\$this->apiRequest->setClientId(\"sdk-php-\" . \\net\\authorize\\api\\constants\\ANetEnvironment::VERSION);"; |
|
| 642 | + $originalNamespace = "net\\authorize\\api\\constants"; |
|
| 643 | + $replacement = "Prefix\\net\\authorize\\api\\constants"; |
|
| 644 | + $contents = "\$this->apiRequest->setClientId(\"sdk-php-\" . \\net\\authorize\\api\\constants\\ANetEnvironment::VERSION);"; |
|
| 645 | 645 | |
| 646 | - $config = $this->createMock(StraussConfig::class); |
|
| 646 | + $config = $this->createMock(StraussConfig::class); |
|
| 647 | 647 | |
| 648 | - $replacer = new Prefixer($config, __DIR__); |
|
| 649 | - $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
| 648 | + $replacer = new Prefixer($config, __DIR__); |
|
| 649 | + $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
| 650 | 650 | |
| 651 | - $expected = "\$this->apiRequest->setClientId(\"sdk-php-\" . \\Prefix\\net\\authorize\\api\\constants\\ANetEnvironment::VERSION);"; |
|
| 651 | + $expected = "\$this->apiRequest->setClientId(\"sdk-php-\" . \\Prefix\\net\\authorize\\api\\constants\\ANetEnvironment::VERSION);"; |
|
| 652 | 652 | |
| 653 | 653 | |
| 654 | - $this->assertEquals($expected, $result); |
|
| 655 | - } |
|
| 654 | + $this->assertEquals($expected, $result); |
|
| 655 | + } |
|
| 656 | 656 | |
| 657 | - /** |
|
| 658 | - * Another mpdf issue where the class "Mpdf" is in the namespace "Mpdf" and incorrect replacements are being made. |
|
| 659 | - */ |
|
| 660 | - public function testClassnameNotConfusedWithNamespace() |
|
| 661 | - { |
|
| 657 | + /** |
|
| 658 | + * Another mpdf issue where the class "Mpdf" is in the namespace "Mpdf" and incorrect replacements are being made. |
|
| 659 | + */ |
|
| 660 | + public function testClassnameNotConfusedWithNamespace() |
|
| 661 | + { |
|
| 662 | 662 | |
| 663 | - $contents = '$default_font_size = $mmsize * (Mpdf::SCALE);'; |
|
| 664 | - $expected = $contents; |
|
| 663 | + $contents = '$default_font_size = $mmsize * (Mpdf::SCALE);'; |
|
| 664 | + $expected = $contents; |
|
| 665 | 665 | |
| 666 | - $config = $this->createMock(StraussConfig::class); |
|
| 666 | + $config = $this->createMock(StraussConfig::class); |
|
| 667 | 667 | |
| 668 | - $replacer = new Prefixer($config, __DIR__); |
|
| 669 | - $result = $replacer->replaceNamespace($contents, 'Mpdf', 'BrianHenryIE\Strauss\Mpdf'); |
|
| 668 | + $replacer = new Prefixer($config, __DIR__); |
|
| 669 | + $result = $replacer->replaceNamespace($contents, 'Mpdf', 'BrianHenryIE\Strauss\Mpdf'); |
|
| 670 | 670 | |
| 671 | - $this->assertEquals($expected, $result); |
|
| 672 | - } |
|
| 671 | + $this->assertEquals($expected, $result); |
|
| 672 | + } |
|
| 673 | 673 | |
| 674 | - public function testClassExtendsNamspacedClassIsPrefixed() |
|
| 675 | - { |
|
| 674 | + public function testClassExtendsNamspacedClassIsPrefixed() |
|
| 675 | + { |
|
| 676 | 676 | |
| 677 | - $contents = 'class BarcodeException extends \Mpdf\MpdfException'; |
|
| 678 | - $expected = 'class BarcodeException extends \BrianHenryIE\Strauss\Mpdf\MpdfException'; |
|
| 677 | + $contents = 'class BarcodeException extends \Mpdf\MpdfException'; |
|
| 678 | + $expected = 'class BarcodeException extends \BrianHenryIE\Strauss\Mpdf\MpdfException'; |
|
| 679 | 679 | |
| 680 | - $config = $this->createMock(StraussConfig::class); |
|
| 680 | + $config = $this->createMock(StraussConfig::class); |
|
| 681 | 681 | |
| 682 | - $replacer = new Prefixer($config, __DIR__); |
|
| 683 | - $result = $replacer->replaceNamespace($contents, 'Mpdf', 'BrianHenryIE\Strauss\Mpdf'); |
|
| 682 | + $replacer = new Prefixer($config, __DIR__); |
|
| 683 | + $result = $replacer->replaceNamespace($contents, 'Mpdf', 'BrianHenryIE\Strauss\Mpdf'); |
|
| 684 | 684 | |
| 685 | - $this->assertEquals($expected, $result); |
|
| 686 | - } |
|
| 685 | + $this->assertEquals($expected, $result); |
|
| 686 | + } |
|
| 687 | 687 | |
| 688 | - /** |
|
| 689 | - * Prefix namespaced classnames after `new` keyword. |
|
| 690 | - * |
|
| 691 | - * @see https://github.com/BrianHenryIE/strauss/issues/11 |
|
| 692 | - */ |
|
| 693 | - public function testNewNamespacedClassIsPrefixed() |
|
| 694 | - { |
|
| 688 | + /** |
|
| 689 | + * Prefix namespaced classnames after `new` keyword. |
|
| 690 | + * |
|
| 691 | + * @see https://github.com/BrianHenryIE/strauss/issues/11 |
|
| 692 | + */ |
|
| 693 | + public function testNewNamespacedClassIsPrefixed() |
|
| 694 | + { |
|
| 695 | 695 | |
| 696 | - $contents = '$ioc->register( new \Carbon_Fields\Provider\Container_Condition_Provider() );'; |
|
| 697 | - $expected = '$ioc->register( new \BrianHenryIE\Strauss\Carbon_Fields\Provider\Container_Condition_Provider() );'; |
|
| 696 | + $contents = '$ioc->register( new \Carbon_Fields\Provider\Container_Condition_Provider() );'; |
|
| 697 | + $expected = '$ioc->register( new \BrianHenryIE\Strauss\Carbon_Fields\Provider\Container_Condition_Provider() );'; |
|
| 698 | 698 | |
| 699 | - $config = $this->createMock(StraussConfig::class); |
|
| 699 | + $config = $this->createMock(StraussConfig::class); |
|
| 700 | 700 | |
| 701 | - $replacer = new Prefixer($config, __DIR__); |
|
| 702 | - $result = $replacer->replaceNamespace($contents, 'Carbon_Fields\Provider', 'BrianHenryIE\Strauss\Carbon_Fields\Provider'); |
|
| 701 | + $replacer = new Prefixer($config, __DIR__); |
|
| 702 | + $result = $replacer->replaceNamespace($contents, 'Carbon_Fields\Provider', 'BrianHenryIE\Strauss\Carbon_Fields\Provider'); |
|
| 703 | 703 | |
| 704 | - $this->assertEquals($expected, $result); |
|
| 705 | - } |
|
| 704 | + $this->assertEquals($expected, $result); |
|
| 705 | + } |
|
| 706 | 706 | |
| 707 | 707 | |
| 708 | 708 | |
| 709 | - /** |
|
| 710 | - * Prefix namespaced classnames after `static` keyword. |
|
| 711 | - * |
|
| 712 | - * @see https://github.com/BrianHenryIE/strauss/issues/11 |
|
| 713 | - */ |
|
| 714 | - public function testStaticNamespacedClassIsPrefixed() |
|
| 715 | - { |
|
| 709 | + /** |
|
| 710 | + * Prefix namespaced classnames after `static` keyword. |
|
| 711 | + * |
|
| 712 | + * @see https://github.com/BrianHenryIE/strauss/issues/11 |
|
| 713 | + */ |
|
| 714 | + public function testStaticNamespacedClassIsPrefixed() |
|
| 715 | + { |
|
| 716 | 716 | |
| 717 | - $contents = '@method static \Carbon_Fields\Container\Comment_Meta_Container'; |
|
| 718 | - $expected = '@method static \BrianHenryIE\Strauss\Carbon_Fields\Container\Comment_Meta_Container'; |
|
| 717 | + $contents = '@method static \Carbon_Fields\Container\Comment_Meta_Container'; |
|
| 718 | + $expected = '@method static \BrianHenryIE\Strauss\Carbon_Fields\Container\Comment_Meta_Container'; |
|
| 719 | 719 | |
| 720 | - $config = $this->createMock(StraussConfig::class); |
|
| 720 | + $config = $this->createMock(StraussConfig::class); |
|
| 721 | 721 | |
| 722 | - $replacer = new Prefixer($config, __DIR__); |
|
| 723 | - $result = $replacer->replaceNamespace($contents, 'Carbon_Fields\Container', 'BrianHenryIE\Strauss\Carbon_Fields\Container'); |
|
| 722 | + $replacer = new Prefixer($config, __DIR__); |
|
| 723 | + $result = $replacer->replaceNamespace($contents, 'Carbon_Fields\Container', 'BrianHenryIE\Strauss\Carbon_Fields\Container'); |
|
| 724 | 724 | |
| 725 | - $this->assertEquals($expected, $result); |
|
| 726 | - } |
|
| 725 | + $this->assertEquals($expected, $result); |
|
| 726 | + } |
|
| 727 | 727 | |
| 728 | - /** |
|
| 729 | - * Prefix namespaced classnames after return statement. |
|
| 730 | - * |
|
| 731 | - * @see https://github.com/BrianHenryIE/strauss/issues/11 |
|
| 732 | - */ |
|
| 733 | - public function testReturnedNamespacedClassIsPrefixed() |
|
| 734 | - { |
|
| 728 | + /** |
|
| 729 | + * Prefix namespaced classnames after return statement. |
|
| 730 | + * |
|
| 731 | + * @see https://github.com/BrianHenryIE/strauss/issues/11 |
|
| 732 | + */ |
|
| 733 | + public function testReturnedNamespacedClassIsPrefixed() |
|
| 734 | + { |
|
| 735 | 735 | |
| 736 | - $contents = 'return \Carbon_Fields\Carbon_Fields::resolve'; |
|
| 737 | - $expected = 'return \BrianHenryIE\Strauss\Carbon_Fields\Carbon_Fields::resolve'; |
|
| 736 | + $contents = 'return \Carbon_Fields\Carbon_Fields::resolve'; |
|
| 737 | + $expected = 'return \BrianHenryIE\Strauss\Carbon_Fields\Carbon_Fields::resolve'; |
|
| 738 | 738 | |
| 739 | - $config = $this->createMock(StraussConfig::class); |
|
| 739 | + $config = $this->createMock(StraussConfig::class); |
|
| 740 | 740 | |
| 741 | - $replacer = new Prefixer($config, __DIR__); |
|
| 742 | - $result = $replacer->replaceNamespace($contents, 'Carbon_Fields', 'BrianHenryIE\Strauss\Carbon_Fields'); |
|
| 741 | + $replacer = new Prefixer($config, __DIR__); |
|
| 742 | + $result = $replacer->replaceNamespace($contents, 'Carbon_Fields', 'BrianHenryIE\Strauss\Carbon_Fields'); |
|
| 743 | 743 | |
| 744 | - $this->assertEquals($expected, $result); |
|
| 745 | - } |
|
| 744 | + $this->assertEquals($expected, $result); |
|
| 745 | + } |
|
| 746 | 746 | |
| 747 | - /** |
|
| 748 | - * Prefix namespaced classnames between two tabs and colon. |
|
| 749 | - * |
|
| 750 | - * @see https://github.com/BrianHenryIE/strauss/issues/11 |
|
| 751 | - */ |
|
| 752 | - public function testNamespacedStaticIsPrefixed() |
|
| 753 | - { |
|
| 747 | + /** |
|
| 748 | + * Prefix namespaced classnames between two tabs and colon. |
|
| 749 | + * |
|
| 750 | + * @see https://github.com/BrianHenryIE/strauss/issues/11 |
|
| 751 | + */ |
|
| 752 | + public function testNamespacedStaticIsPrefixed() |
|
| 753 | + { |
|
| 754 | 754 | |
| 755 | - $contents = " \\Carbon_Fields\\Carbon_Fields::service( 'legacy_storage' )->enable()"; |
|
| 756 | - $expected = " \\BrianHenryIE\\Strauss\\Carbon_Fields\\Carbon_Fields::service( 'legacy_storage' )->enable()"; |
|
| 755 | + $contents = " \\Carbon_Fields\\Carbon_Fields::service( 'legacy_storage' )->enable()"; |
|
| 756 | + $expected = " \\BrianHenryIE\\Strauss\\Carbon_Fields\\Carbon_Fields::service( 'legacy_storage' )->enable()"; |
|
| 757 | 757 | |
| 758 | - $config = $this->createMock(StraussConfig::class); |
|
| 758 | + $config = $this->createMock(StraussConfig::class); |
|
| 759 | 759 | |
| 760 | - $replacer = new Prefixer($config, __DIR__); |
|
| 761 | - $result = $replacer->replaceNamespace( |
|
| 762 | - $contents, |
|
| 763 | - 'Carbon_Fields', |
|
| 764 | - 'BrianHenryIE\Strauss\Carbon_Fields' |
|
| 765 | - ); |
|
| 760 | + $replacer = new Prefixer($config, __DIR__); |
|
| 761 | + $result = $replacer->replaceNamespace( |
|
| 762 | + $contents, |
|
| 763 | + 'Carbon_Fields', |
|
| 764 | + 'BrianHenryIE\Strauss\Carbon_Fields' |
|
| 765 | + ); |
|
| 766 | 766 | |
| 767 | - $this->assertEquals($expected, $result); |
|
| 768 | - } |
|
| 767 | + $this->assertEquals($expected, $result); |
|
| 768 | + } |
|
| 769 | 769 | |
| 770 | - /** |
|
| 771 | - * Sometimes the namespace in a string should be replaced, but sometimes not. |
|
| 772 | - * |
|
| 773 | - * @see https://github.com/BrianHenryIE/strauss/issues/15 |
|
| 774 | - */ |
|
| 775 | - public function testDoNotReplaceInStringThatIsNotCode() |
|
| 776 | - { |
|
| 777 | - $originalNamespace = "TrustedLogin"; |
|
| 778 | - $replacement = "Prefix\\TrustedLogin"; |
|
| 779 | - $contents = "esc_html__( 'Learn about TrustedLogin', 'trustedlogin' )"; |
|
| 770 | + /** |
|
| 771 | + * Sometimes the namespace in a string should be replaced, but sometimes not. |
|
| 772 | + * |
|
| 773 | + * @see https://github.com/BrianHenryIE/strauss/issues/15 |
|
| 774 | + */ |
|
| 775 | + public function testDoNotReplaceInStringThatIsNotCode() |
|
| 776 | + { |
|
| 777 | + $originalNamespace = "TrustedLogin"; |
|
| 778 | + $replacement = "Prefix\\TrustedLogin"; |
|
| 779 | + $contents = "esc_html__( 'Learn about TrustedLogin', 'trustedlogin' )"; |
|
| 780 | 780 | |
| 781 | - $config = $this->createMock(StraussConfig::class); |
|
| 781 | + $config = $this->createMock(StraussConfig::class); |
|
| 782 | 782 | |
| 783 | - $replacer = new Prefixer($config, __DIR__); |
|
| 784 | - $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
| 783 | + $replacer = new Prefixer($config, __DIR__); |
|
| 784 | + $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement); |
|
| 785 | 785 | |
| 786 | - $expected = "esc_html__( 'Learn about TrustedLogin', 'trustedlogin' )"; |
|
| 786 | + $expected = "esc_html__( 'Learn about TrustedLogin', 'trustedlogin' )"; |
|
| 787 | 787 | |
| 788 | - $this->assertEquals($expected, $result); |
|
| 789 | - } |
|
| 788 | + $this->assertEquals($expected, $result); |
|
| 789 | + } |
|
| 790 | 790 | |
| 791 | 791 | |
| 792 | - /** |
|
| 793 | - * |
|
| 794 | - * |
|
| 795 | - * @see https://github.com/BrianHenryIE/strauss/issues/19 |
|
| 796 | - * |
|
| 797 | - */ |
|
| 798 | - public function testDoNotReplaceInVariableNames() |
|
| 799 | - { |
|
| 800 | - $originalClassname = 'object'; |
|
| 801 | - $classnamePrefix = 'Strauss_Issue19_'; |
|
| 802 | - $contents = "public static function objclone(\$object) {"; |
|
| 792 | + /** |
|
| 793 | + * |
|
| 794 | + * |
|
| 795 | + * @see https://github.com/BrianHenryIE/strauss/issues/19 |
|
| 796 | + * |
|
| 797 | + */ |
|
| 798 | + public function testDoNotReplaceInVariableNames() |
|
| 799 | + { |
|
| 800 | + $originalClassname = 'object'; |
|
| 801 | + $classnamePrefix = 'Strauss_Issue19_'; |
|
| 802 | + $contents = "public static function objclone(\$object) {"; |
|
| 803 | 803 | |
| 804 | - $config = $this->createMock(StraussConfig::class); |
|
| 804 | + $config = $this->createMock(StraussConfig::class); |
|
| 805 | 805 | |
| 806 | - $replacer = new Prefixer($config, __DIR__); |
|
| 807 | - $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
| 806 | + $replacer = new Prefixer($config, __DIR__); |
|
| 807 | + $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix); |
|
| 808 | 808 | |
| 809 | - // NOT public static function objclone($Strauss_Issue19_object) { |
|
| 810 | - $expected = "public static function objclone(\$object) {"; |
|
| 809 | + // NOT public static function objclone($Strauss_Issue19_object) { |
|
| 810 | + $expected = "public static function objclone(\$object) {"; |
|
| 811 | 811 | |
| 812 | - $this->assertEquals($expected, $result); |
|
| 813 | - } |
|
| 812 | + $this->assertEquals($expected, $result); |
|
| 813 | + } |
|
| 814 | 814 | } |
@@ -22,80 +22,80 @@ discard block |
||
| 22 | 22 | { |
| 23 | 23 | |
| 24 | 24 | |
| 25 | - /** |
|
| 26 | - * @covers ::findLicenseFiles |
|
| 27 | - * |
|
| 28 | - */ |
|
| 29 | - public function testFindLicenceFilesPathsAreRelative() |
|
| 30 | - { |
|
| 31 | - $config = $this->createStub(StraussConfig::class); |
|
| 32 | - $workingDir = __DIR__ . DIRECTORY_SEPARATOR; |
|
| 25 | + /** |
|
| 26 | + * @covers ::findLicenseFiles |
|
| 27 | + * |
|
| 28 | + */ |
|
| 29 | + public function testFindLicenceFilesPathsAreRelative() |
|
| 30 | + { |
|
| 31 | + $config = $this->createStub(StraussConfig::class); |
|
| 32 | + $workingDir = __DIR__ . DIRECTORY_SEPARATOR; |
|
| 33 | 33 | |
| 34 | - $dependencies = array(); |
|
| 34 | + $dependencies = array(); |
|
| 35 | 35 | |
| 36 | - $dependency = $this->createStub(ComposerPackage::class); |
|
| 37 | - $dependency->method('getPath')->willReturn('developer-name/project-name/'); |
|
| 38 | - $dependencies[] = $dependency; |
|
| 36 | + $dependency = $this->createStub(ComposerPackage::class); |
|
| 37 | + $dependency->method('getPath')->willReturn('developer-name/project-name/'); |
|
| 38 | + $dependencies[] = $dependency; |
|
| 39 | 39 | |
| 40 | - $sut = new Licenser($config, $workingDir, $dependencies, 'BrianHenryIE'); |
|
| 40 | + $sut = new Licenser($config, $workingDir, $dependencies, 'BrianHenryIE'); |
|
| 41 | 41 | |
| 42 | - $finder = $this->createStub(Finder::class); |
|
| 42 | + $finder = $this->createStub(Finder::class); |
|
| 43 | 43 | |
| 44 | - $file = $this->createStub(\SplFileInfo::class); |
|
| 45 | - $file->method('getPathname') |
|
| 46 | - ->willReturn(__DIR__.'/vendor/developer-name/project-name/license.md'); |
|
| 44 | + $file = $this->createStub(\SplFileInfo::class); |
|
| 45 | + $file->method('getPathname') |
|
| 46 | + ->willReturn(__DIR__.'/vendor/developer-name/project-name/license.md'); |
|
| 47 | 47 | |
| 48 | - $finderArrayIterator = new ArrayIterator(array( |
|
| 49 | - $file |
|
| 50 | - )); |
|
| 48 | + $finderArrayIterator = new ArrayIterator(array( |
|
| 49 | + $file |
|
| 50 | + )); |
|
| 51 | 51 | |
| 52 | - $finder->method('getIterator')->willReturn($finderArrayIterator); |
|
| 52 | + $finder->method('getIterator')->willReturn($finderArrayIterator); |
|
| 53 | 53 | |
| 54 | - // Make the rest fluent. |
|
| 55 | - $callableConstraintNotGetIterator = function ($methodName) { |
|
| 56 | - return 'getIterator' !== $methodName; |
|
| 57 | - }; |
|
| 58 | - $finder->method(new Callback($callableConstraintNotGetIterator))->willReturn($finder); |
|
| 54 | + // Make the rest fluent. |
|
| 55 | + $callableConstraintNotGetIterator = function ($methodName) { |
|
| 56 | + return 'getIterator' !== $methodName; |
|
| 57 | + }; |
|
| 58 | + $finder->method(new Callback($callableConstraintNotGetIterator))->willReturn($finder); |
|
| 59 | 59 | |
| 60 | - $sut->findLicenseFiles($finder); |
|
| 60 | + $sut->findLicenseFiles($finder); |
|
| 61 | 61 | |
| 62 | - $result = $sut->getDiscoveredLicenseFiles(); |
|
| 62 | + $result = $sut->getDiscoveredLicenseFiles(); |
|
| 63 | 63 | |
| 64 | - // Currently contains an array entry: /Users/brianhenry/Sites/mozart/mozart/tests/Unit/developer-name/project-name/license.md |
|
| 65 | - $this->assertStringContainsString('developer-name/project-name/license.md', $result[0]); |
|
| 66 | - } |
|
| 64 | + // Currently contains an array entry: /Users/brianhenry/Sites/mozart/mozart/tests/Unit/developer-name/project-name/license.md |
|
| 65 | + $this->assertStringContainsString('developer-name/project-name/license.md', $result[0]); |
|
| 66 | + } |
|
| 67 | 67 | |
| 68 | - /** |
|
| 69 | - * Licence files should be found regardless of case and regardless of British/US-English spelling. |
|
| 70 | - * |
|
| 71 | - * @see https://www.phpliveregex.com/p/A5y |
|
| 72 | - */ |
|
| 68 | + /** |
|
| 69 | + * Licence files should be found regardless of case and regardless of British/US-English spelling. |
|
| 70 | + * |
|
| 71 | + * @see https://www.phpliveregex.com/p/A5y |
|
| 72 | + */ |
|
| 73 | 73 | |
| 74 | - // https://schibsted.com/blog/mocking-the-file-system-using-phpunit-and-vfsstream/ |
|
| 74 | + // https://schibsted.com/blog/mocking-the-file-system-using-phpunit-and-vfsstream/ |
|
| 75 | 75 | |
| 76 | 76 | |
| 77 | - /** |
|
| 78 | - * @see https://github.com/AuthorizeNet/sdk-php/blob/a3e76f96f674d16e892f87c58bedb99dada4b067/lib/net/authorize/api/contract/v1/ANetApiRequestType.php |
|
| 79 | - * |
|
| 80 | - * @covers ::addChangeDeclarationToPhpString |
|
| 81 | - */ |
|
| 82 | - public function testAppendHeaderCommentInformationNoHeader() |
|
| 83 | - { |
|
| 77 | + /** |
|
| 78 | + * @see https://github.com/AuthorizeNet/sdk-php/blob/a3e76f96f674d16e892f87c58bedb99dada4b067/lib/net/authorize/api/contract/v1/ANetApiRequestType.php |
|
| 79 | + * |
|
| 80 | + * @covers ::addChangeDeclarationToPhpString |
|
| 81 | + */ |
|
| 82 | + public function testAppendHeaderCommentInformationNoHeader() |
|
| 83 | + { |
|
| 84 | 84 | |
| 85 | - $author = 'BrianHenryIE'; |
|
| 85 | + $author = 'BrianHenryIE'; |
|
| 86 | 86 | |
| 87 | - $config = $this->createStub(StraussConfig::class); |
|
| 88 | - $sut = new Licenser($config, __DIR__, array(), $author); |
|
| 87 | + $config = $this->createStub(StraussConfig::class); |
|
| 88 | + $sut = new Licenser($config, __DIR__, array(), $author); |
|
| 89 | 89 | |
| 90 | - $given = <<<'EOD' |
|
| 90 | + $given = <<<'EOD' |
|
| 91 | 91 | <?php |
| 92 | 92 | |
| 93 | 93 | namespace net\authorize\api\contract\v1; |
| 94 | 94 | EOD; |
| 95 | 95 | |
| 96 | - // "license": "proprietary", |
|
| 96 | + // "license": "proprietary", |
|
| 97 | 97 | |
| 98 | - $expected = <<<'EOD' |
|
| 98 | + $expected = <<<'EOD' |
|
| 99 | 99 | <?php |
| 100 | 100 | /** |
| 101 | 101 | * @license proprietary |
@@ -107,22 +107,22 @@ discard block |
||
| 107 | 107 | namespace net\authorize\api\contract\v1; |
| 108 | 108 | EOD; |
| 109 | 109 | |
| 110 | - $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary'); |
|
| 110 | + $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary'); |
|
| 111 | 111 | |
| 112 | - $this->assertEquals($expected, $actual); |
|
| 113 | - } |
|
| 112 | + $this->assertEquals($expected, $actual); |
|
| 113 | + } |
|
| 114 | 114 | |
| 115 | - /** |
|
| 116 | - * @covers ::addChangeDeclarationToPhpString |
|
| 117 | - */ |
|
| 118 | - public function testWithLicenceAlreadyInHeader(): void |
|
| 119 | - { |
|
| 115 | + /** |
|
| 116 | + * @covers ::addChangeDeclarationToPhpString |
|
| 117 | + */ |
|
| 118 | + public function testWithLicenceAlreadyInHeader(): void |
|
| 119 | + { |
|
| 120 | 120 | |
| 121 | - $config = $this->createStub(StraussConfig::class); |
|
| 122 | - $author = 'BrianHenryIE'; |
|
| 123 | - $sut = new Licenser($config, __DIR__, array(), $author); |
|
| 121 | + $config = $this->createStub(StraussConfig::class); |
|
| 122 | + $author = 'BrianHenryIE'; |
|
| 123 | + $sut = new Licenser($config, __DIR__, array(), $author); |
|
| 124 | 124 | |
| 125 | - $given = <<<'EOD' |
|
| 125 | + $given = <<<'EOD' |
|
| 126 | 126 | <?php // phpcs:ignore WordPress.Files.FileName |
| 127 | 127 | /** |
| 128 | 128 | * Handles dismissing admin notices. |
@@ -137,7 +137,7 @@ discard block |
||
| 137 | 137 | namespace Yeah; |
| 138 | 138 | EOD; |
| 139 | 139 | |
| 140 | - $expected = <<<'EOD' |
|
| 140 | + $expected = <<<'EOD' |
|
| 141 | 141 | <?php // phpcs:ignore WordPress.Files.FileName |
| 142 | 142 | /** |
| 143 | 143 | * Handles dismissing admin notices. |
@@ -155,25 +155,25 @@ discard block |
||
| 155 | 155 | namespace Yeah; |
| 156 | 156 | EOD; |
| 157 | 157 | |
| 158 | - $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'GPL-2.0-or-later'); |
|
| 158 | + $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'GPL-2.0-or-later'); |
|
| 159 | 159 | |
| 160 | - $this->assertEquals($expected, $actual); |
|
| 161 | - } |
|
| 160 | + $this->assertEquals($expected, $actual); |
|
| 161 | + } |
|
| 162 | 162 | |
| 163 | 163 | |
| 164 | - /** |
|
| 165 | - * Shouldn't matter too much but y'know regexes. |
|
| 166 | - * |
|
| 167 | - * @covers ::addChangeDeclarationToPhpString |
|
| 168 | - */ |
|
| 169 | - public function testWithTwoCommentsBeforeFirstCode() |
|
| 170 | - { |
|
| 164 | + /** |
|
| 165 | + * Shouldn't matter too much but y'know regexes. |
|
| 166 | + * |
|
| 167 | + * @covers ::addChangeDeclarationToPhpString |
|
| 168 | + */ |
|
| 169 | + public function testWithTwoCommentsBeforeFirstCode() |
|
| 170 | + { |
|
| 171 | 171 | |
| 172 | - $config = $this->createStub(StraussConfig::class); |
|
| 173 | - $author = 'BrianHenryIE'; |
|
| 174 | - $sut = new Licenser($config, __DIR__, array(), $author); |
|
| 172 | + $config = $this->createStub(StraussConfig::class); |
|
| 173 | + $author = 'BrianHenryIE'; |
|
| 174 | + $sut = new Licenser($config, __DIR__, array(), $author); |
|
| 175 | 175 | |
| 176 | - $given = <<<'EOD' |
|
| 176 | + $given = <<<'EOD' |
|
| 177 | 177 | <?php |
| 178 | 178 | /** |
| 179 | 179 | * WP Dependency Installer |
@@ -194,7 +194,7 @@ discard block |
||
| 194 | 194 | if ( ! defined( 'WPINC' ) ) { |
| 195 | 195 | EOD; |
| 196 | 196 | |
| 197 | - $expected = <<<'EOD' |
|
| 197 | + $expected = <<<'EOD' |
|
| 198 | 198 | <?php |
| 199 | 199 | /** |
| 200 | 200 | * WP Dependency Installer |
@@ -219,22 +219,22 @@ discard block |
||
| 219 | 219 | EOD; |
| 220 | 220 | |
| 221 | 221 | |
| 222 | - $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT'); |
|
| 222 | + $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT'); |
|
| 223 | 223 | |
| 224 | - $this->assertEquals($expected, $actual); |
|
| 225 | - } |
|
| 224 | + $this->assertEquals($expected, $actual); |
|
| 225 | + } |
|
| 226 | 226 | |
| 227 | - /** |
|
| 228 | - * @covers ::addChangeDeclarationToPhpString |
|
| 229 | - */ |
|
| 230 | - public function testUnusualHeaderCommentStyle() |
|
| 231 | - { |
|
| 227 | + /** |
|
| 228 | + * @covers ::addChangeDeclarationToPhpString |
|
| 229 | + */ |
|
| 230 | + public function testUnusualHeaderCommentStyle() |
|
| 231 | + { |
|
| 232 | 232 | |
| 233 | - $config = $this->createStub(StraussConfig::class); |
|
| 234 | - $author = 'BrianHenryIE'; |
|
| 235 | - $sut = new Licenser($config, __DIR__, array(), $author); |
|
| 233 | + $config = $this->createStub(StraussConfig::class); |
|
| 234 | + $author = 'BrianHenryIE'; |
|
| 235 | + $sut = new Licenser($config, __DIR__, array(), $author); |
|
| 236 | 236 | |
| 237 | - $given = <<<'EOD' |
|
| 237 | + $given = <<<'EOD' |
|
| 238 | 238 | <?php |
| 239 | 239 | /******************************************************************************* |
| 240 | 240 | * FPDF * |
@@ -247,7 +247,7 @@ discard block |
||
| 247 | 247 | define('FPDF_VERSION','1.82'); |
| 248 | 248 | EOD; |
| 249 | 249 | |
| 250 | - $expected = <<<'EOD' |
|
| 250 | + $expected = <<<'EOD' |
|
| 251 | 251 | <?php |
| 252 | 252 | /******************************************************************************* |
| 253 | 253 | * FPDF * |
@@ -264,22 +264,22 @@ discard block |
||
| 264 | 264 | define('FPDF_VERSION','1.82'); |
| 265 | 265 | EOD; |
| 266 | 266 | |
| 267 | - $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary'); |
|
| 267 | + $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary'); |
|
| 268 | 268 | |
| 269 | - $this->assertEquals($expected, $actual); |
|
| 270 | - } |
|
| 269 | + $this->assertEquals($expected, $actual); |
|
| 270 | + } |
|
| 271 | 271 | |
| 272 | - /** |
|
| 273 | - * @covers ::addChangeDeclarationToPhpString |
|
| 274 | - */ |
|
| 275 | - public function testCommentWithLicenseWord() |
|
| 276 | - { |
|
| 272 | + /** |
|
| 273 | + * @covers ::addChangeDeclarationToPhpString |
|
| 274 | + */ |
|
| 275 | + public function testCommentWithLicenseWord() |
|
| 276 | + { |
|
| 277 | 277 | |
| 278 | - $config = $this->createStub(StraussConfig::class); |
|
| 279 | - $author = 'BrianHenryIE'; |
|
| 280 | - $sut = new Licenser($config, __DIR__, array(), $author); |
|
| 278 | + $config = $this->createStub(StraussConfig::class); |
|
| 279 | + $author = 'BrianHenryIE'; |
|
| 280 | + $sut = new Licenser($config, __DIR__, array(), $author); |
|
| 281 | 281 | |
| 282 | - $given = <<<'EOD' |
|
| 282 | + $given = <<<'EOD' |
|
| 283 | 283 | <?php |
| 284 | 284 | |
| 285 | 285 | /** |
@@ -297,7 +297,7 @@ discard block |
||
| 297 | 297 | namespace Your_Domain\Assert; |
| 298 | 298 | EOD; |
| 299 | 299 | |
| 300 | - $expected = <<<'EOD' |
|
| 300 | + $expected = <<<'EOD' |
|
| 301 | 301 | <?php |
| 302 | 302 | |
| 303 | 303 | /** |
@@ -318,27 +318,27 @@ discard block |
||
| 318 | 318 | namespace Your_Domain\Assert; |
| 319 | 319 | EOD; |
| 320 | 320 | |
| 321 | - $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT'); |
|
| 321 | + $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT'); |
|
| 322 | 322 | |
| 323 | - $this->assertEquals($expected, $actual); |
|
| 324 | - } |
|
| 323 | + $this->assertEquals($expected, $actual); |
|
| 324 | + } |
|
| 325 | 325 | |
| 326 | - /** |
|
| 327 | - * This was matching the "no header comment" regex. |
|
| 328 | - * |
|
| 329 | - * FOCK: The test passed. How do I debug when the test passes?! The test is passing but actual output is incorrect. |
|
| 330 | - * @see https://www.youtube.com/watch?v=QnxpHIl5Ynw |
|
| 331 | - * |
|
| 332 | - * Seems files loaded are treated different to strings passed. |
|
| 333 | - */ |
|
| 334 | - public function testIncorrectlyMatching() |
|
| 335 | - { |
|
| 326 | + /** |
|
| 327 | + * This was matching the "no header comment" regex. |
|
| 328 | + * |
|
| 329 | + * FOCK: The test passed. How do I debug when the test passes?! The test is passing but actual output is incorrect. |
|
| 330 | + * @see https://www.youtube.com/watch?v=QnxpHIl5Ynw |
|
| 331 | + * |
|
| 332 | + * Seems files loaded are treated different to strings passed. |
|
| 333 | + */ |
|
| 334 | + public function testIncorrectlyMatching() |
|
| 335 | + { |
|
| 336 | 336 | |
| 337 | - $config = $this->createStub(StraussConfig::class); |
|
| 338 | - $author = 'BrianHenryIE'; |
|
| 339 | - $sut = new Licenser($config, __DIR__, array(), $author); |
|
| 337 | + $config = $this->createStub(StraussConfig::class); |
|
| 338 | + $author = 'BrianHenryIE'; |
|
| 339 | + $sut = new Licenser($config, __DIR__, array(), $author); |
|
| 340 | 340 | |
| 341 | - $fileContents = <<<'EOD' |
|
| 341 | + $fileContents = <<<'EOD' |
|
| 342 | 342 | <?php |
| 343 | 343 | /** |
| 344 | 344 | * WP Dependency Installer |
@@ -361,12 +361,12 @@ discard block |
||
| 361 | 361 | } |
| 362 | 362 | EOD; |
| 363 | 363 | |
| 364 | - // Attempt to replicate the failing test, since the contents seem the same but the input manner is different. |
|
| 365 | - $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
| 366 | - file_put_contents($tmpfname, $fileContents); |
|
| 367 | - $given = file_get_contents($tmpfname); |
|
| 364 | + // Attempt to replicate the failing test, since the contents seem the same but the input manner is different. |
|
| 365 | + $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
| 366 | + file_put_contents($tmpfname, $fileContents); |
|
| 367 | + $given = file_get_contents($tmpfname); |
|
| 368 | 368 | |
| 369 | - $expected = <<<'EOD' |
|
| 369 | + $expected = <<<'EOD' |
|
| 370 | 370 | <?php |
| 371 | 371 | /** |
| 372 | 372 | * WP Dependency Installer |
@@ -393,22 +393,22 @@ discard block |
||
| 393 | 393 | EOD; |
| 394 | 394 | |
| 395 | 395 | |
| 396 | - $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT'); |
|
| 396 | + $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT'); |
|
| 397 | 397 | |
| 398 | - $this->assertEquals($expected, $actual); |
|
| 399 | - } |
|
| 398 | + $this->assertEquals($expected, $actual); |
|
| 399 | + } |
|
| 400 | 400 | |
| 401 | - /** |
|
| 402 | - * The licence was being inserted after every `<?php` in the file. |
|
| 403 | - */ |
|
| 404 | - public function testLicenseDetailsOnlyInsertedOncePerFile() |
|
| 405 | - { |
|
| 401 | + /** |
|
| 402 | + * The licence was being inserted after every `<?php` in the file. |
|
| 403 | + */ |
|
| 404 | + public function testLicenseDetailsOnlyInsertedOncePerFile() |
|
| 405 | + { |
|
| 406 | 406 | |
| 407 | - $config = $this->createStub(StraussConfig::class); |
|
| 408 | - $author = 'BrianHenryIE'; |
|
| 409 | - $sut = new Licenser($config, __DIR__, array(), $author); |
|
| 407 | + $config = $this->createStub(StraussConfig::class); |
|
| 408 | + $author = 'BrianHenryIE'; |
|
| 409 | + $sut = new Licenser($config, __DIR__, array(), $author); |
|
| 410 | 410 | |
| 411 | - $fileContents = <<<'EOD' |
|
| 411 | + $fileContents = <<<'EOD' |
|
| 412 | 412 | <?php |
| 413 | 413 | |
| 414 | 414 | ?> |
@@ -418,7 +418,7 @@ discard block |
||
| 418 | 418 | ?> |
| 419 | 419 | EOD; |
| 420 | 420 | |
| 421 | - $expected = <<<'EOD' |
|
| 421 | + $expected = <<<'EOD' |
|
| 422 | 422 | <?php |
| 423 | 423 | /** |
| 424 | 424 | * @license MIT |
@@ -435,8 +435,8 @@ discard block |
||
| 435 | 435 | EOD; |
| 436 | 436 | |
| 437 | 437 | |
| 438 | - $actual = $sut->addChangeDeclarationToPhpString($fileContents, '25-April-2021', 'MIT'); |
|
| 438 | + $actual = $sut->addChangeDeclarationToPhpString($fileContents, '25-April-2021', 'MIT'); |
|
| 439 | 439 | |
| 440 | - $this->assertEquals($expected, $actual); |
|
| 441 | - } |
|
| 440 | + $this->assertEquals($expected, $actual); |
|
| 441 | + } |
|
| 442 | 442 | } |
@@ -11,194 +11,194 @@ |
||
| 11 | 11 | class ComposeTest extends TestCase |
| 12 | 12 | { |
| 13 | 13 | |
| 14 | - /** |
|
| 15 | - * When composer.json is absent, instead of failing with: |
|
| 16 | - * "failed to open stream: No such file or directory" |
|
| 17 | - * a better message should be written to the OutputInterface. |
|
| 18 | - * |
|
| 19 | - * @test |
|
| 20 | - */ |
|
| 21 | - public function it_fails_gracefully_when_composer_json_absent(): void |
|
| 22 | - { |
|
| 23 | - chdir(sys_get_temp_dir()); |
|
| 24 | - |
|
| 25 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
| 26 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
| 27 | - |
|
| 28 | - $outputInterfaceMock->expects($this->exactly(1)) |
|
| 29 | - ->method('write'); |
|
| 30 | - |
|
| 31 | - new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
| 32 | - public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
| 33 | - { |
|
| 34 | - parent::__construct(); |
|
| 35 | - |
|
| 36 | - $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
| 37 | - } |
|
| 38 | - }; |
|
| 39 | - } |
|
| 40 | - |
|
| 41 | - /** |
|
| 42 | - * When json_decode fails, instead of |
|
| 43 | - * "Trying to get property 'extra' of non-object" |
|
| 44 | - * a better message should be written to the OutputInterface. |
|
| 45 | - * |
|
| 46 | - * @test |
|
| 47 | - */ |
|
| 48 | - public function it_handles_malformed_json_with_grace(): void |
|
| 49 | - { |
|
| 50 | - |
|
| 51 | - $badComposerJson = '{ "name": "coenjacobs/mozart", }'; |
|
| 52 | - |
|
| 53 | - $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
| 54 | - file_put_contents($tmpfname, $badComposerJson); |
|
| 55 | - chdir(dirname($tmpfname)); |
|
| 56 | - |
|
| 57 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
| 58 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
| 59 | - |
|
| 60 | - $outputInterfaceMock->expects($this->exactly(1)) |
|
| 61 | - ->method('write'); |
|
| 62 | - |
|
| 63 | - new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
| 64 | - public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
| 65 | - { |
|
| 66 | - parent::__construct(); |
|
| 67 | - |
|
| 68 | - $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
| 69 | - } |
|
| 70 | - }; |
|
| 71 | - } |
|
| 72 | - |
|
| 73 | - /** |
|
| 74 | - * When composer.json->extra is absent, instead of |
|
| 75 | - * "Undefined property: stdClass::$extra" |
|
| 76 | - * a better message should be written to the OutputInterface. |
|
| 77 | - * |
|
| 78 | - * When package name is not set, `\Composer\Composer::getPackage()->getName()` returns '__root__'. |
|
| 79 | - * |
|
| 80 | - */ |
|
| 81 | - public function test_it_handles_absent_extra_config_with_grace(): void |
|
| 82 | - { |
|
| 83 | - |
|
| 84 | - $badComposerJson = '{ }'; |
|
| 85 | - |
|
| 86 | - $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
| 87 | - file_put_contents($tmpfname, $badComposerJson); |
|
| 88 | - chdir(dirname($tmpfname)); |
|
| 89 | - |
|
| 90 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
| 91 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
| 92 | - |
|
| 93 | - $outputInterfaceMock->expects($this->exactly(1)) |
|
| 94 | - ->method('write'); |
|
| 95 | - |
|
| 96 | - new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
| 97 | - public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
| 98 | - { |
|
| 99 | - parent::__construct(); |
|
| 100 | - |
|
| 101 | - $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
| 102 | - } |
|
| 103 | - }; |
|
| 104 | - } |
|
| 105 | - |
|
| 106 | - |
|
| 107 | - /** |
|
| 108 | - * When composer.json->extra is not an object, instead of |
|
| 109 | - * "Trying to get property 'mozart' of non-object" |
|
| 110 | - * a better message should be written to the OutputInterface. |
|
| 111 | - * |
|
| 112 | - * @test |
|
| 113 | - */ |
|
| 114 | - public function it_handles_malformed_extra_config_with_grace(): void |
|
| 115 | - { |
|
| 116 | - |
|
| 117 | - $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": [] }'; |
|
| 118 | - |
|
| 119 | - $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
| 120 | - file_put_contents($tmpfname, $badComposerJson); |
|
| 121 | - chdir(dirname($tmpfname)); |
|
| 122 | - |
|
| 123 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
| 124 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
| 125 | - |
|
| 126 | - $outputInterfaceMock->expects($this->exactly(1)) |
|
| 127 | - ->method('write'); |
|
| 128 | - |
|
| 129 | - new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
| 130 | - public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
| 131 | - { |
|
| 132 | - parent::__construct(); |
|
| 133 | - |
|
| 134 | - $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
| 135 | - } |
|
| 136 | - }; |
|
| 137 | - } |
|
| 138 | - |
|
| 139 | - /** |
|
| 140 | - * When composer.json->extra->mozart is absent, instead of |
|
| 141 | - * "Undefined property: stdClass::$mozart" |
|
| 142 | - * a better message should be written to the OutputInterface. |
|
| 143 | - * |
|
| 144 | - * @test |
|
| 145 | - */ |
|
| 146 | - public function it_handles_absent_mozart_config_with_grace(): void |
|
| 147 | - { |
|
| 148 | - |
|
| 149 | - $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "moozart": {} } }'; |
|
| 150 | - |
|
| 151 | - $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
| 152 | - file_put_contents($tmpfname, $badComposerJson); |
|
| 153 | - chdir(dirname($tmpfname)); |
|
| 154 | - |
|
| 155 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
| 156 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
| 157 | - |
|
| 158 | - $outputInterfaceMock->expects($this->exactly(1)) |
|
| 159 | - ->method('write'); |
|
| 160 | - |
|
| 161 | - new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
| 162 | - public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
| 163 | - { |
|
| 164 | - parent::__construct(); |
|
| 165 | - |
|
| 166 | - $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
| 167 | - } |
|
| 168 | - }; |
|
| 169 | - } |
|
| 170 | - |
|
| 171 | - /** |
|
| 172 | - * When composer.json->extra->mozart is malformed, instead of |
|
| 173 | - * "Undefined property: stdClass::$mozart" |
|
| 174 | - * a better message should be written to the OutputInterface. |
|
| 175 | - * |
|
| 176 | - * is_object() added. |
|
| 177 | - * |
|
| 178 | - * @test |
|
| 179 | - */ |
|
| 180 | - public function it_handles_malformed_mozart_config__with_grace(): void |
|
| 181 | - { |
|
| 182 | - |
|
| 183 | - $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "mozart": [] }'; |
|
| 184 | - |
|
| 185 | - $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
| 186 | - file_put_contents($tmpfname, $badComposerJson); |
|
| 187 | - chdir(dirname($tmpfname)); |
|
| 188 | - |
|
| 189 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
| 190 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
| 191 | - |
|
| 192 | - $outputInterfaceMock->expects($this->exactly(1)) |
|
| 193 | - ->method('write'); |
|
| 194 | - |
|
| 195 | - new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
| 196 | - public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
| 197 | - { |
|
| 198 | - parent::__construct(); |
|
| 199 | - |
|
| 200 | - $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
| 201 | - } |
|
| 202 | - }; |
|
| 203 | - } |
|
| 14 | + /** |
|
| 15 | + * When composer.json is absent, instead of failing with: |
|
| 16 | + * "failed to open stream: No such file or directory" |
|
| 17 | + * a better message should be written to the OutputInterface. |
|
| 18 | + * |
|
| 19 | + * @test |
|
| 20 | + */ |
|
| 21 | + public function it_fails_gracefully_when_composer_json_absent(): void |
|
| 22 | + { |
|
| 23 | + chdir(sys_get_temp_dir()); |
|
| 24 | + |
|
| 25 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
| 26 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
| 27 | + |
|
| 28 | + $outputInterfaceMock->expects($this->exactly(1)) |
|
| 29 | + ->method('write'); |
|
| 30 | + |
|
| 31 | + new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
| 32 | + public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
| 33 | + { |
|
| 34 | + parent::__construct(); |
|
| 35 | + |
|
| 36 | + $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
| 37 | + } |
|
| 38 | + }; |
|
| 39 | + } |
|
| 40 | + |
|
| 41 | + /** |
|
| 42 | + * When json_decode fails, instead of |
|
| 43 | + * "Trying to get property 'extra' of non-object" |
|
| 44 | + * a better message should be written to the OutputInterface. |
|
| 45 | + * |
|
| 46 | + * @test |
|
| 47 | + */ |
|
| 48 | + public function it_handles_malformed_json_with_grace(): void |
|
| 49 | + { |
|
| 50 | + |
|
| 51 | + $badComposerJson = '{ "name": "coenjacobs/mozart", }'; |
|
| 52 | + |
|
| 53 | + $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
| 54 | + file_put_contents($tmpfname, $badComposerJson); |
|
| 55 | + chdir(dirname($tmpfname)); |
|
| 56 | + |
|
| 57 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
| 58 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
| 59 | + |
|
| 60 | + $outputInterfaceMock->expects($this->exactly(1)) |
|
| 61 | + ->method('write'); |
|
| 62 | + |
|
| 63 | + new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
| 64 | + public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
| 65 | + { |
|
| 66 | + parent::__construct(); |
|
| 67 | + |
|
| 68 | + $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
| 69 | + } |
|
| 70 | + }; |
|
| 71 | + } |
|
| 72 | + |
|
| 73 | + /** |
|
| 74 | + * When composer.json->extra is absent, instead of |
|
| 75 | + * "Undefined property: stdClass::$extra" |
|
| 76 | + * a better message should be written to the OutputInterface. |
|
| 77 | + * |
|
| 78 | + * When package name is not set, `\Composer\Composer::getPackage()->getName()` returns '__root__'. |
|
| 79 | + * |
|
| 80 | + */ |
|
| 81 | + public function test_it_handles_absent_extra_config_with_grace(): void |
|
| 82 | + { |
|
| 83 | + |
|
| 84 | + $badComposerJson = '{ }'; |
|
| 85 | + |
|
| 86 | + $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
| 87 | + file_put_contents($tmpfname, $badComposerJson); |
|
| 88 | + chdir(dirname($tmpfname)); |
|
| 89 | + |
|
| 90 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
| 91 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
| 92 | + |
|
| 93 | + $outputInterfaceMock->expects($this->exactly(1)) |
|
| 94 | + ->method('write'); |
|
| 95 | + |
|
| 96 | + new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
| 97 | + public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
| 98 | + { |
|
| 99 | + parent::__construct(); |
|
| 100 | + |
|
| 101 | + $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
| 102 | + } |
|
| 103 | + }; |
|
| 104 | + } |
|
| 105 | + |
|
| 106 | + |
|
| 107 | + /** |
|
| 108 | + * When composer.json->extra is not an object, instead of |
|
| 109 | + * "Trying to get property 'mozart' of non-object" |
|
| 110 | + * a better message should be written to the OutputInterface. |
|
| 111 | + * |
|
| 112 | + * @test |
|
| 113 | + */ |
|
| 114 | + public function it_handles_malformed_extra_config_with_grace(): void |
|
| 115 | + { |
|
| 116 | + |
|
| 117 | + $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": [] }'; |
|
| 118 | + |
|
| 119 | + $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
| 120 | + file_put_contents($tmpfname, $badComposerJson); |
|
| 121 | + chdir(dirname($tmpfname)); |
|
| 122 | + |
|
| 123 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
| 124 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
| 125 | + |
|
| 126 | + $outputInterfaceMock->expects($this->exactly(1)) |
|
| 127 | + ->method('write'); |
|
| 128 | + |
|
| 129 | + new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
| 130 | + public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
| 131 | + { |
|
| 132 | + parent::__construct(); |
|
| 133 | + |
|
| 134 | + $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
| 135 | + } |
|
| 136 | + }; |
|
| 137 | + } |
|
| 138 | + |
|
| 139 | + /** |
|
| 140 | + * When composer.json->extra->mozart is absent, instead of |
|
| 141 | + * "Undefined property: stdClass::$mozart" |
|
| 142 | + * a better message should be written to the OutputInterface. |
|
| 143 | + * |
|
| 144 | + * @test |
|
| 145 | + */ |
|
| 146 | + public function it_handles_absent_mozart_config_with_grace(): void |
|
| 147 | + { |
|
| 148 | + |
|
| 149 | + $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "moozart": {} } }'; |
|
| 150 | + |
|
| 151 | + $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
| 152 | + file_put_contents($tmpfname, $badComposerJson); |
|
| 153 | + chdir(dirname($tmpfname)); |
|
| 154 | + |
|
| 155 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
| 156 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
| 157 | + |
|
| 158 | + $outputInterfaceMock->expects($this->exactly(1)) |
|
| 159 | + ->method('write'); |
|
| 160 | + |
|
| 161 | + new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
| 162 | + public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
| 163 | + { |
|
| 164 | + parent::__construct(); |
|
| 165 | + |
|
| 166 | + $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
| 167 | + } |
|
| 168 | + }; |
|
| 169 | + } |
|
| 170 | + |
|
| 171 | + /** |
|
| 172 | + * When composer.json->extra->mozart is malformed, instead of |
|
| 173 | + * "Undefined property: stdClass::$mozart" |
|
| 174 | + * a better message should be written to the OutputInterface. |
|
| 175 | + * |
|
| 176 | + * is_object() added. |
|
| 177 | + * |
|
| 178 | + * @test |
|
| 179 | + */ |
|
| 180 | + public function it_handles_malformed_mozart_config__with_grace(): void |
|
| 181 | + { |
|
| 182 | + |
|
| 183 | + $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "mozart": [] }'; |
|
| 184 | + |
|
| 185 | + $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__); |
|
| 186 | + file_put_contents($tmpfname, $badComposerJson); |
|
| 187 | + chdir(dirname($tmpfname)); |
|
| 188 | + |
|
| 189 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
| 190 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
| 191 | + |
|
| 192 | + $outputInterfaceMock->expects($this->exactly(1)) |
|
| 193 | + ->method('write'); |
|
| 194 | + |
|
| 195 | + new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose { |
|
| 196 | + public function __construct($inputInterfaceMock, $outputInterfaceMock) |
|
| 197 | + { |
|
| 198 | + parent::__construct(); |
|
| 199 | + |
|
| 200 | + $this->execute($inputInterfaceMock, $outputInterfaceMock); |
|
| 201 | + } |
|
| 202 | + }; |
|
| 203 | + } |
|
| 204 | 204 | } |
@@ -9,22 +9,22 @@ |
||
| 9 | 9 | class ApplicationTest extends TestCase |
| 10 | 10 | { |
| 11 | 11 | |
| 12 | - /** |
|
| 13 | - * Test the Symfony\Component\Console\Application instance contains the Compose command. |
|
| 14 | - */ |
|
| 15 | - public function testInstantiation() |
|
| 16 | - { |
|
| 12 | + /** |
|
| 13 | + * Test the Symfony\Component\Console\Application instance contains the Compose command. |
|
| 14 | + */ |
|
| 15 | + public function testInstantiation() |
|
| 16 | + { |
|
| 17 | 17 | |
| 18 | - $version = '1.0.0'; |
|
| 18 | + $version = '1.0.0'; |
|
| 19 | 19 | |
| 20 | - $sut = new Application($version); |
|
| 20 | + $sut = new Application($version); |
|
| 21 | 21 | |
| 22 | - $commands = $sut->all(); |
|
| 22 | + $commands = $sut->all(); |
|
| 23 | 23 | |
| 24 | - $containsComposeCommand = array_reduce($commands, function ($carry, $item) { |
|
| 25 | - return $carry || $item instanceof Compose; |
|
| 26 | - }, false); |
|
| 24 | + $containsComposeCommand = array_reduce($commands, function ($carry, $item) { |
|
| 25 | + return $carry || $item instanceof Compose; |
|
| 26 | + }, false); |
|
| 27 | 27 | |
| 28 | - $this->assertTrue($containsComposeCommand); |
|
| 29 | - } |
|
| 28 | + $this->assertTrue($containsComposeCommand); |
|
| 29 | + } |
|
| 30 | 30 | } |
@@ -18,54 +18,54 @@ |
||
| 18 | 18 | */ |
| 19 | 19 | class IntegrationTestCase extends TestCase |
| 20 | 20 | { |
| 21 | - protected $testsWorkingDir; |
|
| 21 | + protected $testsWorkingDir; |
|
| 22 | 22 | |
| 23 | - public function setUp(): void |
|
| 24 | - { |
|
| 25 | - parent::setUp(); |
|
| 23 | + public function setUp(): void |
|
| 24 | + { |
|
| 25 | + parent::setUp(); |
|
| 26 | 26 | |
| 27 | - $this->testsWorkingDir = rtrim(sys_get_temp_dir(), DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR |
|
| 28 | - . 'strausstestdir' . DIRECTORY_SEPARATOR; |
|
| 27 | + $this->testsWorkingDir = rtrim(sys_get_temp_dir(), DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR |
|
| 28 | + . 'strausstestdir' . DIRECTORY_SEPARATOR; |
|
| 29 | 29 | |
| 30 | - if (file_exists($this->testsWorkingDir)) { |
|
| 31 | - $this->deleteDir($this->testsWorkingDir); |
|
| 32 | - } |
|
| 30 | + if (file_exists($this->testsWorkingDir)) { |
|
| 31 | + $this->deleteDir($this->testsWorkingDir); |
|
| 32 | + } |
|
| 33 | 33 | |
| 34 | - @mkdir($this->testsWorkingDir); |
|
| 35 | - } |
|
| 34 | + @mkdir($this->testsWorkingDir); |
|
| 35 | + } |
|
| 36 | 36 | |
| 37 | 37 | |
| 38 | - /** |
|
| 39 | - * Delete $this->testsWorkingDir after each test. |
|
| 40 | - * |
|
| 41 | - * @see https://stackoverflow.com/questions/3349753/delete-directory-with-files-in-it |
|
| 42 | - */ |
|
| 43 | - public function tearDown(): void |
|
| 44 | - { |
|
| 45 | - parent::tearDown(); |
|
| 38 | + /** |
|
| 39 | + * Delete $this->testsWorkingDir after each test. |
|
| 40 | + * |
|
| 41 | + * @see https://stackoverflow.com/questions/3349753/delete-directory-with-files-in-it |
|
| 42 | + */ |
|
| 43 | + public function tearDown(): void |
|
| 44 | + { |
|
| 45 | + parent::tearDown(); |
|
| 46 | 46 | |
| 47 | - $dir = $this->testsWorkingDir; |
|
| 47 | + $dir = $this->testsWorkingDir; |
|
| 48 | 48 | |
| 49 | - $this->deleteDir($dir); |
|
| 50 | - } |
|
| 49 | + $this->deleteDir($dir); |
|
| 50 | + } |
|
| 51 | 51 | |
| 52 | - protected function deleteDir($dir) |
|
| 53 | - { |
|
| 54 | - if (!file_exists($dir)) { |
|
| 55 | - return; |
|
| 56 | - } |
|
| 52 | + protected function deleteDir($dir) |
|
| 53 | + { |
|
| 54 | + if (!file_exists($dir)) { |
|
| 55 | + return; |
|
| 56 | + } |
|
| 57 | 57 | |
| 58 | - $it = new RecursiveDirectoryIterator($dir, RecursiveDirectoryIterator::SKIP_DOTS); |
|
| 59 | - $files = new RecursiveIteratorIterator($it, RecursiveIteratorIterator::CHILD_FIRST); |
|
| 60 | - foreach ($files as $file) { |
|
| 61 | - if (is_link($file)) { |
|
| 62 | - unlink($file); |
|
| 63 | - } elseif ($file->isDir()) { |
|
| 64 | - rmdir($file->getRealPath()); |
|
| 65 | - } else { |
|
| 66 | - unlink($file->getRealPath()); |
|
| 67 | - } |
|
| 68 | - } |
|
| 69 | - rmdir($dir); |
|
| 70 | - } |
|
| 58 | + $it = new RecursiveDirectoryIterator($dir, RecursiveDirectoryIterator::SKIP_DOTS); |
|
| 59 | + $files = new RecursiveIteratorIterator($it, RecursiveIteratorIterator::CHILD_FIRST); |
|
| 60 | + foreach ($files as $file) { |
|
| 61 | + if (is_link($file)) { |
|
| 62 | + unlink($file); |
|
| 63 | + } elseif ($file->isDir()) { |
|
| 64 | + rmdir($file->getRealPath()); |
|
| 65 | + } else { |
|
| 66 | + unlink($file->getRealPath()); |
|
| 67 | + } |
|
| 68 | + } |
|
| 69 | + rmdir($dir); |
|
| 70 | + } |
|
| 71 | 71 | } |
@@ -18,10 +18,10 @@ discard block |
||
| 18 | 18 | class CopierIntegrationTest extends IntegrationTestCase |
| 19 | 19 | { |
| 20 | 20 | |
| 21 | - public function testsPrepareTarget() |
|
| 22 | - { |
|
| 21 | + public function testsPrepareTarget() |
|
| 22 | + { |
|
| 23 | 23 | |
| 24 | - $composerJsonString = <<<'EOD' |
|
| 24 | + $composerJsonString = <<<'EOD' |
|
| 25 | 25 | { |
| 26 | 26 | "name": "brianhenryie/strauss", |
| 27 | 27 | "require": { |
@@ -37,50 +37,50 @@ discard block |
||
| 37 | 37 | } |
| 38 | 38 | EOD; |
| 39 | 39 | |
| 40 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
| 40 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
| 41 | 41 | |
| 42 | - chdir($this->testsWorkingDir); |
|
| 42 | + chdir($this->testsWorkingDir); |
|
| 43 | 43 | |
| 44 | - exec('composer install'); |
|
| 44 | + exec('composer install'); |
|
| 45 | 45 | |
| 46 | - $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
| 46 | + $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
| 47 | 47 | |
| 48 | - $dependencies = array_map(function ($element) { |
|
| 49 | - $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
| 50 | - return new ComposerPackage($dir); |
|
| 51 | - }, $projectComposerPackage->getRequiresNames()); |
|
| 48 | + $dependencies = array_map(function ($element) { |
|
| 49 | + $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
| 50 | + return new ComposerPackage($dir); |
|
| 51 | + }, $projectComposerPackage->getRequiresNames()); |
|
| 52 | 52 | |
| 53 | - $workingDir = $this->testsWorkingDir; |
|
| 54 | - $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR; |
|
| 53 | + $workingDir = $this->testsWorkingDir; |
|
| 54 | + $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR; |
|
| 55 | 55 | |
| 56 | - $config = $this->createStub(StraussConfig::class); |
|
| 56 | + $config = $this->createStub(StraussConfig::class); |
|
| 57 | 57 | |
| 58 | - $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
| 59 | - $fileEnumerator->compileFileList(); |
|
| 60 | - $filepaths = $fileEnumerator->getAllFilesAndDependencyList(); |
|
| 58 | + $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
| 59 | + $fileEnumerator->compileFileList(); |
|
| 60 | + $filepaths = $fileEnumerator->getAllFilesAndDependencyList(); |
|
| 61 | 61 | |
| 62 | - $copier = new Copier($filepaths, $workingDir, $relativeTargetDir); |
|
| 62 | + $copier = new Copier($filepaths, $workingDir, $relativeTargetDir); |
|
| 63 | 63 | |
| 64 | - $file = 'ContainerAwareTrait.php'; |
|
| 65 | - $relativePath = 'league/container/src/'; |
|
| 66 | - $targetPath = $this->testsWorkingDir . $relativeTargetDir . $relativePath; |
|
| 67 | - $targetFile = $targetPath . $file; |
|
| 64 | + $file = 'ContainerAwareTrait.php'; |
|
| 65 | + $relativePath = 'league/container/src/'; |
|
| 66 | + $targetPath = $this->testsWorkingDir . $relativeTargetDir . $relativePath; |
|
| 67 | + $targetFile = $targetPath . $file; |
|
| 68 | 68 | |
| 69 | - mkdir(rtrim($targetPath, DIRECTORY_SEPARATOR), 0777, true); |
|
| 69 | + mkdir(rtrim($targetPath, DIRECTORY_SEPARATOR), 0777, true); |
|
| 70 | 70 | |
| 71 | - file_put_contents($targetFile, 'dummy file'); |
|
| 71 | + file_put_contents($targetFile, 'dummy file'); |
|
| 72 | 72 | |
| 73 | - assert(file_exists($targetFile)); |
|
| 73 | + assert(file_exists($targetFile)); |
|
| 74 | 74 | |
| 75 | - $copier->prepareTarget(); |
|
| 75 | + $copier->prepareTarget(); |
|
| 76 | 76 | |
| 77 | - $this->assertFileDoesNotExist($targetFile); |
|
| 78 | - } |
|
| 77 | + $this->assertFileDoesNotExist($targetFile); |
|
| 78 | + } |
|
| 79 | 79 | |
| 80 | - public function testsCopy() |
|
| 81 | - { |
|
| 80 | + public function testsCopy() |
|
| 81 | + { |
|
| 82 | 82 | |
| 83 | - $composerJsonString = <<<'EOD' |
|
| 83 | + $composerJsonString = <<<'EOD' |
|
| 84 | 84 | { |
| 85 | 85 | "name": "brianhenryie/strauss", |
| 86 | 86 | "require": { |
@@ -96,183 +96,183 @@ discard block |
||
| 96 | 96 | } |
| 97 | 97 | EOD; |
| 98 | 98 | |
| 99 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
| 99 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
| 100 | 100 | |
| 101 | - chdir($this->testsWorkingDir); |
|
| 101 | + chdir($this->testsWorkingDir); |
|
| 102 | 102 | |
| 103 | - exec('composer install'); |
|
| 103 | + exec('composer install'); |
|
| 104 | 104 | |
| 105 | - $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
| 105 | + $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
| 106 | 106 | |
| 107 | - $dependencies = array_map(function ($element) { |
|
| 108 | - $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
| 109 | - return new ComposerPackage($dir); |
|
| 110 | - }, $projectComposerPackage->getRequiresNames()); |
|
| 107 | + $dependencies = array_map(function ($element) { |
|
| 108 | + $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
| 109 | + return new ComposerPackage($dir); |
|
| 110 | + }, $projectComposerPackage->getRequiresNames()); |
|
| 111 | 111 | |
| 112 | - $workingDir = $this->testsWorkingDir; |
|
| 113 | - $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR; |
|
| 114 | - $config = $this->createStub(StraussConfig::class); |
|
| 115 | - $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
| 116 | - $fileEnumerator->compileFileList(); |
|
| 117 | - $filepaths = $fileEnumerator->getAllFilesAndDependencyList(); |
|
| 112 | + $workingDir = $this->testsWorkingDir; |
|
| 113 | + $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR; |
|
| 114 | + $config = $this->createStub(StraussConfig::class); |
|
| 115 | + $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
| 116 | + $fileEnumerator->compileFileList(); |
|
| 117 | + $filepaths = $fileEnumerator->getAllFilesAndDependencyList(); |
|
| 118 | 118 | |
| 119 | - $copier = new Copier($filepaths, $workingDir, $relativeTargetDir); |
|
| 119 | + $copier = new Copier($filepaths, $workingDir, $relativeTargetDir); |
|
| 120 | 120 | |
| 121 | - $file = 'Client.php'; |
|
| 122 | - $relativePath = 'google/apiclient/src/'; |
|
| 123 | - $targetPath = $this->testsWorkingDir . $relativeTargetDir . $relativePath; |
|
| 124 | - $targetFile = $targetPath . $file; |
|
| 121 | + $file = 'Client.php'; |
|
| 122 | + $relativePath = 'google/apiclient/src/'; |
|
| 123 | + $targetPath = $this->testsWorkingDir . $relativeTargetDir . $relativePath; |
|
| 124 | + $targetFile = $targetPath . $file; |
|
| 125 | 125 | |
| 126 | - $copier->prepareTarget(); |
|
| 126 | + $copier->prepareTarget(); |
|
| 127 | 127 | |
| 128 | - $copier->copy(); |
|
| 128 | + $copier->copy(); |
|
| 129 | 129 | |
| 130 | - $this->assertFileExists($targetFile); |
|
| 131 | - } |
|
| 130 | + $this->assertFileExists($targetFile); |
|
| 131 | + } |
|
| 132 | 132 | |
| 133 | 133 | |
| 134 | 134 | |
| 135 | 135 | |
| 136 | - /** |
|
| 137 | - * Set up a common settings object. |
|
| 138 | - * @see MoverTest.php |
|
| 139 | - */ |
|
| 140 | - protected function createComposer(): void |
|
| 141 | - { |
|
| 136 | + /** |
|
| 137 | + * Set up a common settings object. |
|
| 138 | + * @see MoverTest.php |
|
| 139 | + */ |
|
| 140 | + protected function createComposer(): void |
|
| 141 | + { |
|
| 142 | 142 | // parent::setUp(); |
| 143 | 143 | |
| 144 | - $this->testsWorkingDir = __DIR__ . '/temptestdir/'; |
|
| 145 | - if (!file_exists($this->testsWorkingDir)) { |
|
| 146 | - mkdir($this->testsWorkingDir); |
|
| 147 | - } |
|
| 148 | - |
|
| 149 | - $mozartConfig = new stdClass(); |
|
| 150 | - $mozartConfig->dep_directory = "/dep_directory/"; |
|
| 151 | - $mozartConfig->classmap_directory = "/classmap_directory/"; |
|
| 152 | - $mozartConfig->packages = array( |
|
| 153 | - "pimple/pimple", |
|
| 154 | - "ezyang/htmlpurifier" |
|
| 155 | - ); |
|
| 156 | - |
|
| 157 | - $pimpleAutoload = new stdClass(); |
|
| 158 | - $pimpleAutoload->{'psr-0'} = new stdClass(); |
|
| 159 | - $pimpleAutoload->{'psr-0'}->Pimple = "src/"; |
|
| 160 | - |
|
| 161 | - $htmlpurifierAutoload = new stdClass(); |
|
| 162 | - $htmlpurifierAutoload->classmap = new stdClass(); |
|
| 163 | - $htmlpurifierAutoload->classmap->Pimple = "library/"; |
|
| 164 | - |
|
| 165 | - $mozartConfig->override_autoload = array(); |
|
| 166 | - $mozartConfig->override_autoload["pimple/pimple"] = $pimpleAutoload; |
|
| 167 | - $mozartConfig->override_autoload["ezyang/htmlpurifier"] = $htmlpurifierAutoload; |
|
| 168 | - |
|
| 169 | - $composer = new stdClass(); |
|
| 170 | - $composer->extra = new stdClass(); |
|
| 171 | - $composer->extra->mozart = $mozartConfig; |
|
| 172 | - |
|
| 173 | - $composerFilepath = $this->testsWorkingDir . '/composer.json'; |
|
| 174 | - $composerJson = json_encode($composer) ; |
|
| 175 | - file_put_contents($composerFilepath, $composerJson); |
|
| 176 | - |
|
| 177 | - $this->config = StraussConfig::loadFromFile($composerFilepath); |
|
| 178 | - } |
|
| 179 | - |
|
| 180 | - /** |
|
| 181 | - * If the specified `dep_directory` or `classmap_directory` are absent, create them. |
|
| 182 | - * @see MoverTest.php |
|
| 183 | - * @test |
|
| 184 | - */ |
|
| 185 | - public function it_creates_absent_dirs(): void |
|
| 186 | - { |
|
| 187 | - $this->markTestIncomplete(); |
|
| 188 | - |
|
| 189 | - $mover = new Mover($this->testsWorkingDir, $this->config); |
|
| 190 | - |
|
| 191 | - // Make sure the directories don't exist. |
|
| 192 | - assert(! file_exists($this->testsWorkingDir . $this->config->gett()), "{$this->testsWorkingDir}{$this->config->getDepDirectory()} already exists"); |
|
| 193 | - assert(! file_exists($this->testsWorkingDir . $this->config->getClassmapDirectory())); |
|
| 194 | - |
|
| 195 | - $packages = array(); |
|
| 196 | - |
|
| 197 | - $mover->deleteTargetDirs($packages); |
|
| 198 | - |
|
| 199 | - $this->assertTrue(file_exists($this->testsWorkingDir |
|
| 200 | - . $this->config->getDepDirectory())); |
|
| 201 | - $this->assertTrue(file_exists($this->testsWorkingDir |
|
| 202 | - . $this->config->getClassmapDirectory())); |
|
| 203 | - } |
|
| 204 | - |
|
| 205 | - /** |
|
| 206 | - * If the specified `dep_directory` or `classmap_directory` already exists with contents, it is not an issue. |
|
| 207 | - * |
|
| 208 | - * @see MoverTest.php |
|
| 209 | - * |
|
| 210 | - * @test |
|
| 211 | - */ |
|
| 212 | - public function it_is_unpertrubed_by_existing_dirs(): void |
|
| 213 | - { |
|
| 214 | - $this->markTestIncomplete(); |
|
| 215 | - |
|
| 216 | - $mover = new Mover($this->testsWorkingDir, $this->config); |
|
| 217 | - |
|
| 218 | - if (!file_exists($this->testsWorkingDir . $this->config->getDepDirectory())) { |
|
| 219 | - mkdir($this->testsWorkingDir . $this->config->getDepDirectory()); |
|
| 220 | - } |
|
| 221 | - if (!file_exists($this->testsWorkingDir . $this->config->getClassmapDirectory())) { |
|
| 222 | - mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory()); |
|
| 223 | - } |
|
| 224 | - |
|
| 225 | - $this->assertDirectoryExists($this->testsWorkingDir . $this->config->getDepDirectory()); |
|
| 226 | - $this->assertDirectoryExists($this->testsWorkingDir . $this->config->getClassmapDirectory()); |
|
| 227 | - |
|
| 228 | - $packages = array(); |
|
| 229 | - |
|
| 230 | - ob_start(); |
|
| 231 | - |
|
| 232 | - $mover->deleteTargetDirs($packages); |
|
| 233 | - |
|
| 234 | - $output = ob_get_clean(); |
|
| 235 | - |
|
| 236 | - $this->assertEmpty($output); |
|
| 237 | - } |
|
| 238 | - |
|
| 239 | - /** |
|
| 240 | - * If the specified `dep_directory` or `classmap_directory` contains a subdir we are going to need when moving, |
|
| 241 | - * delete the subdir. aka: If subfolders exist for dependencies we are about to manage, delete those subfolders. |
|
| 242 | - * |
|
| 243 | - * @see MoverTest.php |
|
| 244 | - * |
|
| 245 | - * @test |
|
| 246 | - */ |
|
| 247 | - public function it_deletes_subdirs_for_packages_about_to_be_moved(): void |
|
| 248 | - { |
|
| 249 | - $this->markTestIncomplete(); |
|
| 250 | - |
|
| 251 | - $mover = new Mover($this->testsWorkingDir, $this->config); |
|
| 252 | - |
|
| 253 | - @mkdir($this->testsWorkingDir . $this->config->getDepDirectory()); |
|
| 254 | - @mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory()); |
|
| 255 | - |
|
| 256 | - @mkdir($this->testsWorkingDir . $this->config->getDepDirectory() . 'Pimple'); |
|
| 257 | - @mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory() . 'ezyang'); |
|
| 258 | - |
|
| 259 | - $packages = array(); |
|
| 260 | - foreach ($this->config->getPackages() as $packageString) { |
|
| 261 | - $testDummyComposerDir = $this->testsWorkingDir . 'vendor' |
|
| 262 | - . DIRECTORY_SEPARATOR . $packageString; |
|
| 263 | - @mkdir($testDummyComposerDir, 0777, true); |
|
| 264 | - $testDummyComposerPath = $testDummyComposerDir . DIRECTORY_SEPARATOR . 'composer.json'; |
|
| 265 | - $testDummyComposerContents = json_encode(new stdClass()); |
|
| 266 | - |
|
| 267 | - file_put_contents($testDummyComposerPath, $testDummyComposerContents); |
|
| 268 | - $parsedPackage = new ComposerPackageConfig($testDummyComposerDir, $this->config->getOverrideAutoload()[$packageString]); |
|
| 269 | - $parsedPackage->findAutoloaders(); |
|
| 270 | - $packages[] = $parsedPackage; |
|
| 271 | - } |
|
| 272 | - |
|
| 273 | - $mover->deleteTargetDirs($packages); |
|
| 274 | - |
|
| 275 | - $this->assertDirectoryDoesNotExist($this->testsWorkingDir . $this->config->getDepDirectory() . 'Pimple'); |
|
| 276 | - $this->assertDirectoryDoesNotExist($this->testsWorkingDir . $this->config->getDepDirectory() . 'ezyang'); |
|
| 277 | - } |
|
| 144 | + $this->testsWorkingDir = __DIR__ . '/temptestdir/'; |
|
| 145 | + if (!file_exists($this->testsWorkingDir)) { |
|
| 146 | + mkdir($this->testsWorkingDir); |
|
| 147 | + } |
|
| 148 | + |
|
| 149 | + $mozartConfig = new stdClass(); |
|
| 150 | + $mozartConfig->dep_directory = "/dep_directory/"; |
|
| 151 | + $mozartConfig->classmap_directory = "/classmap_directory/"; |
|
| 152 | + $mozartConfig->packages = array( |
|
| 153 | + "pimple/pimple", |
|
| 154 | + "ezyang/htmlpurifier" |
|
| 155 | + ); |
|
| 156 | + |
|
| 157 | + $pimpleAutoload = new stdClass(); |
|
| 158 | + $pimpleAutoload->{'psr-0'} = new stdClass(); |
|
| 159 | + $pimpleAutoload->{'psr-0'}->Pimple = "src/"; |
|
| 160 | + |
|
| 161 | + $htmlpurifierAutoload = new stdClass(); |
|
| 162 | + $htmlpurifierAutoload->classmap = new stdClass(); |
|
| 163 | + $htmlpurifierAutoload->classmap->Pimple = "library/"; |
|
| 164 | + |
|
| 165 | + $mozartConfig->override_autoload = array(); |
|
| 166 | + $mozartConfig->override_autoload["pimple/pimple"] = $pimpleAutoload; |
|
| 167 | + $mozartConfig->override_autoload["ezyang/htmlpurifier"] = $htmlpurifierAutoload; |
|
| 168 | + |
|
| 169 | + $composer = new stdClass(); |
|
| 170 | + $composer->extra = new stdClass(); |
|
| 171 | + $composer->extra->mozart = $mozartConfig; |
|
| 172 | + |
|
| 173 | + $composerFilepath = $this->testsWorkingDir . '/composer.json'; |
|
| 174 | + $composerJson = json_encode($composer) ; |
|
| 175 | + file_put_contents($composerFilepath, $composerJson); |
|
| 176 | + |
|
| 177 | + $this->config = StraussConfig::loadFromFile($composerFilepath); |
|
| 178 | + } |
|
| 179 | + |
|
| 180 | + /** |
|
| 181 | + * If the specified `dep_directory` or `classmap_directory` are absent, create them. |
|
| 182 | + * @see MoverTest.php |
|
| 183 | + * @test |
|
| 184 | + */ |
|
| 185 | + public function it_creates_absent_dirs(): void |
|
| 186 | + { |
|
| 187 | + $this->markTestIncomplete(); |
|
| 188 | + |
|
| 189 | + $mover = new Mover($this->testsWorkingDir, $this->config); |
|
| 190 | + |
|
| 191 | + // Make sure the directories don't exist. |
|
| 192 | + assert(! file_exists($this->testsWorkingDir . $this->config->gett()), "{$this->testsWorkingDir}{$this->config->getDepDirectory()} already exists"); |
|
| 193 | + assert(! file_exists($this->testsWorkingDir . $this->config->getClassmapDirectory())); |
|
| 194 | + |
|
| 195 | + $packages = array(); |
|
| 196 | + |
|
| 197 | + $mover->deleteTargetDirs($packages); |
|
| 198 | + |
|
| 199 | + $this->assertTrue(file_exists($this->testsWorkingDir |
|
| 200 | + . $this->config->getDepDirectory())); |
|
| 201 | + $this->assertTrue(file_exists($this->testsWorkingDir |
|
| 202 | + . $this->config->getClassmapDirectory())); |
|
| 203 | + } |
|
| 204 | + |
|
| 205 | + /** |
|
| 206 | + * If the specified `dep_directory` or `classmap_directory` already exists with contents, it is not an issue. |
|
| 207 | + * |
|
| 208 | + * @see MoverTest.php |
|
| 209 | + * |
|
| 210 | + * @test |
|
| 211 | + */ |
|
| 212 | + public function it_is_unpertrubed_by_existing_dirs(): void |
|
| 213 | + { |
|
| 214 | + $this->markTestIncomplete(); |
|
| 215 | + |
|
| 216 | + $mover = new Mover($this->testsWorkingDir, $this->config); |
|
| 217 | + |
|
| 218 | + if (!file_exists($this->testsWorkingDir . $this->config->getDepDirectory())) { |
|
| 219 | + mkdir($this->testsWorkingDir . $this->config->getDepDirectory()); |
|
| 220 | + } |
|
| 221 | + if (!file_exists($this->testsWorkingDir . $this->config->getClassmapDirectory())) { |
|
| 222 | + mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory()); |
|
| 223 | + } |
|
| 224 | + |
|
| 225 | + $this->assertDirectoryExists($this->testsWorkingDir . $this->config->getDepDirectory()); |
|
| 226 | + $this->assertDirectoryExists($this->testsWorkingDir . $this->config->getClassmapDirectory()); |
|
| 227 | + |
|
| 228 | + $packages = array(); |
|
| 229 | + |
|
| 230 | + ob_start(); |
|
| 231 | + |
|
| 232 | + $mover->deleteTargetDirs($packages); |
|
| 233 | + |
|
| 234 | + $output = ob_get_clean(); |
|
| 235 | + |
|
| 236 | + $this->assertEmpty($output); |
|
| 237 | + } |
|
| 238 | + |
|
| 239 | + /** |
|
| 240 | + * If the specified `dep_directory` or `classmap_directory` contains a subdir we are going to need when moving, |
|
| 241 | + * delete the subdir. aka: If subfolders exist for dependencies we are about to manage, delete those subfolders. |
|
| 242 | + * |
|
| 243 | + * @see MoverTest.php |
|
| 244 | + * |
|
| 245 | + * @test |
|
| 246 | + */ |
|
| 247 | + public function it_deletes_subdirs_for_packages_about_to_be_moved(): void |
|
| 248 | + { |
|
| 249 | + $this->markTestIncomplete(); |
|
| 250 | + |
|
| 251 | + $mover = new Mover($this->testsWorkingDir, $this->config); |
|
| 252 | + |
|
| 253 | + @mkdir($this->testsWorkingDir . $this->config->getDepDirectory()); |
|
| 254 | + @mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory()); |
|
| 255 | + |
|
| 256 | + @mkdir($this->testsWorkingDir . $this->config->getDepDirectory() . 'Pimple'); |
|
| 257 | + @mkdir($this->testsWorkingDir . $this->config->getClassmapDirectory() . 'ezyang'); |
|
| 258 | + |
|
| 259 | + $packages = array(); |
|
| 260 | + foreach ($this->config->getPackages() as $packageString) { |
|
| 261 | + $testDummyComposerDir = $this->testsWorkingDir . 'vendor' |
|
| 262 | + . DIRECTORY_SEPARATOR . $packageString; |
|
| 263 | + @mkdir($testDummyComposerDir, 0777, true); |
|
| 264 | + $testDummyComposerPath = $testDummyComposerDir . DIRECTORY_SEPARATOR . 'composer.json'; |
|
| 265 | + $testDummyComposerContents = json_encode(new stdClass()); |
|
| 266 | + |
|
| 267 | + file_put_contents($testDummyComposerPath, $testDummyComposerContents); |
|
| 268 | + $parsedPackage = new ComposerPackageConfig($testDummyComposerDir, $this->config->getOverrideAutoload()[$packageString]); |
|
| 269 | + $parsedPackage->findAutoloaders(); |
|
| 270 | + $packages[] = $parsedPackage; |
|
| 271 | + } |
|
| 272 | + |
|
| 273 | + $mover->deleteTargetDirs($packages); |
|
| 274 | + |
|
| 275 | + $this->assertDirectoryDoesNotExist($this->testsWorkingDir . $this->config->getDepDirectory() . 'Pimple'); |
|
| 276 | + $this->assertDirectoryDoesNotExist($this->testsWorkingDir . $this->config->getDepDirectory() . 'ezyang'); |
|
| 277 | + } |
|
| 278 | 278 | } |
@@ -23,10 +23,10 @@ discard block |
||
| 23 | 23 | class ReplacerIntegrationTest extends IntegrationTestCase |
| 24 | 24 | { |
| 25 | 25 | |
| 26 | - public function testReplaceNamespace() |
|
| 27 | - { |
|
| 26 | + public function testReplaceNamespace() |
|
| 27 | + { |
|
| 28 | 28 | |
| 29 | - $composerJsonString = <<<'EOD' |
|
| 29 | + $composerJsonString = <<<'EOD' |
|
| 30 | 30 | { |
| 31 | 31 | "name": "brianhenryie/strauss", |
| 32 | 32 | "require": { |
@@ -41,56 +41,56 @@ discard block |
||
| 41 | 41 | } |
| 42 | 42 | EOD; |
| 43 | 43 | |
| 44 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
| 44 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
| 45 | 45 | |
| 46 | - chdir($this->testsWorkingDir); |
|
| 46 | + chdir($this->testsWorkingDir); |
|
| 47 | 47 | |
| 48 | - exec('composer install'); |
|
| 48 | + exec('composer install'); |
|
| 49 | 49 | |
| 50 | - $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
| 51 | - $config = $projectComposerPackage->getStraussConfig(); |
|
| 50 | + $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
| 51 | + $config = $projectComposerPackage->getStraussConfig(); |
|
| 52 | 52 | |
| 53 | - $dependencies = array_map(function ($element) { |
|
| 54 | - $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
| 55 | - return new ComposerPackage($dir); |
|
| 56 | - }, $projectComposerPackage->getRequiresNames()); |
|
| 53 | + $dependencies = array_map(function ($element) { |
|
| 54 | + $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
| 55 | + return new ComposerPackage($dir); |
|
| 56 | + }, $projectComposerPackage->getRequiresNames()); |
|
| 57 | 57 | |
| 58 | - $workingDir = $this->testsWorkingDir; |
|
| 59 | - $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR; |
|
| 60 | - $absoluteTargetDir = $workingDir . $relativeTargetDir; |
|
| 58 | + $workingDir = $this->testsWorkingDir; |
|
| 59 | + $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR; |
|
| 60 | + $absoluteTargetDir = $workingDir . $relativeTargetDir; |
|
| 61 | 61 | |
| 62 | 62 | // $config = $this->createStub(StraussConfig::class); |
| 63 | 63 | // $config->method('getTargetDirectory')->willReturn('strauss' . DIRECTORY_SEPARATOR); |
| 64 | 64 | |
| 65 | - $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
| 66 | - $fileEnumerator->compileFileList(); |
|
| 67 | - $fileList = $fileEnumerator->getAllFilesAndDependencyList(); |
|
| 68 | - $phpFileList = $fileEnumerator->getPhpFilesAndDependencyList(); |
|
| 65 | + $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
| 66 | + $fileEnumerator->compileFileList(); |
|
| 67 | + $fileList = $fileEnumerator->getAllFilesAndDependencyList(); |
|
| 68 | + $phpFileList = $fileEnumerator->getPhpFilesAndDependencyList(); |
|
| 69 | 69 | |
| 70 | - $copier = new Copier($fileList, $workingDir, $relativeTargetDir); |
|
| 71 | - $copier->prepareTarget(); |
|
| 72 | - $copier->copy(); |
|
| 70 | + $copier = new Copier($fileList, $workingDir, $relativeTargetDir); |
|
| 71 | + $copier->prepareTarget(); |
|
| 72 | + $copier->copy(); |
|
| 73 | 73 | |
| 74 | - $changeEnumerator = new ChangeEnumerator($config); |
|
| 75 | - $changeEnumerator->findInFiles($absoluteTargetDir, $phpFileList); |
|
| 74 | + $changeEnumerator = new ChangeEnumerator($config); |
|
| 75 | + $changeEnumerator->findInFiles($absoluteTargetDir, $phpFileList); |
|
| 76 | 76 | |
| 77 | - $namespaces = $changeEnumerator->getDiscoveredNamespaceReplacements(); |
|
| 78 | - $classes = $changeEnumerator->getDiscoveredClasses(); |
|
| 77 | + $namespaces = $changeEnumerator->getDiscoveredNamespaceReplacements(); |
|
| 78 | + $classes = $changeEnumerator->getDiscoveredClasses(); |
|
| 79 | 79 | |
| 80 | - $replacer = new Prefixer($config, $workingDir); |
|
| 80 | + $replacer = new Prefixer($config, $workingDir); |
|
| 81 | 81 | |
| 82 | - $replacer->replaceInFiles($namespaces, $classes, $phpFileList); |
|
| 82 | + $replacer->replaceInFiles($namespaces, $classes, $phpFileList); |
|
| 83 | 83 | |
| 84 | - $updatedFile = file_get_contents($absoluteTargetDir . 'google/apiclient/src/Client.php'); |
|
| 84 | + $updatedFile = file_get_contents($absoluteTargetDir . 'google/apiclient/src/Client.php'); |
|
| 85 | 85 | |
| 86 | - $this->assertStringContainsString('use BrianHenryIE\Strauss\Google\AccessToken\Revoke;', $updatedFile); |
|
| 87 | - } |
|
| 86 | + $this->assertStringContainsString('use BrianHenryIE\Strauss\Google\AccessToken\Revoke;', $updatedFile); |
|
| 87 | + } |
|
| 88 | 88 | |
| 89 | 89 | |
| 90 | - public function testReplaceClass() |
|
| 91 | - { |
|
| 90 | + public function testReplaceClass() |
|
| 91 | + { |
|
| 92 | 92 | |
| 93 | - $composerJsonString = <<<'EOD' |
|
| 93 | + $composerJsonString = <<<'EOD' |
|
| 94 | 94 | { |
| 95 | 95 | "name": "brianhenryie/strauss", |
| 96 | 96 | "require": { |
@@ -106,18 +106,18 @@ discard block |
||
| 106 | 106 | } |
| 107 | 107 | EOD; |
| 108 | 108 | |
| 109 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
| 109 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
| 110 | 110 | |
| 111 | - chdir($this->testsWorkingDir); |
|
| 111 | + chdir($this->testsWorkingDir); |
|
| 112 | 112 | |
| 113 | - exec('composer install'); |
|
| 113 | + exec('composer install'); |
|
| 114 | 114 | |
| 115 | - $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
| 116 | - $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
| 115 | + $inputInterfaceMock = $this->createMock(InputInterface::class); |
|
| 116 | + $outputInterfaceMock = $this->createMock(OutputInterface::class); |
|
| 117 | 117 | |
| 118 | - $mozartCompose = new Compose(); |
|
| 118 | + $mozartCompose = new Compose(); |
|
| 119 | 119 | |
| 120 | - $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
| 120 | + $result = $mozartCompose->run($inputInterfaceMock, $outputInterfaceMock); |
|
| 121 | 121 | |
| 122 | 122 | |
| 123 | 123 | // $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
@@ -150,8 +150,8 @@ discard block |
||
| 150 | 150 | // |
| 151 | 151 | // $replacer->replaceInFiles($namespaces, $classes, $phpFileList); |
| 152 | 152 | |
| 153 | - $updatedFile = file_get_contents($this->testsWorkingDir . '/strauss/' . 'setasign/fpdf/fpdf.php'); |
|
| 153 | + $updatedFile = file_get_contents($this->testsWorkingDir . '/strauss/' . 'setasign/fpdf/fpdf.php'); |
|
| 154 | 154 | |
| 155 | - $this->assertStringContainsString('class BrianHenryIE_Strauss_FPDF', $updatedFile); |
|
| 156 | - } |
|
| 155 | + $this->assertStringContainsString('class BrianHenryIE_Strauss_FPDF', $updatedFile); |
|
| 156 | + } |
|
| 157 | 157 | } |
@@ -19,10 +19,10 @@ discard block |
||
| 19 | 19 | class FileEnumeratorIntegrationTest extends IntegrationTestCase |
| 20 | 20 | { |
| 21 | 21 | |
| 22 | - public function testBuildFileList() |
|
| 23 | - { |
|
| 22 | + public function testBuildFileList() |
|
| 23 | + { |
|
| 24 | 24 | |
| 25 | - $composerJsonString = <<<'EOD' |
|
| 25 | + $composerJsonString = <<<'EOD' |
|
| 26 | 26 | { |
| 27 | 27 | "name": "brianhenryie/strauss", |
| 28 | 28 | "require": { |
@@ -38,41 +38,41 @@ discard block |
||
| 38 | 38 | } |
| 39 | 39 | EOD; |
| 40 | 40 | |
| 41 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
| 41 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
| 42 | 42 | |
| 43 | - chdir($this->testsWorkingDir); |
|
| 43 | + chdir($this->testsWorkingDir); |
|
| 44 | 44 | |
| 45 | - exec('composer install'); |
|
| 45 | + exec('composer install'); |
|
| 46 | 46 | |
| 47 | - $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
| 47 | + $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
| 48 | 48 | |
| 49 | - // Only one because we haven't run "flat dependency list". |
|
| 50 | - $dependencies = array_map(function ($element) { |
|
| 51 | - $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
| 52 | - return new ComposerPackage($dir); |
|
| 53 | - }, $projectComposerPackage->getRequiresNames()); |
|
| 49 | + // Only one because we haven't run "flat dependency list". |
|
| 50 | + $dependencies = array_map(function ($element) { |
|
| 51 | + $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
| 52 | + return new ComposerPackage($dir); |
|
| 53 | + }, $projectComposerPackage->getRequiresNames()); |
|
| 54 | 54 | |
| 55 | - $workingDir = $this->testsWorkingDir; |
|
| 56 | - $config = $this->createStub(StraussConfig::class); |
|
| 55 | + $workingDir = $this->testsWorkingDir; |
|
| 56 | + $config = $this->createStub(StraussConfig::class); |
|
| 57 | 57 | |
| 58 | - $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
| 58 | + $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
| 59 | 59 | |
| 60 | - $fileEnumerator->compileFileList(); |
|
| 60 | + $fileEnumerator->compileFileList(); |
|
| 61 | 61 | |
| 62 | - $list = array_keys($fileEnumerator->getAllFilesAndDependencyList()); |
|
| 62 | + $list = array_keys($fileEnumerator->getAllFilesAndDependencyList()); |
|
| 63 | 63 | |
| 64 | - $this->assertContains('google/apiclient/src/aliases.php', $list); |
|
| 65 | - } |
|
| 64 | + $this->assertContains('google/apiclient/src/aliases.php', $list); |
|
| 65 | + } |
|
| 66 | 66 | |
| 67 | 67 | |
| 68 | - public function testClassmapAutoloader() |
|
| 69 | - { |
|
| 70 | - $this->markTestIncomplete(); |
|
| 71 | - } |
|
| 68 | + public function testClassmapAutoloader() |
|
| 69 | + { |
|
| 70 | + $this->markTestIncomplete(); |
|
| 71 | + } |
|
| 72 | 72 | |
| 73 | 73 | |
| 74 | - public function testFilesAutoloader() |
|
| 75 | - { |
|
| 76 | - $this->markTestIncomplete(); |
|
| 77 | - } |
|
| 74 | + public function testFilesAutoloader() |
|
| 75 | + { |
|
| 76 | + $this->markTestIncomplete(); |
|
| 77 | + } |
|
| 78 | 78 | } |
@@ -18,13 +18,13 @@ discard block |
||
| 18 | 18 | class ChangeEnumeratorIntegrationTest extends IntegrationTestCase |
| 19 | 19 | { |
| 20 | 20 | |
| 21 | - /** |
|
| 22 | - * Given a list of files, find all the global classes and the namespaces. |
|
| 23 | - */ |
|
| 24 | - public function testOne() |
|
| 25 | - { |
|
| 21 | + /** |
|
| 22 | + * Given a list of files, find all the global classes and the namespaces. |
|
| 23 | + */ |
|
| 24 | + public function testOne() |
|
| 25 | + { |
|
| 26 | 26 | |
| 27 | - $composerJsonString = <<<'EOD' |
|
| 27 | + $composerJsonString = <<<'EOD' |
|
| 28 | 28 | { |
| 29 | 29 | "name": "brianhenryie/strauss", |
| 30 | 30 | "require": { |
@@ -40,53 +40,53 @@ discard block |
||
| 40 | 40 | } |
| 41 | 41 | EOD; |
| 42 | 42 | |
| 43 | - file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
| 43 | + file_put_contents($this->testsWorkingDir . '/composer.json', $composerJsonString); |
|
| 44 | 44 | |
| 45 | - chdir($this->testsWorkingDir); |
|
| 45 | + chdir($this->testsWorkingDir); |
|
| 46 | 46 | |
| 47 | - exec('composer install'); |
|
| 47 | + exec('composer install'); |
|
| 48 | 48 | |
| 49 | - $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
| 49 | + $projectComposerPackage = new ProjectComposerPackage($this->testsWorkingDir); |
|
| 50 | 50 | |
| 51 | - $dependencies = array_map(function ($element) { |
|
| 52 | - $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
| 53 | - return new ComposerPackage($dir); |
|
| 54 | - }, $projectComposerPackage->getRequiresNames()); |
|
| 51 | + $dependencies = array_map(function ($element) { |
|
| 52 | + $dir = $this->testsWorkingDir . 'vendor'. DIRECTORY_SEPARATOR . $element; |
|
| 53 | + return new ComposerPackage($dir); |
|
| 54 | + }, $projectComposerPackage->getRequiresNames()); |
|
| 55 | 55 | |
| 56 | - $workingDir = $this->testsWorkingDir; |
|
| 57 | - $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR; |
|
| 56 | + $workingDir = $this->testsWorkingDir; |
|
| 57 | + $relativeTargetDir = 'strauss' . DIRECTORY_SEPARATOR; |
|
| 58 | 58 | |
| 59 | - $config = $this->createStub(StraussConfig::class); |
|
| 59 | + $config = $this->createStub(StraussConfig::class); |
|
| 60 | 60 | |
| 61 | - $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
| 61 | + $fileEnumerator = new FileEnumerator($dependencies, $workingDir, $config); |
|
| 62 | 62 | |
| 63 | - $fileEnumerator->compileFileList(); |
|
| 63 | + $fileEnumerator->compileFileList(); |
|
| 64 | 64 | |
| 65 | - $copier = new Copier($fileEnumerator->getAllFilesAndDependencyList(), $workingDir, $relativeTargetDir); |
|
| 65 | + $copier = new Copier($fileEnumerator->getAllFilesAndDependencyList(), $workingDir, $relativeTargetDir); |
|
| 66 | 66 | |
| 67 | - $copier->prepareTarget(); |
|
| 67 | + $copier->prepareTarget(); |
|
| 68 | 68 | |
| 69 | - $copier->copy(); |
|
| 69 | + $copier->copy(); |
|
| 70 | 70 | |
| 71 | - $config = $this->createStub(StraussConfig::class); |
|
| 71 | + $config = $this->createStub(StraussConfig::class); |
|
| 72 | 72 | |
| 73 | - $config->method('getExcludePackagesFromPrefixing')->willReturn(array()); |
|
| 74 | - $config->method('getExcludeNamespacesFromPrefixing')->willReturn(array()); |
|
| 73 | + $config->method('getExcludePackagesFromPrefixing')->willReturn(array()); |
|
| 74 | + $config->method('getExcludeNamespacesFromPrefixing')->willReturn(array()); |
|
| 75 | 75 | |
| 76 | - $changeEnumerator = new ChangeEnumerator($config); |
|
| 76 | + $changeEnumerator = new ChangeEnumerator($config); |
|
| 77 | 77 | |
| 78 | - $phpFileList = $fileEnumerator->getPhpFilesAndDependencyList(); |
|
| 78 | + $phpFileList = $fileEnumerator->getPhpFilesAndDependencyList(); |
|
| 79 | 79 | |
| 80 | - $changeEnumerator->findInFiles($workingDir . $relativeTargetDir, $phpFileList); |
|
| 80 | + $changeEnumerator->findInFiles($workingDir . $relativeTargetDir, $phpFileList); |
|
| 81 | 81 | |
| 82 | 82 | |
| 83 | - $classes = $changeEnumerator->getDiscoveredClasses(); |
|
| 83 | + $classes = $changeEnumerator->getDiscoveredClasses(); |
|
| 84 | 84 | |
| 85 | - $namespaces = $changeEnumerator->getDiscoveredNamespaceReplacements(); |
|
| 85 | + $namespaces = $changeEnumerator->getDiscoveredNamespaceReplacements(); |
|
| 86 | 86 | |
| 87 | - $this->assertNotEmpty($classes); |
|
| 88 | - $this->assertNotEmpty($namespaces); |
|
| 87 | + $this->assertNotEmpty($classes); |
|
| 88 | + $this->assertNotEmpty($namespaces); |
|
| 89 | 89 | |
| 90 | - $this->assertContains('Google_Task_Composer', $classes); |
|
| 91 | - } |
|
| 90 | + $this->assertContains('Google_Task_Composer', $classes); |
|
| 91 | + } |
|
| 92 | 92 | } |