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

Unified Diff: test/version_solver_test.dart

Issue 1556243002: Fix analyzer hints and warnings in test/. (Closed) Base URL: git@github.com:dart-lang/pub.git@master
Patch Set: Created 4 years, 12 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 | « test/validator/sdk_constraint_test.dart ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/version_solver_test.dart
diff --git a/test/version_solver_test.dart b/test/version_solver_test.dart
index f4a24651545d27ff6b9e3274fbbec85c86a1c900..23e853ee9fcb55fa4330fdfc8eb5187e28cdad87 100644
--- a/test/version_solver_test.dart
+++ b/test/version_solver_test.dart
@@ -18,8 +18,6 @@ import 'package:pub/src/utils.dart';
import 'package:pub_semver/pub_semver.dart';
import 'package:test/test.dart';
-import 'test_pub.dart';
-
MockSource source1;
MockSource source2;
@@ -1110,27 +1108,9 @@ void downgrade() {
testResolve(String description, Map packages, {
Map lockfile, Map overrides, Map result, FailMatcherBuilder error,
int maxTries, bool downgrade: false}) {
- _testResolve(test, description, packages, lockfile: lockfile,
- overrides: overrides, result: result, error: error, maxTries: maxTries,
- downgrade: downgrade);
-}
-
-solo_testResolve(String description, Map packages, {
- Map lockfile, Map overrides, Map result, FailMatcherBuilder error,
- int maxTries, bool downgrade: false}) {
- log.verbosity = log.Verbosity.SOLVER;
- _testResolve(solo_test, description, packages, lockfile: lockfile,
- overrides: overrides, result: result, error: error, maxTries: maxTries,
- downgrade: downgrade);
-}
-
-_testResolve(void testFn(String description, Function body),
- String description, Map packages, {
- Map lockfile, Map overrides, Map result, FailMatcherBuilder error,
- int maxTries, bool downgrade: false}) {
if (maxTries == null) maxTries = 1;
- testFn(description, () {
+ test(description, () {
var cache = new SystemCache('.');
source1 = new MockSource('mock1');
source2 = new MockSource('mock2');
@@ -1439,7 +1419,7 @@ class MockSource extends CachedSource {
List<Package> getCachedPackages() =>
throw new UnsupportedError('Cannot get mock packages');
- Future<Pair<List<Package>, List<Package>>> repairCachedPackages() =>
+ Future<Pair<List<PackageId>, List<PackageId>>> repairCachedPackages() =>
throw new UnsupportedError('Cannot repair mock packages');
void addPackage(String description, Package package) {
« no previous file with comments | « test/validator/sdk_constraint_test.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698