Skip to content

Commit c951fa5

Browse files
committed
Merge branch '7.1' into 7.2
* 7.1: fix functional tests fix merge fix XSD to allow to configure locks without resources
2 parents daa7b62 + 69e306c commit c951fa5

File tree

3 files changed

+9
-5
lines changed

3 files changed

+9
-5
lines changed

Resources/config/schema/symfony-1.0.xsd

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -531,7 +531,7 @@
531531

532532
<xsd:complexType name="lock">
533533
<xsd:sequence>
534-
<xsd:element name="resource" type="lock_resource" minOccurs="1" maxOccurs="unbounded" />
534+
<xsd:element name="resource" type="lock_resource" minOccurs="0" maxOccurs="unbounded" />
535535
</xsd:sequence>
536536
<xsd:attribute name="enabled" type="xsd:boolean" />
537537
</xsd:complexType>

Tests/DependencyInjection/Fixtures/xml/lock.xml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,6 @@
88
<framework:config http-method-override="false" handle-all-throwables="true">
99
<framework:annotations enabled="false" />
1010
<framework:php-errors log="true" />
11-
<framework:lock>
12-
<framework:resource>semaphore</framework:resource>
13-
</framework:lock>
11+
<framework:lock />
1412
</framework:config>
1513
</container>

Tests/DependencyInjection/FrameworkExtensionTestCase.php

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,7 @@
5757
use Symfony\Component\HttpFoundation\IpUtils;
5858
use Symfony\Component\HttpKernel\DependencyInjection\LoggerPass;
5959
use Symfony\Component\HttpKernel\Fragment\FragmentUriGeneratorInterface;
60+
use Symfony\Component\Lock\Store\SemaphoreStore;
6061
use Symfony\Component\Messenger\Bridge\AmazonSqs\Transport\AmazonSqsTransportFactory;
6162
use Symfony\Component\Messenger\Bridge\Amqp\Transport\AmqpTransportFactory;
6263
use Symfony\Component\Messenger\Bridge\Beanstalkd\Transport\BeanstalkdTransportFactory;
@@ -2399,7 +2400,12 @@ public function testDefaultLock()
23992400

24002401
self::assertTrue($container->hasDefinition('lock.default.factory'));
24012402
$storeDef = $container->getDefinition($container->getDefinition('lock.default.factory')->getArgument(0));
2402-
self::assertEquals(new Reference('semaphore'), $storeDef->getArgument(0));
2403+
2404+
if (class_exists(SemaphoreStore::class) && SemaphoreStore::isSupported()) {
2405+
self::assertEquals(new Reference('semaphore'), $storeDef->getArgument(0));
2406+
} else {
2407+
self::assertEquals(new Reference('flock'), $storeDef->getArgument(0));
2408+
}
24032409
}
24042410

24052411
public function testNamedLocks()

0 commit comments

Comments
 (0)