Unverified Commit 49249693 authored by godofredoc's avatar godofredoc Committed by GitHub

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

This reverts commit a66e5a7f.
parent adb3f138
...@@ -442,7 +442,8 @@ Future<void> _runExampleProjectBuildTests(FileSystemEntity exampleDirectory) asy ...@@ -442,7 +442,8 @@ 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()) {
// TODO(jmagman): Re-add Windows build tests when Visual Studio is available https://github.com/flutter/flutter/issues/81956. await _flutterBuildWin32(examplePath, release: false, additionalArgs: additionalArgs, verifyCaching: verifyCaching);
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');
} }
...@@ -506,6 +507,21 @@ Future<void> _flutterBuildMacOS(String relativePathToApplication, { ...@@ -506,6 +507,21 @@ 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": true "enabled": false
}, },
{ {
"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