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

Unified Diff: tests/lib/math/random_test.dart

Issue 1398453004: Add a 'secure' constructor to Random in dart:math returning a cryptographically (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: address review comments Created 5 years, 2 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/lib/math/random_test.dart
diff --git a/tests/lib/math/random_test.dart b/tests/lib/math/random_test.dart
index f769efb793e691f69c13232324e0ad58d3b55ab2..d969e4b5e0ad33b313f1063f93b99a2589cffde1 100644
--- a/tests/lib/math/random_test.dart
+++ b/tests/lib/math/random_test.dart
@@ -56,6 +56,7 @@ void checkSequence() {
Expect.equals( 226275094, rnd.nextInt(i *= 2));
Expect.equals(1639629168, rnd.nextInt(i *= 2));
Expect.equals(0x100000000, i);
+ Expect.equals( 1552504971, rnd.nextInt(i));
Lasse Reichstein Nielsen 2015/10/15 09:32:33 Why? We already called with that value of max abov
regis 2015/10/15 17:12:02 Aargh. Left over from debugging. Reverted. I was
// If max is too large expect an ArgumentError.
Expect.throws(() => rnd.nextInt(i + 1), (e) => e is ArgumentError);
« tests/lib/math/random_secure_test.dart ('K') | « tests/lib/math/random_secure_test.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698