Unverified Commit a66e5a7f authored by Jenn Magder's avatar Jenn Magder Committed by GitHub

Turn on win_build_tests_2_3 shard, skip 'build windows' tests (#82386)

parent fe43a8c6
...@@ -442,8 +442,7 @@ Future<void> _runExampleProjectBuildTests(FileSystemEntity exampleDirectory) asy ...@@ -442,8 +442,7 @@ Future<void> _runExampleProjectBuildTests(FileSystemEntity exampleDirectory) asy
} }
if (Platform.isWindows) { if (Platform.isWindows) {
if (Directory(path.join(examplePath, 'windows')).existsSync()) { if (Directory(path.join(examplePath, 'windows')).existsSync()) {
await _flutterBuildWin32(examplePath, release: false, additionalArgs: additionalArgs, verifyCaching: verifyCaching); // TODO(jmagman): Re-add Windows build tests when Visual Studio is available https://github.com/flutter/flutter/issues/81956.
await _flutterBuildWin32(examplePath, release: true, additionalArgs: additionalArgs, verifyCaching: verifyCaching);
} else { } else {
print('Example project ${path.basename(examplePath)} has no windows directory, skipping Win32'); print('Example project ${path.basename(examplePath)} has no windows directory, skipping Win32');
} }
...@@ -507,21 +506,6 @@ Future<void> _flutterBuildMacOS(String relativePathToApplication, { ...@@ -507,21 +506,6 @@ Future<void> _flutterBuildMacOS(String relativePathToApplication, {
); );
} }
Future<void> _flutterBuildWin32(String relativePathToApplication, {
@required bool release,
bool verifyCaching = false,
List<String> additionalArgs = const <String>[],
}) async {
assert(Platform.isWindows);
await runCommand(flutter, <String>['config', '--enable-windows-desktop']);
print('${green}Testing Windows build$reset for $cyan$relativePathToApplication$reset...');
await _flutterBuild(relativePathToApplication, 'Windows', 'windows',
release: release,
verifyCaching: verifyCaching,
additionalArgs: additionalArgs
);
}
Future<void> _flutterBuild( Future<void> _flutterBuild(
String relativePathToApplication, String relativePathToApplication,
String platformLabel, String platformLabel,
......
...@@ -579,7 +579,7 @@ ...@@ -579,7 +579,7 @@
"name": "Windows build_tests_2_3", "name": "Windows build_tests_2_3",
"repo": "flutter", "repo": "flutter",
"task_name": "win_build_tests_2_3", "task_name": "win_build_tests_2_3",
"enabled": false "enabled": true
}, },
{ {
"name": "Windows build_tests_3_3", "name": "Windows build_tests_3_3",
......
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