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

Unified Diff: tests/compiler/dart2js/serialization_impact_test.dart

Issue 1870133002: Refactor serialization test files. (Closed) Base URL: https://github.com/dart-lang/sdk.git@master
Patch Set: Fix Created 4 years, 8 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: tests/compiler/dart2js/serialization_impact_test.dart
diff --git a/tests/compiler/dart2js/serialization_impact_test.dart b/tests/compiler/dart2js/serialization_impact_test.dart
index 2fbd212a1729239aa99cc4720c3dc1923e98a0ff..05d91d6dd981f8366432a33079a4365e3da03567 100644
--- a/tests/compiler/dart2js/serialization_impact_test.dart
+++ b/tests/compiler/dart2js/serialization_impact_test.dart
@@ -14,7 +14,7 @@ import 'package:compiler/src/filenames.dart';
import 'package:compiler/src/serialization/equivalence.dart';
import 'memory_compiler.dart';
import 'serialization_helper.dart';
-import 'serialization_test.dart';
+import 'serialization_test_helper.dart';
main(List<String> arguments) {
asyncTest(() async {
@@ -47,65 +47,5 @@ Future check(
deserialize(compilerDeserialized, serializedData);
await compilerDeserialized.run(entryPoint);
- checkResolutionImpacts(compilerNormal, compilerDeserialized, verbose: true);
+ checkAllImpacts(compilerNormal, compilerDeserialized, verbose: true);
}
-
-/// Check equivalence of [impact1] and [impact2].
-void checkImpacts(Element element1, Element element2,
- ResolutionImpact impact1, ResolutionImpact impact2,
- {bool verbose: false}) {
- if (impact1 == null || impact2 == null) return;
-
- if (verbose) {
- print('Checking impacts for $element1 vs $element2');
- }
-
- testResolutionImpactEquivalence(impact1, impact2, const CheckStrategy());
-}
-
-
-/// Check equivalence between all resolution impacts common to [compiler1] and
-/// [compiler2].
-void checkResolutionImpacts(
- Compiler compiler1,
- Compiler compiler2,
- {bool verbose: false}) {
-
- void checkMembers(Element member1, Element member2) {
- if (member1.isClass && member2.isClass) {
- ClassElement class1 = member1;
- ClassElement class2 = member2;
- class1.forEachLocalMember((m1) {
- checkMembers(m1, class2.lookupLocalMember(m1.name));
- });
- return;
- }
-
- if (!compiler1.resolution.hasResolutionImpact(member1)) {
- return;
- }
-
- if (member2 == null) {
- return;
- }
-
- if (areElementsEquivalent(member1, member2)) {
- checkImpacts(
- member1, member2,
- compiler1.resolution.getResolutionImpact(member1),
- compiler2.serialization.deserializer.getResolutionImpact(member2),
- verbose: verbose);
- }
- }
-
- for (LibraryElement library1 in compiler1.libraryLoader.libraries) {
- LibraryElement library2 =
- compiler2.libraryLoader.lookupLibrary(library1.canonicalUri);
- if (library2 != null) {
- library1.forEachLocalMember((Element member1) {
- checkMembers(member1, library2.localLookup(member1.name));
- });
-
- }
- }
-}
« no previous file with comments | « tests/compiler/dart2js/serialization_analysis_test.dart ('k') | tests/compiler/dart2js/serialization_model_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698