Clean up usage of temporary directories (#20682)
All temporary directory start with `flutter_` and have their random component separated from the name by a period, as in `flutter_test_bundle.YFYQMY`.
I've tried to find some of the places where we didn't cleanly delete temporary directories, too. This greatly reduces, though it does not entirely eliminate, the directories we leave behind when running tests, especially `flutter_tools` tests.
While I was at it I standardized on `tempDir` as the variable name for temporary directories, since it was the most common, removing occurrences of `temp` and `tmp`, among others.
Also I factored out some common code that used to catch exceptions that happen on Windows, and made more places use that pattern.
diff --git a/packages/flutter_tools/test/analytics_test.dart b/packages/flutter_tools/test/analytics_test.dart
index b61cd35..f6f2b75 100644
--- a/packages/flutter_tools/test/analytics_test.dart
+++ b/packages/flutter_tools/test/analytics_test.dart
@@ -21,7 +21,7 @@
void main() {
group('analytics', () {
- Directory temp;
+ Directory tempDir;
setUpAll(() {
Cache.disableLocking();
@@ -29,11 +29,11 @@
setUp(() {
Cache.flutterRoot = '../..';
- temp = fs.systemTempDirectory.createTempSync('flutter_tools');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_tools_analytics_test.');
});
tearDown(() {
- temp.deleteSync(recursive: true);
+ tryToDelete(tempDir);
});
// Ensure we don't send anything when analytics is disabled.
@@ -42,11 +42,11 @@
flutterUsage.onSend.listen((Map<String, dynamic> data) => count++);
flutterUsage.enabled = false;
- await createProject(temp);
+ await createProject(tempDir);
expect(count, 0);
flutterUsage.enabled = true;
- await createProject(temp);
+ await createProject(tempDir);
expect(count, flutterUsage.isFirstRun ? 0 : 2);
count = 0;
@@ -57,7 +57,7 @@
expect(count, 0);
}, overrides: <Type, Generator>{
FlutterVersion: () => new FlutterVersion(const Clock()),
- Usage: () => new Usage(configDirOverride: temp.path),
+ Usage: () => new Usage(configDirOverride: tempDir.path),
});
// Ensure we don't send for the 'flutter config' command.
@@ -76,7 +76,7 @@
expect(count, 0);
}, overrides: <Type, Generator>{
FlutterVersion: () => new FlutterVersion(const Clock()),
- Usage: () => new Usage(configDirOverride: temp.path),
+ Usage: () => new Usage(configDirOverride: tempDir.path),
});
});
@@ -151,9 +151,14 @@
});
group('analytics bots', () {
- Directory temp;
+ Directory tempDir;
+
setUp(() {
- temp = fs.systemTempDirectory.createTempSync('flutter_tools');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_tools_analytics_bots_test.');
+ });
+
+ tearDown(() {
+ tryToDelete(tempDir);
});
testUsingContext('don\'t send on bots', () async {
@@ -166,7 +171,7 @@
Usage: () => new Usage(
settingsName: 'flutter_bot_test',
versionOverride: 'dev/unknown',
- configDirOverride: temp.path,
+ configDirOverride: tempDir.path,
),
});
@@ -181,7 +186,7 @@
Usage: () => new Usage(
settingsName: 'flutter_bot_test',
versionOverride: 'dev/unknown',
- configDirOverride: temp.path,
+ configDirOverride: tempDir.path,
),
});
});
diff --git a/packages/flutter_tools/test/android/android_sdk_test.dart b/packages/flutter_tools/test/android/android_sdk_test.dart
index 179db33..870a0bf 100644
--- a/packages/flutter_tools/test/android/android_sdk_test.dart
+++ b/packages/flutter_tools/test/android/android_sdk_test.dart
@@ -30,8 +30,10 @@
Directory sdkDir;
tearDown(() {
- sdkDir?.deleteSync(recursive: true);
- sdkDir = null;
+ if (sdkDir != null) {
+ tryToDelete(sdkDir);
+ sdkDir = null;
+ }
});
testUsingContext('parse sdk', () {
diff --git a/packages/flutter_tools/test/android/android_workflow_test.dart b/packages/flutter_tools/test/android/android_workflow_test.dart
index 4883165..70bc842 100644
--- a/packages/flutter_tools/test/android/android_workflow_test.dart
+++ b/packages/flutter_tools/test/android/android_workflow_test.dart
@@ -39,7 +39,6 @@
testUsingContext('licensesAccepted throws if cannot run sdkmanager', () async {
processManager.succeed = false;
- MockAndroidSdk.createSdkDirectory();
when(sdk.sdkManagerPath).thenReturn('/foo/bar/sdkmanager');
final AndroidWorkflow androidWorkflow = new AndroidWorkflow();
expect(androidWorkflow.licensesAccepted, throwsToolExit());
@@ -52,7 +51,6 @@
});
testUsingContext('licensesAccepted handles garbage/no output', () async {
- MockAndroidSdk.createSdkDirectory();
when(sdk.sdkManagerPath).thenReturn('/foo/bar/sdkmanager');
final AndroidWorkflow androidWorkflow = new AndroidWorkflow();
final LicensesAccepted result = await androidWorkflow.licensesAccepted;
@@ -68,7 +66,6 @@
});
testUsingContext('licensesAccepted works for all licenses accepted', () async {
- MockAndroidSdk.createSdkDirectory();
when(sdk.sdkManagerPath).thenReturn('/foo/bar/sdkmanager');
processManager.processFactory = processMetaFactory(<String>[
'[=======================================] 100% Computing updates... ',
@@ -87,7 +84,6 @@
});
testUsingContext('licensesAccepted works for some licenses accepted', () async {
- MockAndroidSdk.createSdkDirectory();
when(sdk.sdkManagerPath).thenReturn('/foo/bar/sdkmanager');
processManager.processFactory = processMetaFactory(<String>[
'[=======================================] 100% Computing updates... ',
@@ -107,7 +103,6 @@
});
testUsingContext('licensesAccepted works for no licenses accepted', () async {
- MockAndroidSdk.createSdkDirectory();
when(sdk.sdkManagerPath).thenReturn('/foo/bar/sdkmanager');
processManager.processFactory = processMetaFactory(<String>[
'[=======================================] 100% Computing updates... ',
@@ -127,7 +122,6 @@
});
testUsingContext('runLicenseManager succeeds for version >= 26', () async {
- MockAndroidSdk.createSdkDirectory();
when(sdk.sdkManagerPath).thenReturn('/foo/bar/sdkmanager');
when(sdk.sdkManagerVersion).thenReturn('26.0.0');
@@ -141,7 +135,6 @@
});
testUsingContext('runLicenseManager errors for version < 26', () async {
- MockAndroidSdk.createSdkDirectory();
when(sdk.sdkManagerPath).thenReturn('/foo/bar/sdkmanager');
when(sdk.sdkManagerVersion).thenReturn('25.0.0');
@@ -155,7 +148,6 @@
});
testUsingContext('runLicenseManager errors correctly for null version', () async {
- MockAndroidSdk.createSdkDirectory();
when(sdk.sdkManagerPath).thenReturn('/foo/bar/sdkmanager');
when(sdk.sdkManagerVersion).thenReturn(null);
@@ -169,7 +161,6 @@
});
testUsingContext('runLicenseManager errors when sdkmanager is not found', () async {
- MockAndroidSdk.createSdkDirectory();
when(sdk.sdkManagerPath).thenReturn('/foo/bar/sdkmanager');
processManager.succeed = false;
diff --git a/packages/flutter_tools/test/artifacts_test.dart b/packages/flutter_tools/test/artifacts_test.dart
index 7767736..ac1c8fe 100644
--- a/packages/flutter_tools/test/artifacts_test.dart
+++ b/packages/flutter_tools/test/artifacts_test.dart
@@ -18,12 +18,12 @@
CachedArtifacts artifacts;
setUp(() {
- tempDir = fs.systemTempDirectory.createTempSync('flutter_temp');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_tools_artifacts_test_cached.');
artifacts = new CachedArtifacts();
});
tearDown(() {
- tempDir.deleteSync(recursive: true);
+ tryToDelete(tempDir);
});
testUsingContext('getArtifactPath', () {
@@ -69,7 +69,7 @@
LocalEngineArtifacts artifacts;
setUp(() {
- tempDir = fs.systemTempDirectory.createTempSync('flutter_temp');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_tools_artifacts_test_local.');
artifacts = new LocalEngineArtifacts(tempDir.path,
fs.path.join(tempDir.path, 'out', 'android_debug_unopt'),
fs.path.join(tempDir.path, 'out', 'host_debug_unopt'),
@@ -77,7 +77,7 @@
});
tearDown(() {
- tempDir.deleteSync(recursive: true);
+ tryToDelete(tempDir);
});
testUsingContext('getArtifactPath', () {
diff --git a/packages/flutter_tools/test/asset_bundle_package_fonts_test.dart b/packages/flutter_tools/test/asset_bundle_package_fonts_test.dart
index dc7c135..4804dba 100644
--- a/packages/flutter_tools/test/asset_bundle_package_fonts_test.dart
+++ b/packages/flutter_tools/test/asset_bundle_package_fonts_test.dart
@@ -95,20 +95,14 @@
Directory oldCurrentDir;
setUp(() async {
- tempDir = await fs.systemTempDirectory.createTemp('asset_bundle_tests');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_asset_bundle_test.');
oldCurrentDir = fs.currentDirectory;
fs.currentDirectory = tempDir;
});
tearDown(() {
fs.currentDirectory = oldCurrentDir;
- try {
- tempDir?.deleteSync(recursive: true);
- tempDir = null;
- } on FileSystemException catch (e) {
- // Do nothing, windows sometimes has trouble deleting.
- print('Ignored exception during tearDown: $e');
- }
+ tryToDelete(tempDir);
});
group('AssetBundle fonts from packages', () {
diff --git a/packages/flutter_tools/test/asset_bundle_package_test.dart b/packages/flutter_tools/test/asset_bundle_package_test.dart
index 6246076..4361bba 100644
--- a/packages/flutter_tools/test/asset_bundle_package_test.dart
+++ b/packages/flutter_tools/test/asset_bundle_package_test.dart
@@ -106,20 +106,14 @@
Directory oldCurrentDir;
setUp(() async {
- tempDir = await fs.systemTempDirectory.createTemp('asset_bundle_tests');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_asset_bundle_test.');
oldCurrentDir = fs.currentDirectory;
fs.currentDirectory = tempDir;
});
tearDown(() {
fs.currentDirectory = oldCurrentDir;
- try {
- tempDir?.deleteSync(recursive: true);
- tempDir = null;
- } on FileSystemException catch (e) {
- // Do nothing, windows sometimes has trouble deleting.
- print('Ignored exception during tearDown: $e');
- }
+ tryToDelete(tempDir);
});
group('AssetBundle assets from packages', () {
diff --git a/packages/flutter_tools/test/asset_bundle_test.dart b/packages/flutter_tools/test/asset_bundle_test.dart
index e4fdf8e..afafc71 100644
--- a/packages/flutter_tools/test/asset_bundle_test.dart
+++ b/packages/flutter_tools/test/asset_bundle_test.dart
@@ -25,20 +25,14 @@
Directory oldCurrentDir;
setUp(() async {
- tempDir = await fs.systemTempDirectory.createTemp('asset_bundle_tests');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_asset_bundle_test.');
oldCurrentDir = fs.currentDirectory;
fs.currentDirectory = tempDir;
});
tearDown(() {
fs.currentDirectory = oldCurrentDir;
- try {
- tempDir?.deleteSync(recursive: true);
- tempDir = null;
- } on FileSystemException catch (e) {
- // Do nothing, windows sometimes has trouble deleting.
- print('Ignored exception during tearDown: $e');
- }
+ tryToDelete(tempDir);
});
testUsingContext('nonempty', () async {
diff --git a/packages/flutter_tools/test/asset_bundle_variant_test.dart b/packages/flutter_tools/test/asset_bundle_variant_test.dart
index e58f0e7..afdbcc6 100644
--- a/packages/flutter_tools/test/asset_bundle_variant_test.dart
+++ b/packages/flutter_tools/test/asset_bundle_variant_test.dart
@@ -20,20 +20,14 @@
Directory oldCurrentDir;
setUp(() async {
- tempDir = await fs.systemTempDirectory.createTemp('asset_bundle_tests');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_asset_bundle_variant_test.');
oldCurrentDir = fs.currentDirectory;
fs.currentDirectory = tempDir;
});
tearDown(() {
fs.currentDirectory = oldCurrentDir;
- try {
- tempDir?.deleteSync(recursive: true);
- tempDir = null;
- } on FileSystemException catch (e) {
- // Do nothing, windows sometimes has trouble deleting.
- print('Ignored exception during tearDown: $e');
- }
+ tryToDelete(tempDir);
});
group('AssetBundle asset variants', () {
diff --git a/packages/flutter_tools/test/base/os_utils_test.dart b/packages/flutter_tools/test/base/os_utils_test.dart
index 5f6987c..27c9849 100644
--- a/packages/flutter_tools/test/base/os_utils_test.dart
+++ b/packages/flutter_tools/test/base/os_utils_test.dart
@@ -11,18 +11,18 @@
void main() {
group('OperatingSystemUtils', () {
- Directory temp;
+ Directory tempDir;
setUp(() {
- temp = fs.systemTempDirectory.createTempSync('flutter_tools');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_tools_os_utils_test.');
});
tearDown(() {
- temp.deleteSync(recursive: true);
+ tryToDelete(tempDir);
});
testUsingContext('makeExecutable', () async {
- final File file = fs.file(fs.path.join(temp.path, 'foo.script'));
+ final File file = fs.file(fs.path.join(tempDir.path, 'foo.script'));
file.writeAsStringSync('hello world');
os.makeExecutable(file);
diff --git a/packages/flutter_tools/test/commands/analyze_continuously_test.dart b/packages/flutter_tools/test/commands/analyze_continuously_test.dart
index dfb1972..018dc85 100644
--- a/packages/flutter_tools/test/commands/analyze_continuously_test.dart
+++ b/packages/flutter_tools/test/commands/analyze_continuously_test.dart
@@ -20,11 +20,11 @@
setUp(() {
FlutterCommandRunner.initFlutterRoot();
- tempDir = fs.systemTempDirectory.createTempSync('analysis_test');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_analysis_test.');
});
tearDown(() {
- tempDir?.deleteSync(recursive: true);
+ tryToDelete(tempDir);
return server?.dispose();
});
diff --git a/packages/flutter_tools/test/commands/analyze_once_test.dart b/packages/flutter_tools/test/commands/analyze_once_test.dart
index 3736082..3b0ee80 100644
--- a/packages/flutter_tools/test/commands/analyze_once_test.dart
+++ b/packages/flutter_tools/test/commands/analyze_once_test.dart
@@ -28,18 +28,13 @@
setUpAll(() {
Cache.disableLocking();
- tempDir = fs.systemTempDirectory.createTempSync('analyze_once_test_').absolute;
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_analyze_once_test_1.').absolute;
projectPath = fs.path.join(tempDir.path, 'flutter_project');
libMain = fs.file(fs.path.join(projectPath, 'lib', 'main.dart'));
});
tearDownAll(() {
- try {
- tempDir?.deleteSync(recursive: true);
- } on FileSystemException catch (e) {
- // ignore errors deleting the temporary directory
- print('Ignored exception during tearDown: $e');
- }
+ tryToDelete(tempDir);
});
// Create a project to be analyzed
@@ -134,7 +129,7 @@
}, timeout: allowForSlowAnalyzeTests);
testUsingContext('no duplicate issues', () async {
- final Directory tempDir = fs.systemTempDirectory.createTempSync('analyze_once_test_').absolute;
+ final Directory tempDir = fs.systemTempDirectory.createTempSync('flutter_analyze_once_test_2.').absolute;
try {
final File foo = fs.file(fs.path.join(tempDir.path, 'foo.dart'));
@@ -163,7 +158,7 @@
toolExit: true,
);
} finally {
- tempDir.deleteSync(recursive: true);
+ tryToDelete(tempDir);
}
});
@@ -171,7 +166,7 @@
const String contents = '''
StringBuffer bar = StringBuffer('baz');
''';
- final Directory tempDir = fs.systemTempDirectory.createTempSync();
+ final Directory tempDir = fs.systemTempDirectory.createTempSync('flutter_analyze_once_test_3.');
tempDir.childFile('main.dart').writeAsStringSync(contents);
try {
await runCommand(
@@ -180,7 +175,7 @@
statusTextContains: <String>['No issues found!'],
);
} finally {
- tempDir.deleteSync(recursive: true);
+ tryToDelete(tempDir);
}
});
});
diff --git a/packages/flutter_tools/test/commands/analyze_test.dart b/packages/flutter_tools/test/commands/analyze_test.dart
index bae562c..d86b574 100644
--- a/packages/flutter_tools/test/commands/analyze_test.dart
+++ b/packages/flutter_tools/test/commands/analyze_test.dart
@@ -20,7 +20,11 @@
fs = new MemoryFileSystem();
fs.directory(_kFlutterRoot).createSync(recursive: true);
Cache.flutterRoot = _kFlutterRoot;
- tempDir = fs.systemTempDirectory.createTempSync('analysis_test');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_analysis_test.');
+ });
+
+ tearDown(() {
+ tryToDelete(tempDir);
});
group('analyze', () {
diff --git a/packages/flutter_tools/test/commands/create_test.dart b/packages/flutter_tools/test/commands/create_test.dart
index d71d4fb..18edcce 100644
--- a/packages/flutter_tools/test/commands/create_test.dart
+++ b/packages/flutter_tools/test/commands/create_test.dart
@@ -24,7 +24,7 @@
void main() {
group('create', () {
- Directory temp;
+ Directory tempDir;
Directory projectDir;
FlutterVersion mockFlutterVersion;
LoggingProcessManager loggingProcessManager;
@@ -35,18 +35,13 @@
setUp(() {
loggingProcessManager = new LoggingProcessManager();
- temp = fs.systemTempDirectory.createTempSync('flutter_tools');
- projectDir = temp.childDirectory('flutter_project');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_tools_create_test.');
+ projectDir = tempDir.childDirectory('flutter_project');
mockFlutterVersion = new MockFlutterVersion();
});
tearDown(() {
- try {
- temp.deleteSync(recursive: true);
- } on FileSystemException catch (e) {
- // ignore errors deleting the temporary directory
- print('Ignored exception during tearDown: $e');
- }
+ tryToDelete(tempDir);
});
// Verify that we create a project that is well-formed.
diff --git a/packages/flutter_tools/test/commands/drive_test.dart b/packages/flutter_tools/test/commands/drive_test.dart
index 9507805..c15bbc9 100644
--- a/packages/flutter_tools/test/commands/drive_test.dart
+++ b/packages/flutter_tools/test/commands/drive_test.dart
@@ -24,7 +24,7 @@
DriveCommand command;
Device mockDevice;
MemoryFileSystem fs;
- Directory cwd;
+ Directory tempDir;
void withMockDevice([Device mock]) {
mockDevice = mock ?? new MockDevice();
@@ -40,8 +40,8 @@
command = new DriveCommand();
applyMocksToCommand(command);
fs = new MemoryFileSystem();
- cwd = fs.systemTempDirectory.createTempSync('some_app_');
- fs.currentDirectory = cwd;
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_drive_test.');
+ fs.currentDirectory = tempDir;
fs.directory('test').createSync();
fs.directory('test_driver').createSync();
fs.file('pubspec.yaml')..createSync();
@@ -68,13 +68,14 @@
restoreAppStopper();
restoreTestRunner();
restoreTargetDeviceFinder();
+ tryToDelete(tempDir);
});
testUsingContext('returns 1 when test file is not found', () async {
withMockDevice();
- final String testApp = fs.path.join(cwd.path, 'test', 'e2e.dart');
- final String testFile = fs.path.join(cwd.path, 'test_driver', 'e2e_test.dart');
+ final String testApp = fs.path.join(tempDir.path, 'test', 'e2e.dart');
+ final String testFile = fs.path.join(tempDir.path, 'test_driver', 'e2e_test.dart');
fs.file(testApp).createSync(recursive: true);
final List<String> args = <String>[
@@ -96,8 +97,8 @@
withMockDevice();
appStarter = expectAsync1((DriveCommand command) async => null);
- final String testApp = fs.path.join(cwd.path, 'test_driver', 'e2e.dart');
- final String testFile = fs.path.join(cwd.path, 'test_driver', 'e2e_test.dart');
+ final String testApp = fs.path.join(tempDir.path, 'test_driver', 'e2e.dart');
+ final String testFile = fs.path.join(tempDir.path, 'test_driver', 'e2e_test.dart');
final MemoryFileSystem memFs = fs;
await memFs.file(testApp).writeAsString('main() { }');
@@ -119,7 +120,7 @@
});
testUsingContext('returns 1 when app file is outside package', () async {
- final String appFile = fs.path.join(cwd.dirname, 'other_app', 'app.dart');
+ final String appFile = fs.path.join(tempDir.dirname, 'other_app', 'app.dart');
fs.file(appFile).createSync(recursive: true);
final List<String> args = <String>[
'drive',
@@ -131,7 +132,7 @@
} on ToolExit catch (e) {
expect(e.exitCode ?? 1, 1);
expect(testLogger.errorText, contains(
- 'Application file $appFile is outside the package directory ${cwd.path}',
+ 'Application file $appFile is outside the package directory ${tempDir.path}',
));
}
}, overrides: <Type, Generator>{
@@ -139,7 +140,7 @@
});
testUsingContext('returns 1 when app file is in the root dir', () async {
- final String appFile = fs.path.join(cwd.path, 'main.dart');
+ final String appFile = fs.path.join(tempDir.path, 'main.dart');
fs.file(appFile).createSync(recursive: true);
final List<String> args = <String>[
'drive',
@@ -162,8 +163,8 @@
testUsingContext('returns 0 when test ends successfully', () async {
withMockDevice();
- final String testApp = fs.path.join(cwd.path, 'test', 'e2e.dart');
- final String testFile = fs.path.join(cwd.path, 'test_driver', 'e2e_test.dart');
+ final String testApp = fs.path.join(tempDir.path, 'test', 'e2e.dart');
+ final String testFile = fs.path.join(tempDir.path, 'test_driver', 'e2e_test.dart');
appStarter = expectAsync1((DriveCommand command) async {
return new LaunchResult.succeeded();
@@ -193,8 +194,8 @@
testUsingContext('returns exitCode set by test runner', () async {
withMockDevice();
- final String testApp = fs.path.join(cwd.path, 'test', 'e2e.dart');
- final String testFile = fs.path.join(cwd.path, 'test_driver', 'e2e_test.dart');
+ final String testApp = fs.path.join(tempDir.path, 'test', 'e2e.dart');
+ final String testFile = fs.path.join(tempDir.path, 'test_driver', 'e2e_test.dart');
appStarter = expectAsync1((DriveCommand command) async {
return new LaunchResult.succeeded();
diff --git a/packages/flutter_tools/test/commands/format_test.dart b/packages/flutter_tools/test/commands/format_test.dart
index 43e7b5c..752e720 100644
--- a/packages/flutter_tools/test/commands/format_test.dart
+++ b/packages/flutter_tools/test/commands/format_test.dart
@@ -12,19 +12,19 @@
void main() {
group('format', () {
- Directory temp;
+ Directory tempDir;
setUp(() {
Cache.disableLocking();
- temp = fs.systemTempDirectory.createTempSync('flutter_tools');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_tools_format_test.');
});
tearDown(() {
- temp.deleteSync(recursive: true);
+ tryToDelete(tempDir);
});
testUsingContext('a file', () async {
- final String projectPath = await createProject(temp);
+ final String projectPath = await createProject(tempDir);
final File srcFile = fs.file(fs.path.join(projectPath, 'lib', 'main.dart'));
final String original = srcFile.readAsStringSync();
@@ -39,7 +39,7 @@
});
testUsingContext('dry-run', () async {
- final String projectPath = await createProject(temp);
+ final String projectPath = await createProject(tempDir);
final File srcFile = fs.file(
fs.path.join(projectPath, 'lib', 'main.dart'));
@@ -56,7 +56,7 @@
});
testUsingContext('dry-run with set-exit-if-changed', () async {
- final String projectPath = await createProject(temp);
+ final String projectPath = await createProject(tempDir);
final File srcFile = fs.file(
fs.path.join(projectPath, 'lib', 'main.dart'));
diff --git a/packages/flutter_tools/test/commands/ide_config_test.dart b/packages/flutter_tools/test/commands/ide_config_test.dart
index aa97518..79e799f 100644
--- a/packages/flutter_tools/test/commands/ide_config_test.dart
+++ b/packages/flutter_tools/test/commands/ide_config_test.dart
@@ -15,15 +15,15 @@
void main() {
group('ide_config', () {
- Directory temp;
+ Directory tempDir;
Directory templateDir;
Directory intellijDir;
Directory toolsDir;
Map<String, String> _getFilesystemContents([Directory root]) {
- final String tempPath = temp.absolute.path;
+ final String tempPath = tempDir.absolute.path;
final List<String> paths =
- (root ?? temp).listSync(recursive: true).map((FileSystemEntity entity) {
+ (root ?? tempDir).listSync(recursive: true).map((FileSystemEntity entity) {
final String relativePath = fs.path.relative(entity.path, from: tempPath);
return relativePath;
}).toList();
@@ -40,7 +40,7 @@
}
Map<String, String> _getManifest(Directory base, String marker, {bool isTemplate = false}) {
- final String basePath = fs.path.relative(base.path, from: temp.absolute.path);
+ final String basePath = fs.path.relative(base.path, from: tempDir.absolute.path);
final String suffix = isTemplate ? Template.copyTemplateExtension : '';
return <String, String>{
fs.path.join(basePath, '.idea'): 'dir',
@@ -59,12 +59,12 @@
void _populateDir(Map<String, String> manifest) {
for (String key in manifest.keys) {
if (manifest[key] == 'dir') {
- temp.childDirectory(key)..createSync(recursive: true);
+ tempDir.childDirectory(key)..createSync(recursive: true);
}
}
for (String key in manifest.keys) {
if (manifest[key] != 'dir') {
- temp.childFile(key)
+ tempDir.childFile(key)
..createSync(recursive: true)
..writeAsStringSync(manifest[key]);
}
@@ -72,7 +72,7 @@
}
bool _fileOrDirectoryExists(String path) {
- final String absPath = fs.path.join(temp.absolute.path, path);
+ final String absPath = fs.path.join(tempDir.absolute.path, path);
return fs.file(absPath).existsSync() || fs.directory(absPath).existsSync();
}
@@ -82,15 +82,15 @@
Map<String, String> expectedContents = const <String, String>{},
List<String> unexpectedPaths = const <String>[],
}) async {
- dir ??= temp;
+ dir ??= tempDir;
final IdeConfigCommand command = new IdeConfigCommand();
final CommandRunner<Null> runner = createTestCommandRunner(command);
- final List<String> finalArgs = <String>['--flutter-root=${temp.absolute.path}', 'ide-config'];
+ final List<String> finalArgs = <String>['--flutter-root=${tempDir.absolute.path}', 'ide-config'];
finalArgs.addAll(args);
await runner.run(finalArgs);
for (String path in expectedContents.keys) {
- final String absPath = fs.path.join(temp.absolute.path, path);
+ final String absPath = fs.path.join(tempDir.absolute.path, path);
expect(_fileOrDirectoryExists(fs.path.join(dir.path, path)), true,
reason: "$path doesn't exist");
if (fs.file(absPath).existsSync()) {
@@ -108,15 +108,15 @@
});
setUp(() {
- temp = fs.systemTempDirectory.createTempSync('flutter_tools_');
- final Directory packagesDir = temp.childDirectory('packages')..createSync(recursive: true);
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_tools_ide_config_test.');
+ final Directory packagesDir = tempDir.childDirectory('packages')..createSync(recursive: true);
toolsDir = packagesDir.childDirectory('flutter_tools')..createSync();
templateDir = toolsDir.childDirectory('ide_templates')..createSync();
intellijDir = templateDir.childDirectory('intellij')..createSync();
});
tearDown(() {
- temp.deleteSync(recursive: true);
+ tryToDelete(tempDir);
});
testUsingContext("doesn't touch existing files without --overwrite", () async {
@@ -126,7 +126,7 @@
isTemplate: true,
);
final Map<String, String> flutterManifest = _getManifest(
- temp,
+ tempDir,
'existing',
);
_populateDir(templateManifest);
@@ -144,7 +144,7 @@
isTemplate: true,
);
final Map<String, String> flutterManifest = _getManifest(
- temp,
+ tempDir,
'template',
);
_populateDir(templateManifest);
@@ -162,13 +162,13 @@
isTemplate: true,
);
final Map<String, String> flutterManifest = _getManifest(
- temp,
+ tempDir,
'existing',
);
_populateDir(templateManifest);
_populateDir(flutterManifest);
final Map<String, String> overwrittenManifest = _getManifest(
- temp,
+ tempDir,
'template',
);
final Map<String, String> expectedContents = templateManifest;
@@ -196,7 +196,7 @@
_populateDir(templateManifest);
templateManifest[flutterIml] = 'flutter existing';
final Map<String, String> flutterManifest = _getManifest(
- temp,
+ tempDir,
'existing',
);
_populateDir(flutterManifest);
@@ -216,7 +216,7 @@
);
_populateDir(templateManifest);
final Map<String, String> flutterManifest = _getManifest(
- temp,
+ tempDir,
'existing',
);
_populateDir(flutterManifest);
@@ -241,7 +241,7 @@
);
_populateDir(templateManifest);
final Map<String, String> flutterManifest = _getManifest(
- temp,
+ tempDir,
'existing',
);
flutterManifest.remove('flutter.iml');
@@ -275,7 +275,7 @@
);
_populateDir(templateManifest);
final Map<String, String> flutterManifest = _getManifest(
- temp,
+ tempDir,
'existing',
);
flutterManifest.remove(fs.path.join('packages', 'new', 'deep.iml'));
diff --git a/packages/flutter_tools/test/commands/packages_test.dart b/packages/flutter_tools/test/commands/packages_test.dart
index 7d5d670..efa71dd 100644
--- a/packages/flutter_tools/test/commands/packages_test.dart
+++ b/packages/flutter_tools/test/commands/packages_test.dart
@@ -35,18 +35,18 @@
void main() {
Cache.disableLocking();
group('packages get/upgrade', () {
- Directory temp;
+ Directory tempDir;
setUp(() {
- temp = fs.systemTempDirectory.createTempSync('flutter_tools');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_tools_packages_test.');
});
tearDown(() {
- temp.deleteSync(recursive: true);
+ tryToDelete(tempDir);
});
Future<String> createProjectWithPlugin(String plugin) async {
- final String projectPath = await createProject(temp);
+ final String projectPath = await createProject(tempDir);
final File pubspec = fs.file(fs.path.join(projectPath, 'pubspec.yaml'));
String content = await pubspec.readAsString();
content = content.replaceFirst(
@@ -168,7 +168,7 @@
}
testUsingContext('get fetches packages', () async {
- final String projectPath = await createProject(temp);
+ final String projectPath = await createProject(tempDir);
removeGeneratedFiles(projectPath);
await runCommandIn(projectPath, 'get');
@@ -178,7 +178,7 @@
}, timeout: allowForRemotePubInvocation);
testUsingContext('get --offline fetches packages', () async {
- final String projectPath = await createProject(temp);
+ final String projectPath = await createProject(tempDir);
removeGeneratedFiles(projectPath);
await runCommandIn(projectPath, 'get', args: <String>['--offline']);
@@ -188,7 +188,7 @@
}, timeout: allowForCreateFlutterProject);
testUsingContext('upgrade fetches packages', () async {
- final String projectPath = await createProject(temp);
+ final String projectPath = await createProject(tempDir);
removeGeneratedFiles(projectPath);
await runCommandIn(projectPath, 'upgrade');
@@ -210,7 +210,7 @@
}, timeout: allowForRemotePubInvocation, skip: true);
testUsingContext('get fetches packages and injects plugin in plugin project', () async {
final String projectPath = await createProject(
- temp,
+ tempDir,
arguments: <String>['-t', 'plugin', '--no-pub'],
);
final String exampleProjectPath = fs.path.join(projectPath, 'example');
diff --git a/packages/flutter_tools/test/commands/upgrade_test.dart b/packages/flutter_tools/test/commands/upgrade_test.dart
index 16469be..f4a0f0d 100644
--- a/packages/flutter_tools/test/commands/upgrade_test.dart
+++ b/packages/flutter_tools/test/commands/upgrade_test.dart
@@ -35,18 +35,18 @@
});
group('findProjectRoot', () {
- Directory temp;
+ Directory tempDir;
setUp(() async {
- temp = fs.systemTempDirectory.createTempSync('flutter_tools');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_tools_upgrade_test.');
});
tearDown(() {
- temp.deleteSync(recursive: true);
+ tryToDelete(tempDir);
});
testUsingContext('in project', () async {
- final String projectPath = await createProject(temp);
+ final String projectPath = await createProject(tempDir);
expect(findProjectRoot(projectPath), projectPath);
expect(findProjectRoot(fs.path.join(projectPath, 'lib')), projectPath);
@@ -56,7 +56,7 @@
});
testUsingContext('outside project', () async {
- final String projectPath = await createProject(temp);
+ final String projectPath = await createProject(tempDir);
expect(findProjectRoot(fs.directory(projectPath).parent.path), null);
expect(findProjectRoot(Cache.flutterRoot), null);
});
diff --git a/packages/flutter_tools/test/config_test.dart b/packages/flutter_tools/test/config_test.dart
index e476cb8..30edb71 100644
--- a/packages/flutter_tools/test/config_test.dart
+++ b/packages/flutter_tools/test/config_test.dart
@@ -9,13 +9,18 @@
void main() {
Config config;
+ Directory tempDir;
setUp(() {
- final Directory tempDirectory = fs.systemTempDirectory.createTempSync('flutter_test');
- final File file = fs.file(fs.path.join(tempDirectory.path, '.settings'));
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_config_test.');
+ final File file = fs.file(fs.path.join(tempDir.path, '.settings'));
config = new Config(file);
});
+ tearDown(() {
+ tryToDelete(tempDir);
+ });
+
group('config', () {
test('get set value', () async {
expect(config.getValue('foo'), null);
diff --git a/packages/flutter_tools/test/dependency_checker_test.dart b/packages/flutter_tools/test/dependency_checker_test.dart
index 0c55782..1fdbc7c 100644
--- a/packages/flutter_tools/test/dependency_checker_test.dart
+++ b/packages/flutter_tools/test/dependency_checker_test.dart
@@ -93,16 +93,18 @@
/// Tests that the flutter tool doesn't crash and displays a warning when its own location
/// changed since it was last referenced to in a package's .packages file.
testUsingContext('moved flutter sdk', () async {
- final Directory destinationPath = fs.systemTempDirectory.createTempSync('dependency_checker_test_');
+ final Directory tempDir = fs.systemTempDirectory.createTempSync('flutter_dependency_checker_test.');
+
// Copy the golden input and let the test run in an isolated temporary in-memory file system.
const LocalFileSystem localFileSystem = LocalFileSystem();
final Directory sourcePath = localFileSystem.directory(localFileSystem.path.join(dataPath, 'changed_sdk_location'));
- copyDirectorySync(sourcePath, destinationPath);
- fs.currentDirectory = destinationPath;
+ copyDirectorySync(sourcePath, tempDir);
+ fs.currentDirectory = tempDir;
// Doesn't matter what commands we run. Arbitrarily list devices here.
await createTestCommandRunner(new DevicesCommand()).run(<String>['devices']);
expect(testLogger.errorText, contains('.packages'));
+ tryToDelete(tempDir);
}, overrides: <Type, Generator>{
FileSystem: () => testFileSystem,
});
diff --git a/packages/flutter_tools/test/devfs_test.dart b/packages/flutter_tools/test/devfs_test.dart
index b2fb445..6a9d309 100644
--- a/packages/flutter_tools/test/devfs_test.dart
+++ b/packages/flutter_tools/test/devfs_test.dart
@@ -459,15 +459,14 @@
final Map <String, Uri> _packages = <String, Uri>{};
Directory _newTempDir(FileSystem fs) {
- final Directory tempDir = fs.systemTempDirectory.createTempSync('devfs${_tempDirs.length}');
+ final Directory tempDir = fs.systemTempDirectory.createTempSync('flutter_devfs${_tempDirs.length}_test.');
_tempDirs.add(tempDir);
return tempDir;
}
void _cleanupTempDirs() {
- while (_tempDirs.isNotEmpty) {
- _tempDirs.removeLast().deleteSync(recursive: true);
- }
+ while (_tempDirs.isNotEmpty)
+ tryToDelete(_tempDirs.removeLast());
}
Future<Null> _createPackage(FileSystem fs, String pkgName, String pkgFileName, { bool doubleSlash = false }) async {
diff --git a/packages/flutter_tools/test/integration/expression_evaluation_test.dart b/packages/flutter_tools/test/integration/expression_evaluation_test.dart
index 677bd29..e68b7c2 100644
--- a/packages/flutter_tools/test/integration/expression_evaluation_test.dart
+++ b/packages/flutter_tools/test/integration/expression_evaluation_test.dart
@@ -13,24 +13,21 @@
import 'test_data/basic_project.dart';
import 'test_driver.dart';
-BasicProject _project = new BasicProject();
-FlutterTestDriver _flutter;
-
void main() {
group('expression evaluation', () {
+ Directory tempDir;
+ final BasicProject _project = new BasicProject();
+ FlutterTestDriver _flutter;
+
setUp(() async {
- final Directory tempDir = await fs.systemTempDirectory.createTemp('test_app');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_expression_test.');
await _project.setUpIn(tempDir);
_flutter = new FlutterTestDriver(tempDir);
});
tearDown(() async {
- try {
- await _flutter.stop();
- _project.cleanup();
- } catch (e) {
- // Don't fail tests if we failed to clean up temp folder.
- }
+ await _flutter.stop();
+ tryToDelete(tempDir);
});
Future<VMIsolate> breakInBuildMethod(FlutterTestDriver flutter) async {
diff --git a/packages/flutter_tools/test/integration/flutter_attach_test.dart b/packages/flutter_tools/test/integration/flutter_attach_test.dart
index 6a2bace..68fb836 100644
--- a/packages/flutter_tools/test/integration/flutter_attach_test.dart
+++ b/packages/flutter_tools/test/integration/flutter_attach_test.dart
@@ -11,26 +11,22 @@
import 'test_data/basic_project.dart';
import 'test_driver.dart';
-FlutterTestDriver _flutterRun, _flutterAttach;
-BasicProject _project = new BasicProject();
-
void main() {
+ FlutterTestDriver _flutterRun, _flutterAttach;
+ final BasicProject _project = new BasicProject();
+ Directory tempDir;
setUp(() async {
- final Directory tempDir = await fs.systemTempDirectory.createTemp('test_app');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_attach_test.');
await _project.setUpIn(tempDir);
_flutterRun = new FlutterTestDriver(tempDir);
_flutterAttach = new FlutterTestDriver(tempDir);
});
tearDown(() async {
- try {
- await _flutterRun.stop();
- await _flutterAttach.stop();
- _project.cleanup();
- } catch (e) {
- // Don't fail tests if we failed to clean up temp folder.
- }
+ await _flutterRun.stop();
+ await _flutterAttach.stop();
+ tryToDelete(tempDir);
});
group('attached process', () {
diff --git a/packages/flutter_tools/test/integration/flutter_tester_test.dart b/packages/flutter_tools/test/integration/flutter_tester_test.dart
index e3d2575..6b98e19 100644
--- a/packages/flutter_tools/test/integration/flutter_tester_test.dart
+++ b/packages/flutter_tools/test/integration/flutter_tester_test.dart
@@ -19,19 +19,14 @@
Directory oldCurrentDir;
setUp(() async {
- tempDir = await fs.systemTempDirectory.createTemp('flutter_tester_device');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_tester_device_test.');
oldCurrentDir = fs.currentDirectory;
fs.currentDirectory = tempDir;
});
tearDown(() {
fs.currentDirectory = oldCurrentDir;
- try {
- tempDir?.deleteSync(recursive: true);
- tempDir = null;
- } catch (e) {
- // Ignored.
- }
+ tryToDelete(tempDir);
});
group('FlutterTesterDevice', () {
diff --git a/packages/flutter_tools/test/integration/hot_reload_test.dart b/packages/flutter_tools/test/integration/hot_reload_test.dart
index 58e6ddf..7d721ca 100644
--- a/packages/flutter_tools/test/integration/hot_reload_test.dart
+++ b/packages/flutter_tools/test/integration/hot_reload_test.dart
@@ -12,24 +12,21 @@
import 'test_data/basic_project.dart';
import 'test_driver.dart';
-BasicProject _project = new BasicProject();
-FlutterTestDriver _flutter;
-
void main() {
group('hot reload', () {
+ Directory tempDir;
+ final BasicProject _project = new BasicProject();
+ FlutterTestDriver _flutter;
+
setUp(() async {
- final Directory tempDir = await fs.systemTempDirectory.createTemp('test_app');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_hot_reload_test_app.');
await _project.setUpIn(tempDir);
_flutter = new FlutterTestDriver(tempDir);
});
tearDown(() async {
- try {
- await _flutter.stop();
- _project.cleanup();
- } catch (e) {
- // Don't fail tests if we failed to clean up temp folder.
- }
+ await _flutter.stop();
+ tryToDelete(tempDir);
});
test('works without error', () async {
diff --git a/packages/flutter_tools/test/integration/lifetime_test.dart b/packages/flutter_tools/test/integration/lifetime_test.dart
index 10336a6..79576da 100644
--- a/packages/flutter_tools/test/integration/lifetime_test.dart
+++ b/packages/flutter_tools/test/integration/lifetime_test.dart
@@ -12,9 +12,6 @@
import 'test_data/basic_project.dart';
import 'test_driver.dart';
-BasicProject _project = new BasicProject();
-FlutterTestDriver _flutter;
-
/// This duration is arbitrary but is ideally:
/// a) long enough to ensure that if the app is crashing at startup, we notice
/// b) as short as possible, to avoid inflating build times
@@ -22,15 +19,19 @@
void main() {
group('flutter run', () {
+ final BasicProject _project = new BasicProject();
+ FlutterTestDriver _flutter;
+ Directory tempDir;
+
setUp(() async {
- final Directory tempDir = await fs.systemTempDirectory.createTemp('test_app');
+ tempDir = fs.systemTempDirectory.createTempSync('flutter_lifetime_test.');
await _project.setUpIn(tempDir);
_flutter = new FlutterTestDriver(tempDir);
});
tearDown(() async {
await _flutter.stop();
- _project.cleanup();
+ tryToDelete(tempDir);
});
test('does not terminate when a debugger is attached', () async {
diff --git a/packages/flutter_tools/test/integration/test_data/test_project.dart b/packages/flutter_tools/test/integration/test_data/test_project.dart
index af825d2..1b73c45 100644
--- a/packages/flutter_tools/test/integration/test_data/test_project.dart
+++ b/packages/flutter_tools/test/integration/test_data/test_project.dart
@@ -26,10 +26,6 @@
await getPackages(dir.path);
}
- void cleanup() {
- dir?.deleteSync(recursive: true);
- }
-
int lineContaining(String contents, String search) {
final int index = contents.split('\n').indexWhere((String l) => l.contains(search));
if (index == -1)
diff --git a/packages/flutter_tools/test/integration/test_driver.dart b/packages/flutter_tools/test/integration/test_driver.dart
index 0df8792..ee9a76d 100644
--- a/packages/flutter_tools/test/integration/test_driver.dart
+++ b/packages/flutter_tools/test/integration/test_driver.dart
@@ -23,6 +23,8 @@
const Duration quitTimeout = Duration(seconds: 5);
class FlutterTestDriver {
+ FlutterTestDriver(this._projectFolder);
+
final Directory _projectFolder;
Process _proc;
int _procPid;
@@ -36,8 +38,6 @@
int _vmServicePort;
bool _hasExited = false;
- FlutterTestDriver(this._projectFolder);
-
VMServiceClient vmService;
String get lastErrorInfo => _errorBuffer.toString();
int get vmServicePort => _vmServicePort;
@@ -318,24 +318,24 @@
int id = 1;
Future<dynamic> _sendRequest(String method, dynamic params) async {
final int requestId = id++;
- final Map<String, dynamic> req = <String, dynamic>{
+ final Map<String, dynamic> request = <String, dynamic>{
'id': requestId,
'method': method,
'params': params
};
- final String jsonEncoded = json.encode(<Map<String, dynamic>>[req]);
+ final String jsonEncoded = json.encode(<Map<String, dynamic>>[request]);
_debugPrint(jsonEncoded);
// Set up the response future before we send the request to avoid any
// races.
final Future<Map<String, dynamic>> responseFuture = _waitFor(id: requestId);
_proc.stdin.writeln(jsonEncoded);
- final Map<String, dynamic> resp = await responseFuture;
+ final Map<String, dynamic> response = await responseFuture;
- if (resp['error'] != null || resp['result'] == null)
+ if (response['error'] != null || response['result'] == null)
_throwErrorResponse('Unexpected error response');
- return resp['result'];
+ return response['result'];
}
void _throwErrorResponse(String msg) {
diff --git a/packages/flutter_tools/test/src/common.dart b/packages/flutter_tools/test/src/common.dart
index 3aa0331..b932cf8 100644
--- a/packages/flutter_tools/test/src/common.dart
+++ b/packages/flutter_tools/test/src/common.dart
@@ -22,6 +22,17 @@
// TODO(ianh): Remove this once https://github.com/dart-lang/matcher/issues/98 is fixed
Matcher isInstanceOf<T>() => new test_package.TypeMatcher<T>(); // ignore: prefer_const_constructors, https://github.com/dart-lang/sdk/issues/32544
+void tryToDelete(Directory directory) {
+ // This should not be necessary, but it turns out that
+ // on Windows it's common for deletions to fail due to
+ // bogus (we think) "access denied" errors.
+ try {
+ directory.deleteSync(recursive: true);
+ } on FileSystemException catch (error) {
+ print('Failed to delete ${directory.path}: $error');
+ }
+}
+
/// Gets the path to the root of the Flutter repository.
///
/// This will first look for a `FLUTTER_ROOT` environment variable. If the
diff --git a/packages/flutter_tools/test/src/context.dart b/packages/flutter_tools/test/src/context.dart
index 0136da1..c366633 100644
--- a/packages/flutter_tools/test/src/context.dart
+++ b/packages/flutter_tools/test/src/context.dart
@@ -48,13 +48,16 @@
// leak a sticky $HOME/.flutter_settings behind!
Directory configDir;
tearDown(() {
- configDir?.deleteSync(recursive: true);
- configDir = null;
+ if (configDir != null) {
+ tryToDelete(configDir);
+ configDir = null;
+ }
});
Config buildConfig(FileSystem fs) {
- configDir = fs.systemTempDirectory.createTempSync('config-dir');
+ configDir = fs.systemTempDirectory.createTempSync('flutter_config_dir_test.');
final File settingsFile = fs.file(
- fs.path.join(configDir.path, '.flutter_settings'));
+ fs.path.join(configDir.path, '.flutter_settings')
+ );
return new Config(settingsFile);
}
diff --git a/packages/flutter_tools/test/src/mocks.dart b/packages/flutter_tools/test/src/mocks.dart
index 19e6e45..a4326e0 100644
--- a/packages/flutter_tools/test/src/mocks.dart
+++ b/packages/flutter_tools/test/src/mocks.dart
@@ -44,7 +44,7 @@
bool withNdkSysroot = false,
bool withSdkManager = true,
}) {
- final Directory dir = fs.systemTempDirectory.createTempSync('android-sdk');
+ final Directory dir = fs.systemTempDirectory.createTempSync('flutter_mock_android_sdk.');
_createSdkFile(dir, 'platform-tools/adb');
@@ -66,18 +66,23 @@
if (withNdkDir != null) {
final String ndkCompiler = fs.path.join(
- 'ndk-bundle',
- 'toolchains',
- 'arm-linux-androideabi-4.9',
- 'prebuilt',
- withNdkDir,
- 'bin',
- 'arm-linux-androideabi-gcc');
+ 'ndk-bundle',
+ 'toolchains',
+ 'arm-linux-androideabi-4.9',
+ 'prebuilt',
+ withNdkDir,
+ 'bin',
+ 'arm-linux-androideabi-gcc',
+ );
_createSdkFile(dir, ndkCompiler);
}
if (withNdkSysroot) {
- final String armPlatform =
- fs.path.join('ndk-bundle', 'platforms', 'android-9', 'arch-arm');
+ final String armPlatform = fs.path.join(
+ 'ndk-bundle',
+ 'platforms',
+ 'android-9',
+ 'arch-arm',
+ );
_createDir(dir, armPlatform);
}
