[Move] Tera Blast Added Ability To Ignore Held Items And Abilities During Stat Calculation

[geeilhan]

What are the changes the user will see?

There are no superficial changes.

What are the changes from a developer perspective?

I added the variable ignoreHeldItems: boolean = false to getEffectiveStat() in order to skip held items during the effective stats calculation.

Also added ability ignore and held items ignore parameter in TeraMoveCategoryAttr so abilities and held items are properly ignored.

Added tests to tera_blast.test.ts to check if my implementation works properly and some more.

Removed .partial()

Screenshots/Videos

No Visual changes are available but I have screenshots of my console (I added some console.logcomments)

Cubone: ATK: 10, SPATK: 11, Held Items: Thick Club (Doubles Cubone's ATK)
Before Implementation:

After Implementation:

(Automated tests cover the same for abilities instead of held items)

How to test the changes?

const overrides = {
  OPP_SPECIES_OVERRIDE: Species.SNORLAX,
  OPP_MOVESET_OVERRIDE: [ Moves.SPLASH ],
  OPP_ABILITY_OVERRIDE: Abilities.ROUGH_SKIN,
  OPP_LEVEL_OVERRIDE: 20,
  STARTER_SPECIES_OVERRIDE: Species.CUBONE,
  MOVESET_OVERRIDE: [ Moves.TERA_BLAST ],
  STARTING_HELD_ITEMS_OVERRIDE: [
    { name: "SPECIES_STAT_BOOSTER", type: "THICK_CLUB" },
    { name: "TERA_SHARD", type: Type.FIRE }
  ],
} satisfies Partial<InstanceType<typeof DefaultOverrides>>;

Checklist

• I'm using beta as my base branch
• There is no overlap with another PR?
• The PR is self-contained and cannot be split into smaller PRs?
• Have I provided a clear explanation of the changes?
• Have I tested the changes manually?
• Are all unit tests still passing? (npm run test)
  • Have I created new automated tests (npm run create-test) or updated existing tests related to the PR's changes?
• Have I provided screenshots/videos of the changes (if applicable)?
  • Have I made sure that any UI change works for both UI themes (default and legacy)?