Download as
Browse Code »

Commit 212e8b0b59314e43a4df748ad8976fe18f8b77fb

Authored by Antonio Terceiro
1 parent da02b7f5
Exists in master and in 22 other branches angular_poc, api-article-archived, article-readonly, backup-7-jan-2016, captcha_serpro_plugin, content-manager-hostspot, export-comment-paragraph, fix_event_date_issue, login-captcha, master_rails3, new_video_plugin, pg_search_rank, production, refactor_with_role, refactor_with_role_scope, remove_profile_cat_icons, resend_confirmation_email, staging, staging_rails3, tasks_keep_filter_params, theme-brasil-digital-from-staging, travis

[rails3] Adjust attr_accessible

Showing 3 changed files with 6 additions and 2 deletions   Show diff stats
app/models/mailing.rb
  Diff comments   View file @212e8b0
@@ -2,6 +2,7 @@ require 'mailing_job' @@ -2,6 +2,7 @@ require 'mailing_job'
2 2
3 class Mailing < ActiveRecord::Base 3 class Mailing < ActiveRecord::Base
4 4
  5 + attr_accessible :subject, :body
5 validates_presence_of :source_id, :subject, :body 6 validates_presence_of :source_id, :subject, :body
6 belongs_to :source, :foreign_key => :source_id, :polymorphic => true 7 belongs_to :source, :foreign_key => :source_id, :polymorphic => true
7 belongs_to :person 8 belongs_to :person
app/models/mailing_sent.rb
  Diff comments   View file @212e8b0
1 class MailingSent < ActiveRecord::Base 1 class MailingSent < ActiveRecord::Base
  2 + attr_accessible :person
2 belongs_to :mailing 3 belongs_to :mailing
3 belongs_to :person 4 belongs_to :person
4 end 5 end
test/unit/mailing_sent_test.rb
  Diff comments   View file @212e8b0
@@ -4,8 +4,10 @@ class MailingSentTest &lt; ActiveSupport::TestCase @@ -4,8 +4,10 @@ class MailingSentTest &lt; ActiveSupport::TestCase
4 4
5 should 'return mailing and person' do 5 should 'return mailing and person' do
6 person = fast_create(Person) 6 person = fast_create(Person)
7 - mailing = Mailing.create(:source => Environment.default, :subject => 'Hello', :body => 'We have some news')  
8 - sent = MailingSent.create(:mailing => mailing, :person => person) 7 + environment = Environment.default
  8 + mailing = environment.mailings.create(:subject => 'Hello', :body => 'We have some news')
  9 +
  10 + sent = mailing.mailing_sents.create(:person => person)
9 11
10 mailing_sent = MailingSent.find(sent.id) 12 mailing_sent = MailingSent.find(sent.id)
11 assert_equal [mailing, person], [mailing_sent.mailing, mailing_sent.person] 13 assert_equal [mailing, person], [mailing_sent.mailing, mailing_sent.person]