Unverified Commit 32000db5 authored by Christopher Fujino's avatar Christopher Fujino Committed by GitHub

[flutter_tools] update libimobiledevice (#59512)

parent 9a2659ad
73b9c81e02f83060aa44bece963a62827908de68
6150fcd888a0ee8a69dd47328509ee608c2e02dc
569016da35ccea9d9b7c81954342fabb98149fb5
28812e5bed8621bd637b4129ab15a9c1abb9f57d
20a2f8dbddcf1a96ad4c720b9afd1d0876d17ffc
2ef3f4c83bce209247479e707eebd5a84f606e28
e2e09d9fba1187f8d6aafaa34d4172f56f1ffb72
eb45fd21469fa35e9bbc3b0a15a962460b8d23a2
ce98c346b7c1dc2a21faea4fd3f32c88e27ca2af
......@@ -510,7 +510,7 @@ class IOSDevice extends Device {
@override
Future<void> takeScreenshot(File outputFile) async {
await _iMobileDevice.takeScreenshot(outputFile);
await _iMobileDevice.takeScreenshot(outputFile, id);
}
@override
......@@ -832,8 +832,8 @@ class IOSDevicePortForwarder extends DevicePortForwarder {
process = await _processUtils.start(
<String>[
_iproxyPath,
hostPort.toString(),
devicePort.toString(),
'$hostPort:$devicePort',
'--udid',
_id,
],
environment: Map<String, String>.fromEntries(
......
......@@ -66,11 +66,13 @@ class IMobileDevice {
}
/// Captures a screenshot to the specified outputFile.
Future<void> takeScreenshot(File outputFile) {
Future<void> takeScreenshot(File outputFile, String deviceID) {
return _processUtils.run(
<String>[
_idevicescreenshotPath,
outputFile.path,
'--udid',
deviceID,
],
throwOnError: true,
environment: Map<String, String>.fromEntries(
......
......@@ -22,14 +22,14 @@ void main() {
const int devicePort = 456;
final FakeProcessManager processManager = FakeProcessManager.list(<FakeCommand>[
const FakeCommand(
command: <String>['iproxy', '49154', '456', '1234'],
command: <String>['iproxy', '49154:456', '--udid', '1234'],
// iproxy does not exit with 0 when it cannot forward.
exitCode: 0,
stdout: null, // no stdout indicates failure.
environment: kDyLdLibEntry,
),
const FakeCommand(
command: <String>['iproxy', '49155', '456', '1234'],
command: <String>['iproxy', '49155:456', '--udid', '1234'],
exitCode: 0,
stdout: 'not empty',
environment: kDyLdLibEntry,
......
......@@ -85,7 +85,7 @@ void main() {
logger: logger,
);
expect(() async => await iMobileDevice.takeScreenshot(mockOutputFile), throwsA(anything));
expect(() async => await iMobileDevice.takeScreenshot(mockOutputFile, '1234'), throwsA(anything));
});
testWithoutContext('idevicescreenshot captures and returns screenshot', () async {
......@@ -100,8 +100,8 @@ void main() {
logger: logger,
);
await iMobileDevice.takeScreenshot(mockOutputFile);
verify(mockProcessManager.run(<String>[idevicescreenshotPath, outputPath],
await iMobileDevice.takeScreenshot(mockOutputFile, '1234');
verify(mockProcessManager.run(<String>[idevicescreenshotPath, outputPath, '--udid', '1234'],
environment: <String, String>{'DYLD_LIBRARY_PATH': libimobiledevicePath},
workingDirectory: null,
));
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment