mirror of
https://github.com/flutter/flutter.git
synced 2026-02-20 02:29:02 +08:00
Rebase ios-experimental branch onto main. This will make the PRs experimenting with newer versions of Xcode (like https://github.com/flutter/flutter/pull/173123) smaller and easier to reason about. Rebases #168860 and #170274 ``` $ git rebase main -Xtheirs ``` --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: gaaclarke <30870216+gaaclarke@users.noreply.github.com> Co-authored-by: Siva <a-siva@users.noreply.github.com> Co-authored-by: engine-flutter-autoroll <engine-flutter-autoroll@skia.org> Co-authored-by: Jamil Saadeh <jssaadeh@outlook.com> Co-authored-by: Dara Adedeji <76637177+SunkenInTime@users.noreply.github.com> Co-authored-by: Greg Price <gnprice@gmail.com> Co-authored-by: Ben Konyi <bkonyi@google.com> Co-authored-by: Ricardo Dalarme <ricardodalarme@outlook.com> Co-authored-by: Flutter GitHub Bot <fluttergithubbot@gmail.com> Co-authored-by: Justin McCandless <jmccandless@google.com> Co-authored-by: Alex Talebi <31685655+SalehTZ@users.noreply.github.com> Co-authored-by: Qun Cheng <36861262+QuncCccccc@users.noreply.github.com> Co-authored-by: Mouad Debbar <mdebbar@google.com> Co-authored-by: Zuckjet <1083941774@qq.com> Co-authored-by: Loïc Sharma <737941+loic-sharma@users.noreply.github.com> Co-authored-by: auto-submit[bot] <98614782+auto-submit[bot]@users.noreply.github.com> Co-authored-by: auto-submit[bot] <flutter-engprod-team@google.com> Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com> Co-authored-by: yim <ybz975218925@gmail.com> Co-authored-by: bufffun <chenmingding.cmd@alibaba-inc.com> Co-authored-by: Chinmay Garde <chinmaygarde@google.com> Co-authored-by: Hannah Jin <jhy03261997@gmail.com> Co-authored-by: Kate Lovett <katelovett@google.com> Co-authored-by: Valentin Vignal <32538273+ValentinVignal@users.noreply.github.com> Co-authored-by: Derek Xu <derekx@google.com> Co-authored-by: Yash Dhrangdhariya <72062416+Yash-Dhrangdhariya@users.noreply.github.com> Co-authored-by: bungeman <bungeman@chromium.org> Co-authored-by: Ahmed Mohamed Sameh <ahmedsameha1@gmail.com> Co-authored-by: John "codefu" McDole <codefu@google.com> Co-authored-by: Dmitry Grand <dmgr@google.com> Co-authored-by: Kostia Sokolovskyi <sokolovskyi.konstantin@gmail.com> Co-authored-by: Reid Baker <1063596+reidbaker@users.noreply.github.com> Co-authored-by: Matthew Kosarek <matt.kosarek@canonical.com> Co-authored-by: Jason Simmons <jason-simmons@users.noreply.github.com> Co-authored-by: Jim Graham <flar@google.com> Co-authored-by: Michael Goderbauer <goderbauer@google.com> Co-authored-by: Gray Mackall <34871572+gmackall@users.noreply.github.com> Co-authored-by: Gray Mackall <mackall@google.com> Co-authored-by: Tong Mu <dkwingsmt@users.noreply.github.com> Co-authored-by: Jon Ihlas <jon.i@hotmail.fr> Co-authored-by: Micael Cid <micaelcid10@gmail.com> Co-authored-by: Alexander Aprelev <aam@google.com> Co-authored-by: hellohuanlin <41930132+hellohuanlin@users.noreply.github.com> Co-authored-by: Luke Memet <1598289+lukemmtt@users.noreply.github.com> Co-authored-by: Victoria Ashworth <15619084+vashworth@users.noreply.github.com> Co-authored-by: Mairramer <50643541+Mairramer@users.noreply.github.com> Co-authored-by: Florin Malita <fmalita@gmail.com> Co-authored-by: chunhtai <47866232+chunhtai@users.noreply.github.com> Co-authored-by: Salem Iranloye <127918074+salemiranloye@users.noreply.github.com> Co-authored-by: Kevin Moore <kevmoo@google.com> Co-authored-by: Sydney Bao <sydneybao@google.com> Co-authored-by: Wdestroier <Wdestroier@gmail.com> Co-authored-by: Matt Boetger <matt.boetger@gmail.com> Co-authored-by: Reid Baker <reidbaker@google.com> Co-authored-by: Victor Sanni <victorsanniay@gmail.com> Co-authored-by: Jessy Yameogo <jessy.yameogo@gmail.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: romain.gyh <11901536+romaingyh@users.noreply.github.com> Co-authored-by: Robert Ancell <robert.ancell@canonical.com> Co-authored-by: TheLastFlame <131446187+TheLastFlame@users.noreply.github.com> Co-authored-by: masato <returnymgstokh@icloud.com> Co-authored-by: Albin PK <56157868+albinpk@users.noreply.github.com> Co-authored-by: Huy <huy@nevercode.io> Co-authored-by: Matan Lurey <matanlurey@users.noreply.github.com> Co-authored-by: Azat Chorekliyev <azat24680@gmail.com> Co-authored-by: EdwynZN <edwinzn9@gmail.com> Co-authored-by: Bruno Leroux <bruno.leroux@gmail.com> Co-authored-by: Dev TtangKong <ttankkeo112@gmail.com> Co-authored-by: LongCatIsLooong <31859944+LongCatIsLooong@users.noreply.github.com> Co-authored-by: Houssem Eddine Fadhli <houssemeddinefadhli81@gmail.com>
151 lines
4.3 KiB
Dart
151 lines
4.3 KiB
Dart
// Copyright 2014 The Flutter Authors. All rights reserved.
|
|
// Use of this source code is governed by a BSD-style license that can be
|
|
// found in the LICENSE file.
|
|
|
|
@TestOn('vm')
|
|
library;
|
|
|
|
import 'dart:convert';
|
|
import 'dart:io' as io;
|
|
|
|
import 'package:dev_tools/check_engine_version.dart' as lib;
|
|
import 'package:file/memory.dart';
|
|
import 'package:process_runner/process_runner.dart';
|
|
import 'package:test/fake.dart';
|
|
import 'package:test/test.dart';
|
|
|
|
void main() {
|
|
late MemoryFileSystem fileSystem;
|
|
late StringBuffer stderr;
|
|
late String versionPath;
|
|
late String scriptPath;
|
|
|
|
setUp(() {
|
|
fileSystem = MemoryFileSystem.test();
|
|
versionPath = fileSystem.path.join('bin', 'internal', 'engine.version');
|
|
scriptPath = fileSystem.path.join('bin', 'internal', 'last_engine_commit.sh');
|
|
|
|
stderr = StringBuffer();
|
|
});
|
|
|
|
tearDown(() {
|
|
printOnFailure('$stderr');
|
|
});
|
|
|
|
test('skips check, returning true, if the file is missing', () async {
|
|
final ProcessRunner noRuns = FakeProcessRunner(<String, ProcessRunnerResult>{});
|
|
await expectLater(
|
|
lib.checkEngineVersion(
|
|
versionPath: versionPath,
|
|
scriptPath: scriptPath,
|
|
fileSystem: fileSystem,
|
|
runner: noRuns,
|
|
stderr: stderr,
|
|
),
|
|
completion(isTrue),
|
|
);
|
|
|
|
expect('$stderr', contains('does not exist, skipping engine.version check'));
|
|
});
|
|
|
|
test('skips check, returning true, if file was not changed', () async {
|
|
fileSystem
|
|
.directory('bin')
|
|
.childDirectory('internal')
|
|
.childFile('engine.version')
|
|
.createSync(recursive: true);
|
|
|
|
final ProcessRunner gitExec = FakeProcessRunner(<String, ProcessRunnerResult>{
|
|
'git diff --name-only --relative master...HEAD -- $versionPath': ProcessRunnerResult(
|
|
0,
|
|
utf8.encode('dev/another/file.txt\n'),
|
|
<int>[],
|
|
<int>[],
|
|
),
|
|
});
|
|
|
|
await expectLater(
|
|
lib.checkEngineVersion(
|
|
versionPath: versionPath,
|
|
scriptPath: scriptPath,
|
|
fileSystem: fileSystem,
|
|
runner: gitExec,
|
|
stderr: stderr,
|
|
),
|
|
completion(isTrue),
|
|
);
|
|
|
|
expect('$stderr', contains('has not changed, skipping engine.version check'));
|
|
});
|
|
|
|
test('fails if the SHAs are different', () async {
|
|
fileSystem.directory('bin').childDirectory('internal').childFile('engine.version')
|
|
..createSync(recursive: true)
|
|
..writeAsStringSync('def456');
|
|
|
|
final ProcessRunner scriptExec = FakeProcessRunner(<String, ProcessRunnerResult>{
|
|
scriptPath: ProcessRunnerResult(0, utf8.encode('abc123'), <int>[], <int>[]),
|
|
});
|
|
|
|
await expectLater(
|
|
lib.checkEngineVersion(
|
|
versionPath: versionPath,
|
|
scriptPath: scriptPath,
|
|
fileSystem: fileSystem,
|
|
runner: scriptExec,
|
|
stderr: stderr,
|
|
onlyIfVersionChanged: false,
|
|
),
|
|
completion(isFalse),
|
|
);
|
|
|
|
expect('$stderr', stringContainsInOrder(<String>['output abc123', 'is def456']));
|
|
});
|
|
|
|
test('succeeds if the SHAs are the same', () async {
|
|
fileSystem.directory('bin').childDirectory('internal').childFile('engine.version')
|
|
..createSync(recursive: true)
|
|
..writeAsStringSync('abc123');
|
|
|
|
final ProcessRunner scriptExec = FakeProcessRunner(<String, ProcessRunnerResult>{
|
|
scriptPath: ProcessRunnerResult(0, utf8.encode('abc123'), <int>[], <int>[]),
|
|
});
|
|
|
|
await expectLater(
|
|
lib.checkEngineVersion(
|
|
versionPath: versionPath,
|
|
scriptPath: scriptPath,
|
|
fileSystem: fileSystem,
|
|
runner: scriptExec,
|
|
stderr: stderr,
|
|
onlyIfVersionChanged: false,
|
|
),
|
|
completion(isTrue),
|
|
);
|
|
|
|
expect('$stderr', isEmpty);
|
|
});
|
|
}
|
|
|
|
final class FakeProcessRunner extends Fake implements ProcessRunner {
|
|
FakeProcessRunner(this._cannedResponses);
|
|
final Map<String, ProcessRunnerResult> _cannedResponses;
|
|
|
|
@override
|
|
Future<ProcessRunnerResult> runProcess(
|
|
List<String> commandLine, {
|
|
io.Directory? workingDirectory,
|
|
bool? printOutput,
|
|
bool failOk = false,
|
|
Stream<List<int>>? stdin,
|
|
bool runInShell = false,
|
|
io.ProcessStartMode startMode = io.ProcessStartMode.normal,
|
|
}) async {
|
|
final ProcessRunnerResult? command = _cannedResponses[commandLine.join(' ')];
|
|
if (command == null) {
|
|
fail('Unexpected process: ${commandLine.join(' ')}');
|
|
}
|
|
return command;
|
|
}
|
|
}
|