did a todo item, now event edits can be marked trivial to avoid sending

email to participants. tried to add some tests to test this, but they don't
quite work yet


git-svn-id: http://svn.pdxruby.org/repos/www/trunk@206 f0fbaf97-c700-0410-a5eb-8ea856f8537e

Author: caleb

Rakefile

@@ -8,5 +8,4 @@ task :default => [ :test_units, :test_functional ]
 require 'rake'
 require 'rake/testtask'
 require 'rake/rdoctask'
-
 require 'tasks/rails'

app/controllers/events_controller.rb

@@ -84,7 +84,7 @@ def update
     end
     if @event.update_attributes(params[:event])
       flash[:notice] = 'Event was successfully updated.'
-      MailBot::deliver_change_message(self, @event)
+      MailBot::deliver_change_message(self, @event) unless params[:trivial]
       redirect_to :action => 'show', :id => @event
     else
       render :action => 'edit'

app/views/events/edit.rhtml

@@ -7,6 +7,7 @@
         render_partial 'form_ended'
       end
   %>
+  <p><%= check_box_tag 'trivial' %> <label for="trivial_change">Trivial Change</label></p>
   <%= submit_tag 'Edit' %>
 <%= end_form_tag %>
 

doc/TODO

@@ -11,10 +11,6 @@ _Spamming_
 
   - All members are notified of new events
 
-  - Should be able to check "minor edit" when editing an event
-    to avoid spam going to everyone. Ideally, there would be a
-    preview for textilized fields too.
-
 _Other_
 
   - Bug in location view code when editing an event. Prevented me from 

test/functional/events_controller_test.rb

@@ -85,4 +85,16 @@ def test_destroy
       Event.find(1)
     }
   end
+
+  def setup_for_mail_tests
+    ActionMailer::Base.delivery_method = :test
+    ActionMailer::Base.perform_deliveries = true
+    ActionMailer::Base.deliveries = []
+  end
+
+  def test_trivial_change
+    setup_for_mail_tests
+    post :update, {"commit"=>"Edit", "event"=>{"minutes"=>"Testing this stupid thing. Dont Send mail."}, "action"=>"update", "id"=>"1", "controller"=>"events", "trivial"=>"1"}
+    assert ActionMailer::Base.deliveries.empty?
+  end
 end

test/unit/article_test.rb

@@ -1,7 +1,7 @@
 require File.dirname(__FILE__) + '/../test_helper'
 
 class ArticleTest < Test::Unit::TestCase
-  fixtures :articles
+  fixtures :articles, :members
 
   def setup
     @article = Article.new