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

Unified Diff: pkg/watcher/test/path_set_test.dart

Issue 68683004: Make path_set_test use Windows-friendly paths where necessary. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: typo Created 7 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/watcher/test/path_set_test.dart
diff --git a/pkg/watcher/test/path_set_test.dart b/pkg/watcher/test/path_set_test.dart
index cc8932d78bfd2d9cf55f52650fb61a931d08afa1..f433ad3c35cfae2d60038b4139cbd848edbc3429 100644
--- a/pkg/watcher/test/path_set_test.dart
+++ b/pkg/watcher/test/path_set_test.dart
@@ -54,7 +54,7 @@ void main() {
test("that's in the set removes and returns that path", () {
set.add("root/path/to/file");
expect(set.remove("root/path/to/file"),
- unorderedEquals(["root/path/to/file"]));
+ unorderedEquals([p.normalize("root/path/to/file")]));
expect(set, isNot(containsPath("root/path/to/file")));
});
@@ -73,7 +73,7 @@ void main() {
"root/path/to/one",
"root/path/to/two",
"root/path/to/sub/three"
- ]));
+ ].map(p.normalize)));
expect(set, containsPath("root/outside"));
expect(set, isNot(containsPath("root/path/to/one")));
@@ -93,7 +93,7 @@ void main() {
"root/path/to/one",
"root/path/to/two",
"root/path/to/sub/three"
- ]));
+ ].map(p.normalize)));
expect(set, isNot(containsPath("root/path")));
expect(set, isNot(containsPath("root/path/to/one")));
@@ -104,13 +104,13 @@ void main() {
test("that's not normalized removes and returns the normalized path", () {
set.add("root/path/to/file");
expect(set.remove("root/../root/path/to/../to/./file"),
- unorderedEquals(["root/path/to/file"]));
+ unorderedEquals([p.normalize("root/path/to/file")]));
});
test("that's absolute removes and returns the normalized path", () {
set.add("root/path/to/file");
expect(set.remove(p.absolute("root/path/to/file")),
- unorderedEquals(["root/path/to/file"]));
+ unorderedEquals([p.normalize("root/path/to/file")]));
});
test("that's not beneath the root throws an error", () {
@@ -210,7 +210,7 @@ void main() {
"root/path",
"root/path/to/one",
"root/path/to/two",
- ]));
+ ].map(p.normalize)));
});
test("doesn't return paths removed from the set", () {
@@ -218,7 +218,7 @@ void main() {
set.add("root/path/to/two");
set.remove("root/path/to/two");
- expect(set.toSet(), unorderedEquals(["root/path/to/one"]));
+ expect(set.toSet(), unorderedEquals([p.normalize("root/path/to/one")]));
});
});
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698