Skip to content

Commit

Permalink
Update notes_controler_test.rb (#9355)
Browse files Browse the repository at this point in the history
Removed unused variables
  • Loading branch information
Tanfeil committed Mar 28, 2021
1 parent e30adfa commit 1902557
Showing 1 changed file with 9 additions and 12 deletions.
21 changes: 9 additions & 12 deletions test/functional/notes_controller_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -463,7 +463,7 @@ def teardown
end

test 'returning json errors on xhr note update' do
user = UserSession.create(users(:jeff))
UserSession.create(users(:jeff))

post :update,
params: {
Expand Down Expand Up @@ -525,15 +525,14 @@ def teardown
# no emails sent for first-time posters, as it's held in moderation
assert users(:bob).first_time_poster
assert_emails 0 do
user = UserSession.create(users(:bob))
UserSession.create(users(:bob))
post :create,
params: {
title: title,
body: 'Spectrometer question',
tags: 'question:spectrometer',
redirect: 'question'
}
node = nodes(:blog)
end
end

Expand Down Expand Up @@ -561,7 +560,7 @@ def teardown
end

test 'should display /post template when editing a note in legacy mode' do
user = UserSession.create(users(:jeff))
UserSession.create(users(:jeff))
note = nodes(:blog)
get :edit,
params: {
Expand All @@ -573,7 +572,7 @@ def teardown
end

test 'should display /post template when editing a question in legacy mode' do
user = UserSession.create(users(:jeff))
UserSession.create(users(:jeff))
note = nodes(:question)
note.add_tag('nice', users(:jeff))
get :edit,
Expand All @@ -586,7 +585,7 @@ def teardown
end

test 'should display /post template when editing a note' do
user = UserSession.create(users(:jeff))
UserSession.create(users(:jeff))
note = nodes(:blog)
get :edit,
params: {
Expand All @@ -598,7 +597,7 @@ def teardown
end

test 'should display /post template when editing a question' do
user = UserSession.create(users(:jeff))
UserSession.create(users(:jeff))
note = nodes(:question)
note.add_tag('nice', users(:jeff))
get :edit,
Expand All @@ -611,7 +610,7 @@ def teardown
end

test 'should redirect to questions show page when editing an existing question' do
user = UserSession.create(users(:jeff))
UserSession.create(users(:jeff))
note = nodes(:question)
post :update, params: { id: note.nid, title: note.title, body: 'Spectrometer doubts', tags: 'question:spectrometer', redirect: 'question' }

Expand All @@ -620,7 +619,7 @@ def teardown


test 'should render a text/plain when the note is edited through xhr' do
user = UserSession.create(users(:jeff))
UserSession.create(users(:jeff))
note = nodes(:one)
post :update, params: { id: note.nid, title: note.title, body: 'Canon A1200 IR Conversion is working' }, xhr: true
assert_equal I18n.t('notes_controller.edits_saved'), flash[:notice]
Expand Down Expand Up @@ -756,7 +755,7 @@ def teardown
node = nodes(:about)
length=node.authors.uniq.length
assert_not_equal 1,length
user = UserSession.create(users(:jeff))
UserSession.create(users(:jeff))

assert_no_difference 'Node.count' do
get :delete, params: { id: node.nid }
Expand Down Expand Up @@ -790,13 +789,11 @@ def test_get_rss_feed
end

test 'draft should not be shown when no user' do
node = nodes(:draft)
get :show, params: { id: '21',title: 'Draft note' }
assert_response :missing
end

test 'draft should not be shown when user is not author' do
node = nodes(:draft)
UserSession.create(users(:bob))
get :show, params: { id: '21',title: 'Draft note' }
assert_response :missing
Expand Down

0 comments on commit 1902557

Please sign in to comment.