Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wild battle tests + tests for exp points #3342

Merged
merged 6 commits into from
Sep 27, 2023

Conversation

DizzyEggg
Copy link
Collaborator

  • Adds support for wild battle tests. Because the test runner uses recorded battles, they are now changed to support wild battles. So, in theory, this would also let the player record wild battles in-game. Not tested, but it should work 🤔
  • Adds new queue event EXPERIENCE_BAR which works similarly to HP_BAR
  • Adds new Pokemon function OtName which changes original trainer's name. Its use is to generate a traded pokemon.
  • Adds new tests for experience points

What next? I would say that'd be wild double battles, trainer battles and ai vs ai trainer battles. Though it's probably gonna take some more work, so I'd rather put it outside the scope of this PR.

@@ -153,6 +153,7 @@ BattleScript_PrintCaughtMonInfo::
getexp BS_TARGET
sethword gBattle_BG2_X, 0
BattleScript_TryPrintCaughtMonInfo:
jumpifbattletype BATTLE_TYPE_RECORDED, BattleScript_GiveCaughtMonEnd
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tabs to spaces.

include/test/battle.h Outdated Show resolved Hide resolved
include/test/battle.h Outdated Show resolved Hide resolved
include/test/battle.h Show resolved Hide resolved
test/test_runner_battle.c Outdated Show resolved Hide resolved
test/test_runner_battle.c Outdated Show resolved Hide resolved
test/test_runner_battle.c Outdated Show resolved Hide resolved
test/test_runner_battle.c Outdated Show resolved Hide resolved
include/test/battle.h Show resolved Hide resolved
@mrgriffin
Copy link
Collaborator

Forgot to thank you in my review, but this is awesome work! 😅

@DizzyEggg
Copy link
Collaborator Author

Should be ready for review again

@mrgriffin mrgriffin merged commit 2fcb9bb into rh-hideout:upcoming Sep 27, 2023
1 check passed
fdeblasio pushed a commit to fdeblasio/pokeemerald-expansion that referenced this pull request Oct 1, 2023
* Add WIld Battles to test runner + exp tests
@DizzyEggg DizzyEggg deleted the wild_battle_tests branch June 23, 2024 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants