[8.x] Explicitly set chromedriver port. (#1124)

fixes #1123 fixes #1122 Before 128, ChromeDriver will use the default 9515 port, however this is no longer the case and would cause chromedriver to run on port 0. https://issues.chromium.org/issues/361370192 Signed-off-by: Mior Muhammad Zaki <[email protected]>
laravel · Aug 26, 2024 · e641800 · e641800
1 parent 4e76666
commit e641800
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/.github/workflows/browser-tests.yml b/.github/workflows/browser-tests.yml
@@ -51,7 +51,7 @@ jobs:
         run: php vendor/bin/testbench dusk:chrome-driver --detect
 
       - name: Start Chrome Driver
-        run: ./bin/chromedriver-linux &
+        run: ./bin/chromedriver-linux --port=9515 &
 
       - name: Run Laravel Server
         run: php vendor/bin/testbench serve --no-reload &

diff --git a/src/TestCase.php b/src/TestCase.php
@@ -43,7 +43,8 @@ protected function setUp(): void
     protected function driver()
     {
         return RemoteWebDriver::create(
-            'http://localhost:9515', DesiredCapabilities::chrome()
+            $_ENV['DUSK_DRIVER_URL'] ?? env('DUSK_DRIVER_URL') ?? 'http://localhost:9515',
+            DesiredCapabilities::chrome()
         );
     }
 

diff --git a/stubs/DuskTestCase.stub b/stubs/DuskTestCase.stub
@@ -20,7 +20,7 @@ abstract class DuskTestCase extends BaseTestCase
     public static function prepare(): void
     {
         if (! static::runningInSail()) {
-            static::startChromeDriver();
+            static::startChromeDriver(['--port=9515']);
         }
     }