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

Unified Diff: utils/tests/pub/lock_file_test.dart

Issue 10541039: Update pub to new expect API. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 6 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 | « no previous file | utils/tests/pub/version_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: utils/tests/pub/lock_file_test.dart
diff --git a/utils/tests/pub/lock_file_test.dart b/utils/tests/pub/lock_file_test.dart
index 5cb978573e86b798b2295ea2ced7a7af77d208aa..da492fed5a3a3d475e68f3ed163f3668d1c2be33 100644
--- a/utils/tests/pub/lock_file_test.dart
+++ b/utils/tests/pub/lock_file_test.dart
@@ -33,12 +33,12 @@ main() {
group('parse()', () {
test('returns an empty lockfile if the contents are empty', () {
var lockFile = new LockFile.parse('', sources);
- expect(lockFile.packages.length).equals(0);
gram 2012/06/06 22:48:29 isZero?
+ expect(lockFile.packages.length, equals(0));
});
test('returns an empty lockfile if the contents are whitespace', () {
var lockFile = new LockFile.parse(' \t\n ', sources);
- expect(lockFile.packages.length).equals(0);
+ expect(lockFile.packages.length, equals(0));
});
test('parses a series of package descriptions', () {
@@ -54,19 +54,19 @@ packages:
description: foo desc
''', sources);
- expect(lockFile.packages.length).equals(2);
+ expect(lockFile.packages.length, equals(2));
var bar = lockFile.packages['bar'];
- expect(bar.name).equals('bar');
- expect(bar.version).equals(new Version(1, 2, 3));
- expect(bar.source).equals(mockSource);
- expect(bar.description).equals('bar desc');
+ expect(bar.name, equals('bar'));
+ expect(bar.version, equals(new Version(1, 2, 3)));
+ expect(bar.source, equals(mockSource));
+ expect(bar.description, equals('bar desc'));
var foo = lockFile.packages['foo'];
- expect(foo.name).equals('foo');
- expect(foo.version).equals(new Version(2, 3, 4));
- expect(foo.source).equals(mockSource);
- expect(foo.description).equals('foo desc');
+ expect(foo.name, equals('foo'));
+ expect(foo.version, equals(new Version(2, 3, 4)));
+ expect(foo.source, equals(mockSource));
+ expect(foo.description, equals('foo desc'));
});
test("throws if the version is missing", () {
« no previous file with comments | « no previous file | utils/tests/pub/version_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698