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

Unified Diff: packages/usage/test/hit_types_test.dart

Issue 2990843002: Removed fixed dependencies (Closed)
Patch Set: Created 3 years, 5 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 | « packages/usage/test/all.dart ('k') | packages/usage/test/src/common.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: packages/usage/test/hit_types_test.dart
diff --git a/packages/usage/test/hit_types_test.dart b/packages/usage/test/hit_types_test.dart
index e8bf241c2d845a462bd1a6ba5700865992690086..4bfb608b772815cfc2000789ba6537b5c7966bb2 100644
--- a/packages/usage/test/hit_types_test.dart
+++ b/packages/usage/test/hit_types_test.dart
@@ -21,13 +21,6 @@ void defineTests() {
expect(mock.mockProperties['clientId'], isNotNull);
expect(mock.mockPostHandler.sentValues, isNot(isEmpty));
});
- test('with parameters', () {
- AnalyticsImplMock mock = createMock();
- mock.sendScreenView('withParams', parameters: {'cd1': 'foo'});
- expect(mock.mockProperties['clientId'], isNotNull);
- expect(mock.mockPostHandler.sentValues, isNot(isEmpty));
- has(mock.last, 'cd1');
- });
});
group('event', () {
@@ -40,16 +33,6 @@ void defineTests() {
has(mock.last, 'ea');
});
- test('with parameters', () {
- AnalyticsImplMock mock = createMock();
- mock.sendEvent('withParams', 'save', parameters: {'cd1': 'foo'});
- expect(mock.mockPostHandler.sentValues, isNot(isEmpty));
- was(mock.last, 'event');
- has(mock.last, 'ec');
- has(mock.last, 'ea');
- has(mock.last, 'cd1');
- });
-
test('optional args', () {
AnalyticsImplMock mock = createMock();
mock.sendEvent('files', 'save', label: 'File Save', value: 23);
@@ -95,25 +78,26 @@ void defineTests() {
has(mock.last, 'utl');
});
- test('timer', () async {
+ test('timer', () {
AnalyticsImplMock mock = createMock();
AnalyticsTimer timer =
mock.startTimer('compile', category: 'Build', label: 'Compile');
- await new Future.delayed(new Duration(milliseconds: 20));
-
- await timer.finish();
- expect(mock.mockPostHandler.sentValues, isNot(isEmpty));
- was(mock.last, 'timing');
- has(mock.last, 'utv');
- has(mock.last, 'utt');
- has(mock.last, 'utc');
- has(mock.last, 'utl');
- int time = timer.currentElapsedMillis;
- expect(time, greaterThan(10));
-
- await new Future.delayed(new Duration(milliseconds: 10));
- expect(timer.currentElapsedMillis, time);
+ return new Future.delayed(new Duration(milliseconds: 20), () {
+ return timer.finish().then((_) {
+ expect(mock.mockPostHandler.sentValues, isNot(isEmpty));
+ was(mock.last, 'timing');
+ has(mock.last, 'utv');
+ has(mock.last, 'utt');
+ has(mock.last, 'utc');
+ has(mock.last, 'utl');
+ int time = timer.currentElapsedMillis;
+ expect(time, greaterThan(10));
+ return new Future.delayed(new Duration(milliseconds: 10), () {
+ expect(timer.currentElapsedMillis, time);
+ });
+ });
+ });
});
});
@@ -140,5 +124,12 @@ void defineTests() {
mock.sendException('foo bar (file:///Users/foobar/tmp/error.dart:3:13)');
expect(mock.last['exd'], 'foo bar (');
});
+
+ test('long description trimmed', () {
+ String str = '0123456789abcdefghijklmnopqrstuvwxyz';
+ AnalyticsImplMock mock = createMock();
+ mock.sendException(str + str + str + str + str);
+ expect(mock.last['exd'].length, 100);
+ });
});
}
« no previous file with comments | « packages/usage/test/all.dart ('k') | packages/usage/test/src/common.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698