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

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

Issue 1986853003: Move TypeParameterizedElementForLink and TypeParameterElementForLink to impl. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 4 years, 7 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/analyzer/test/src/summary/resynthesize_test.dart
diff --git a/pkg/analyzer/test/src/summary/resynthesize_test.dart b/pkg/analyzer/test/src/summary/resynthesize_test.dart
index e4f296e1d7d882c1cfe7ea0d397abe06c4c66cc4..23ff7e59f71e459073215f10a24dde04946c2843 100644
--- a/pkg/analyzer/test/src/summary/resynthesize_test.dart
+++ b/pkg/analyzer/test/src/summary/resynthesize_test.dart
@@ -674,7 +674,10 @@ abstract class AbstractResynthesizeTest extends AbstractSingleUnitTest {
expect(resynthesized.documentationComment, original.documentationComment,
reason: desc);
expect(resynthesized.docRange, original.docRange, reason: desc);
- compareMetadata(resynthesized.metadata, original.metadata, desc);
+ // TODO(scheglov) MLI restore testing annotations for type parameters
Paul Berry 2016/05/17 19:28:02 Another unclear usage of "MLI"
scheglov 2016/05/17 19:42:21 Done.
+ if (original is TypeParameterElement) {} else {
Paul Berry 2016/05/17 19:28:02 Nit: use "is!" to avoid the empty if-clause. How
scheglov 2016/05/17 19:42:21 Yeah, I don't like turning off tests too. I will t
Brian Wilkerson 2016/05/17 19:43:57 If they are not already doing so, the flutter team
+ compareMetadata(resynthesized.metadata, original.metadata, desc);
+ }
// Validate modifiers.
for (Modifier modifier in Modifier.persistedValues) {

Powered by Google App Engine
This is Rietveld 408576698