8000 minor #49747 [FrameworkBundle]  fix tests (xabbuh) · symfony/symfony@ef385d6 · GitHub
[go: up one dir, main page]
Skip to content
8000

Commit ef385d6

Browse files
nicolas-grekasnicolas-grekas
committed
minor #49747 [FrameworkBundle]  fix tests (xabbuh)
This PR was merged into the 6.3 branch. Discussion ---------- [FrameworkBundle]  fix tests | Q | A | ------------- | --- | Branch? | 6.3 | Bug fix? | no | New feature? | no | Deprecations? | no | Tickets | | License | MIT | Doc PR | Commits ------- e8fa3f4 fix tests
2 parents 78a7c5b + e8fa3f4 commit ef385d6

File tree

4 files changed

+26
-6
lines changed

4 files changed

+26
-6
lines changed

‎src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Fixtures/php/notifier_without_messenger.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,4 +28,5 @@
2828
['email' => 'test@test.de', 'phone' => '+490815',],
2929
]
3030
],
31+
'scheduler' => false,
3132
]);

‎src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Fixtures/xml/notifier_without_messenger.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,5 +17,6 @@
1717
<framework:admin-recipient email="test@test.de" phone="+490815" />
1818
</framework:admin-recipients>
1919
</framework:notifier>
20+
<framework:scheduler enabled="false" />
2021
</framework:config>
2122
</container>

‎src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Fixtures/yml/notifier_without_messenger.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,3 +16,4 @@ framework:
1616
high: ['slack', 'twilio']
1717
admin_recipients:
1818
- { email: 'test@test.de', phone: '+490815' }
19+
scheduler: false

‎src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/FrameworkExtensionTestCase.php

Lines changed: 23 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,10 @@
5757
use Symfony\Component\HttpClient\ScopingHttpClient;
5858
use Symfony\Component\HttpKernel\DependencyInjection\LoggerPass;
5959
use Symfony\Component\HttpKernel\Fragment\FragmentUriGeneratorInterface;
60+
use Symfony\Component\Messenger\Bridge\AmazonSqs\Transport\AmazonSqsTransportFactory;
61+
use Symfony\Component\Messenger\Bridge\Amqp\Transport\AmqpTransportFactory;
62+
use Symfony\Component\Messenger\Bridge\Beanstalkd\Transport\BeanstalkdTransportFactory;
63+
use Symfony\Component\Messenger\Bridge\Redis\Transport\RedisTransportFactory;
6064
use Symfony\Component\Messenger\Transport\TransportFactory;
6165
use Symfony\Component\Notifier\ChatterInterface;
6266
use Symfony\Component\Notifier\TexterInterface;
@@ -814,14 +818,27 @@ public function testMessenger()
814818

815819
$expectedFactories = [
816820
new Reference('scheduler.messenger_transport_factory'),
817-
new Reference('messenger.transport.amqp.factory'),
818-
new Reference('messenger.transport.redis.factory'),
819-
new Reference('messenger.transport.sync.factory'),
820-
new Reference('messenger.transport.in_memory.factory'),
821-
new Reference('messenger.transport.sqs.factory'),
822-
new Reference('messenger.transport.beanstalkd.factory'),
823821
];
824822

823+
if (class_exists(AmqpTransportFactory::class)) {
824+
$expectedFactories[] = 'messenger.transport.amqp.factory';
825+
}
826+
827+
if (class_exists(RedisTransportFactory::class)) {
828+
$expectedFactories[] = 'messenger.transport.redis.factory';
829+
}
830+
831+
$expectedFactories[] = 'messenger.transport.sync.factory';
832+
$expectedFactories[] = 'messenger.transport.in_memory.factory';
833+
834+
if (class_exists(AmazonSqsTransportFactory::class)) {
835+
$expectedFactories[] = 'messenger.transport.sqs.factory';
836+
}
837+
838+
if (class_exists(BeanstalkdTransportFactory::class)) {
839+
$expectedFactories[] = 'messenger.transport.beanstalkd.factory';
840+
}
841+
825842
$this->assertTrue($container->hasDefinition('messenger.receiver_locator'));
826843
$this->assertTrue($container->hasDefinition('console.command.messenger_consume_messages'));
827844
$this->assertTrue($container->hasAlias('messenger.default_bus'));

0 commit comments

Comments
 (0)
0