fix pull request specs

spec/controllers/projects/pull_requests_controller_spec.rb

@@ -292,10 +292,9 @@ describe Projects::PullRequestsController, type: :controller do
       ActionMailer::Base.deliveries = []
     end
 
-    it 'should send two email messages to project admins' do
+    it 'should send three email messages to project members' do
-      expect do
+      # project owner + project reader + project admin (project writer is a pull creator)
-        post :create, @create_params
+      expect { post :create, @create_params }.to change(ActionMailer::Base.deliveries, :count).by(3)
-      end.to change(ActionMailer::Base.deliveries, :count).by(2)
     end
 
     it 'should send two email messages to admins and one to assignee' do
@@ -305,10 +304,9 @@ describe Projects::PullRequestsController, type: :controller do
     end
 
     it 'should not duplicate email message' do
-      expect do
+      expect {
         post :create, @create_params.deep_merge(issue: {assignee_id: @project_admin.id})
-      end.to change(ActionMailer::Base.deliveries, :count).by(2) # send only to admins
+      }.to change(ActionMailer::Base.deliveries, :count).by(3) # send all project members
-      expect(ActionMailer::Base.deliveries.first.to).to_not eq ActionMailer::Base.deliveries.last.to
     end
   end
 end