Compare commits
No commits in common. "297a8454f61bc8358b82ea4a08ea1d4c663a463a" and "eb427043807f37cddc3bf2068b6336959b85fa4f" have entirely different histories.
297a8454f6
...
eb42704380
|
@ -190,12 +190,12 @@ impl DicePoolRoll {
|
|||
}
|
||||
|
||||
pub fn successes(&self) -> i32 {
|
||||
let successes: usize = self
|
||||
let successes = self
|
||||
.rolls
|
||||
.iter()
|
||||
.filter(|&roll| *roll >= self.modifiers.success_on)
|
||||
.cloned()
|
||||
.filter(|&roll| roll >= self.modifiers.success_on)
|
||||
.count();
|
||||
|
||||
i32::try_from(successes).unwrap_or(0)
|
||||
}
|
||||
|
||||
|
@ -689,32 +689,4 @@ mod tests {
|
|||
fmt_rolls(&result)
|
||||
);
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn shows_more_than_10_dice_test() {
|
||||
//Make sure we display more than 10 dice when below the display limit (15).
|
||||
let result = DicePoolRoll {
|
||||
modifiers: DicePoolModifiers::default(),
|
||||
rolls: vec![1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14],
|
||||
};
|
||||
|
||||
assert_eq!(
|
||||
"1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14",
|
||||
fmt_rolls(&result)
|
||||
);
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn shows_exactly_15_dice_test() {
|
||||
//If we are at format limit (15), make sure all are shown
|
||||
let result = DicePoolRoll {
|
||||
modifiers: DicePoolModifiers::default(),
|
||||
rolls: vec![1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15],
|
||||
};
|
||||
|
||||
assert_eq!(
|
||||
"1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15",
|
||||
fmt_rolls(&result)
|
||||
);
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue