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

Uproar test potentially wrong #5445

Closed
ghoulslash opened this issue Sep 30, 2024 · 2 comments
Closed

Uproar test potentially wrong #5445

ghoulslash opened this issue Sep 30, 2024 · 2 comments
Labels
bug Bug category: battle-mechanic Pertains to battle mechanics category: battle-tests Related to the automated test environment status: unconfirmed This bug has not been reproduced yet

Comments

@ghoulslash
Copy link
Collaborator

Description

The uproar test (https://github.com/rh-hideout/pokeemerald-expansion/blob/master/test/battle/move_effect/uproar.c#L9) assigns Voltorb soundproof but also expects it to wake up from soundproof, which is incorrect behavior according to

&& GetBattlerAbility(gBattlerAttacker) != ABILITY_SOUNDPROOF)
.

Conversation/GIF starts here: https://discord.com/channels/419213663107416084/1077009799293710357/1290119643612184718

Version

1.9.2 (Latest release)

Upcoming/master Version

No response

Discord contact info

No response

@ghoulslash ghoulslash added bug Bug category: battle-mechanic Pertains to battle mechanics status: unconfirmed This bug has not been reproduced yet category: battle-tests Related to the automated test environment labels Sep 30, 2024
@AlexOn1ine
Copy link
Collaborator

While a Pokémon is making an uproar, Pokémon cannot fall asleep (even if they have Soundproof or are not adjacent to the Pokémon making the uproar). Pokémon can be made drowsy by Yawn, but a drowsy Pokémon cannot fall asleep during an uproar (even if it has Soundproof).

According to Bulbapedia this is correct behavior

@AlexOn1ine
Copy link
Collaborator

discussed on discord

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug category: battle-mechanic Pertains to battle mechanics category: battle-tests Related to the automated test environment status: unconfirmed This bug has not been reproduced yet
Projects
None yet
Development

No branches or pull requests

2 participants