firka/flutter
firka/flutter
5
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove dead ensureDirectoryExists (#76829)

Browse Source
This commit is contained in:
Jenn Magder 2021-02-25 18:11:01 -08:00 committed by GitHub
parent cc813bf153
commit f5903a91e0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 0 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
packages/flutter_tools/lib/src/base/file_system.dart
View File

@ -8,7 +8,6 @@ import 'package:file/file.dart';
import 'package:file/local.dart' as local_fs; import 'package:file/local.dart' as local_fs;
import 'package:meta/meta.dart'; import 'package:meta/meta.dart';
import 'common.dart' show throwToolExit;
import 'io.dart'; import 'io.dart';
import 'platform.dart'; import 'platform.dart';
import 'process.dart'; import 'process.dart';
@ -41,19 +40,6 @@ class FileSystemUtils {
final Platform _platform; final Platform _platform;
/// Create the ancestor directories of a file path if they do not already exist.
void ensureDirectoryExists(String filePath) {
final String dirPath = _fileSystem.path.dirname(filePath);
if (_fileSystem.isDirectorySync(dirPath)) {
return;
}
try {
_fileSystem.directory(dirPath).createSync(recursive: true);
} on FileSystemException catch (e) {
throwToolExit('Failed to create directory "$dirPath": ${e.osError.message}');
}
}
/// Appends a number to a filename in order to make it unique under a /// Appends a number to a filename in order to make it unique under a
/// directory. /// directory.
File getUniqueFile(Directory dir, String baseName, String ext) { File getUniqueFile(Directory dir, String baseName, String ext) {

10
packages/flutter_tools/test/general.shard/base/file_system_test.dart
View File

@ -30,16 +30,6 @@ void main() {
); );
}); });
testWithoutContext('ensureDirectoryExists recursively creates a directory if it does not exist', () async {
fsUtils.ensureDirectoryExists('foo/bar/baz.flx');
expect(fs.isDirectorySync('foo/bar'), true);
});
testWithoutContext('ensureDirectoryExists throws tool exit on failure to create', () async {
fs.file('foo').createSync();
expect(() => fsUtils.ensureDirectoryExists('foo/bar.flx'), throwsToolExit());
});
testWithoutContext('getUniqueFile creates a unique file name', () async { testWithoutContext('getUniqueFile creates a unique file name', () async {
final File fileA = fsUtils.getUniqueFile(fs.currentDirectory, 'foo', 'json') final File fileA = fsUtils.getUniqueFile(fs.currentDirectory, 'foo', 'json')
..createSync(); ..createSync();