Skip to content

Commit 0f9a43d

Browse files
committed
libcore: Fix tests.
1 parent f51b6d6 commit 0f9a43d

File tree

18 files changed

+135
-113
lines changed

18 files changed

+135
-113
lines changed

src/libcore/comm.rs

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -596,9 +596,10 @@ mod test {
596596
597597
c1.send(~"abc");
598598

599-
match (p1, p2).select() {
600-
Right(_) => fail!(),
601-
_ => ()
599+
let mut tuple = (p1, p2);
600+
match tuple.select() {
601+
Right(_) => fail!(),
602+
_ => (),
602603
}
603604

604605
c2.send(123);

src/libcore/flate.rs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -84,10 +84,11 @@ pub fn inflate_bytes(bytes: &const [u8]) -> ~[u8] {
8484
#[test]
8585
#[allow(non_implicitly_copyable_typarams)]
8686
fn test_flate_round_trip() {
87-
let r = rand::rng();
87+
let mut r = rand::rng();
8888
let mut words = ~[];
8989
for 20.times {
90-
words.push(r.gen_bytes(r.gen_uint_range(1, 10)));
90+
let range = r.gen_uint_range(1, 10);
91+
words.push(r.gen_bytes(range));
9192
}
9293
for 20.times {
9394
let mut in = ~[];

src/libcore/hash.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -473,10 +473,10 @@ mod tests {
473473
let k1 = 0x_0f_0e_0d_0c_0b_0a_09_08_u64;
474474
let mut buf : ~[u8] = ~[];
475475
let mut t = 0;
476-
let stream_inc = &State(k0,k1);
477-
let stream_full = &State(k0,k1);
476+
let mut stream_inc = SipState::new(k0, k1);
477+
let mut stream_full = SipState::new(k0, k1);
478478

479-
fn to_hex_str(r: &[u8, ..8]) -> ~str {
479+
fn to_hex_str(r: &[u8, ..8]) -> ~str {
480480
let mut s = ~"";
481481
for vec::each(*r) |b| {
482482
s += uint::to_str_radix(*b as uint, 16u);

src/libcore/io.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1886,15 +1886,15 @@ mod tests {
18861886
fn bytes_buffer_overwrite() {
18871887
let wr = BytesWriter();
18881888
wr.write(~[0u8, 1u8, 2u8, 3u8]);
1889-
assert!(wr.bytes == ~[0u8, 1u8, 2u8, 3u8]);
1889+
assert!(*wr.bytes == ~[0u8, 1u8, 2u8, 3u8]);
18901890
wr.seek(-2, SeekCur);
18911891
wr.write(~[4u8, 5u8, 6u8, 7u8]);
1892-
assert!(wr.bytes == ~[0u8, 1u8, 4u8, 5u8, 6u8, 7u8]);
1892+
assert!(*wr.bytes == ~[0u8, 1u8, 4u8, 5u8, 6u8, 7u8]);
18931893
wr.seek(-2, SeekEnd);
18941894
wr.write(~[8u8]);
18951895
wr.seek(1, SeekSet);
18961896
wr.write(~[9u8]);
1897-
assert!(wr.bytes == ~[0u8, 9u8, 4u8, 5u8, 8u8, 7u8]);
1897+
assert!(*wr.bytes == ~[0u8, 9u8, 4u8, 5u8, 8u8, 7u8]);
18981898
}
18991899
19001900
#[test]

src/libcore/os.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1435,7 +1435,7 @@ mod tests {
14351435
}
14361436
14371437
fn make_rand_name() -> ~str {
1438-
let rng = rand::rng();
1438+
let mut rng = rand::rng();
14391439
let n = ~"TEST" + rng.gen_str(10u);
14401440
assert!(getenv(n).is_none());
14411441
n

src/libcore/pipes.rs

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -894,9 +894,10 @@ mod test {
894894
895895
c1.send(~"abc");
896896

897-
match (p1, p2).select() {
898-
Right(_) => fail!(),
899-
_ => ()
897+
let mut tuple = (p1, p2);
898+
match tuple.select() {
899+
Right(_) => fail!(),
900+
_ => (),
900901
}
901902

902903
c2.send(123);

src/libcore/rand.rs

Lines changed: 23 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -879,24 +879,24 @@ mod tests {
879879
#[test]
880880
fn test_rng_seeded() {
881881
let seed = seed();
882-
let ra = IsaacRng::new_seeded(seed);
883-
let rb = IsaacRng::new_seeded(seed);
882+
let mut ra = IsaacRng::new_seeded(seed);
883+
let mut rb = IsaacRng::new_seeded(seed);
884884
assert!(ra.gen_str(100u) == rb.gen_str(100u));
885885
}
886886

887887
#[test]
888888
fn test_rng_seeded_custom_seed() {
889889
// much shorter than generated seeds which are 1024 bytes
890890
let seed = [2u8, 32u8, 4u8, 32u8, 51u8];
891-
let ra = IsaacRng::new_seeded(seed);
892-
let rb = IsaacRng::new_seeded(seed);
891+
let mut ra = IsaacRng::new_seeded(seed);
892+
let mut rb = IsaacRng::new_seeded(seed);
893893
assert!(ra.gen_str(100u) == rb.gen_str(100u));
894894
}
895895

896896
#[test]
897897
fn test_rng_seeded_custom_seed2() {
898898
let seed = [2u8, 32u8, 4u8, 32u8, 51u8];
899-
let ra = IsaacRng::new_seeded(seed);
899+
let mut ra = IsaacRng::new_seeded(seed);
900900
// Regression test that isaac is actually using the above vector
901901
let r = ra.next();
902902
error!("%?", r);
@@ -906,7 +906,7 @@ mod tests {
906906

907907
#[test]
908908
fn test_gen_int_range() {
909-
let r = rng();
909+
let mut r = rng();
910910
let a = r.gen_int_range(-3, 42);
911911
assert!(a >= -3 && a < 42);
912912
assert!(r.gen_int_range(0, 1) == 0);
@@ -917,12 +917,13 @@ mod tests {
917917
#[should_fail]
918918
#[ignore(cfg(windows))]
919919
fn test_gen_int_from_fail() {
920-
rng().gen_int_range(5, -2);
920+
let mut r = rng();
921+
r.gen_int_range(5, -2);
921922
}
922923

923924
#[test]
924925
fn test_gen_uint_range() {
925-
let r = rng();
926+
let mut r = rng();
926927
let a = r.gen_uint_range(3u, 42u);
927928
assert!(a >= 3u && a < 42u);
928929
assert!(r.gen_uint_range(0u, 1u) == 0u);
@@ -933,27 +934,28 @@ mod tests {
933934
#[should_fail]
934935
#[ignore(cfg(windows))]
935936
fn test_gen_uint_range_fail() {
936-
rng().gen_uint_range(5u, 2u);
937+
let mut r = rng();
938+
r.gen_uint_range(5u, 2u);
937939
}
938940

939941
#[test]
940942
fn test_gen_float() {
941-
let r = rng();
943+
let mut r = rng();
942944
let a = r.gen::<float>();
943945
let b = r.gen::<float>();
944946
debug!((a, b));
945947
}
946948

947949
#[test]
948950
fn test_gen_weighted_bool() {
949-
let r = rng();
951+
let mut r = rng();
950952
assert!(r.gen_weighted_bool(0u) == true);
951953
assert!(r.gen_weighted_bool(1u) == true);
952954
}
953955

954956
#[test]
955957
fn test_gen_str() {
956-
let r = rng();
958+
let mut r = rng();
957959
debug!(r.gen_str(10u));
958960
debug!(r.gen_str(10u));
959961
debug!(r.gen_str(10u));
@@ -964,29 +966,29 @@ mod tests {
964966

965967
#[test]
966968
fn test_gen_bytes() {
967-
let r = rng();
969+
let mut r = rng();
968970
assert!(r.gen_bytes(0u).len() == 0u);
969971
assert!(r.gen_bytes(10u).len() == 10u);
970972
assert!(r.gen_bytes(16u).len() == 16u);
971973
}
972974

973975
#[test]
974976
fn test_choose() {
975-
let r = rng();
977+
let mut r = rng();
976978
assert!(r.choose([1, 1, 1]) == 1);
977979
}
978980

979981
#[test]
980982
fn test_choose_option() {
981-
let r = rng();
983+
let mut r = rng();
982984
let x: Option<int> = r.choose_option([]);
983985
assert!(x.is_none());
984986
assert!(r.choose_option([1, 1, 1]) == Some(1));
985987
}
986988

987989
#[test]
988990
fn test_choose_weighted() {
989-
let r = rng();
991+
let mut r = rng();
990992
assert!(r.choose_weighted(~[
991993
Weighted { weight: 1u, item: 42 },
992994
]) == 42);
@@ -998,7 +1000,7 @@ mod tests {
9981000

9991001
#[test]
10001002
fn test_choose_weighted_option() {
1001-
let r = rng();
1003+
let mut r = rng();
10021004
assert!(r.choose_weighted_option(~[
10031005
Weighted { weight: 1u, item: 42 },
10041006
]) == Some(42));
@@ -1012,7 +1014,7 @@ mod tests {
10121014

10131015
#[test]
10141016
fn test_weighted_vec() {
1015-
let r = rng();
1017+
let mut r = rng();
10161018
let empty: ~[int] = ~[];
10171019
assert!(r.weighted_vec(~[]) == empty);
10181020
assert!(r.weighted_vec(~[
@@ -1024,15 +1026,15 @@ mod tests {
10241026

10251027
#[test]
10261028
fn test_shuffle() {
1027-
let r = rng();
1029+
let mut r = rng();
10281030
let empty: ~[int] = ~[];
10291031
assert!(r.shuffle(~[]) == empty);
10301032
assert!(r.shuffle(~[1, 1, 1]) == ~[1, 1, 1]);
10311033
}
10321034

10331035
#[test]
10341036
fn test_task_rng() {
1035-
let r = task_rng();
1037+
let mut r = task_rng();
10361038
r.gen::<int>();
10371039
assert!(r.shuffle(~[1, 1, 1]) == ~[1, 1, 1]);
10381040
assert!(r.gen_uint_range(0u, 1u) == 0u);
@@ -1079,7 +1081,7 @@ mod tests {
10791081
let rt_rng = do vec::as_imm_buf(seed) |p, sz| {
10801082
rustrt::rand_new_seeded(p, sz as size_t)
10811083
};
1082-
let rng = IsaacRng::new_seeded(seed);
1084+
let mut rng = IsaacRng::new_seeded(seed);
10831085

10841086
for 10000.times {
10851087
assert_eq!(rng.next(), rustrt::rand_next(rt_rng));

0 commit comments

Comments
 (0)