Merge develop into acf

* Initial commit

* Fixed the test

* Linter

* Whoops, forgot to rename the test cases from Kuni Lab!

* add first draft of card with test

I modified the pilgramage code to work with henshin mysteries currently is
failing the test for when the province is broken. but is stopping the
ring from being claimed otherwise as intented

* added a test to see if the ring effect was working

* fix bug in broken province test
the issue was a copy paste error the card was working as the whole time

* card implementation and test

wrote the card implementation with 2 static effects. one for mil and
one for pol. then I wrote one test to check each effect was working

* fix bug for him counting himself

I made it to where he filters himself out when modifying his skill and add a
test to make sure it is working

* changing to use .getNumberOfParticipants

* used `getNumberOfParticipants`

instead of filtering the cards in play for each player instead just call
getNumberOfParticipants on the game object to make it a lot less LOC

* Templates based on other cards

* Palace guard implementation & test cases

* Implementation and tests for quarrelsome youth

* Added test for equal cards

* Initial test suite for Outflank.

* Failing test - able to select the same defender twice with two copies.

* Linting

* Updated outflank based on discussions.
Added test for border fortress.

* wrote the card and a test for the targetting

* added test for movement and switched to a conditional game actions to get it to work

* first draft of card implementation

wrote out the logic for the reaction on the card and created an
empty file for the tests

* added tests

added four test: one to show targetting is working, one for showing triggering if
conditions are met, one for cards in hand and one show the character is brought
into play

* using selectCard

the card doesnt target does so the selectCard action is now being
used instead

* add test for less cards in hand

* Added Deduction, plus a change to the returnRings cost in costs.js to support the card

* Updated the test case

Merge dev into acf

merge dev into dor

* Initial commit

* Added tests for Hida O-Ushi

* Added way of the dragon for the max 1 per round test

* Removed the dummy files for DoR.

* Bug fix for Suppress effects

* Update fiveringsdb-data

* Base implementation and tests

* Fixed Matsu Agetoki tests

* Initial tests and implementation for Purifier Apprentice

* Fixed the tests

* Linting fix

* Initial tests & implementation

* Updated the tests

* Implementation and tets for Seventh Tower (not working, needs a json update)

* Fixed the test case

* Linting issue

* Tests and implementation for Subterranean Guile

* Replaced .forEach some .some for checking unbroken provinces

* Implementation and tests for Fruitful Respite

* Updated reaction title

* Tests and implementation for watchtower of valor

* Added a test for losing a conflict

* Added the unlimited clause

* Added test for multiple activations

* Tests and implementation for Slovenly Scavenger

* Linting cleanup

* Tests and implementation for Kansen Haunt

* Added test for negative strength modifier

* Not working at all!

* Implementation and tests for frontline engineer

* Linter cleanup

* Updated to getDynastyCardsInProvince() and added tests for Back Alley Hideaway

* Linting cleanup

* Updated after having done other cards

