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

Create a TestCompareHands.java #40

Closed
dvanmali opened this issue Jul 29, 2016 · 4 comments
Closed

Create a TestCompareHands.java #40

dvanmali opened this issue Jul 29, 2016 · 4 comments

Comments

@dvanmali
Copy link
Contributor

dvanmali commented Jul 29, 2016 •
edited
Loading

Create a file to test all combinations of hands and table cards to see if comparing actually works.

Estimate: ~150 pts
@dvanmali dvanmali changed the title TestCompareHands.java Test CompareHands.java Jul 29, 2016
@dvanmali dvanmali changed the title Test CompareHands.java Add junit Functionality Jul 29, 2016
@mcaccamo
Copy link
Contributor

m16 okay

@dvanmali dvanmali changed the title Add junit Functionality Create a TestCompareHands.java Jul 29, 2016
@jalexanderqed
Copy link
Contributor

F16 OK (150 pts)

@jalexanderqed jalexanderqed mentioned this issue Sep 28, 2016
Closed
gwhiter added a commit to gwhiter/cs56-games-poker that referenced this issue Nov 9, 2016
@gwhiter
Fixed Issues ucsb-cs56-projects#36 and ucsb-cs56-projects#40
2985215
@jalexanderqed
Copy link
Contributor

The new test file looks good but does not seem to be run by ant test because it does not follow the naming convention *Test.java

@jalexanderqed jalexanderqed mentioned this issue Nov 14, 2016
Merged
@jalexanderqed
Copy link
Contributor

I'm not sure why, but the test file still does not appear to run. To test it, try putting something in the test file that should DEFINITELY fail and run ant test. As far as I can tell, it's still not invoked at all.

jalexanderqed added a commit that referenced this issue Nov 15, 2016
@jalexanderqed
Merge pull request #50 from gwhiter/f16
6bed267 
Fixed Issues #36, #37, #40, #10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jalexanderqed @dvanmali @mcaccamo