@@ -23,6 +23,7 @@ module SocialFeed end def unsubscribe_from_feed_event(event_class) + self.feed_event_subscriptions_will_change! self.feed_event_subscriptions.delete event_class.to_s end @@ -36,6 +37,7 @@ module SocialFeed end def unsubscribe_from_email(event_class) + self.email_subscriptions_will_change! self.email_subscriptions.delete event_class.to_s end @@ -45,6 +47,7 @@ module SocialFeed end def disable_feed_event(event_class) + self.enabled_feed_events_will_change! self.enabled_feed_events.delete event_class.to_s end lib/social_feed/user_extension.rb @@ -32,6 +32,20 @@ describe User, 'enable feed events' do it "should return false if an event is disabled" do @user.should_not be_feed_event_enabled(TestEvent) end + + it "should mark the attibute dirty when disabling" do + @user.enabled_feed_events = ['TestEvent'] + @user.save! + @user.disable_feed_event TestEvent + @user.should be_enabled_feed_events_changed + end + + it "should mark the attibute dirty when enabling" do + @user.enabled_feed_events = ['TestEvent1'] + @user.save! + @user.enable_feed_event TestEvent + @user.should be_enabled_feed_events_changed + end end describe User, 'subscribe to feed events' do @@ -62,10 +76,24 @@ describe User, 'subscribe to feed events' do @user.should be_subscribed_to_feed_event(TestEvent) end - it "should return false if user is not subscribed" do @user.should_not be_subscribed_to_feed_event(TestEvent) end + + it "should mark the attibute dirty when subscribing" do + @user.feed_event_subscriptions = ['TestEvent1'] + @user.save! + @user.subscribe_to_feed_event TestEvent + @user.should be_feed_event_subscriptions_changed + end + + it "should mark the attibute dirty when unsubscribing" do + @user.feed_event_subscriptions = ['TestEvent'] + @user.save! + @user.unsubscribe_from_feed_event TestEvent + @user.should be_feed_event_subscriptions_changed + end + end describe User, 'subscribe to emails' do @@ -96,10 +124,23 @@ describe User, 'subscribe to emails' do @user.should be_subscribed_to_email(TestEvent) end - it "should return false if user is not subscribed" do @user.should_not be_subscribed_to_email(TestEvent) end + + it "should mark the attibute dirty when subscribing" do + @user.email_subscriptions = ['TestEvent1'] + @user.save! + @user.subscribe_to_email TestEvent + @user.should be_email_subscriptions_changed + end + + it "should mark the attibute dirty when unsubscribing" do + @user.email_subscriptions = ['TestEvent'] + @user.save! + @user.unsubscribe_from_email TestEvent + @user.should be_email_subscriptions_changed + end end spec/models/user_spec.rb 26eea0cd67b7a3ba27737ba657888d6dbc1b8d38 Alexander Lang alex@skywalker.local http://github.com/langalex/social_feed/commit/e47f1eeb3be54bd44a87808704048b994cfd319b e47f1eeb3be54bd44a87808704048b994cfd319b 2008-07-07T03:23:16-07:00 2008-07-07T03:23:16-07:00 fixed disabling subscrptions/feed events to work with activerecord 2.1 partial updates 04f88ced084cf28adbdc446f9f1b8b226dc5d403 Alexander Lang alex@skywalker.local