[Bug] Moves copied by Dancer should not consume PP

[DayKev]

What are the changes the user will see?

Dancer will no longer use PP when triggered.

Why am I making these changes?

Dancer is not supposed to consume PP.

What are the changes from a developer perspective?

The ignorePP parameter is passed as true to the MovePhase created by the Dancer ability, and the ignorePP parameter is made to actually function in MovePhase:start().domove().

How to test the changes?

Override the battle type to doubles, have a team of Oricorio + another Pokémon and add Swords Dance via the moves override. Use Swords Dance with the non-Oricorio Pokémon and have Oricorio use a different move. Example:

const overrides = {
  MOVESET_OVERRIDE: [Moves.SPLASH, Moves.SWORDS_DANCE],
  OPP_SPECIES_OVERRIDE: Species.MAGIKARP,
  OPP_MOVESET_OVERRIDE: Array(4).fill(Moves.SPLASH),
  BATTLE_TYPE_OVERRIDE: "double"
} satisfies Partial<InstanceType<typeof DefaultOverrides>>;

Screenshots/Videos

Before fix:

dancer.bug.mp4

After fix:

dancer.fix.mp4

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 considered writing automated tests for the issue?
• [ ] If I have text, did I add placeholders for them in locales?
• Have I tested the changes (manually)?
  • Are all unit tests still passing? (npm run test)
• [ ] Are the changes visual?
  • Have I provided screenshots/videos of the changes?

[Snailman11]

Link to Discord's bug report https://discord.com/channels/1125469663833370665/1274672573556330526 (Dancer Ability fails if the move it copied is at 0pp)

[frutescens]

Is there a unit test for Dancer?

[frutescens]

HI! could you resolve the merge conflicts?