added mockito dep
improved ActionTest.testPerformAllowedAction and removed side-effecty things
removed side-effecty ActionTest.used
got rid of duplicated string literal
Added a system to draw a card when a creature dies
Merge pull request #59 from janosgyerik/improved-actiontest

Improved actiontest, using Mockito. Thanks @janosgyerik
Minor cleanup of various methods in CardHand and BattlefieldZoneView
Removed some minor commends
Removed code duplication by making `removeCardActive` inside `CardView`
Clearing targetable and clearing active is the same thing
`setTargetable` is now an abstract method in `CardView`
Avoiding FindBugs warning. cardId *is* initialized but findbugs doesn't
know it because it's done with reflection.
Changed package of some view classes
Corrected location of FXML documents
Starting a new game no longer starts a new connection. Can now continue
being in lobby while playing, and also after playing a game.
Avoiding NPE when a game ends
Changed how opening a new game window works so that it is now opened
when the Server sends a `NewGameMessage`
Possibly fixed a possible test failure
merging with develop
Made card declaration more explicit to avoid confusion