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

Draco meteor should decrease the user's special attack by 2. #379

Open
dyc3 opened this issue May 15, 2021 · 0 comments
Open

Draco meteor should decrease the user's special attack by 2. #379

dyc3 opened this issue May 15, 2021 · 0 comments
Labels
bug/inaccuracy Behavior doesn't match what is in-game bug Something isn't working component/battle Pertains to battles in particular has-premade-tests This issue has premade tests that are already present on the main branch, just disabled.

Comments

@dyc3
Copy link
Member

dyc3 commented May 15, 2021

Current Behavior

Draco meteor does not change the user's stat modifiers.

Expected Behavior

Draco meteor should decrease the user's special attack by 2.

https://bulbapedia.bulbagarden.net/wiki/Draco_Meteor_(move)

Draco meteor has the meta category: MoveMetaCategoryDamageRaise, so all moves in this category should have this behavior (damage and chance to modify the user's stat)

Additional context

Premade test:

PIt("should deal damage and modify the user's stat (MoveMetaCategoryDamageRaise)", func() {
b := New1v1Battle(
GeneratePokemon(PkmnMightyena, WithLevel(20), WithMoves(TestMoveDamageAndStatChangeSelf)), &a1,
GeneratePokemon(PkmnPonyta, WithLevel(20), WithMoves(TestMoveNoDamage)), &a1,
)
b.rng = AlwaysRNG()
Expect(b.Start()).To(Succeed())
t, _ := b.SimulateRound()
Expect(t).To(HaveTransactionsInOrder(
DamageTransaction{
Target: target{1, 0},
},
ModifyStatTransaction{
Target: target{0, 0},
SelfInflicted: true,
Stat: int(b.getPokemon(target{0, 0}).Moves[0].AffectedStat()),
Stages: int(b.getPokemon(target{0, 0}).Moves[0].StatStages()),
},
))
})

@dyc3 dyc3 added bug Something isn't working bug/inaccuracy Behavior doesn't match what is in-game component/battle Pertains to battles in particular labels May 15, 2021
@dyc3 dyc3 added the has-premade-tests This issue has premade tests that are already present on the main branch, just disabled. label May 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug/inaccuracy Behavior doesn't match what is in-game bug Something isn't working component/battle Pertains to battles in particular has-premade-tests This issue has premade tests that are already present on the main branch, just disabled.
Projects
None yet
Development

No branches or pull requests

1 participant