fix bug where updating any project would destroy all the other notifi…

…ers for all of the other project

lib/integrity/notifier.rb

@@ -18,7 +18,7 @@ def self.available
     end
 
     def self.enable_notifiers(project, enabled, config={})
-      all.destroy!
+      all(:project_id => project).destroy!
       list_of_enabled_notifiers(enabled).each do |name|
         create! :project_id => project, :name => name, :config => config[name]
       end

spec/notifier_spec.rb

@@ -15,6 +15,10 @@ def sample_notifier(attributes={})
   def sample_project
     @project ||= Integrity::Project.create(:name => "Blah", :uri => "blah.blah")
   end
+
+  def other_project
+    @other_project ||= Integrity::Project.create(:name => "Blah Blah Blah", :uri => "blah.blah")
+  end
 
   describe "configuring a notifier" do
     it "should not be valid by default" do
@@ -52,6 +56,20 @@ def sample_project
         sample_project.enable_notifiers("Blah", "Blah" => { "foo" => "bar" })
       }.should change(Integrity::Notifier, :count).by(1)
     end
+
+    it "should destroy all of the previous notifiers for that project" do
+      sample_project.enable_notifiers("Blah", "Blah" => { "foo" => "bar" })
+      sample_project.enable_notifiers(["Cuack", "Test"], "Cuack" => { "foo" => "bar"},
+                                                         "Test" => { "bar" => "baz "})
+
+      sample_project.notifiers.length.should == 2
+    end
+
+    it "should not destroy all of the other notifiers that exist for other projects" do
+      other_project.enable_notifiers("Blah", "Blah" => { "foo" => "bar" })
+      sample_project.enable_notifiers("Blah", "Blah" => { "foo" => "bar" })
+      other_project.notifiers.length.should == 1
+    end
   end
 
   describe "Notifying the world of a build" do