Skip to content

Commit

Permalink
Add BROWSER_NAME_ALIAS_MAP and revert some changes.
Browse files Browse the repository at this point in the history
  • Loading branch information
saidatom committed Aug 5, 2024
1 parent ad11f06 commit 11fcbef
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 6 deletions.
5 changes: 1 addition & 4 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ jobs:
strategy:
matrix:
php: [ '7.4', '8.0', '8.1', '8.2' ]
browser: [ 'firefox', 'chrome']
browser: [ 'firefox', 'chrome' ]
selenium: [ '2.53.1', '3', '4' ]
include:
- php: '7.4'
Expand Down Expand Up @@ -82,9 +82,6 @@ jobs:
WEB_FIXTURES_BROWSER: ${{ matrix.browser }}
DRIVER_MACHINE_BASE_PATH: /fixtures/
run: |
if [ "${{ matrix.browser }}" = "chromium" ]; then
WEB_FIXTURES_BROWSER="chrome"
fi
vendor/bin/phpunit -v --coverage-clover=coverage.xml --colors=always --testdox
- name: Upload coverage
Expand Down
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
"php": ">=7.4",
"ext-json": "*",
"behat/mink": "^1.11@dev",
"php-webdriver/webdriver": "dev-main"
"php-webdriver/webdriver": "^1.14"
},
"require-dev": {
"mink/driver-testsuite": "dev-master",
Expand Down
4 changes: 3 additions & 1 deletion docker-compose.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
version: "3"

services:
selenium:
image: ${SELENIUM_IMAGE:-selenium/standalone-chromium:4}
image: ${SELENIUM_IMAGE:-selenium/standalone-chrome:4}
hostname: selenium
shm_size: 4g
environment:
Expand Down
1 change: 1 addition & 0 deletions src/WebdriverClassicDriver.php
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,7 @@ class WebdriverClassicDriver extends CoreDriver
private const BROWSER_NAME_ALIAS_MAP = [
'edge' => WebDriverBrowserType::MICROSOFT_EDGE,
'chrome' => WebDriverBrowserType::CHROME,
'chromium' => WebDriverBrowserType::CHROME,
'firefox' => WebDriverBrowserType::FIREFOX,
];

Expand Down

0 comments on commit 11fcbef

Please sign in to comment.