Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Commit 51af96b

Browse files
author
nturgut
committed
change enable flag for isUnitTestsScreenshotsAvailable
1 parent 338e577 commit 51af96b

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

lib/web_ui/dev/test_runner.dart

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -191,7 +191,7 @@ class TestCommand extends Command<bool> with ArgUtils {
191191
environment.webUiTestResultsDirectory.createSync(recursive: true);
192192

193193
// If screenshot tests are available, fetch the screenshot goldens.
194-
if (isScreenhotTestsAvailable) {
194+
if (isScreenshotTestsAvailable) {
195195
final GoldensRepoFetcher goldensRepoFetcher = GoldensRepoFetcher(
196196
environment.webUiGoldensRepositoryDirectory,
197197
path.join(environment.webUiDevDir.path, 'goldens_lock.yaml'));
@@ -381,15 +381,13 @@ class TestCommand extends Command<bool> with ArgUtils {
381381
isSafariIntegrationTestAvailable;
382382

383383
// Whether the tests will do screenshot testing.
384-
bool get isScreenhotTestsAvailable =>
384+
bool get isScreenshotTestsAvailable =>
385385
isIntegrationTestsAvailable || isUnitTestsScreenshotsAvailable;
386386

387387
// For unit tests screenshot tests and smoke tests only run on:
388388
// "Chrome/iOS" for LUCI/local.
389389
bool get isUnitTestsScreenshotsAvailable =>
390-
((isChrome && isLuci && io.Platform.isLinux) ||
391-
(isChrome || isSafariIOS) && !isLuci) ||
392-
(isSafariIOS && isLuci);
390+
isChrome && (io.Platform.isLinux || !isLuci) || isSafariIOS;
393391

394392
/// Use system flutter instead of cloning the repository.
395393
///

0 commit comments

Comments
 (0)