Skip to content

Friday finish - #259

Open
erb16 wants to merge 4 commits into
makersacademy:mainfrom
erb16:main
Open

Friday finish#259
erb16 wants to merge 4 commits into
makersacademy:mainfrom
erb16:main

Conversation

@erb16

@erb16 erb16 commented Jul 11, 2022

Copy link
Copy Markdown

As far as I got on Friday

Comment thread app.rb
# end

get '/' do
erb(:index)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I forgot the brackets lol thanks

@RonanHughes1 RonanHughes1 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well formatted and easy to follow

Comment thread spec/player_spec.rb
end
end
end

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very tidy and easy to follow

click_button 'Submit'
click_button 'ROCK'
expect(page).to have_content 'rock'
end

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great capybara test format

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants