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

Unified Diff: tests/html/xhr_test.dart

Issue 12050010: Revert "Adding ease-of-use methods to HttpRequest." (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 11 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/html/xhr_test.dart
diff --git a/tests/html/xhr_test.dart b/tests/html/xhr_test.dart
index ef1d94eecec14a8ff251721945c37a85f547e4c3..10df44a6fa83212b901f126764c982e02505514d 100644
--- a/tests/html/xhr_test.dart
+++ b/tests/html/xhr_test.dart
@@ -8,12 +8,6 @@ import '../../pkg/unittest/lib/html_individual_config.dart';
import 'dart:html';
import 'dart:json' as json;
-void fail(message) {
- guardAsync(() {
- expect(false, isTrue, reason: message);
- });
-}
-
main() {
useHtmlIndividualConfiguration();
var url = "/tests/html/xhr_cross_origin_data.txt";
@@ -65,77 +59,34 @@ main() {
xhr.send();
});
- test('XHR.request No file', () {
- HttpRequest.request('NonExistingFile').then(
- (_) { fail('Request should not have succeeded.'); },
- onError: expectAsync1((e) {
- var xhr = e.error.target;
- expect(xhr.readyState, equals(HttpRequest.DONE));
- validate404(xhr);
- }));
- });
-
- test('XHR.request file', () {
- HttpRequest.request(url).then(expectAsync1((xhr) {
+ test('XHR.get No file', () {
+ new HttpRequest.get("NonExistingFile", expectAsync1((xhr) {
expect(xhr.readyState, equals(HttpRequest.DONE));
- validate200Response(xhr);
+ validate404(xhr);
}));
});
- test('XHR.request onProgress', () {
- var progressCalled = false;
- HttpRequest.request(url,
- onProgress: (_) {
- progressCalled = true;
- }).then(expectAsync1(
- (xhr) {
- expect(xhr.readyState, equals(HttpRequest.DONE));
- expect(progressCalled, isTrue);
- validate200Response(xhr);
- }));
+ test('XHR.get file', () {
+ var xhr = new HttpRequest.get(url, expectAsync1((event) {
+ expect(event.readyState, equals(HttpRequest.DONE));
+ validate200Response(event);
+ }));
});
- test('XHR.request withCredentials No file', () {
- HttpRequest.request('NonExistingFile', withCredentials: true).then(
- (_) { fail('Request should not have succeeded.'); },
- onError: expectAsync1((e) {
- var xhr = e.error.target;
- expect(xhr.readyState, equals(HttpRequest.DONE));
- validate404(xhr);
- }));
+ test('XHR.getWithCredentials No file', () {
+ new HttpRequest.getWithCredentials("NonExistingFile", expectAsync1((xhr) {
+ expect(xhr.readyState, equals(HttpRequest.DONE));
+ validate404(xhr);
+ }));
});
- test('XHR.request withCredentials file', () {
- HttpRequest.request(url, withCredentials: true).then(expectAsync1((xhr) {
+ test('XHR.getWithCredentials file', () {
+ new HttpRequest.getWithCredentials(url, expectAsync1((xhr) {
expect(xhr.readyState, equals(HttpRequest.DONE));
validate200Response(xhr);
}));
});
- test('XHR.getString file', () {
- HttpRequest.getString(url).then(expectAsync1((str) {}));
- });
-
- test('XHR.getString No file', () {
- HttpRequest.getString('NonExistingFile').then(
- (_) { fail('Succeeded for non-existing file.'); },
- onError: expectAsync1((e) {
- validate404(e.error.target);
- }));
- });
-
- test('XHR.request responseType', () {
- if (ArrayBuffer.supported) {
- HttpRequest.request(url, responseType: 'ArrayBuffer').then(
- expectAsync1((xhr) {
- validate200Response(xhr);
- var arrayBuffer = xhr.response;
- expect(arrayBuffer, new isInstanceOf<ArrayBuffer>());
- expect(arrayBuffer, isNotNull);
- }));
- }
- });
-
test('HttpRequestProgressEvent', () {
var expectation = HttpRequestProgressEvent.supported ?
returnsNormally : throws;
« no previous file with comments | « sdk/lib/html/dartium/html_dartium.dart ('k') | tools/dom/templates/html/impl/impl_XMLHttpRequest.darttemplate » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698