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

Unified Diff: pkg/analyzer/test/src/summary/summary_common.dart

Issue 2669863003: Handle invalid URIs in summaries. (Closed)
Patch Set: Remove uri.dart, tweak for summarize_ast_test. Created 3 years, 11 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
« no previous file with comments | « pkg/analyzer/test/src/summary/summarize_ast_test.dart ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/analyzer/test/src/summary/summary_common.dart
diff --git a/pkg/analyzer/test/src/summary/summary_common.dart b/pkg/analyzer/test/src/summary/summary_common.dart
index b8336556d85781ad099ea2cf18728040ff67a830..5f69ff10b9c3ef1a245f557091e689fec78b1134 100644
--- a/pkg/analyzer/test/src/summary/summary_common.dart
+++ b/pkg/analyzer/test/src/summary/summary_common.dart
@@ -6481,6 +6481,14 @@ class B extends A {}
expect(unlinkedExports[0].configurations, isEmpty);
}
+ test_export_uri_invalid() {
+ String uriString = '[invalid uri]';
+ String libraryText = 'export "$uriString";';
+ serializeLibraryText(libraryText);
+ expect(unlinkedUnits[0].publicNamespace.exports, hasLength(1));
+ expect(unlinkedUnits[0].publicNamespace.exports[0].uri, uriString);
+ }
+
test_export_uri_nullStringValue() {
String libraryText = r'''
export "${'a'}.dart";
@@ -8404,6 +8412,15 @@ class D extends p.C {} // Prevent "unused import" warning
expect(unlinkedUnits[0].imports[0].uri, 'dart:async');
}
+ test_import_uri_invalid() {
+ String uriString = '[invalid uri]';
+ String libraryText = 'import "$uriString";';
+ serializeLibraryText(libraryText);
+ // Second import is the implicit import of dart:core
+ expect(unlinkedUnits[0].imports, hasLength(2));
+ expect(unlinkedUnits[0].imports[0].uri, uriString);
+ }
+
test_import_uri_nullStringValue() {
String libraryText = r'''
import "${'a'}.dart";
@@ -9602,6 +9619,14 @@ part "${'a'}.dart"; // <-part
expect(unlinkedUnits[1].isPartOf, isTrue);
}
+ test_part_uri_invalid() {
+ String uriString = '[invalid uri]';
+ String libraryText = 'part "$uriString";';
+ serializeLibraryText(libraryText);
+ expect(unlinkedUnits[0].publicNamespace.parts, hasLength(1));
+ expect(unlinkedUnits[0].publicNamespace.parts[0], uriString);
+ }
+
test_parts_defining_compilation_unit() {
serializeLibraryText('');
expect(linked.units, hasLength(1));
« no previous file with comments | « pkg/analyzer/test/src/summary/summarize_ast_test.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698