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 {
|
pub fn successes(&self) -> i32 {
|
||||||
let successes: usize = self
|
let successes = self
|
||||||
.rolls
|
.rolls
|
||||||
.iter()
|
.iter()
|
||||||
.filter(|&roll| *roll >= self.modifiers.success_on)
|
.cloned()
|
||||||
|
.filter(|&roll| roll >= self.modifiers.success_on)
|
||||||
.count();
|
.count();
|
||||||
|
|
||||||
i32::try_from(successes).unwrap_or(0)
|
i32::try_from(successes).unwrap_or(0)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -689,32 +689,4 @@ mod tests {
|
||||||
fmt_rolls(&result)
|
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