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

Unified Diff: packages/usage/test/web_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/uuid_test.dart ('k') | packages/usage/tool/drone.sh » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: packages/usage/test/web_test.dart
diff --git a/packages/usage/test/web_test.dart b/packages/usage/test/web_test.dart
index 8d91ae9980ec1be13e2af5fe051d597970482555..c2aaa9a5f7b6be55c660bdd64a79954d8e781063 100644
--- a/packages/usage/test/web_test.dart
+++ b/packages/usage/test/web_test.dart
@@ -6,7 +6,6 @@
library usage.web_test;
import 'dart:async';
-import 'dart:html';
import 'package:test/test.dart';
import 'package:usage/src/usage_impl_html.dart';
@@ -23,34 +22,32 @@ void main() {
usage_impl_test.defineTests();
uuid_test.defineTests();
- // Define some web specific tests.
+ // Define some web specfic tests.
defineWebTests();
}
void defineWebTests() {
group('HtmlPostHandler', () {
- test('sendPost', () async {
+ test('sendPost', () {
MockRequestor client = new MockRequestor();
- HtmlPostHandler postHandler =
- new HtmlPostHandler(mockRequestor: client.request);
+ HtmlPostHandler postHandler = new HtmlPostHandler(
+ mockRequestor: client.request);
Map<String, dynamic> args = {'utv': 'varName', 'utt': 123};
-
- await postHandler.sendPost('http://www.google.com', args);
- expect(client.sendCount, 1);
+ return postHandler.sendPost('http://www.google.com', args).then((_) {
+ expect(client.sendCount, 1);
+ });
});
});
group('HtmlPersistentProperties', () {
test('add', () {
- HtmlPersistentProperties props =
- new HtmlPersistentProperties('foo_props');
+ HtmlPersistentProperties props = new HtmlPersistentProperties('foo_props');
props['foo'] = 'bar';
expect(props['foo'], 'bar');
});
test('remove', () {
- HtmlPersistentProperties props =
- new HtmlPersistentProperties('foo_props');
+ HtmlPersistentProperties props = new HtmlPersistentProperties('foo_props');
props['foo'] = 'bar';
expect(props['foo'], 'bar');
props['foo'] = null;
@@ -62,7 +59,7 @@ void defineWebTests() {
class MockRequestor {
int sendCount = 0;
- Future<HttpRequest> request(String url, {String method, sendData}) {
+ Future request(String url, {String method, String sendData}) {
expect(url, isNotEmpty);
expect(method, isNotEmpty);
expect(sendData, isNotEmpty);
« no previous file with comments | « packages/usage/test/uuid_test.dart ('k') | packages/usage/tool/drone.sh » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698