Skip to content

Commit ac21f9e

Browse files
authored
Merge pull request #76 from vudaltsov/patch-1
Service/alias names might be fqcns
2 parents 54b1242 + ef2f06a commit ac21f9e

File tree

2 files changed

+13
-1
lines changed

2 files changed

+13
-1
lines changed

PhpUnit/ContainerBuilderHasAliasConstraint.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ private function evaluateServiceId(ContainerBuilder $containerBuilder, $returnRe
8080
*/
8181
$actualServiceId = (string) $alias;
8282

83-
$constraint = new IsEqual(strtolower($this->expectedServiceId));
83+
$constraint = new IsEqual($this->expectedServiceId);
8484
if (!$constraint->evaluate($actualServiceId, '', true)) {
8585
if ($returnResult) {
8686
return false;

Tests/PhpUnit/ContainerBuilderHasAliasConstraintTest.php

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,4 +78,16 @@ public function it_expects_a_string_for_service_id()
7878

7979
new ContainerBuilderHasAliasConstraint('alias_id', new \stdClass());
8080
}
81+
82+
/**
83+
* @test
84+
*/
85+
public function it_does_not_change_case_of_aliased_service_ids()
86+
{
87+
$containerBuilder = new ContainerBuilder();
88+
$containerBuilder->setAlias('Interface', 'InterfaceImplementationService');
89+
$constraint = new ContainerBuilderHasAliasConstraint('Interface', 'InterfaceImplementationService');
90+
91+
$this->assertTrue($constraint->evaluate($containerBuilder, null, true));
92+
}
8193
}

0 commit comments

Comments
 (0)