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

Unified Diff: tests/html/crypto_test.dart

Issue 12326071: Adding supported checks and annotations to Crypto API. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 10 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/crypto_test.dart
diff --git a/tests/html/crypto_test.dart b/tests/html/crypto_test.dart
index 0e0672355af2f7cf3771e85d8c47f7886027b255..57a293024acd83152e0028e8c694e13a87f14745 100644
--- a/tests/html/crypto_test.dart
+++ b/tests/html/crypto_test.dart
@@ -4,33 +4,45 @@
library crypto_test;
import '../../pkg/unittest/lib/unittest.dart';
-import '../../pkg/unittest/lib/html_config.dart';
+import '../../pkg/unittest/lib/html_individual_config.dart';
import 'dart:html';
main() {
- useHtmlConfiguration();
+ useHtmlIndividualConfiguration();
- test('exists', () {
- var crypto = window.crypto;
- expect(crypto is Crypto, isTrue);
+ group('supported', () {
+ test('supported', () {
+ expect(Crypto.supported, true);
+ });
});
- test('successful call', () {
- var crypto = window.crypto;
- var data = new Uint8Array(100);
- expect(data.every((e) => e == 0), isTrue);
- crypto.getRandomValues(data);
- // In theory this is flaky. However, in practice you will get 100 zeroes
- // in a row from a cryptographically secure random number generator so
- // rarely that we don't have to worry about it.
- expect(data.any((e) => e != 0), isTrue);
- });
+ group('functional', () {
+ if (Crypto.supported) {
+ // This will actually pass on FF since it has a Crypto API, but it is
+ // incompatible.
+ test('exists', () {
+ var crypto = window.crypto;
+ expect(crypto is Crypto, isTrue);
+ });
+
+ test('successful call', () {
+ var crypto = window.crypto;
+ var data = new Uint8Array(100);
+ expect(data.every((e) => e == 0), isTrue);
+ crypto.getRandomValues(data);
+ // In theory this is flaky. However, in practice you will get 100 zeroes
+ // in a row from a cryptographically secure random number generator so
+ // rarely that we don't have to worry about it.
+ expect(data.any((e) => e != 0), isTrue);
+ });
- test('type mismatch', () {
- var crypto = window.crypto;
- var data = new Float32Array(100);
- expect(() {
- crypto.getRandomValues(data);
- }, throws, reason: 'Only typed array views with integer types allowed');
+ test('type mismatch', () {
+ var crypto = window.crypto;
+ var data = new Float32Array(100);
+ expect(() {
+ crypto.getRandomValues(data);
+ }, throws, reason: 'Only typed array views with integer types allowed');
+ });
+ }
});
}

Powered by Google App Engine
This is Rietveld 408576698