Send project transfer email to all members. Added specs for project move email

Dmitriy Zaporozhets
1 parent dc0f08f8
Showing 3 changed files with 24 additions and 6 deletions Show diff stats
app/mailers/emails/projects.rb
app/models/project.rb
spec/mailers/notify_spec.rb
@@ -8,10 +8,10 @@ module Emails
     end
-    def project_was_moved_email(user_project_id)
-      @users_project = UsersProject.find user_project_id
-      @project = @users_project.project
-      mail(to: @users_project.user.email,
+    def project_was_moved_email(project_id, user_id)
+      @user = User.find user_id
+      @project = Project.find project_id
+      mail(to: @user.email,
            subject: subject("project was moved"))
     end
   end
@@ -246,8 +246,8 @@ class Project &lt; ActiveRecord::Base
   end
   def send_move_instructions
-    self.users_projects.each do |member|
-      Notify.delay.project_was_moved_email(member.id)
+    team.members.each do |user|
+      Notify.delay.project_was_moved_email(self.id, user.id)
     end
   end
@@ -218,6 +218,24 @@ describe Notify do
       end
     end
+    describe 'project was moved' do
+      let(:project) { create(:project) }
+      let(:user) { create(:user) }
+      subject { Notify.project_was_moved_email(project.id, user.id) }
+
+      it 'has the correct subject' do
+        should have_subject /project was moved/
+      end
+
+      it 'contains name of project' do
+        should have_body_text /#{project.name_with_namespace}/
+      end
+
+      it 'contains new user role' do
+        should have_body_text /#{project.ssh_url_to_repo}/
+      end
+    end
+
     describe 'project access changed' do
       let(:project) { create(:project) }
       let(:user) { create(:user) }
	@@ -218,6 +218,24 @@ describe Notify do		@@ -218,6 +218,24 @@ describe Notify do
218	end	218	end
219	end	219	end
220		220
		221	+ describe 'project was moved' do
		222	+ let(:project) { create(:project) }
		223	+ let(:user) { create(:user) }
		224	+ subject { Notify.project_was_moved_email(project.id, user.id) }
		225	+
		226	+ it 'has the correct subject' do
		227	+ should have_subject /project was moved/
		228	+ end
		229	+
		230	+ it 'contains name of project' do
		231	+ should have_body_text /#{project.name_with_namespace}/
		232	+ end
		233	+
		234	+ it 'contains new user role' do
		235	+ should have_body_text /#{project.ssh_url_to_repo}/
		236	+ end
		237	+ end
		238	+
221	describe 'project access changed' do	239	describe 'project access changed' do
222	let(:project) { create(:project) }	240	let(:project) { create(:project) }
223	let(:user) { create(:user) }	241	let(:user) { create(:user) }