Skip to content

Commit c598da4

Browse files
[macOS] Add Chrome to arm64 image (#8526)
1 parent 9e6658b commit c598da4

File tree

5 files changed

+44
-18
lines changed

5 files changed

+44
-18
lines changed

images/macos/provision/core/chrome.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,8 @@ CHROMEDRIVER_DIR="/usr/local/share/chromedriver-${CHROME_PLATFORM}"
2626
CHROMEDRIVER_BIN="$CHROMEDRIVER_DIR/chromedriver"
2727

2828
download_with_retries "$CHROMEDRIVER_URL" "/tmp" "$CHROMEDRIVER_ARCHIVE"
29-
unzip -qq /tmp/$CHROMEDRIVER_ARCHIVE -d /usr/local/share
30-
chmod +x $CHROMEDRIVER_BIN
29+
unzip -qq /tmp/$CHROMEDRIVER_ARCHIVE -d /tmp/
30+
sudo mv "/tmp/chromedriver-${CHROME_PLATFORM}" "$CHROMEDRIVER_DIR"
3131
ln -s "$CHROMEDRIVER_BIN" /usr/local/bin/chromedriver
3232
echo "export CHROMEWEBDRIVER=$CHROMEDRIVER_DIR" >> "${HOME}/.bashrc"
3333

images/macos/software-report/SoftwareReport.Browsers.psm1

Lines changed: 26 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,30 @@
11
function Build-BrowserSection {
2-
return @(
2+
3+
$nodes = @()
4+
$os = Get-OSVersion
5+
6+
$nodes += @(
37
[ToolVersionNode]::new("Safari", $(Get-SafariVersion))
48
[ToolVersionNode]::new("SafariDriver", $(Get-SafariDriverVersion))
59
[ToolVersionNode]::new("Google Chrome", $(Get-ChromeVersion))
610
[ToolVersionNode]::new("Google Chrome for Testing", $(Get-ChromeForTestingVersion))
711
[ToolVersionNode]::new("ChromeDriver", $(Get-ChromeDriverVersion))
8-
[ToolVersionNode]::new("Microsoft Edge", $(Get-EdgeVersion))
9-
[ToolVersionNode]::new("Microsoft Edge WebDriver", $(Get-EdgeDriverVersion))
10-
[ToolVersionNode]::new("Mozilla Firefox", $(Get-FirefoxVersion))
11-
[ToolVersionNode]::new("geckodriver", $(Get-GeckodriverVersion))
12+
)
13+
14+
if (-not $os.IsVenturaArm64) {
15+
$nodes += @(
16+
[ToolVersionNode]::new("Microsoft Edge", $(Get-EdgeVersion))
17+
[ToolVersionNode]::new("Microsoft Edge WebDriver", $(Get-EdgeDriverVersion))
18+
[ToolVersionNode]::new("Mozilla Firefox", $(Get-FirefoxVersion))
19+
[ToolVersionNode]::new("geckodriver", $(Get-GeckodriverVersion))
20+
)
21+
}
22+
23+
$nodes += @(
1224
[ToolVersionNode]::new("Selenium server", $(Get-SeleniumVersion))
1325
)
26+
27+
return $nodes
1428
}
1529

1630
function Get-SafariVersion {
@@ -64,7 +78,13 @@ function Get-GeckodriverVersion {
6478
}
6579

6680
function Get-SeleniumVersion {
67-
$seleniumVersion = (Get-ChildItem -Path "/usr/local/Cellar/selenium-server*/*").Name
81+
$os = Get-OSVersion
82+
if ($os.IsVenturaArm64) {
83+
$cellarPath = "/opt/homebrew/Cellar"
84+
} else {
85+
$cellarPath = "/usr/local/Cellar"
86+
}
87+
$seleniumVersion = (Get-ChildItem -Path "$cellarPath/selenium-server*/*").Name
6888
return $seleniumVersion
6989
}
7090

images/macos/software-report/SoftwareReport.Generator.ps1

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -202,12 +202,11 @@ if ((-not $os.IsVentura) -and (-not $os.IsVenturaArm64)) {
202202
$linters.AddToolVersion("Yamllint", $(Get-YamllintVersion))
203203
}
204204

205-
if (-not $os.IsVenturaArm64) {
206-
# Browsers
207-
$browsers = $installedSoftware.AddHeader("Browsers")
208-
$browsers.AddNodes($(Build-BrowserSection))
209-
$browsers.AddNode($(Build-BrowserWebdriversEnvironmentTable))
210-
}
205+
# Browsers
206+
$browsers = $installedSoftware.AddHeader("Browsers")
207+
$browsers.AddNodes($(Build-BrowserSection))
208+
$browsers.AddNode($(Build-BrowserWebdriversEnvironmentTable))
209+
211210
# Java
212211
$java = $installedSoftware.AddHeader("Java")
213212
$java.AddTable($(Get-JavaVersions))

images/macos/templates/macOS-13.arm64.anka.pkr.hcl

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -202,6 +202,7 @@ build {
202202
"./provision/core/gcc.sh",
203203
"./provision/core/cocoapods.sh",
204204
"./provision/core/safari.sh",
205+
"./provision/core/chrome.sh",
205206
"./provision/core/bicep.sh",
206207
"./provision/core/codeql-bundle.sh"
207208
]

images/macos/tests/Browsers.Tests.ps1

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
Import-Module "$PSScriptRoot/../helpers/Common.Helpers.psm1"
22
$os = Get-OSVersion
33

4-
Describe "Chrome" -Skip:($os.IsVenturaArm64) {
4+
Describe "Chrome" {
55
BeforeAll {
66
$chromeLocation = "/Applications/Google Chrome.app/Contents/MacOS/Google Chrome"
77
$chromeForTestingLocation = "/Applications/Google Chrome for Testing.app/Contents/MacOS/Google Chrome for Testing"
@@ -28,9 +28,15 @@ Describe "Chrome" -Skip:($os.IsVenturaArm64) {
2828
}
2929
}
3030

31-
Describe "Selenium server" -Skip:($os.IsVenturaArm64) {
31+
Describe "Selenium server" {
3232
It "Selenium server" {
33-
(Get-ChildItem -Path "/usr/local/Cellar/selenium-server*/*").Name | Should -BeLike "4.*"
33+
$os = Get-OSVersion
34+
if ($os.IsVenturaArm64) {
35+
$cellarPath = "/opt/homebrew/Cellar"
36+
} else {
37+
$cellarPath = "/usr/local/Cellar"
38+
}
39+
(Get-ChildItem -Path "$cellarPath/selenium-server*/*").Name | Should -BeLike "4.*"
3440
}
3541
}
3642

@@ -66,4 +72,4 @@ Describe "Safari" {
6672
$commandResult.ExitCode | Should -Be 0
6773
$commandResult.Output | Should -Be "true"
6874
}
69-
}
75+
}

0 commit comments

Comments
 (0)