Skip to content

Commit

Permalink
Merge pull request #42 from tienvx/fix-tests-failed-with-symfony-6.1
Browse files Browse the repository at this point in the history
Fix tests failed with Symfony 6.1
  • Loading branch information
tienvx committed Jun 30, 2022
2 parents 1d2cda9 + 4a8619f commit 56c8ed1
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
php-versions: ['7.4', '8.0']
php-versions: ['7.4', '8.0', '8.1']
dependencies: [ 'lowest', 'locked', 'highest' ]
name: PHP ${{ matrix.php-versions }} with ${{ matrix.dependencies }} dependencies
steps:
Expand Down
4 changes: 2 additions & 2 deletions tests/Kernel/AppKernelTrait.php
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,12 @@

trait AppKernelTrait
{
public function getCacheDir()
public function getCacheDir(): string
{
return $this->createTmpDir('cache');
}

public function getLogDir()
public function getLogDir(): string
{
return $this->createTmpDir('logs');
}
Expand Down
2 changes: 1 addition & 1 deletion tests/Kernel/EmptyAppKernel.php
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ class EmptyAppKernel extends Kernel
{
use AppKernelTrait;

public function registerBundles()
public function registerBundles(): iterable
{
return [new CollectionJsBundle()];
}
Expand Down
2 changes: 1 addition & 1 deletion tests/Kernel/FrameworkAppKernel.php
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ class FrameworkAppKernel extends Kernel
{
use AppKernelTrait;

public function registerBundles()
public function registerBundles(): iterable
{
return [new FrameworkBundle(), new CollectionJsBundle()];
}
Expand Down
2 changes: 1 addition & 1 deletion tests/Kernel/TwigAppKernel.php
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ class TwigAppKernel extends Kernel
{
use AppKernelTrait;

public function registerBundles()
public function registerBundles(): iterable
{
return [new FrameworkBundle(), new TwigBundle(), new CollectionJsBundle()];
}
Expand Down

0 comments on commit 56c8ed1

Please sign in to comment.