Skip to content

Commit 04eef87

Browse files
committed
Replace panic with unreachable in tests when it can't be related to the tested function
1 parent a390d58 commit 04eef87

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

src/map.rs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3915,28 +3915,28 @@ mod test_map {
39153915
a.insert(key, value);
39163916

39173917
match a.entry(key) {
3918-
Vacant(_) => panic!(),
39193918
Occupied(e) => {
39203919
e.replace_entry_with(|k, v| {
39213920
assert_eq!(k, &key);
39223921
assert_eq!(v, value);
39233922
Some(new_value)
39243923
});
39253924
}
3925+
Vacant(_) => unreachable!(),
39263926
}
39273927

39283928
assert_eq!(a[key], new_value);
39293929
assert_eq!(a.len(), 1);
39303930

39313931
match a.entry(key) {
3932-
Vacant(_) => panic!(),
39333932
Occupied(e) => {
39343933
e.replace_entry_with(|k, v| {
39353934
assert_eq!(k, &key);
39363935
assert_eq!(v, new_value);
39373936
None
39383937
});
39393938
}
3939+
Vacant(_) => panic!(),
39403940
}
39413941

39423942
assert!(!a.contains_key(key));
@@ -3955,7 +3955,7 @@ mod test_map {
39553955
e @ Vacant(_) => {
39563956
e.and_replace_entry_with(|_, _| panic!("Can't replace a VacantEntry"));
39573957
}
3958-
Occupied(_) => panic!(),
3958+
Occupied(_) => unreachable!(),
39593959
}
39603960

39613961
a.insert(key, value);
@@ -3990,28 +3990,28 @@ mod test_map {
39903990
a.insert(key, value);
39913991

39923992
match a.raw_entry_mut().from_key(&key) {
3993-
RawEntryMut::Vacant(_) => panic!(),
39943993
RawEntryMut::Occupied(e) => {
39953994
e.replace_entry_with(|k, v| {
39963995
assert_eq!(k, &key);
39973996
assert_eq!(v, value);
39983997
Some(new_value)
39993998
});
40003999
}
4000+
RawEntryMut::Vacant(_) => unreachable!(),
40014001
}
40024002

40034003
assert_eq!(a[key], new_value);
40044004
assert_eq!(a.len(), 1);
40054005

40064006
match a.raw_entry_mut().from_key(&key) {
4007-
RawEntryMut::Vacant(_) => panic!(),
40084007
RawEntryMut::Occupied(e) => {
40094008
e.replace_entry_with(|k, v| {
40104009
assert_eq!(k, &key);
40114010
assert_eq!(v, new_value);
40124011
None
40134012
});
40144013
}
4014+
RawEntryMut::Vacant(_) => panic!(),
40154015
}
40164016

40174017
assert!(!a.contains_key(key));
@@ -4030,7 +4030,7 @@ mod test_map {
40304030
e @ RawEntryMut::Vacant(_) => {
40314031
e.and_replace_entry_with(|_, _| panic!("Can't replace a VacantEntry"));
40324032
}
4033-
RawEntryMut::Occupied(_) => panic!(),
4033+
RawEntryMut::Occupied(_) => unreachable!(),
40344034
}
40354035

40364036
a.insert(key, value);

0 commit comments

Comments
 (0)