[Test] Move selectTarget from gameManager to moveHelper

[torranx]

What are the changes the user will see?

none

Why am I making these changes?

selectTarget is outdated and is only being used by moxie test

What are the changes from a developer perspective?

see PR title, added private tag to the method to prevent someone from using it
added public tag to some methods

Screenshots/Videos

How to test the changes?

npm run test

Checklist

• ⚠️ If this is a PR for main (such as a hotfix), has the game version been updated (npm run update-version:patch / npm run update version:minor?
• Otherwise: 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:silent)
  • Have I created new automated tests (npm run test:create) 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 (dark and light)?

Are there any localization additions or changes? If so:

• Has a locales PR been created on the locales repo?
  • If so, please leave a link to it here:
• Have I added the Localization tag to this PR?

If there are no locale changes:

• Have I made sure not to commit any changes to the locale repo on this branch?