6 Commits

Author SHA1 Message Date
Paliak
d320c2e27b FIX: broken tests after changing default boss preset (#8327)
Due to #8271 changing the default boss preset many tests that relied on
enemy stats broke. This pr fixes that by changing the boss configuration
back to "None" for affected tests.

Additionally this commit adds an extra test for scorching ray exposure
when using the automatic maximum sustainable stages mode.
2024-11-18 16:56:01 +11:00
Nighty
684bd3a6d5 indent space -> tab (#8058) 2024-07-31 14:07:24 +10:00
Paliak
e9dd799221 Fix maximum sustainable stages calculations ignoring certain buffs/mods (#7678)
* FIX: maximum sustainable stacks calculations ignoring certain buffs/mods

Essentially an improved version of
https://github.com/PathOfBuildingCommunity/PathOfBuilding/pull/5164 that
utilizes skill cache functionality.

* FIX: spelling

* TEST: add tests for schorching ray and blight interactions

* FIX: formatting

* DOCS: add fancy annotation for new function

* FIX: calculate the correct skill

* FIX: annotation

* FIX: use activation frequency for penance brand
2024-07-22 06:15:37 +10:00
Paliak
41ba38d925 Implement wisps support (#7532)
* FEAT: impl wisps support

* FIX: add display stat for summon wisps

* FIX: less damage mod not applying

* FIX: use mirage archer style calcs

Sacred wisps causes the Summon Sacred Wisps skill to be triggered when a
projectile spell is fired from a wand. This triggers 2 wisps which then
have an x percent to use the skill that was used to trigger them when
you use it. They likely do not care about whether you actually hit
something with the projectile or not. They're also likely not a trigger
which means they're not under the same tick rounding contrstains like
other triggers.

* TEST: add a test for Sacred Wisps

* TEST: fix test for sacred wisps

Sacred wisps support is two part so the SkillTriggerRate check will fail

* FIX: add stat maps to .txt skill files

* Separate stat display in sidebar

* Change flag to mirageArcher

---------

Co-authored-by: LocalIdentity <localidentity2@gmail.com>
2024-06-28 02:56:12 +10:00
Paliak
f7f55c96ea Move mirage calcs to a new file (#6637)
* FIX: move mirage calcs to a new file

* FIX: exerts applying to skills used by mirages

* FIX:ignore source if SkillType.OtherThingUsesSkill

* TEST: test mirage archer + manaforged
2023-09-11 09:45:00 +10:00
Wires77
8d4f46a570 test: Adding unit test 2022-01-27 12:29:41 -06:00