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.
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
Kay D rps challenge #2135
base: main
Are you sure you want to change the base?
Kay D rps challenge #2135
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
really clear how your program works, and the thorough testing further adds to that. nice job!
redirect ('/result') | ||
end | ||
|
||
get '/result' do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is a clear and concise way of displaying who has won the game.
end | ||
end | ||
|
||
feature 'player 1 chooses rock' do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps the second and third tests here are redundant if the first one is working?
end | ||
|
||
feature 'player 1 picks rock and player 2 picks paper' do | ||
scenario 'shows player 1 as the winner' do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this line should say "shows player 2 as the winner"
@player2 = player2 | ||
end | ||
|
||
def winner |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this solution to the coding of the game of rock, paper, scissors
No description provided.