Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Tests/moretests #25

Merged
merged 3 commits into from Jan 17, 2019
Merged

Tests/moretests #25

merged 3 commits into from Jan 17, 2019

Conversation

Xephyz
Copy link
Contributor

@Xephyz Xephyz commented Jan 17, 2019

Changes

  • 2 new tests in GameTest.java
    • Added buyFieldPlayerIsOnTest()
    • Added getPlayerFieldTypeTest
  • 2 new tests in ChanceCardTest.java
    • Added getPaidCardTest1()
    • Added getPaidCardTest2()
    • Added two more players to Game testGame for better testing
  • GetPaidCard constructor changed to public: public GetPaidCard(...)

@Xephyz
New GameTest tests
999860d 
- Added ```buyFieldPlayerIsOnTest()```
- Added ```getPlayerFieldTypeTest```
@Xephyz
Made GetPaidCard() public
89023ff 
- Made ```GetPaidCard()``` public for easier testing
@Xephyz
More tests for ChanceCards
7e14338 
- Added ```getPaidCardTest1()```
- Added ```getPaidCardTest2()```
- Added two more players to ```testGame``` for better testing
@Xephyz Xephyz added the enhancement New feature or request label Jan 17, 2019
@Xephyz Xephyz self-assigned this Jan 17, 2019
@Xephyz Xephyz merged commit 18f27ca into dev Jan 17, 2019
@Xephyz Xephyz deleted the tests/moretests branch January 17, 2019 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Xephyz Xephyz

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@Xephyz