Cloyir/pokeemmo
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fixes Z and Max moves dmg reduction against side protection - upcoming (#6698)

Browse Source
This commit is contained in:
Alex 2025-04-26 23:37:15 +02:00 committed by GitHub
parent d1aa49cca3
commit 87bd59b6f4
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 78 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/battle_util.c
View File

@ -8971,9 +8971,16 @@ static inline uq4_12_t GetGlaiveRushModifier(u32 battlerDef)
static inline uq4_12_t GetZMaxMoveAgainstProtectionModifier(struct DamageCalculationData *damageCalcData)
{
if ((IsZMove(damageCalcData->move) || IsMaxMove(damageCalcData->move))
&& gProtectStructs[damageCalcData->battlerDef].protected != PROTECT_NONE
&& gProtectStructs[damageCalcData->battlerDef].protected != PROTECT_MAX_GUARD)
if (!IsZMove(damageCalcData->move) && !IsMaxMove(damageCalcData->move))
return UQ_4_12(1.0);
u32 protected = gProtectStructs[damageCalcData->battlerDef].protected;
if (protected != PROTECT_NONE
&& protected != PROTECT_WIDE_GUARD
&& protected != PROTECT_QUICK_GUARD
&& protected != PROTECT_CRAFTY_SHIELD
&& protected != PROTECT_MAT_BLOCK
&& protected != PROTECT_MAX_GUARD)
return UQ_4_12(0.25);
return UQ_4_12(1.0);
}

81
test/battle/move_effect/protect.c
View File

@ -20,7 +20,7 @@ ASSUMPTIONS
ASSUME(!(MoveMakesContact(MOVE_WATER_GUN)));
}
SINGLE_BATTLE_TEST("Protect, Detect, Spiky Shield, Baneful Bunker and Burning Bulwark protect from all moves")
SINGLE_BATTLE_TEST("Protect: Protect, Detect, Spiky Shield, Baneful Bunker and Burning Bulwark protect from all moves")
{
u32 j;
static const u16 protectMoves[] = {
@ -59,7 +59,7 @@ SINGLE_BATTLE_TEST("Protect, Detect, Spiky Shield, Baneful Bunker and Burning Bu
}
}
SINGLE_BATTLE_TEST("King's Shield, Silk Trap and Obstruct protect from damaging moves and lower stats on contact")
SINGLE_BATTLE_TEST("Protect: King's Shield, Silk Trap and Obstruct protect from damaging moves and lower stats on contact")
{
u32 j;
static const u16 protectMoves[][3] =
@ -125,7 +125,7 @@ SINGLE_BATTLE_TEST("King's Shield, Silk Trap and Obstruct protect from damaging
}
}
SINGLE_BATTLE_TEST("Spiky Shield does 1/8 dmg of max hp of attackers making contact and may faint them")
SINGLE_BATTLE_TEST("Protect: Spiky Shield does 1/8 dmg of max hp of attackers making contact and may faint them")
{
u16 usedMove = MOVE_NONE;
u16 hp = 400, maxHp = 400;
@ -162,7 +162,7 @@ SINGLE_BATTLE_TEST("Spiky Shield does 1/8 dmg of max hp of attackers making cont
}
}
SINGLE_BATTLE_TEST("Baneful Bunker poisons pokemon for moves making contact")
SINGLE_BATTLE_TEST("Protect: Baneful Bunker poisons pokemon for moves making contact")
{
u16 usedMove = MOVE_NONE;
@ -194,7 +194,7 @@ SINGLE_BATTLE_TEST("Baneful Bunker poisons pokemon for moves making contact")
}
}
SINGLE_BATTLE_TEST("Burning Bulwark burns pokemon for moves making contact")
SINGLE_BATTLE_TEST("Protect: Burning Bulwark burns pokemon for moves making contact")
{
u16 usedMove = MOVE_NONE;
@ -226,7 +226,7 @@ SINGLE_BATTLE_TEST("Burning Bulwark burns pokemon for moves making contact")
}
}
SINGLE_BATTLE_TEST("Recoil damage is not applied if target was protected")
SINGLE_BATTLE_TEST("Protect: Recoil damage is not applied if target was protected")
{
u32 j, k;
static const u16 protectMoves[] = { MOVE_PROTECT, MOVE_DETECT, MOVE_KINGS_SHIELD, MOVE_BANEFUL_BUNKER, MOVE_SILK_TRAP, MOVE_OBSTRUCT, MOVE_SPIKY_SHIELD };
@ -269,7 +269,7 @@ SINGLE_BATTLE_TEST("Recoil damage is not applied if target was protected")
}
}
SINGLE_BATTLE_TEST("Multi-hit moves don't hit a protected target and fail only once")
SINGLE_BATTLE_TEST("Protect: Multi-hit moves don't hit a protected target and fail only once")
{
u16 move = MOVE_NONE;
@ -316,7 +316,7 @@ SINGLE_BATTLE_TEST("Multi-hit moves don't hit a protected target and fail only o
}
}
DOUBLE_BATTLE_TEST("Wide Guard protects self and ally from multi-target moves")
DOUBLE_BATTLE_TEST("Protect: Wide Guard protects self and ally from multi-target moves")
{
u16 move = MOVE_NONE;
@ -358,7 +358,7 @@ DOUBLE_BATTLE_TEST("Wide Guard protects self and ally from multi-target moves")
}
}
DOUBLE_BATTLE_TEST("Wide Guard can not fail on consecutive turns")
DOUBLE_BATTLE_TEST("Protect: Wide Guard can not fail on consecutive turns")
{
u8 turns;
@ -386,7 +386,7 @@ DOUBLE_BATTLE_TEST("Wide Guard can not fail on consecutive turns")
}
}
DOUBLE_BATTLE_TEST("Quick Guard protects self and ally from priority moves")
DOUBLE_BATTLE_TEST("Protect: Quick Guard protects self and ally from priority moves")
{
u16 move = MOVE_NONE;
struct BattlePokemon *targetOpponent = NULL;
@ -421,7 +421,7 @@ DOUBLE_BATTLE_TEST("Quick Guard protects self and ally from priority moves")
}
}
DOUBLE_BATTLE_TEST("Quick Guard can not fail on consecutive turns")
DOUBLE_BATTLE_TEST("Protect: Quick Guard can not fail on consecutive turns")
{
u8 turns;
@ -446,7 +446,7 @@ DOUBLE_BATTLE_TEST("Quick Guard can not fail on consecutive turns")
}
}
DOUBLE_BATTLE_TEST("Crafty Shield protects self and ally from status moves")
DOUBLE_BATTLE_TEST("Protect: Crafty Shield protects self and ally from status moves")
{
u16 move = MOVE_NONE;
struct BattlePokemon *targetOpponent = NULL;
@ -487,7 +487,7 @@ DOUBLE_BATTLE_TEST("Crafty Shield protects self and ally from status moves")
}
}
SINGLE_BATTLE_TEST("Protect does not block Confide or Decorate")
SINGLE_BATTLE_TEST("Protect: Protect does not block Confide or Decorate")
{
u32 move;
PARAMETRIZE { move = MOVE_CONFIDE; }
@ -563,3 +563,58 @@ DOUBLE_BATTLE_TEST("Crafty Shield does not protect against moves that target all
MESSAGE("The opposing Sunflora's Defense rose!");
}
}
SINGLE_BATTLE_TEST("Protect: Quick Guard, Wide Guard and Crafty Shield don't reduce Max Move demage", s16 damage)
{
s16 dmg[2];
u32 move;
PARAMETRIZE { move = MOVE_WIDE_GUARD; }
PARAMETRIZE { move = MOVE_QUICK_GUARD; }
PARAMETRIZE { move = MOVE_CRAFTY_SHIELD; }
GIVEN {
PLAYER(SPECIES_WOBBUFFET);
OPPONENT(SPECIES_WOBBUFFET);
} WHEN {
TURN { MOVE(player, MOVE_TACKLE, gimmick: GIMMICK_DYNAMAX); }
TURN { MOVE(player, MOVE_TACKLE); MOVE(opponent, move); }
} SCENE {
HP_BAR(opponent, captureDamage: &dmg[0]);
HP_BAR(opponent, captureDamage: &dmg[1]);
} FINALLY {
EXPECT_EQ(dmg[0], dmg[1]);
}
}
SINGLE_BATTLE_TEST("Protect: Quick Guard, Wide Guard and Crafty Shield don't reduce Z-Move demage", s16 damage)
{
bool32 protected;
u32 move;
PARAMETRIZE { protected = TRUE; move = MOVE_WIDE_GUARD; }
PARAMETRIZE { protected = FALSE; move = MOVE_WIDE_GUARD; }
PARAMETRIZE { protected = TRUE; move = MOVE_QUICK_GUARD; }
PARAMETRIZE { protected = FALSE; move = MOVE_QUICK_GUARD; }
PARAMETRIZE { protected = TRUE; move = MOVE_CRAFTY_SHIELD; }
PARAMETRIZE { protected = FALSE; move = MOVE_CRAFTY_SHIELD; }
GIVEN {
ASSUME(GetMoveType(MOVE_TACKLE) == TYPE_NORMAL);
PLAYER(SPECIES_WOBBUFFET) { Item(ITEM_NORMALIUM_Z); }
OPPONENT(SPECIES_WOBBUFFET);
} WHEN {
if (protected)
TURN { MOVE(player, MOVE_TACKLE, gimmick: GIMMICK_Z_MOVE); MOVE(opponent, move); }
else
TURN { MOVE(player, MOVE_TACKLE, gimmick: GIMMICK_Z_MOVE); }
} SCENE {
HP_BAR(opponent, captureDamage: &results[i].damage);
} FINALLY {
EXPECT_EQ(results[0].damage, results[1].damage);
EXPECT_EQ(results[2].damage, results[3].damage);
EXPECT_EQ(results[4].damage, results[5].damage);
}
}