Skip to content

Scrabble Pull Request #7

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
Open
wants to merge 38 commits into
base: master
Choose a base branch
from
Open

Scrabble Pull Request #7

wants to merge 38 commits into from

Conversation

@jadevance
Copy link

@jadevance jadevance commented Mar 12, 2016

No optionals done

abrown152 and others added 30 commits March 7, 2016 16:33
@abrown152
Created spec helper file.
7fdb17a
@abrown152
Created rakefile and spec file.
a93e551
@abrown152
Created specs folder.
7c6dce7
@abrown152
created spec sheets and scoring class. Also created scoring hash cons…
a182434 
…tant.
@jadevance
Added notes for tomorrow.
1eef307
@jadevance
created spec files for class scoring and class player
5e4ca83
@jadevance
created player.rb
44340fc
@jadevance
Delete .DS_Store
6daec94
@jadevance
Delete .DS_Store
70b7672
@jadevance
Update and rename spec_helper.rb to scrabble.rb
dc75d6d
@jadevance
Completed baseline requirements
f59f9b7
@jadevance
Wrote a test for 50 point bonus in scoring class method. Also wrote a…
2dc2c3b 
… test to add that score to the total score and return correctly.
@abrown152
Fixed self.score method and associated tests.
c936248
@jadevance
started working on higest score method.
bf2dc05
@abrown152
Finished wave 1.
c19f615
@abrown152
Started tie-breaker test.
0588c81
@jadevance
solved the tie breaker problem and all tests pass
4a196fc
@abrown152
Completed Wave 1 requirements and polished tie-breaker scenarios.
a8d79d8
@abrown152
Clarified tie breaker code can handle more than two tied words.
fe312d8
@jadevance
Cleaned up and clarified scoring_spec tests, arranged constants to be…
bc3a04d 
… reflective of the test they are performing
@jadevance
Removed extra comments, made all test cases consistent
61d161f
@jadevance
created a player_spec and player file
522c699
@jadevance
git didn't commit player_spec, doing it again
ac3ecf2
@abrown152
Linked Scoring class to Player class to calculate score of word entered.
cbdf019
@jadevance
Added in module scrabble.
7e33806
@jadevance
Roll back of changes due to confusion
27dd6ad
@jadevance
Pushing notes and tests
47f53ea
@jadevance
Three tests written for player_spec.
e2950bc
@jadevance
Scoring test for scores over 100 points returns false.
c4f012b
@jadevance
wrote a test for returning an array of words played
6f5bd41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jadevance @abrown152