8000 [Meta] fix issues found via PHPStan in Config component · symfony/symfony@54cc2b5 · GitHub
[go: up one dir, main page]
Skip to content

Commit 54cc2b5

Browse files
dkarlovidkarlovi
committed
[Meta] fix issues found via PHPStan in Config component
1 parent 9183062 commit 54cc2b5

File tree

5 files changed

+15
-12
lines changed

5 files changed

+15
-12
lines changed

‎phpstan.neon

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ parameters:
55
- vendor/bin/.phpunit/phpunit-5.7/vendor/autoload.php
66
ignoreErrors:
77
- '#__construct\(\) does not call parent constructor from .+#'
8+
- '#Function opcache_[^\s]* not found.#'
89
- '#Function pcntl_[^\s]* not found.#'
910

1011
# not errors, actually expected to fail
@@ -21,6 +22,8 @@ parameters:
2122
- src/Symfony/Bundle/FrameworkBundle/Tests/Templating/Helper/Resources
2223

2324
- */src/Symfony/Component/*/Tests/Fixtures/*
25+
- */src/Symfony/Component/*/Tests/*/Fixtures/*
26+
- */src/Symfony/Component/*/Tests/Resource/*
2427

2528
- vendor/
2629

@@ -30,3 +33,6 @@ parameters:
3033
# temporary, currently crashing PHPStan 0.9.1
3134
- src/Symfony/Bundle/FrameworkBundle/Tests/Controller/AbstractControllerTest.php
3235
- src/Symfony/Bundle/FrameworkBundle/Tests/Controller/ControllerTest.php
36+
37+
# temporary, loading a NotLoadableClass throws a fatal error
38+
- src/Symfony/Component/VarDumper/Tests/Caster/ReflectionCasterTest.php

‎src/Symfony/Component/Config/Tests/Definition/Builder/TreeBuilderTest.php

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -12,12 +12,9 @@
1212
namespace Symfony\Component\Config\Tests\Definition\Builder;
1313

1414
use PHPUnit\Framework\TestCase;
15-
use Symfony\Component\Config\Tests\Definition\Builder\NodeBuilder as CustomNodeBuilder;
15+
use Symfony\Component\Config\Tests\Fixtures\Builder\NodeBuilder as CustomNodeBuilder;
1616
use Symfony\Component\Config\Definition\Builder\TreeBuilder;
17-
18-
require __DIR__.'/../../Fixtures/Builder/NodeBuilder.php';
19-
require __DIR__.'/../../Fixtures/Builder/BarNodeDefinition.php';
20-
require __DIR__.'/../../Fixtures/Builder/VariableNodeDefinition.php';
17+
use Symfony\Component\Config\Tests\Fixtures\Builder\VariableNodeDefinition;
2118

2219
class TreeBuilderTest extends TestCase
2320
{
@@ -28,11 +25,11 @@ public function testUsingACustomNodeBuilder()
2825

2926
$nodeBuilder = $root->children();
3027

31-
$this->assertInstanceOf('Symfony\Component\Config\Tests\Definition\Builder\NodeBuilder', $nodeBuilder);
28+
$this->assertInstanceOf('Symfony\Component\Config\Tests\Fixtures\Builder\NodeBuilder', $nodeBuilder);
3229

3330
$nodeBuilder = $nodeBuilder->arrayNode('deeper')->children();
3431

35-
$this->assertInstanceOf('Symfony\Component\Config\Tests\Definition\Builder\NodeBuilder', $nodeBuilder);
32+
$this->assertInstanceOf('Symfony\Component\Config\Tests\Fixtures\Builder\NodeBuilder', $nodeBuilder);
3633
}
3734

3835
public function testOverrideABuiltInNodeType()
@@ -42,7 +39,7 @@ public function testOverrideABuiltInNodeType()
4239

4340
$definition = $root->children()->variableNode('variable');
4441

45-
$this->assertInstanceOf('Symfony\Component\Config\Tests\Definition\Builder\VariableNodeDefinition', $definition);
42+
$this->assertInstanceOf('Symfony\Component\Config\Tests\Fixtures\Builder\VariableNodeDefinition', $definition);
4643
}
4744

4845
public function testAddANodeType()
@@ -52,7 +49,7 @@ public function testAddANodeType()
5249

5350
$definition = $root->children()->barNode('variable');
5451

55-
$this->assertInstanceOf('Symfony\Component\Config\Tests\Definition\Builder\BarNodeDefinition', $definition);
52+
$this->assertInstanceOf('Symfony\Component\Config\Tests\Fixtures\Builder\BarNodeDefinition', $definition);
5653
}
5754

5855
public function testCreateABuiltInNodeTypeWithACustomNodeBuilder()

‎src/Symfony/Component/Config/Tests/Fixtures/Builder/BarNodeDefinition.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
* file that was distributed with this source code.
1010
*/
1111

12-
namespace Symfony\Component\Config\Tests\Definition\Builder;
12+
namespace Symfony\Component\Config\Tests\Fixtures\Builder;
1313

1414
use Symfony\Component\Config\Definition\Builder\NodeDefinition;
1515
use Symfony\Component\Config\Tests\Fixtures\BarNode;

‎src/Symfony/Component/Config/Tests/Fixtures/Builder/NodeBuilder.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
* file that was distributed with this source code.
1010
*/
1111

12-
namespace Symfony\Component\Config\Tests\Definition\Builder;
12+
namespace Symfony\Component\Config\Tests\Fixtures\Builder;
1313

1414
use Symfony\Component\Config\Definition\Builder\NodeBuilder as BaseNodeBuilder;
1515

‎src/Symfony/Component/Config/Tests/Fixtures/Builder/VariableNodeDefinition.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
* file that was distributed with this source code.
1010
*/
1111

12-
namespace Symfony\Component\Config\Tests\Definition\Builder;
12+
namespace Symfony\Component\Config\Tests\Fixtures\Builder;
1313

1414
use Symfony\Component\Config\Definition\Builder\VariableNodeDefinition as BaseVariableNodeDefinition;
1515

0 commit comments

Comments
 (0)
0