Skip to content

Commit bb7238f

Browse files
n8shn8sh
committed
Issue 18715 - Non-documented unittests should not use unpredictableSeed or default Random alias
1 parent 861e65a commit bb7238f

File tree

5 files changed

+37
-37
lines changed

5 files changed

+37
-37
lines changed

‎std/algorithm/internal.d

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,13 +21,13 @@ version(unittest)
2121

2222
package string[] rndstuff(T : string)()
2323
{
24-
import std.random : Random, unpredictableSeed, uniform;
24+
import std.random : Random = Xorshift, uniform;
2525

2626
static Random rnd;
2727
static bool first = true;
2828
if (first)
2929
{
30-
rnd = Random(unpredictableSeed);
30+
rnd.seed(234_567_891);
3131
first = false;
3232
}
3333
string[] result =
@@ -46,13 +46,13 @@ version(unittest)
4646

4747
package int[] rndstuff(T : int)()
4848
{
49-
import std.random : Random, unpredictableSeed, uniform;
49+
import std.random : Random = Xorshift, uniform;
5050

5151
static Random rnd;
5252
static bool first = true;
5353
if (first)
5454
{
55-
rnd = Random(unpredictableSeed);
55+
rnd = Random(345_678_912);
5656
first = false;
5757
}
5858
int[] result = new int[uniform(minArraySize, maxArraySize, rnd)];

‎std/algorithm/iteration.d

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -691,7 +691,7 @@ private struct MapResult(alias fun, Range)
691691
import std.internal.test.dummyrange;
692692
import std.range;
693693
import std.typecons : tuple;
694-
import std.random : unpredictableSeed, uniform, Random;
694+
import std.random : uniform, Random = Xorshift;
695695

696696
int[] arr1 = [ 1, 2, 3, 4 ];
697697
const int[] arr1Const = arr1;
@@ -749,7 +749,7 @@ private struct MapResult(alias fun, Range)
749749
assert(fibsSquares.front == 9);
750750

751751
auto repeatMap = map!"a"(repeat(1));
752-
auto gen = Random(unpredictableSeed);
752+
auto gen = Random(123_456_789);
753753
auto index = uniform(0, 1024, gen);
754754
static assert(isInfinite!(typeof(repeatMap)));
755755
assert(repeatMap[index] == 1);

‎std/algorithm/mutation.d

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -278,11 +278,11 @@ if (isInputRange!InputRange && isForwardRange!ForwardRange)
278278
{
279279
import std.algorithm.comparison : equal;
280280
import std.conv : text;
281-
import std.random : Random, unpredictableSeed, uniform;
281+
import std.random : Random = Xorshift, uniform;
282282

283283
// a more elaborate test
284284
{
285-
auto rnd = Random(unpredictableSeed);
285+
auto rnd = Random(123_456_789);
286286
int[] a = new int[uniform(100, 200, rnd)];
287287
int[] b = new int[uniform(100, 200, rnd)];
288288
foreach (ref e; a) e = uniform(-100, 100, rnd);

‎std/algorithm/sorting.d

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -746,8 +746,8 @@ if (isRandomAccessRange!Range && hasLength!Range && hasSlicing!Range && hasAssig
746746
import std.algorithm.iteration : map;
747747
import std.random;
748748
import std.stdio;
749-
auto s = unpredictableSeed;
750-
auto g = Random(s);
749+
auto s = 123_456_789;
750+
auto g = Xorshift(s);
751751
a = iota(0, uniform(1, 1000, g))
752752
.map!(_ => uniform(-1000, 1000, g))
753753
.array;
@@ -883,9 +883,9 @@ if (ss == SwapStrategy.unstable && isRandomAccessRange!Range
883883

884884
@safe unittest
885885
{
886-
import std.random : Random, uniform, unpredictableSeed;
886+
import std.random : Random = Xorshift, uniform;
887887

888-
immutable uint[] seeds = [3923355730, 1927035882, unpredictableSeed];
888+
immutable uint[] seeds = [3923355730, 1927035882];
889889
foreach (s; seeds)
890890
{
891891
auto r = Random(s);
@@ -1710,7 +1710,7 @@ private void shortSort(alias less, Range)(Range r)
17101710

17111711
@safe unittest
17121712
{
1713-
import std.random : Random, uniform;
1713+
import std.random : Random = Xorshift, uniform;
17141714

17151715
auto rnd = Random(1);
17161716
auto a = new int[uniform(100, 200, rnd)];
@@ -1949,12 +1949,12 @@ if (((ss == SwapStrategy.unstable && (hasSwappableElements!Range ||
19491949
{
19501950
import std.algorithm.internal : rndstuff;
19511951
import std.algorithm.mutation : swapRanges;
1952-
import std.random : Random, unpredictableSeed, uniform;
1952+
import std.random : Random = Xorshift, uniform;
19531953
import std.uni : toUpper;
19541954

19551955
// sort using delegate
19561956
auto a = new int[100];
1957-
auto rnd = Random(unpredictableSeed);
1957+
auto rnd = Random(123_456_789);
19581958
foreach (ref e; a)
19591959
{
19601960
e = uniform(-100, 100, rnd);
@@ -3457,7 +3457,7 @@ private T[] randomArray(Flag!"exactSize" flag = No.exactSize, T = int)(
34573457
T minValue = 0, T maxValue = 255)
34583458
{
34593459
import std.algorithm.iteration : map;
3460-
import std.random : unpredictableSeed, Random, uniform;
3460+
import std.random : uniform;
34613461
auto size = flag == Yes.exactSize ? maxSize : uniform(1, maxSize);
34623462
return iota(0, size).map!(_ => uniform(minValue, maxValue)).array;
34633463
}
@@ -3513,9 +3513,9 @@ private T[] randomArray(Flag!"exactSize" flag = No.exactSize, T = int)(
35133513
{
35143514
import std.algorithm.comparison : max, min;
35153515
import std.algorithm.iteration : reduce;
3516-
import std.random : Random, uniform, unpredictableSeed;
3516+
import std.random : Random = Xorshift, uniform;
35173517

3518-
immutable uint[] seeds = [90027751, 2709791795, 1374631933, 995751648, 3541495258, 984840953, unpredictableSeed];
3518+
immutable uint[] seeds = [90027751, 2709791795, 1374631933, 995751648, 3541495258, 984840953];
35193519
foreach (s; seeds)
35203520
{
35213521
auto r = Random(s);
@@ -3718,10 +3718,10 @@ if (isInputRange!(SRange) && isRandomAccessRange!(TRange)
37183718

37193719
@system unittest
37203720
{
3721-
import std.random : Random, unpredictableSeed, uniform, randomShuffle;
3721+
import std.random : Random = Xorshift, uniform, randomShuffle;
37223722
import std.typecons : Yes;
37233723

3724-
auto r = Random(unpredictableSeed);
3724+
auto r = Random(123_456_789);
37253725
ptrdiff_t[] a = new ptrdiff_t[uniform(1, 1000, r)];
37263726
foreach (i, ref e; a) e = i;
37273727
randomShuffle(a, r);

‎std/random.d

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -583,7 +583,7 @@ alias MinstdRand = LinearCongruentialEngine!(uint, 48_271, 0, 2_147_483_647);
583583
// Check .save works
584584
static foreach (Type; std.meta.AliasSeq!(MinstdRand0, MinstdRand))
585585
{{
586-
auto rnd1 = Type(unpredictableSeed);
586+
auto rnd1 = Type(123_456_789); rnd1.popFront();
587587
auto rnd2 = rnd1.save;
588588
assert(rnd1 == rnd2);
589589
// Enable next test when RNGs are reference types
@@ -991,11 +991,11 @@ alias Mt19937_64 = MersenneTwisterEngine!(ulong, 64, 312, 156, 31,
991991

992992
Mt19937 gen;
993993

994-
assertThrown(gen.seed(map!((a) => unpredictableSeed)(repeat(0, 623))));
994+
assertThrown(gen.seed(map!((a) => 123_456_789U)(repeat(0, 623))));
995995

996-
gen.seed(map!((a) => unpredictableSeed)(repeat(0, 624)));
996+
gen.seed(map!((a) => 123_456_789U)(repeat(0, 624)));
997997
//infinite Range
998-
gen.seed(map!((a) => unpredictableSeed)(repeat(0)));
998+
gen.seed(map!((a) => 123_456_789U)(repeat(0)));
999999
}
10001000

10011001
@safe pure nothrow unittest
@@ -1020,7 +1020,7 @@ alias Mt19937_64 = MersenneTwisterEngine!(ulong, 64, 312, 156, 31,
10201020
// Check .save works
10211021
static foreach (Type; std.meta.AliasSeq!(Mt19937, Mt19937_64))
10221022
{{
1023-
auto gen1 = Type(unpredictableSeed);
1023+
auto gen1 = Type(123_456_789); gen1.popFront();
10241024
auto gen2 = gen1.save;
10251025
assert(gen1 == gen2); // Danger, Will Robinson -- no opEquals for MT
10261026
// Enable next test when RNGs are reference types
@@ -1323,7 +1323,7 @@ alias Xorshift = Xorshift128; /// ditto
13231323
// Check .save works
13241324
foreach (Type; XorshiftTypes)
13251325
{
1326-
auto rnd1 = Type(unpredictableSeed);
1326+
auto rnd1 = Type(123_456_789); rnd1.popFront();
13271327
auto rnd2 = rnd1.save;
13281328
assert(rnd1 == rnd2);
13291329
// Enable next test when RNGs are reference types
@@ -1343,7 +1343,7 @@ alias Xorshift = Xorshift128; /// ditto
13431343
foreach (Rng; PseudoRngTypes)
13441344
{
13451345
static assert(isUniformRNG!Rng);
1346-
auto rng = Rng(unpredictableSeed);
1346+
auto rng = Rng(123_456_789);
13471347
}
13481348
}
13491349

@@ -1720,7 +1720,7 @@ if ((isIntegral!(CommonType!(T1, T2)) || isSomeChar!(CommonType!(T1, T2))) &&
17201720
@safe unittest
17211721
{
17221722
import std.conv : to;
1723-
auto gen = Mt19937(unpredictableSeed);
1723+
auto gen = Mt19937(123_456_789);
17241724
static assert(isForwardRange!(typeof(gen)));
17251725

17261726
auto a = uniform(0, 1024, gen);
@@ -2056,7 +2056,7 @@ do
20562056

20572057
static foreach (T; std.meta.AliasSeq!(float, double, real))
20582058
{{
2059-
UniformRNG rng = UniformRNG(unpredictableSeed);
2059+
UniformRNG rng = UniformRNG(123_456_789);
20602060

20612061
auto a = uniform01();
20622062
assert(is(typeof(a) == double));
@@ -2240,7 +2240,7 @@ if (isRandomAccessRange!Range)
22402240
// Also tests partialShuffle indirectly.
22412241
auto a = [0, 1, 2, 3, 4, 5, 6, 7, 8, 9];
22422242
auto b = a.dup;
2243-
auto gen = RandomGen(unpredictableSeed);
2243+
auto gen = RandomGen(123_456_789);
22442244
randomShuffle(a, gen);
22452245
sort(a);
22462246
assert(a == b);
@@ -2421,7 +2421,7 @@ do
24212421

24222422
@safe unittest
24232423
{
2424-
auto rnd = Random(unpredictableSeed);
2424+
auto rnd = Xorshift(123_456_789);
24252425
auto i = dice(rnd, 0.0, 100.0);
24262426
assert(i == 1);
24272427
i = dice(rnd, 100.0, 0.0);
@@ -2628,11 +2628,11 @@ if (isRandomAccessRange!Range)
26282628
}
26292629
else
26302630
{
2631-
auto rng = UniformRNG(unpredictableSeed);
2631+
auto rng = UniformRNG(123_456_789);
26322632
auto rc = randomCover(a, rng);
26332633
static assert(isForwardRange!(typeof(rc)));
26342634
// check for constructor passed a value-type RNG
2635-
auto rc2 = RandomCover!(int[], UniformRNG)(a, UniformRNG(unpredictableSeed));
2635+
auto rc2 = RandomCover!(int[], UniformRNG)(a, UniformRNG(987_654_321));
26362636
static assert(isForwardRange!(typeof(rc2)));
26372637
auto rcEmpty = randomCover(c, rng);
26382638
assert(rcEmpty.length == 0);
@@ -3199,7 +3199,7 @@ if (isInputRange!Range && hasLength!Range && isUniformRNG!UniformRNG)
31993199
{
32003200
auto sample =
32013201
RandomSample!(TestInputRange, UniformRNG)
3202-
(TestInputRange(), 5, 10, UniformRNG(unpredictableSeed));
3202+
(TestInputRange(), 5, 10, UniformRNG(987_654_321));
32033203
static assert(isInputRange!(typeof(sample)));
32043204
static assert(!isForwardRange!(typeof(sample)));
32053205
}
@@ -3215,7 +3215,7 @@ if (isInputRange!Range && hasLength!Range && isUniformRNG!UniformRNG)
32153215
{
32163216
auto sample =
32173217
RandomSample!(typeof(TestInputRange().takeExactly(10)), UniformRNG)
3218-
(TestInputRange().takeExactly(10), 5, 10, UniformRNG(unpredictableSeed));
3218+
(TestInputRange().takeExactly(10), 5, 10, UniformRNG(654_321_987));
32193219
static assert(isInputRange!(typeof(sample)));
32203220
static assert(!isForwardRange!(typeof(sample)));
32213221
}
@@ -3229,7 +3229,7 @@ if (isInputRange!Range && hasLength!Range && isUniformRNG!UniformRNG)
32293229
{
32303230
auto sample =
32313231
RandomSample!(int[], UniformRNG)
3232-
(a, 5, UniformRNG(unpredictableSeed));
3232+
(a, 5, UniformRNG(321_987_654));
32333233
static assert(isForwardRange!(typeof(sample)));
32343234
}
32353235
}
@@ -3241,7 +3241,7 @@ if (isInputRange!Range && hasLength!Range && isUniformRNG!UniformRNG)
32413241
{
32423242
auto sample =
32433243
RandomSample!(int[], UniformRNG)
3244-
(a, 5, UniformRNG(unpredictableSeed));
3244+
(a, 5, UniformRNG(789_123_456));
32453245
static assert(isInputRange!(typeof(sample)));
32463246
static assert(!isForwardRange!(typeof(sample)));
32473247
}

0 commit comments

Comments
 (0)