…ses locally) (#3718)

* Fix #3592: SYH does not cancel Hitomi duel

* Added test for undefined (dash in mongodb) skill removing characters from combat

* Bug fix for Suppress effects

* Update fiveringsdb-data

* Bug fix for Total Warfare & Let Go. (#3709)

* Bugfix for Chasing the Sun (so you can no longer move the conflict to your own provinces) (#3701)

* Updated the deck validator for new roles (#3700)

The roles have been freed as decreed during the winter court of 2019.

* fix Shrewd Yasuki Null Reference crash (#3685)

* Fix Omoidasu trigger condition (#3719)

* Fix Omoidasu trigger condition

It didn't check who was dishonored, only  by who.

* Fix lint errors

* Rework ResolveAbilityAction (#3706)

* Implement play costs

* Add trigger costs

* Rework ResolveAbilityAction

* Test and bugfixes

* Bug fix for prepare for war (& test updates) (#3732)

Closes #3716.

* Bug fix for incorrect text on Isawa Heiko (#3479)

* Bug fix for incorrect text on Isawa Heiko

* finished this bugfix

for the card implementation I change the condition that it need to be
in a conflict. and then I add one test to check this works and I also
had to change the test that stops you from choosing partiicipating
characters

* Fix #3592: SYH does not cancel Hitomi duel

* Added test for undefined (dash in mongodb) skill removing characters from combat

* Bug fix for Suppress effects

* Update fiveringsdb-data

* Bug fix for Total Warfare & Let Go. (#3709)

* Bugfix for Chasing the Sun (so you can no longer move the conflict to your own provinces) (#3701)

* Updated the deck validator for new roles (#3700)

The roles have been freed as decreed during the winter court of 2019.

* fix Shrewd Yasuki Null Reference crash (#3685)

* Fix Omoidasu trigger condition (#3719)

* Fix Omoidasu trigger condition

It didn't check who was dishonored, only  by who.

* Fix lint errors

* Rework ResolveAbilityAction (#3706)

* Implement play costs

* Add trigger costs

* Rework ResolveAbilityAction

* Test and bugfixes

* Bug fix for prepare for war (& test updates) (#3732)

Closes #3716.

* Bug fix for incorrect text on Isawa Heiko (#3479)

* Bug fix for incorrect text on Isawa Heiko

* finished this bugfix

for the card implementation I change the condition that it need to be
in a conflict. and then I add one test to check this works and I also
had to change the test that stops you from choosing partiicipating
characters

…e an opponent

Introduced in PR #3709

* Base implementation for Shukujo.

* Added shukujo tests

* Updated the tests for Shukujo

* Linting fixes

* Updated to use the new attachmentConditions block

* Added an appropriate chat message

* Tests and implementation for army of the rising wave

* Updated the gameaction to use context instead of this.game

* Linting cleanup

* Tests and implementation for erudite prestige

* Added text output to the tests

* how I think the card implementation should work

the implementation prompt the user for a card and then the card will create a lasting effect
for the conflict which creates a delayed effect that will trigger if the card is played

* forgot to change some copy paste code from gossip

* wrote three basic test to show the card working

frist test show the card triggering a the start of a conflict
the second checks the player gets the name card prompt
the last shows that the player gets to claim the ring if the card is played

* more debugging

* more debugging

* got the card working so far

thanks to jadiel help on discord I got this version of the card working but
there a some issue still.
* need to use `claimRing` instead of `takeRing`
* naming the card should be part of the cost
* figure out a way to make the claimed ring political

* refactoring for cost to be naming the card

trying to get the cost to be naming the card but it appears that the game is
getting stuck on the name card prompt. still needs more debugging

* changed the promptmenu to pass down context

basically added in jadiels update to see if this would help
with having naming the card be part of the cost but it
still needs some work on that front

* got the naming of the card to be the cost

cost field on the reaction has be updated to reflect the
cards text better

still need to claim the ring instead of take it and make it
political

* card claims the ring now

created a new gameaction that will to claim the ring instead of take it
it also takes in the type of the ring. which solves two the issues for the
card;

* added a test for claim ring

it is currently failing the server is logging the following error message
```
2019-11-08T02:02:43.140Z - error:  message=Cannot read property conflictRing of undefined, stack=TypeError: Cannot read property conflictRing of undefined
    at onClaimRing (/home/mattg/projects/ringteki/build/server/game/gamesteps/triggeredabilitywindowtitles.js:6:54)
    at _.filter._.map.event (/home/mattg/projects/ringteki/build/server/game/gamesteps/triggeredabilitywindowtitles.js:46:24)
    at Function._.map._.collect (/home/mattg/projects/ringteki/node_modules/underscore/underscore.js:205:24)
    at Object.getTitle (/home/mattg/projects/ringteki/build/server/game/gamesteps/triggeredabilitywindowtitles.js:43:33)
    at TriggeredAbilityWindow.getPromptProperties (/home/mattg/projects/ringteki/build/server/game/gamesteps/forcedtriggeredabilitywindow.js:66:61)
    at TriggeredAbilityWindow.getPromptForSelectProperties (/home/mattg/projects/ringteki/build/server/game/gamesteps/forcedtriggeredabilitywindow.js:60:64)
    at TriggeredAbilityWindow.getPromptForSelectProperties (/home/mattg/projects/ringteki/build/server/game/gamesteps/triggeredabilitywindow.js:82:31)
    at TriggeredAbilityWindow.promptBetweenSources (/home/mattg/projects/ringteki/build/server/game/gamesteps/forcedtriggeredabilitywindow.js:51:69)
    at TriggeredAbilityWindow.filterChoices (/home/mattg/projects/ringteki/build/server/game/gamesteps/triggeredabilitywindow.js:73:14)
    at TriggeredAbilityWindow.filterChoices (/home/mattg/projects/ringteki/build/server/game/gamesteps/triggeredabilitywindow.js:69:25)
    at TriggeredAbilityWindow.continue (/home/mattg/projects/ringteki/build/server/game/gamesteps/forcedtriggeredabilitywindow.js:21:18)
    at GamePipeline.continue (/home/mattg/projects/ringteki/build/server/game/gamepipeline.js:88:37)
    at EventWindow.continue (/home/mattg/projects/ringteki/build/server/game/gamesteps/basestepwithpipeline.js:28:42)
    at GamePipeline.continue (/home/mattg/projects/ringteki/build/server/game/gamepipeline.js:88:37)
    at InitiateAbilityEventWindow.continue (/home/mattg/projects/ringteki/build/server/game/gamesteps/basestepwithpipeline.js:28:42)
    at GamePipeline.continue (/home/mattg/projects/ringteki/build/server/game/gamepipeline.js:88:37)
    at AbilityResolver.continue (/home/mattg/projects/ringteki/build/server/game/gamesteps/basestepwithpipeline.js:28:42)
    at GamePipeline.continue (/home/mattg/projects/ringteki/build/server/game/gamepipeline.js:88:37)
    at ConflictFlow.continue (/home/mattg/projects/ringteki/build/server/game/gamesteps/basestepwithpipeline.js:28:42)
    at GamePipeline.continue (/home/mattg/projects/ringteki/build/server/game/gamepipeline.js:88:37)
    at EventWindow.continue (/home/mattg/projects/ringteki/build/server/game/gamesteps/basestepwithpipeline.js:28:42)
    at GamePipeline.continue (/home/mattg/projects/ringteki/build/server/game/gamepipeline.js:88:37)
    at EventWindow.continue (/home/mattg/projects/ringteki/build/server/game/gamesteps/basestepwithpipeline.js:28:42)
    at GamePipeline.continue (/home/mattg/projects/ringteki/build/server/game/gamepipeline.js:88:37)
    at ConflictPhase.continue (/home/mattg/projects/ringteki/build/server/game/gamesteps/basestepwithpipeline.js:28:42)
    at GamePipeline.continue (/home/mattg/projects/ringteki/build/server/game/gamepipeline.js:88:37)
    at Game.continue (/home/mattg/projects/ringteki/build/server/game/game.js:974:31)
    at runAndCatchErrors (/home/mattg/projects/ringteki/build/server/gamenode/gameserver.js:279:26)
    at GameServer.runAndCatchErrors (/home/mattg/projects/ringteki/build/server/gamenode/gameserver.js:106:13)
    at GameServer.onGameMessage (/home/mattg/projects/ringteki/build/server/gamenode/gameserver.js:276:14)
    at Socket.onSocketEvent (/home/mattg/projects/ringteki/build/server/socket.js:40:13)
    at emitTwo (events.js:126:13)
    at Socket.emit (events.js:214:7)
    at /home/mattg/projects/ringteki/node_modules/socket.io/lib/socket.js:503:12
    at _combinedTickCallback (internal/process/next_tick.js:131:7)
    at process._tickDomainCallback (internal/process/next_tick.js:218:9)
```

* add a test that the ring type was changed

this is working correctly claim ring action is changing it
to the correct type of political

* small linting fix

* OnClaimRing no longer assumes presence of conflict. Updated reaction abilities accordingly.

* Fix test failures and log message

* Unit tests are written

* Added sacrifice to the cannot, more tests are passing

* Added isKeywordAbility, added an ability to restrict keyword abilities and to restrict non-keyword abilities.

Netsu working.
npm run test working.

* Oops, relic of a bad idea

* Tests and implementation for Desolation

* Added a check to ensure you have an opponent

* Added a test to make sure it doesn't blank your own provinces

* Tests and implementation for Yogo Asami

* Removed the location check.  Need to add Ujina tests.

* Added a restriction for abilities triggered by opponents rather than opponent's triggered abilities.

Added the Ujina test, made sure it passed.

* Linting cleanup

* Tests and implementation for Kyuden Hida

* Made Kyuden Hida do what it says it will.

Working, but feels bad if you cancel & burn the trigger.  Going to do a bit of investigation on UX to see if I can make it work better

* UX is still not great, but it's consistent with Kyuden Isawa.

Also actually works properly now instead of discarding 5 cards if you take nothing.

* Northern Curtain Wall - Base Implementation modifying province strength (this is incorrect).

* Updated the effects engine to include the ability to modify province strength bonus, and updated northern curtain wall with the new effect

* Fixed the modifier so set should actually work now

…ovince (#3742)

* Works, provided I cheat the player effect.  Need to create an effect that targets the player I think

* Updated it a little bit

* Initial implementation & tests

Needs some UI work

* Linting

* Added a holding test

* Added a check for undefined cardname. Covers not known cards and cards you as player shouldn't see.

* Added a comment to the test

* Unit tests are written

* Added sacrifice to the cannot, more tests are passing

* Added isKeywordAbility, added an ability to restrict keyword abilities and to restrict non-keyword abilities.

Netsu working.
npm run test working.

* Oops, relic of a bad idea

* Initial commit.  UX is a bit of a mess when choosing covert, but going to try to update that.

* Oops, this was included in a merge

SD - Moto Outrider

Bug Fix - Gamenode crash during targeting if no opponent

merge develop into shojus-duty

* Tests and implementation for Bayushi Shoju

* Linting update

* Getting somewhere

* Working, but with removed from game

* Fixed the messages a bit

* Basic implementation for Kaiu Shihobu, along with tests, using the Removed from Game zone.

* Added a test case

* Updated the dynasty deck search action

* Passes tests with a fixed location of "UnderStronghold".

Will explore having a generic location, since it should work.

* Consolidating the deck search action.

* UI implementation for SHihobu (cards underneath stronghold)

* Updated the menu prompt title

* Linting updates

Merge Defenders of Rokugan into Develop

* Need to update the exclusion criteria for alternate fate pools, since it has a clause that it stops working if you can't spend fate from rings

* Working

* Added highlighting the current card incase you have multiple students out

* Had to remove the reference to ring.js for some reason

Also added support for action opportunity durations

* Tests and implementation for Daidoji Harrier

* Added a test for having <2 cards

* Base case working without Banzai... going to explore Banzai a bit to try it

* Linting cleanup

* Added a couple new tests.  Now to look at Banzai

* Working with Banzai

* Added Banzai cancel tests, fixed some issues that came up

Merge aCF into Develop

* Tests and implementation for Togashi Ichi

* Updated test

* Nowhere yet

* Play restrictions working, now to test the text

* Tests and implementation for Castigated

* Initial commit (test file is entirely wrong)

* Not working yet

* Oops, duel participants can be arrays

* Done

* Added a final test case

* Tests and implementation for Root Out Heresy.

Added the ability for the discard actions to store the cards that have been discarded.

* Forgot to update the test names

SD - Chrysanthemum Steward

* tests and implementation for Inferno Guard Invoker

* Fix lint errors

* - Rework implementation
- Add tests for multiple delayed effects and province breaking by other means

* Fix lint error