Skip to content

Commit

Permalink
Merge pull request #416 from Lombiq/issue/INFRA-164
Browse files Browse the repository at this point in the history
INFRA-164: GHA/WarpBuild runner OS version pinning
  • Loading branch information
BenedekFarkas authored Oct 31, 2024
2 parents b4fb72c + 84d77a5 commit 7c3b888
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 4 deletions.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
2 changes: 1 addition & 1 deletion Lombiq.Tests.UI/Docs/SecurityScanning.md
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ This is because the Docker installation is configured to use Windows images, whi
# See https://github.com/Lombiq/GitHub-Actions/blob/dev/.github/workflows/build-and-test-orchard-core.yml.
uses: Lombiq/GitHub-Actions/.github/workflows/build-and-test-orchard-core.yml@dev
with:
machine-types: '["windows-latest"]'
machine-types: '["windows-2022"]'
test-filter: "FullyQualifiedName!~SecurityScanningTests"
```
Expand Down
9 changes: 6 additions & 3 deletions Lombiq.Tests.UI/Services/WebDriverFactory.cs
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,6 @@ public static Task<Func<ChromeDriver>> CreateChromeDriverAsync(BrowserConfigurat
Task<Func<ChromeDriver>> CreateDriverInnerAsync(string driverPath = null) =>
Task.FromResult(() =>
{
// Note that no-sandbox should NOT be used, because it causes Chrome processes to remain open, see
// https://github.com/Lombiq/UI-Testing-Toolbox/issues/356.
var chromeConfig = new ChromeConfiguration { Options = new ChromeOptions().SetCommonOptions() };
chromeConfig.Options.SetLoggingPreference(LogType.Browser, LogLevel.Info);
Expand Down Expand Up @@ -160,6 +157,12 @@ private static TDriverOptions SetCommonChromiumOptions<TDriverOptions>(
// Disabling smooth scrolling to avoid large waiting time when taking full-page screenshots.
options.AddArgument("disable-smooth-scrolling");

// Previously this switch caused Chrome processes to remain open after test execution, see
// https://github.com/Lombiq/UI-Testing-Toolbox/issues/356, but it doesn't seem to be case anymore.
// Additionally, Ubuntu 2024-based GitHub Actions runners seem to require this flag to be set, see
// https://github.com/actions/runner-images/issues/8268#issuecomment-2343831000.
options.AddArgument("--no-sandbox");

if (configuration.FakeVideoSource is not null)
{
var fakeCameraSourceFilePath = configuration.FakeVideoSource.SaveVideoToTempFolder();
Expand Down

0 comments on commit 7c3b888

Please sign in to comment.