github/mage
1
0
Fork 0
mirror of https://github.com/correl/mage.git synced 2024-11-28 19:19:55 +00:00
Code Issues Projects Releases Packages Wiki Activity

Fixed Ominous Roost (#9079)

Browse source 
Co-authored-by: Alex Vasile <48962821+Alex-Vasile@users.noreply.github.com>
This commit is contained in:
Alexander Novotny 2022-09-09 19:30:40 -07:00 committed by GitHub
parent e6a7605fd7
commit 63b95bffa5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 63 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
Mage.Sets/src/mage/cards/o/OminousRoost.java
View file

@ -37,7 +37,7 @@ public final class OminousRoost extends CardImpl {
class OminousRoostTriggeredAbility extends TriggeredAbilityImpl {
OminousRoostTriggeredAbility() {
super(Zone.ALL, new CreateTokenEffect(new OminousRoostBirdToken()));
super(Zone.BATTLEFIELD, new CreateTokenEffect(new OminousRoostBirdToken()));
}
private OminousRoostTriggeredAbility(final OminousRoostTriggeredAbility ability) {

62
Mage.Tests/src/test/java/org/mage/test/cards/single/mid/OminousRoostTest.java Normal file
View file

@ -0,0 +1,62 @@
package org.mage.test.cards.single.mid;
import mage.constants.PhaseStep;
import mage.constants.Zone;
import org.junit.Test;
import org.mage.test.serverside.base.CardTestPlayerBase;
/**
* {@link mage.cards.o.OminousRoost Ominous Roost}
* {2}{U}
* Enchantment
* When Ominous Roost enters the battlefield or whenever you cast a spell from your graveyard,
* create a 1/1 blue Bird creature token with flying and “This creature can block only creatures with flying.”
*
* @author alexander-novo, Alex-Vasile
*/
public class OminousRoostTest extends CardTestPlayerBase {
private static final String ominousRoost = "Ominous Roost";
/**
* Reported bug: https://github.com/magefree/mage/issues/9078
* If Ominous Roost is in your library, it will trigger any time you cast a spell from your graveyard.
*/
@Test
public void doesNotTriggerFromOtherZones() {
addCard(Zone.BATTLEFIELD, playerA, "Mountain", 3);
addCard(Zone.EXILED, playerA, ominousRoost);
addCard(Zone.LIBRARY, playerA, ominousRoost);
addCard(Zone.GRAVEYARD, playerA, ominousRoost);
addCard(Zone.HAND, playerA, ominousRoost);
// Flashback {2}{R}
// Create a treasure token
addCard(Zone.GRAVEYARD, playerA, "Strike It Rich");
setStrictChooseMode(true);
activateAbility(1, PhaseStep.PRECOMBAT_MAIN, playerA, "Flashback {2}{R}");
setStopAt(1, PhaseStep.END_TURN);
execute();
assertPermanentCount(playerA, ominousRoost, 0);
assertPermanentCount(playerA, "Bird Token", 0); // None of the cards are on the field, so it should not have triggered
}
/**
* Test that it triggers on ETB
*/
@Test
public void triggersOnOwnETB() {
addCard(Zone.HAND, playerA, ominousRoost);
addCard(Zone.BATTLEFIELD, playerA, "Island", 3);
castSpell(1, PhaseStep.PRECOMBAT_MAIN, playerA, ominousRoost);
setStopAt(1, PhaseStep.END_TURN);
execute();
assertPermanentCount(playerA, ominousRoost, 1);
assertPermanentCount(playerA, "Bird Token", 1);
}
}