Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(176)

Unified Diff: pkg/front_end/test/physical_file_system_test.dart

Issue 2471283002: Add implementations of the front end FileSystem API. (Closed)
Patch Set: Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: pkg/front_end/test/physical_file_system_test.dart
diff --git a/pkg/front_end/test/physical_file_system_test.dart b/pkg/front_end/test/physical_file_system_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..f77d2ff86d1045125a00fe1b0f5fa83aa992bf47
--- /dev/null
+++ b/pkg/front_end/test/physical_file_system_test.dart
@@ -0,0 +1,193 @@
+// Copyright (c) 2016, the Dart project authors. Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+// SharedOptions=--supermixin
+
+library front_end.test.physical_file_system_test;
+
+import 'dart:convert';
+import 'dart:io' as io;
+
+import 'package:front_end/file_system.dart';
+import 'package:front_end/physical_file_system.dart';
+import 'package:path/path.dart' as p;
+import 'package:test/test.dart';
+import 'package:test_reflective_loader/test_reflective_loader.dart';
+
+main() {
+ defineReflectiveSuite(() {
+ defineReflectiveTests(PhysicalFileSystemTest);
+ defineReflectiveTests(FileTest);
+ });
+}
+
+@reflectiveTest
+class FileTest extends _BaseTest {
+ String path;
+ FileSystemEntity file;
+
+ setUp() {
+ super.setUp();
+ path = p.join(tempPath, 'file.txt');
+ file = PhysicalFileSystem.instance.entityForPath(path);
+ }
+
+ test_equals_differentPaths() {
+ expect(
+ file ==
+ PhysicalFileSystem.instance
+ .entityForPath(p.join(tempPath, 'file2.txt')),
+ isFalse);
+ }
+
+ test_equals_samePath() {
+ expect(
+ file ==
+ PhysicalFileSystem.instance
+ .entityForPath(p.join(tempPath, 'file.txt')),
+ isTrue);
+ }
+
+ test_hashCode_samePath() {
+ expect(
+ file.hashCode,
+ PhysicalFileSystem.instance
+ .entityForPath(p.join(tempPath, 'file.txt'))
+ .hashCode);
+ }
+
+ test_path() {
+ expect(file.path, path);
+ }
+
+ test_readAsBytes_badUtf8() async {
+ // A file containing invalid UTF-8 can still be read as raw bytes.
+ List<int> bytes = [0xc0, 0x40]; // Invalid UTF-8
+ new io.File(path).writeAsBytesSync(bytes);
+ expect(await file.readAsBytes(), bytes);
+ }
+
+ test_readAsBytes_doesNotExist() {
+ expect(file.readAsBytes(), throwsException);
+ }
+
+ test_readAsBytes_exists() async {
+ var s = 'contents';
+ new io.File(path).writeAsStringSync(s);
+ expect(await file.readAsBytes(), UTF8.encode(s));
+ }
+
+ test_readAsString_badUtf8() {
+ new io.File(path).writeAsBytesSync([0xc0, 0x40]); // Invalid UTF-8
+ expect(file.readAsString(), throwsException);
+ }
+
+ test_readAsString_doesNotExist() {
+ expect(file.readAsString(), throwsException);
+ }
+
+ test_readAsString_exists() async {
+ var s = 'contents';
+ new io.File(path).writeAsStringSync(s);
+ expect(await file.readAsString(), s);
+ }
+
+ test_readAsString_utf8() async {
+ var bytes = [0xe2, 0x82, 0xac]; // Unicode € symbol (in UTF-8)
+ new io.File(path).writeAsBytesSync(bytes);
+ expect(await file.readAsString(), '\u20ac');
+ }
+}
+
+@reflectiveTest
+class PhysicalFileSystemTest extends _BaseTest {
+ Uri tempUri;
+
+ setUp() {
+ super.setUp();
+ tempUri = new Uri.directory(tempPath);
+ }
+
+ test_entityForPath() {
+ var path = p.join(tempPath, 'file.txt');
+ expect(PhysicalFileSystem.instance.entityForPath(path).path, path);
+ }
+
+ test_entityForPath_absolutize() {
+ expect(PhysicalFileSystem.instance.entityForPath('file.txt').path,
+ new io.File('file.txt').absolute.path);
+ }
+
+ test_entityForPath_normalize_dot() {
+ expect(
+ PhysicalFileSystem.instance
+ .entityForPath(p.join(tempPath, '.', 'file.txt'))
+ .path,
+ p.join(tempPath, 'file.txt'));
+ }
+
+ test_entityForPath_normalize_dotDot() {
+ expect(
+ PhysicalFileSystem.instance
+ .entityForPath(p.join(tempPath, 'foo', '..', 'file.txt'))
+ .path,
+ p.join(tempPath, 'file.txt'));
+ }
+
+ test_entityForUri() {
+ expect(
+ PhysicalFileSystem.instance
+ .entityForUri(Uri.parse('$tempUri/file.txt'))
+ .path,
+ p.join(tempPath, 'file.txt'));
+ }
+
+ test_entityForUri_bareUri_absolute() {
+ expect(
+ () => PhysicalFileSystem.instance.entityForUri(Uri.parse('/file.txt')),
+ throwsA(new isInstanceOf<Error>()));
+ }
+
+ test_entityForUri_bareUri_relative() {
+ expect(
+ () => PhysicalFileSystem.instance.entityForUri(Uri.parse('file.txt')),
+ throwsA(new isInstanceOf<Error>()));
+ }
+
+ test_entityForUri_nonFileUri() {
+ expect(
+ () => PhysicalFileSystem.instance
+ .entityForUri(Uri.parse('package:foo/bar.dart')),
+ throwsA(new isInstanceOf<Error>()));
+ }
+
+ test_entityForUri_normalize_dot() {
+ expect(
+ PhysicalFileSystem.instance
+ .entityForUri(Uri.parse('$tempUri/./file.txt'))
+ .path,
+ p.join(tempPath, 'file.txt'));
+ }
+
+ test_entityForUri_normalize_dotDot() {
+ expect(
+ PhysicalFileSystem.instance
+ .entityForUri(Uri.parse('$tempUri/foo/../file.txt'))
+ .path,
+ p.join(tempPath, 'file.txt'));
+ }
+}
+
+class _BaseTest {
+ io.Directory tempDirectory;
+ String tempPath;
+
+ setUp() {
+ tempDirectory = io.Directory.systemTemp.createTempSync('test_file_system');
+ tempPath = tempDirectory.absolute.path;
+ }
+
+ tearDown() {
+ tempDirectory.deleteSync(recursive: true);
+ }
+}

Powered by Google App Engine
This is Rietveld 408576698