Revert "Merge branch 'fix_email_images' into 'master'"

This reverts commit d66148ef393f1748c669c934eec4e928d92ef36a, reversing
changes made to cdb64a81a8ca96961033b8ab06d5191ef5449634.

This change needed to be reverted,
because not enough email clients support inline images.

1 files changed, 3 insertions, 79 deletions

diff --git a/spec/mailers/notify_spec.rb b/spec/mailers/notify_spec.rb
index 7c4235ab379..6bea0321cb9 100644
--- a/spec/mailers/notify_spec.rb
+++ b/spec/mailers/notify_spec.rb
@@ -191,13 +191,6 @@ describe Notify do
       context 'for issues' do
         let(:issue) { create(:issue, author: current_user, assignee: assignee, project: project) }
         let(:issue_with_description) { create(:issue, author: current_user, assignee: assignee, project: project, description: Faker::Lorem.sentence) }
-        let(:issue_with_image) do
-          create(:issue,
-                 author: current_user,
-                 assignee: assignee,
-                 project: project,
-                 description: "![test](#{Gitlab.config.gitlab.url}/#{project.path_with_namespace}/uploads/12345/test.jpg)")
-        end
 
         describe 'that are new' do
           subject { Notify.new_issue_email(issue.assignee_id, issue.id) }
@@ -222,22 +215,6 @@ describe Notify do
           end
         end
 
-        describe 'that contain images' do
-          let(:png) { File.read("#{Rails.root}/spec/fixtures/dk.png") }
-          let(:png_encoded) { Base64::encode64(png) }
-
-          before :each do
-            file_path = File.join(Rails.root, 'public', 'uploads', issue_with_image.project.path_with_namespace, '12345/test.jpg')
-            allow(File).to receive(:file?).with(file_path).and_return(true)
-            allow(File).to receive(:read).with(file_path).and_return(png)
-          end
-
-          subject { Notify.new_issue_email(issue_with_image.assignee_id, issue_with_image.id) }
-          it 'replaces attached images with inline images' do
-            is_expected.to have_body_text URI.encode(png_encoded)
-          end
-        end
-
         describe 'that have been reassigned' do
           subject { Notify.reassigned_issue_email(recipient.id, issue.id, previous_assignee.id, current_user) }
 
@@ -302,14 +279,6 @@ describe Notify do
         let(:merge_author) { create(:user) }
         let(:merge_request) { create(:merge_request, author: current_user, assignee: assignee, source_project: project, target_project: project) }
         let(:merge_request_with_description) { create(:merge_request, author: current_user, assignee: assignee, source_project: project, target_project: project, description: Faker::Lorem.sentence) }
-        let(:merge_request_with_image) do
-           create(:merge_request,
-                  author: current_user,
-                  assignee: assignee,
-                  source_project: project,
-                  target_project: project,
-                  description: "![test](#{Gitlab.config.gitlab.url}/#{project.path_with_namespace}/uploads/12345/test.jpg)")
-         end
 
         describe 'that are new' do
           subject { Notify.new_merge_request_email(merge_request.assignee_id, merge_request.id) }
@@ -346,22 +315,6 @@ describe Notify do
           end
         end
 
-        describe 'that are new and contain contain images in the description' do
-          let(:png) {File.read("#{Rails.root}/spec/fixtures/dk.png")}
-          let(:png_encoded) { Base64::encode64(png) }
-
-          before :each do
-            file_path = File.join(Rails.root, 'public', 'uploads', merge_request_with_image.project.path_with_namespace, '/12345/test.jpg')
-            allow(File).to receive(:file?).with(file_path).and_return(true)
-            allow(File).to receive(:read).with(file_path).and_return(png)
-          end
-
-          subject { Notify.new_merge_request_email(merge_request_with_image.assignee_id, merge_request_with_image.id) }
-          it 'replaces attached images with inline images' do
-            is_expected.to have_body_text URI.encode(png_encoded)
-          end
-        end
-
         describe 'that are reassigned' do
           subject { Notify.reassigned_merge_request_email(recipient.id, merge_request.id, previous_assignee.id, current_user.id) }
 
@@ -470,12 +423,9 @@ describe Notify do
     describe 'project access changed' do
       let(:project) { create(:project) }
       let(:user) { create(:user) }
-      let(:project_member) do
-        create(:project_member,
-               project: project,
-               user: user)
-      end
-
+      let(:project_member) { create(:project_member,
+                                   project: project,
+                                   user: user) }
       subject { Notify.project_access_granted_email(project_member.id) }
 
       it_behaves_like 'an email sent from GitLab'
@@ -515,32 +465,6 @@ describe Notify do
         end
       end
 
-      describe 'on a commit that contains an image' do
-        let(:commit) { project.repository.commit }
-        let(:note_with_image) do
-          create(:note,
-                 project: project,
-                 author: note_author,
-                 note: "![test](#{Gitlab.config.gitlab.url}/#{project.path_with_namespace}/uploads/12345/test.jpg)")
-        end
-
-        let(:png) {File.read("#{Rails.root}/spec/fixtures/dk.png")}
-        let(:png_encoded) { Base64::encode64(png) }
-
-        before :each do
-          file_path = File.join(Rails.root, 'public', 'uploads', note_with_image.project.path_with_namespace, '12345/test.jpg')
-          allow(File).to receive(:file?).with(file_path).and_return(true)
-          allow(File).to receive(:read).with(file_path).and_return(png)
-          allow(Note).to receive(:find).with(note_with_image.id).and_return(note_with_image)
-          allow(note_with_image).to receive(:noteable).and_return(commit)
-        end
-
-        subject { Notify.note_commit_email(recipient.id, note_with_image.id) }
-        it 'replaces attached images with inline images' do
-          is_expected.to have_body_text URI.encode(png_encoded)
-        end
-      end
-
       describe 'on a commit' do
         let(:commit) { project.repository.commit }