db0/Android-Netrunner-OCTGN — pull request #121 by IanWhalen (2013-01-26 03:52:54+00:00)
Title: Small change to accessing R&D; initial pass at unit tests

This is kind of an unnecessarily huge explanation, but it's also my first pull request here so I want to make sure i was meeting your expectation of 'functional testing':

With regards to c3c063ce3f5823928edab7f1266030dcae4421fb I've made a pass at functional testing as follows:

Start 1 player game as corp
Load Starter deck
Start turn, use 3 clicks gaining credits, end turn
Activate 'debug' function to switch to runner
start turn, run on R&D, finish run successfully
Enter '25' as number of files to be accessed
As expected the chat output is correct for 1-20, and incorrect for 21+

![old_chat1](https://f.cloud.github.com/assets/482962/99451/678f9986-676b-11e2-923f-63f4f72e2e09.PNG)
![old_chat2](https://f.cloud.github.com/assets/482962/99452/67931052-676b-11e2-8ab2-45dbb7ad93e4.PNG)

Then with the attached patch applied, I loaded a new game, performed the same steps, and got the corrected chat logs:

![new_chat1](https://f.cloud.github.com/assets/482962/99457/85963188-676b-11e2-8943-d31060a14861.PNG)
![new_chat2](https://f.cloud.github.com/assets/482962/99458/85976846-676b-11e2-9a22-f06e2ec7a9f8.PNG)

I've *also* included the unit tests in this pull request. Hopefully the commits messages should make sufficient sense about how I'm approaching the problem, but I'm happy to answer any questions/justify decisions :)