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

Add Liquid Ooze, Aromatic Mist Tests #5753

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

ghoulslash
Copy link
Collaborator

Tests If the recipient of Leech Seed's effect were to faint due to Liquid Ooze on the same turn as the victim of Leech Seed, then the victim faints before the recipient. This means that the victim's team loses the battle if both teams had their final Pokémon sent out.

@ghoulslash ghoulslash added the category: battle-tests Related to the automated test environment label Dec 1, 2024
@AlexOn1ine
Copy link
Collaborator

test fails

@ghoulslash ghoulslash changed the title Add Liquid Ooze Test Add Liquid Ooze, Aromatic Mist Tests Dec 2, 2024
@AsparagusEduardo
Copy link
Collaborator

SPECIES_WEEZING_GALARIAN is now SPECIES_WEEZING_GALAR

test/battle/ability/liquid_ooze.c Outdated Show resolved Hide resolved
test/battle/ability/liquid_ooze.c Outdated Show resolved Hide resolved
@AlexOn1ine
Copy link
Collaborator

AlexOn1ine commented Dec 18, 2024

The Liquid Test test message order is my fault. I moved absorb to moveend (which I think is better) but it does not work well with the order. The faint message is printed before the leech seed message.

I will try to find a fix for this or revert the moveend change.

/Edit: Sorry, I just realized this can't be my fault because it is Leech Seed what causes the absorb, not an absorb move 😅

@AsparagusEduardo
Copy link
Collaborator

Let's slap a KNOWN_FAILING and call it a day

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: battle-tests Related to the automated test environment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants