def test_index_with_no_additional_emails
@request.session[:user_id] = 2
- get :index, :params => {
- :user_id => 2
- }
+ get(:index, :params => {:user_id => 2})
assert_response :success
end
@request.session[:user_id] = 2
EmailAddress.create!(:user_id => 2, :address => 'another@somenet.foo')
- get :index, :params => {
- :user_id => 2
- }
+ get(:index, :params => {:user_id => 2})
assert_response :success
assert_select '.email', :text => 'another@somenet.foo'
end
@request.session[:user_id] = 2
EmailAddress.create!(:user_id => 2, :address => 'another@somenet.foo')
- get :index, :params => {
- :user_id => 2
- },
- :xhr => true
+ get(:index, :params => {:user_id => 2}, :xhr => true)
assert_response :success
assert_include 'another@somenet.foo', response.body
end
def test_index_by_admin_should_be_allowed
@request.session[:user_id] = 1
- get :index, :params => {
- :user_id => 2
- }
+ get(:index, :params => {:user_id => 2})
assert_response :success
end
def test_index_by_another_user_should_be_denied
@request.session[:user_id] = 3
- get :index, :params => {
- :user_id => 2
- }
+ get(:index, :params => {:user_id => 2})
assert_response 403
end
def test_create
@request.session[:user_id] = 2
assert_difference 'EmailAddress.count' do
- post :create, :params => {
+ post(
+ :create,
+ :params => {
:user_id => 2,
:email_address => {
:address => 'another@somenet.foo'
}
}
+ )
assert_response 302
assert_redirected_to '/users/2/email_addresses'
end
def test_create_as_js
@request.session[:user_id] = 2
assert_difference 'EmailAddress.count' do
- post :create, :params => {
+ post(
+ :create,
+ :params => {
:user_id => 2,
:email_address => {
:address => 'another@somenet.foo'
}
},
:xhr => true
+ )
assert_response 200
end
end
def test_create_with_failure
@request.session[:user_id] = 2
assert_no_difference 'EmailAddress.count' do
- post :create, :params => {
+ post(
+ :create,
+ :params => {
:user_id => 2,
:email_address => {
:address => 'invalid'
}
}
+ )
assert_response :success
assert_select_error /email is invalid/i
end
with_settings :email_domains_denied => 'black.example' do
assert_no_difference 'EmailAddress.count' do
- post :create, :params => {
+ post(
+ :create,
+ :params => {
:user_id => 2,
:email_address => {
:address => 'another@black.example'
}
}
+ )
assert_response :success
assert_select_error 'Email is invalid'
end
with_settings :email_domains_allowed => 'white.example' do
assert_no_difference 'EmailAddress.count' do
- post :create, :params => {
+ post(
+ :create,
+ :params => {
:user_id => 2,
:email_address => {
:address => 'something@example.fr'
}
}
+ )
assert_response :success
assert_select_error 'Email is invalid'
end
def test_create_should_send_security_notification
@request.session[:user_id] = 2
ActionMailer::Base.deliveries.clear
- post :create, :params => {
+ post(
+ :create,
+ :params => {
:user_id => 2,
:email_address => {
:address => 'something@example.fr'
}
}
+ )
mail = ActionMailer::Base.deliveries.last
assert_not_nil mail
assert_mail_body_match '0.0.0.0', mail
@request.session[:user_id] = 2
email = EmailAddress.create!(:user_id => 2, :address => 'another@somenet.foo')
- put :update, :params => {
+ put(
+ :update,
+ :params => {
:user_id => 2,
:id => email.id,
:notify => '0'
}
+ )
assert_response 302
assert_equal false, email.reload.notify
@request.session[:user_id] = 2
email = EmailAddress.create!(:user_id => 2, :address => 'another@somenet.foo')
- put :update, :params => {
+ put(
+ :update,
+ :params => {
:user_id => 2,
:id => email.id,
:notify => '0'
},
:xhr => true
+ )
assert_response 200
assert_equal false, email.reload.notify
email = EmailAddress.create!(:user_id => 2, :address => 'another@somenet.foo')
ActionMailer::Base.deliveries.clear
- put :update, :params => {
+ put(
+ :update,
+ :params => {
:user_id => 2,
:id => email.id,
:notify => '0'
},
:xhr => true
+ )
mail = ActionMailer::Base.deliveries.last
assert_not_nil mail
assert_mail_body_match I18n.t(:mail_body_security_notification_notify_disabled, value: 'another@somenet.foo'), mail
email = EmailAddress.create!(:user_id => 2, :address => 'another@somenet.foo')
assert_difference 'EmailAddress.count', -1 do
- delete :destroy, :params => {
+ delete(
+ :destroy,
+ :params => {
:user_id => 2,
:id => email.id
}
+ )
assert_response 302
assert_redirected_to '/users/2/email_addresses'
end
email = EmailAddress.create!(:user_id => 2, :address => 'another@somenet.foo')
assert_difference 'EmailAddress.count', -1 do
- delete :destroy, :params => {
+ delete(
+ :destroy,
+ :params => {
:user_id => 2,
:id => email.id
},
:xhr => true
+ )
assert_response 200
end
end
@request.session[:user_id] = 2
assert_no_difference 'EmailAddress.count' do
- delete :destroy, :params => {
+ delete(
+ :destroy,
+ :params => {
:user_id => 2,
:id => User.find(2).email_address.id
}
+ )
assert_response 404
end
end
email = EmailAddress.create!(:user_id => 2, :address => 'another@somenet.foo')
ActionMailer::Base.deliveries.clear
- delete :destroy, :params => {
+ delete(
+ :destroy,
+ :params => {
:user_id => 2,
:id => email.id
},
:xhr => true
+ )
mail = ActionMailer::Base.deliveries.last
assert_not_nil mail
assert_mail_body_match I18n.t(:mail_body_security_notification_remove, field: I18n.t(:field_mail), value: 'another@somenet.foo'), mail