[Misc] Move Revival Blessing func from PlayerPokemon to the move's attr

[DayKev]

What are the changes the user will see?

N/A

Why am I making these changes?

A move shouldn't be adding arbitrary methods to the Pokemon classes.

What are the changes from a developer perspective?

Removed revivalBlessing() method from the PlayerPokemon class and moved its functionality to Revival Blessing's move attr.
Removed an unused parameter from RevivalBlessingAttr's constructor.
Updated some tsdocs.
Added tests for Revival Blessing.

How to test the changes?

npm run test revival_blessing

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 make it translatable and add a key in the English locale file(s)?
• [ ] 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?