dungeonslayers/ds4
dungeonslayers/ds4
3
0
Fork 0
Code Issues 9 Pull requests 6 Packages Projects Releases 10 Activity

Use coups, even if they are higher than the corresponding CTN

Browse source
This commit is contained in:
Johannes Loher 2021-03-13 20:58:59 +01:00
parent e1d376057c
commit eeb1aa61f4
2 changed files with 10 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
spec/rolls/check-evaluation.spec.ts
View file

@ -258,4 +258,12 @@ describe("evaluateCheck with multiple dice and coup / fumble modification", () =
{ result: 15, checkTargetNumber: 20, active: true, discarded: false },
]);
});
it("should use all the dice if they are coups, even if they are higher than the checkTargetNumber", () => {
expect(evaluateCheck([18, 19, 17], 48, { maximumCoupResult: 19 })).toEqual([
{ result: 18, checkTargetNumber: 8, active: true, discarded: false, success: true, count: 8 },
{ result: 19, checkTargetNumber: 20, active: true, discarded: false, success: true, count: 20 },
{ result: 17, checkTargetNumber: 20, active: true, discarded: false, success: true, count: 20 },
]);
});
});

2
src/module/rolls/check-evaluation.ts
View file

@ -104,6 +104,8 @@ function evaluateDiceWithSubChecks(
if (result.result <= maximumCoupResult) {
result.success = true;
result.count = result.checkTargetNumber;
result.active = true;
result.discarded = false;
}
if (index === 0 && canFumble && result.result >= minimumFumbleResult) result.failure = true;
return result;