minor symfony#57998 Replace external FTP server by a local Docker instance (alexandre-daubois)

This PR was merged into the 5.4 branch.

Discussion
----------

Replace external FTP server by a local Docker instance

| Q             | A
| ------------- | ---
| Branch?       | 5.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | -
| License       | MIT

After a suggestion from Nicolas, to avoid being dependent on external FTPs and make some tests flaky.

Commits
-------

37b3ce9 Replace external FTP server by a local docker instance

Author: nicolas-grekas

.github/workflows/integration-tests.yml

@@ -44,6 +44,13 @@ jobs:
           LDAP_PORT_NUMBER: 3389
           LDAP_USERS: a
           LDAP_PASSWORDS: a
+      ftp:
+        image: onekilo79/ftpd_test
+        ports:
+          - 21:21
+          - 30000-30009:30000-30009
+        volumes:
+          - ./:/hostmount
       redis:
         image: redis:6.2.8
         ports:
@@ -144,6 +151,11 @@ jobs:
           curl -s -u Administrator:111111 -X POST http://localhost:8091/pools/default/buckets -d 'ramQuotaMB=100&bucketType=ephemeral&name=cache'
           curl -s -u Administrator:111111 -X POST  http://localhost:8091/pools/default -d 'memoryQuota=256'
 
+      - name: Create FTP fixtures
+        run: |
+          mkdir -p ./ftpusers/test/pub
+          touch ./ftpusers/test/pub/example ./ftpusers/test/readme.txt
+
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
@@ -196,6 +208,7 @@ jobs:
       - name: Run tests
         run: ./phpunit --group integration -v
         env:
+          INTEGRATION_FTP_URL: 'ftp://test:test@localhost'
           REDIS_HOST: 'localhost:16379'
           REDIS_AUTHENTICATED_HOST: 'localhost:16380'
           REDIS_CLUSTER_HOSTS: 'localhost:7000 localhost:7001 localhost:7002 localhost:7003 localhost:7004 localhost:7005'

src/Symfony/Component/Finder/Tests/Iterator/RecursiveDirectoryIteratorTest.php

@@ -24,26 +24,38 @@ protected function setUp(): void
 
     /**
      * @group network
+     * @group integration
      */
     public function testRewindOnFtp()
     {
-        $i = new RecursiveDirectoryIterator('ftp://test.rebex.net/', \RecursiveDirectoryIterator::SKIP_DOTS);
+        if (!getenv('INTEGRATION_FTP_URL')) {
+            self::markTestSkipped('INTEGRATION_FTP_URL env var is not defined.');
+        }
+
+        $i = new RecursiveDirectoryIterator(getenv('INTEGRATION_FTP_URL').\DIRECTORY_SEPARATOR, \RecursiveDirectoryIterator::SKIP_DOTS);
 
         $i->rewind();
 
-        $this->assertTrue(true);
+        $this->expectNotToPerformAssertions();
     }
 
     /**
      * @group network
+     * @group integration
      */
     public function testSeekOnFtp()
     {
-        $i = new RecursiveDirectoryIterator('ftp://test.rebex.net/', \RecursiveDirectoryIterator::SKIP_DOTS);
+        if (!getenv('INTEGRATION_FTP_URL')) {
+            self::markTestSkipped('INTEGRATION_FTP_URL env var is not defined.');
+        }
+
+        $ftpUrl = getenv('INTEGRATION_FTP_URL');
+
+        $i = new RecursiveDirectoryIterator($ftpUrl.\DIRECTORY_SEPARATOR, \RecursiveDirectoryIterator::SKIP_DOTS);
 
         $contains = [
-            'ftp://test.rebex.net'.\DIRECTORY_SEPARATOR.'pub',
-            'ftp://test.rebex.net'.\DIRECTORY_SEPARATOR.'readme.txt',
+            $ftpUrl.\DIRECTORY_SEPARATOR.'pub',
+            $ftpUrl.\DIRECTORY_SEPARATOR.'readme.txt',
         ];
         $actual = [];