Allow Symfony 6

Author: norkunas

.github/workflows/ci.yml

@@ -53,8 +53,22 @@ jobs:
             max-parallel: 10
             fail-fast: true
             matrix:
-                php: ['7.3', '7.4', '8.0']
-                sf_version: ['4.4.*', '5.2.*', '5.3.*']
+                strategy: ['default']
+                php: ['7.3', '7.4', '8.0', '8.1']
+                include:
+                    - php: '7.3'
+                      sf_version: '4.4.*'
+                      strategy: 'lowest'
+                    - php: '7.4'
+                      sf_version: '5.4.*'
+                    - php: '8.0'
+                      sf_version: '5.4.*'
+                    - php: '8.1'
+                      sf_version: '5.4.*'
+                    - php: '8.0'
+                      sf_version: '6.0.*'
+                    - php: '8.1'
+                      sf_version: '6.0.*'
         steps:
             - name: "Setup PHP"
               uses: shivammathur/setup-php@v2
@@ -66,37 +80,14 @@ jobs:
             - name: "Checkout code"
               uses: actions/checkout@v2
 
-            - name: "Install Composer dependencies"
+            - name: "Install Composer dependencies (default)"
+              if: matrix.strategy != 'lowest'
               env:
                   SYMFONY_REQUIRE: ${{ matrix.sf_version }}
               run: composer update --no-interaction --optimize-autoloader
 
-            - name: "Run tests"
-              env:
-                  SYMFONY_DEPRECATIONS_HELPER: 'max[self]=2'
-              run: ./vendor/bin/simple-phpunit -v --testsuite main
-
-    phpunit-lowest:
-        name: "PHPUnit (Lowest)"
-        runs-on: ubuntu-latest
-        strategy:
-            max-parallel: 10
-            fail-fast: true
-            matrix:
-                php: ['7.4']
-                sf_version: ['4.4.*', '5.2.*', '5.3.*']
-        steps:
-            - name: "Setup PHP"
-              uses: shivammathur/setup-php@v2
-              with:
-                  php-version: ${{ matrix.php }}
-                  tools: composer:v2, flex
-                  coverage: none
-
-            - name: "Checkout code"
-              uses: actions/checkout@v2
-
-            - name: "Install Composer dependencies"
+            - name: "Install Composer dependencies (lowest)"
+              if: matrix.strategy == 'lowest'
               env:
                   SYMFONY_REQUIRE: ${{ matrix.sf_version }}
               run: composer update --no-interaction --prefer-stable --prefer-lowest --optimize-autoloader
@@ -113,8 +104,20 @@ jobs:
             max-parallel: 10
             fail-fast: true
             matrix:
-                php: ['7.3', '7.4', '8.0']
-                sf_version: ['4.4.*', '5.2.*', '5.3.*']
+                php: ['7.4', '8.0', '8.1']
+                include:
+                    - php: '7.4'
+                      sf_version: '4.4.*'
+                    - php: '7.4'
+                      sf_version: '5.4.*'
+                    - php: '8.0'
+                      sf_version: '5.4.*'
+                    - php: '8.1'
+                      sf_version: '5.4.*'
+                    - php: '8.0'
+                      sf_version: '6.0.*'
+                    - php: '8.1'
+                      sf_version: '6.0.*'
         steps:
             - name: "Setup PHP"
               uses: shivammathur/setup-php@v2

CHANGELOG.md

@@ -2,6 +2,12 @@
 
 The changelog describes what have been "Added", "Changed", "Removed" or "Fixed" between versions.
 
+## Version 5.16.1
+
+### Fixed
+
+- Allow Symfony 6
+
 ## Version 5.16.0
 
 ### Added

Command/GeocodeCommand.php

