facebook/sapling
1
1
Fork 0
mirror of https://github.com/facebook/sapling.git synced 2024-10-08 15:57:43 +03:00
Code Issues Projects Releases Wiki Activity

tests: remove cargo warnings

Browse Source 
Summary: Fix up some simple cargo test warnings

Reviewed By: quark-zju

Differential Revision: D9136979

fbshipit-source-id: dd10ea6751eb68190190381dc69b3494160cf358
This commit is contained in:
Durham Goode 2018-08-15 15:05:19 -07:00 committed by Facebook Github Bot
parent 99d2313c59
commit 01b81f1217
2 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/revisionstore/src/datapack.rs
View File

@ -461,7 +461,7 @@ mod tests {
),
];
let pack = make_pack(&tempdir, &revisions);
for &(ref delta, ref metadata) in revisions.iter() {
for &(ref delta, ref _metadata) in revisions.iter() {
let missing = pack.get_missing(&[delta.key.clone()]).unwrap();
assert_eq!(missing.len(), 0);
}
@ -534,7 +534,7 @@ mod tests {
];
let pack = make_pack(&tempdir, &revisions);
for &(ref delta, ref metadata) in revisions.iter() {
for &(ref delta, ref _metadata) in revisions.iter() {
let chain = pack.get_delta_chain(&delta.key).unwrap();
assert_eq!(chain[0], *delta);
}

3
lib/revisionstore/src/node.rs
View File

@ -46,13 +46,12 @@ impl Node {
pub fn random(rng: &mut RngCore) -> Self {
let mut bytes = [0; 20];
rng.fill_bytes(&mut bytes);
while true {
loop {
let node = Node::from(&bytes);
if !node.is_null() {
return node;
}
}
panic!("should never get here")
}
}