Skip to content

Commit

Permalink
Merge pull request #580 from vyuldashev/fix/pint-nullable-type-declar…
Browse files Browse the repository at this point in the history
…ations

Fix bottlenecks for tests
  • Loading branch information
adm-bome authored Jan 26, 2024
2 parents 03e1051 + a7de79e commit a6f430f
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 3 deletions.
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
"php": "^8.0",
"ext-json": "*",
"illuminate/queue": "^9.0|^10.0",
"php-amqplib/php-amqplib": "^v3.5.2"
"php-amqplib/php-amqplib": "^v3.6"
},
"require-dev": {
"phpunit/phpunit": "^9.3",
Expand Down
8 changes: 8 additions & 0 deletions pint.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
{
"preset": "laravel",
"rules": {
"nullable_type_declaration_for_default_null_value": {
"use_nullable_type_declaration": false
}
}
}
1 change: 0 additions & 1 deletion src/Queue/Connection/ConnectionFactory.php
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,6 @@ protected static function createStreamConnection($connection, AMQPConnectionConf
'keepalive' => $config->isKeepalive(),
'heartbeat' => $config->getHeartbeat(),
],
$config->getNetworkProtocol(),
$config
);
}
Expand Down
2 changes: 1 addition & 1 deletion tests/Feature/TestCase.php
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@ public function testPushAfterCommit(): void
$this->assertSame(0, Queue::size());
$this->assertNull(Queue::pop());

$transaction->commit('FakeDBConnection');
$transaction->commit('FakeDBConnection', 1, 0);

sleep(1);

Expand Down

0 comments on commit a6f430f

Please sign in to comment.