Skip to content

Commit 8f91266

Browse files
committed
Remove :void return types from tests
1 parent c6b64c3 commit 8f91266

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

tests/DependencyInjection/ConfigurationTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -541,7 +541,7 @@ public function testConsoleFormatterOptionsRename()
541541
/**
542542
* @dataProvider processPsr3MessagesProvider
543543
*/
544-
public function testWithProcessPsr3Messages(array $configuration, array $processedConfiguration): void
544+
public function testWithProcessPsr3Messages(array $configuration, array $processedConfiguration)
545545
{
546546
$configs = [
547547
[

tests/DependencyInjection/MonologExtensionTest.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -788,7 +788,7 @@ public function testProcessorAutoConfiguration()
788788
/**
789789
* @requires PHP 8.0
790790
*/
791-
public function testAsMonologProcessorAutoconfigurationRedeclareMethod(): void
791+
public function testAsMonologProcessorAutoconfigurationRedeclareMethod()
792792
{
793793
if (!class_exists(AsMonologProcessor::class, true)) {
794794
$this->markTestSkipped('Monolog >= 2.3.6 is needed.');
@@ -805,7 +805,7 @@ public function testAsMonologProcessorAutoconfigurationRedeclareMethod(): void
805805
/**
806806
* @requires PHP 8.0
807807
*/
808-
public function testAsMonologProcessorAutoconfiguration(): void
808+
public function testAsMonologProcessorAutoconfiguration()
809809
{
810810
if (!class_exists(AsMonologProcessor::class, true) || property_exists(AsMonologProcessor::class, 'priority')) {
811811
$this->markTestSkipped('Monolog >= 2.3.6 and < 3.4.0 is needed.');
@@ -832,7 +832,7 @@ public function testAsMonologProcessorAutoconfiguration(): void
832832
/**
833833
* @requires PHP 8.0
834834
*/
835-
public function testAsMonologProcessorAutoconfigurationWithPriority(): void
835+
public function testAsMonologProcessorAutoconfigurationWithPriority()
836836
{
837837
if (!class_exists(AsMonologProcessor::class, true) || !property_exists(AsMonologProcessor::class, 'priority')) {
838838
$this->markTestSkipped('Monolog >= 3.4.0 is needed.');
@@ -861,7 +861,7 @@ public function testAsMonologProcessorAutoconfigurationWithPriority(): void
861861
/**
862862
* @requires PHP 8.0
863863
*/
864-
public function testWithLoggerChannelAutoconfiguration(): void
864+
public function testWithLoggerChannelAutoconfiguration()
865865
{
866866
if (!class_exists(WithMonologChannel::class)) {
867867
$this->markTestSkipped('Monolog >= 3.5.0 is needed.');
@@ -917,15 +917,15 @@ public function testElasticsearchAndElasticaHandlers()
917917

918918
// Elasticsearch handler should receive the elasticsearch.client as first argument
919919
$esHandler = $container->getDefinition('monolog.handler.es_handler');
920-
$this->assertSame(ElasticsearchHandler::class,$esHandler->getClass());
920+
$this->assertSame(ElasticsearchHandler::class, $esHandler->getClass());
921921
$esClient = $esHandler->getArgument(0);
922922
$this->assertInstanceOf(Definition::class, $esClient);
923923
$this->assertStringEndsWith('Elasticsearch\Client', $esClient->getClass());
924924
$this->assertSame(['hosts' => ['es:9200']], $esClient->getArgument(0));
925925

926926
// Elastica handler should receive the elastica.client as first argument
927927
$elasticaHandler = $container->getDefinition('monolog.handler.elastica_handler');
928-
$this->assertSame(ElasticaHandler::class,$elasticaHandler->getClass());
928+
$this->assertSame(ElasticaHandler::class, $elasticaHandler->getClass());
929929
$elasticaClient = $elasticaHandler->getArgument(0);
930930
$this->assertInstanceOf(Definition::class, $elasticaClient);
931931
$this->assertSame('Elastica\Client', $elasticaClient->getClass());

0 commit comments

Comments
 (0)