@@ -62,6 +62,8 @@ protected function configure()
 
     /**
      * {@inheritdoc}
+     *
+     * @return int
      */
     protected function execute(InputInterface $input, OutputInterface $output)
     {

DependencyInjection/BazingaGeocoderExtension.php

@@ -59,7 +59,7 @@ public function load(array $configs, ContainerBuilder $container)
             $definition->replaceArgument(2, $config['fake_ip']['use_faker']);
 
             if ($config['fake_ip']['use_faker'] && !class_exists(Generator::class)) {
-                throw new \LogicException('To enable this option, you must install fzaninotto/faker package.');
+                throw new \LogicException('To enable this option, you must install fakerphp/faker package.');
             }
         } else {
             $container->removeDefinition(FakeIpPlugin::class);
@@ -173,7 +173,7 @@ public function configureProviderPlugins(ContainerBuilder $container, array $con
     }
 
     /**
-     * {@inheritdoc}
+     * @return Configuration
      */
     public function getConfiguration(array $config, ContainerBuilder $container)
     {

Doctrine/ORM/GeocoderListener.php

@@ -44,6 +44,8 @@ public function __construct(Provider $geocoder, DriverInterface $driver)
 
     /**
      * {@inheritdoc}
+     *
+     * @return array
      */
     public function getSubscribedEvents()
     {

Plugin/FakeIpPlugin.php

@@ -17,6 +17,7 @@
 use Geocoder\Plugin\Plugin;
 use Geocoder\Query\GeocodeQuery;
 use Geocoder\Query\Query;
+use Http\Promise\Promise;
 
 /**
  * Replace local IP with something better.
@@ -52,7 +53,7 @@ public function __construct(?string $needle, string $replacement = null, bool $u
     }
 
     /**
-     * {@inheritdoc}
+     * @return Promise
      */
     public function handleQuery(Query $query, callable $next, callable $first)
     {

Plugin/ProfilingPlugin.php

@@ -19,6 +19,7 @@
 use Geocoder\Query\GeocodeQuery;
 use Geocoder\Query\Query;
 use Geocoder\Query\ReverseQuery;
+use Http\Promise\Promise;
 
 /**
  * @author Tobias Nyholm <tobias.nyholm@gmail.com>
@@ -40,6 +41,9 @@ public function __construct(string $name)
         $this->name = $name;
     }
 
+    /**
+     * @return Promise
+     */
     public function handleQuery(Query $query, callable $next, callable $first)
     {
         $startTime = microtime(true);

Resources/doc/index.md

@@ -215,7 +215,7 @@ bazinga_geocoder:
         ip: 123.123.123.123
 ```
 
-You can also let [Faker](https://github.com/fzaninotto/Faker) generate fake ip for you.
+You can also let [Faker](https://github.com/fakerphp/faker) generate fake ip for you.
 
 ```yaml
 bazinga_geocoder:

Tests/Command/GeocodeCommandTest.php

@@ -32,7 +32,7 @@ class GeocodeCommandTest extends TestCase
 {
     private static $address = '10 rue Gambetta, Paris, France';
 
-    public function testExecute()
+    public function testExecute(): void
     {
         $coordinates = new Coordinates(1, 2);
         $country = new Country('France', 'FR');
@@ -59,7 +59,7 @@ public function testExecute()
 
         $kernel->expects($this->any())
             ->method('getContainer')
-            ->will($this->returnValue($container));
+            ->willReturn($container);
 
         $kernel->expects($this->any())
             ->method('getBundles')

Tests/DependencyInjection/Compiler/AddProvidersPassTest.php

@@ -17,25 +17,22 @@
 use Geocoder\ProviderAggregator;
 use Http\Client\Curl\Client;
 use PHPUnit\Framework\TestCase;
-use Symfony\Bridge\PhpUnit\SetUpTearDownTrait;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Definition;
 
 class AddProvidersPassTest extends TestCase
 {
-    use SetUpTearDownTrait;
-
     /**
      * @var AddProvidersPass
      */
     private $compilerPass;
 
-    protected function doSetUp()
+    protected function setUp(): void
     {
         $this->compilerPass = new AddProvidersPass();
     }
 
-    public function testRegistersProviders()
+    public function testRegistersProviders(): void
     {
         $containerBuilder = new ContainerBuilder();
         $containerBuilder->setDefinition(ProviderAggregator::class, new Definition(ProviderAggregator::class));