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

Commit ee5adf6

Browse files
authored
Zip and upload Xcode xcresults on scenario test failure (#55093)
The `FLUTTER_ENGINE` has a `/` in it: `ci/ios_debug_unopt_sim`. ``` path.join(storePath, '$iosEngineVariant.zip'); ``` was resolving to `path_to_output/ci/ios_debug_unopt_sim.zip`. `path_to_output` existed, but the `ci` directory didn't: > zip error: Could not create output file (/Volumes/Work/s/w/ir/x/w/rc/flutter_logs_dir/ci/ios_debug_unopt_sim.zip) Change the output zip path to `path_to_output/ci_ios_debug_unopt_sim.zip` with an underscore instead. Fixes flutter/flutter#154956 [C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
1 parent 38e37ce commit ee5adf6

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

testing/scenario_app/bin/run_ios_tests.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -382,7 +382,7 @@ String _zipAndStoreFailedTestResults({
382382
required io.Directory resultBundle,
383383
required String storePath,
384384
}) {
385-
final outputPath = path.join(storePath, '$iosEngineVariant.zip');
385+
final outputPath = path.join(storePath, '${iosEngineVariant.replaceAll('/', '_')}.zip');
386386
final result = io.Process.runSync(
387387
'zip',
388388
[

0 commit comments

Comments
 (0)