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

Unified Diff: pkg/analysis_server/test/context_manager_test.dart

Issue 2323173002: Remove several references to JavaFile (Closed)
Patch Set: Created 4 years, 3 months 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/analysis_server/test/context_manager_test.dart
diff --git a/pkg/analysis_server/test/context_manager_test.dart b/pkg/analysis_server/test/context_manager_test.dart
index 1aa02fef7ad9ee6fa3e671afd19293061e77fd2f..c00648c592980b15ec2e9136afecd374312fa1bf 100644
--- a/pkg/analysis_server/test/context_manager_test.dart
+++ b/pkg/analysis_server/test/context_manager_test.dart
@@ -5,6 +5,7 @@
library test.context.directory.manager;
import 'dart:collection';
+import 'dart:io' as io;
import 'package:analysis_server/src/context_manager.dart';
import 'package:analyzer/file_system/file_system.dart';
@@ -15,7 +16,6 @@ import 'package:analyzer/src/context/builder.dart';
import 'package:analyzer/src/dart/sdk/sdk.dart';
import 'package:analyzer/src/generated/engine.dart';
import 'package:analyzer/src/generated/error.dart';
-import 'package:analyzer/src/generated/java_io.dart';
import 'package:analyzer/src/generated/sdk.dart';
import 'package:analyzer/src/generated/source.dart';
import 'package:analyzer/src/generated/source_io.dart';
@@ -154,8 +154,10 @@ test_pack:lib/''');
var context = contexts[0];
var source = context.sourceFactory.forUri('dart:foobar');
expect(source, isNotNull);
- expect(source.fullName,
- '/my/proj/sdk_ext/entry.dart'.replaceAll('/', JavaFile.separator));
+ expect(
+ source.fullName,
+ '/my/proj/sdk_ext/entry.dart'
+ .replaceAll('/', io.Platform.pathSeparator));
// We can't find dart:core because we didn't list it in our
// embedded_libs map.
expect(context.sourceFactory.forUri('dart:core'), isNull);
@@ -1771,7 +1773,7 @@ abstract class ContextManagerTest {
'**/${AnalysisEngine.ANALYSIS_OPTIONS_YAML_FILE}'
];
return patterns
- .map((pattern) => new Glob(JavaFile.pathContext.separator, pattern))
+ .map((pattern) => new Glob(posix.separator, pattern))
.toList();
}
@@ -2193,8 +2195,10 @@ linter:
// Sanity check embedder libs.
var source = context.sourceFactory.forUri('dart:foobar');
expect(source, isNotNull);
- expect(source.fullName,
- '/my/proj/sdk_ext/entry.dart'.replaceAll('/', JavaFile.separator));
+ expect(
+ source.fullName,
+ '/my/proj/sdk_ext/entry.dart'
+ .replaceAll('/', io.Platform.pathSeparator));
}
test_embedder_options() async {
@@ -2287,8 +2291,10 @@ linter:
// Sanity check embedder libs.
var source = context.sourceFactory.forUri('dart:foobar');
expect(source, isNotNull);
- expect(source.fullName,
- '/my/proj/sdk_ext/entry.dart'.replaceAll('/', JavaFile.separator));
+ expect(
+ source.fullName,
+ '/my/proj/sdk_ext/entry.dart'
+ .replaceAll('/', io.Platform.pathSeparator));
}
test_error_filter_analysis_option() async {
« no previous file with comments | « pkg/analysis_server/lib/src/analysis_server.dart ('k') | pkg/analysis_server/test/single_context_manager_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698