Fix houndci warnings

This commit is contained in:
Vinnie Okada 2014-10-18 18:24:12 -05:00
parent f9e423b499
commit 74c82ae325
4 changed files with 10 additions and 10 deletions

View file

@ -152,8 +152,8 @@ module SharedDiffNote
step 'I should see two separate previews' do
within(diff_file_selector) do
expect(page).to have_css('.js-md-preview', count: 2)
expect(page).to have_content("Should fix it")
expect(page).to have_content("DRY this up")
expect(page).to have_content('Should fix it')
expect(page).to have_content('DRY this up')
end
end

View file

@ -60,7 +60,7 @@ EOT
end
step 'The Markdown preview tab should say there is nothing to do' do
within(".gfm-form") do
within('.gfm-form') do
find('.js-md-preview-button').click
expect(find('.js-md-preview')).to have_content('Nothing to preview.')
end
@ -79,7 +79,7 @@ EOT
end
step 'The Markdown preview tab should display rendered Markdown' do
within(".gfm-form") do
within('.gfm-form') do
find('.js-md-preview-button').click
expect(find('.js-md-preview')).to have_css('img.emoji')
end
@ -92,7 +92,7 @@ EOT
step 'I preview a description text like "Bug fixed :smile:"' do
within('.gfm-form') do
fill_in 'Description', with: 'Bug fixed :smile:'
find('.js-md-preview-button').click()
find('.js-md-preview-button').click
end
end

View file

@ -35,7 +35,7 @@ module SharedNote
step 'I write a comment like ":+1: Nice"' do
within(".js-main-target-form") do
fill_in "note[note]", with: ":+1: Nice"
fill_in 'note[note]', with: ':+1: Nice'
end
end

View file

@ -32,8 +32,8 @@ describe 'Comments' do
end
it 'should have enable submit button and preview button' do
within(".js-main-target-form") do
expect(page).not_to have_css(".js-comment-button[disabled]")
within('.js-main-target-form') do
expect(page).not_to have_css('.js-comment-button[disabled]')
expect(page).to have_css('.js-md-preview-button')
end
end
@ -51,8 +51,8 @@ describe 'Comments' do
it 'should be added and form reset' do
should have_content("This is awsome!")
within(".js-main-target-form") do
expect(page).to have_no_field("note[note]", with: "This is awesome!")
within('.js-main-target-form') do
expect(page).to have_no_field('note[note]', with: 'This is awesome!')
expect(page).not_to have_css('.js-md-preview', visible: true)
end
within(".js-main-target-form") { should have_css(".js-note-text", visible: true) }