Nathaniel Brandes
d7d700f66a
Some clarification around loyalty. Added unit test.
2017-03-09 02:30:52 -08:00
Nathaniel Brandes
dd0c89db1d
Fix Dack Fayden Emblem.
...
The emblem's effect was referencing the permanent directly.
This meant that if the permanent left and came back, the control effect
would start working again.
2017-03-09 02:24:34 -08:00
Justin Herlehy
9dd1e60f2d
Implement Clutch of Undeath
...
Implement Clutch of Undeath and add test to ensure functionality.
2017-03-09 01:48:34 -08:00
Justin Herlehy
1a2724c9b2
Revert: Fix planeswalker loyalty counters.
...
This change was incorrect.
Gatherer Ruling:
10/1/2005: Planeswalkers will enter the battlefield with double the normal amount of loyalty counters. However, if you activate an ability whose cost has you put loyalty counters on a planeswalker, the number you put on isn’t doubled. This is because those counters are put on as a cost, not as an effect.
2017-03-09 01:14:10 -08:00
Nathaniel Brandes
52fa369024
Fix planeswalker loyalty counters.
...
They should be added like normal counters, firing events that can be
intercepted by cards like doubling season.
2017-03-08 23:57:11 -08:00
Nathaniel Brandes
ce1f4a3bf8
Phyrexian mana now correctly a payment choice.
...
Per rule 601.2b, it is not determined at the pay costs step,
but at the "choice" step, long before costs are determined.
This fixes trinisphere interactions and should be consistent with the
rules.
2017-03-08 20:03:28 -08:00
Jeff
209e2d13c1
- Fix for Spellskite in situations where it was also a target. Fixed the test for Spellskite.
2017-03-08 12:05:36 -06:00
Achilles
f5d13944a6
- Test for Auratouched Mage. The tests worked fine in the game, but the test suite can't handle it yet.
2017-03-07 20:52:13 -06:00
Derek M
2ff68c2fb3
Merge pull request #2944 from Alexsandr0x/sakashima-bug
...
SpellStutter Sprite - Sakashima Impostor Bug
2017-03-07 16:10:06 -05:00
Alexsandro
b83bc9d68a
Assert if Sakashima copied SpellStutter by S/T
2017-03-06 21:11:09 -03:00
Derek Monturo
3b6ffe9382
#2157 Linvala tests on activated abilities
2017-03-06 17:02:23 -05:00
Derek Monturo
e28dd5aef7
minor added another assert
2017-03-06 16:34:52 -05:00
Derek Monturo
f93524a0f7
#2135 added test for clever impersonator copying an aura
2017-03-06 16:32:40 -05:00
Alexsandro
d98e43ded2
Create test class to validate error
2017-03-06 16:15:17 -03:00
Alexsandro
6fb344a306
Create test class to validate error
2017-03-06 16:15:02 -03:00
Derek Monturo
c53c0d2e9e
#2832 test for Athreos not triggering on Cartel sac ability
2017-03-06 14:12:38 -05:00
drmDev
2aa1ee2471
minor mod
2017-03-05 12:17:17 -05:00
drmDev
594e22a245
tests for GrandArbiter cost reduction and increase based on reported bug. unconfirmed
2017-03-05 12:16:12 -05:00
drmDev
c8e41bc002
Phyrexian Purge impl and tests
2017-03-05 10:37:05 -05:00
drmDev
4bfa956368
more tests for Nacatl must be blocked
2017-03-05 07:54:55 -05:00
ingmargoudt
b5e8c88153
remove all contains(cardtype.XX) references, instead use the default interface method
2017-03-05 09:14:33 +01:00
Nathaniel Brandes
1d3ebb749d
Fix omniscience effect for colorless mana.
...
Also unified the tests for omniscience into a single file, and
put the effect into a single class so that Tamiyo and Omniscience
can share implementations.
2017-03-04 23:04:11 -08:00
drmDev
6d20c7a996
added test for Nacatl War-Pride
2017-03-04 18:54:29 -05:00
ingmargoudt
5a57731968
added new helper methods to test for CardType, to get rid of the contains(CardType.XXX) everywhere, put the logic of that in one place and use the interface call
2017-03-04 00:16:36 +01:00
ingmargoudt
372584f7ad
changed cardtype from list to enumset
2017-03-03 17:04:32 +01:00
vraskulin
11dc1e10f1
Few lambda's and removed explicit type from creating lists and hashmaps
2017-02-28 13:46:57 +03:00
vraskulin
0a3c95dde5
Inner classes should be static. It reduces memory usage and gives perfomance boost + DeckImporter rollback
2017-02-28 12:40:12 +03:00
doncarton
5eaaa67d1e
Nulls to optional. Also there was bug with AetherChaser "you may pay" was not set to optional
2017-02-26 15:15:54 +03:00
Goesta
e9d103b70e
Fixed BouncingACrewedVehicle test
2017-02-22 23:25:15 +01:00
ingmargoudt
a98df9a28e
fix javadoc
2017-02-19 09:13:19 +01:00
ingmargoudt
9ea690797f
the card should still be exiled.
...
added a method to test exile count per player
added a test for chandrapyromaster
2017-02-19 09:09:43 +01:00
LevelX2
899046cf16
* Gratuitous Violence - FIxed that if the damage of a creature with lifelink was doubeled only the non doubled amount was added to life.
2017-02-18 01:26:02 +01:00
ingmargoudt
7d655ce745
Merge pull request #2866 from goesta/BouncingCrewedVehicleTest
...
Bouncing crewed vehicle test
2017-02-15 23:55:11 +01:00
vraskulin
a15220d51e
Bugfixes with != instead of equals and default toString calls
2017-02-15 19:52:20 +03:00
Goesta
338baa546e
Added bouncing a crewed vehicle will uncrew it test
2017-02-14 21:24:53 +01:00
Goesta
5ece29e5d7
Added assertNotType test assertion
2017-02-14 21:19:47 +01:00
LevelX2
7f866f1b21
Merge pull request #2859 from goesta/FlusterstormTest
...
Added Flusterstorm test
2017-02-13 00:06:45 +01:00
LevelX2
2e8d68aa4b
Merge pull request #2858 from goesta/SpellskiteTest
...
Added Spellskite split damage test
2017-02-13 00:06:17 +01:00
Goesta
0bccc7ab49
Added Flusterstorm test
2017-02-12 05:32:25 +01:00
Goesta
b39996e1eb
Added Spellskite spit damage test
2017-02-12 04:30:01 +01:00
Goesta
ce78f64fab
Added test for Champion of Lambholt
2017-02-11 23:45:20 +01:00
Goesta
a21128932b
Added Test for Kalastria Healer
2017-02-11 20:18:46 +01:00
Jeff
fc3ecdf8d0
- Added Auratouched Mage and test suite.
2017-02-10 10:15:56 -06:00
Jeff
1e82f0750a
Fixed FlashbackTest and made modifications to CascadeAbility allowing it to pass its test suite.
2017-02-09 10:20:21 -06:00
vraskulin
0557b5e89c
list.size > 0 changed to !list.isEmpty
...
We care about if it's empty or not, not about it's size
2017-02-06 14:32:51 +03:00
vraskulin
f60ebfbb1f
All 1-character strings converted to primitives
...
"b" + "r" now changed to 'b' + 'w'. It's more straight-forward, and may cause perfomance improvements - character primitives allocation is faster and less expensive than string creation.
2017-02-06 14:32:51 +03:00
vraskulin
31589778ca
Try/finally refactored to try with resources
2017-02-06 14:32:11 +03:00
LevelX2
ff6d79829e
* Fixed a bug that damage was wrongly still assigned to players during combat that already left the game.
2017-02-04 23:56:03 +01:00
LevelX2
0dacf2a6fa
* Fixed that the cascade converted mana costs comparison did not work correctly with split spells.
2017-02-04 14:26:17 +01:00
LevelX2
067b72dcce
* Added test.
2017-02-03 21:36:33 +01:00
