Remove Redcarpet markdown engine

This engine was replaced with CommonMarker in 11.4, it was deprecated
since then.

6 files changed, 3 insertions, 157 deletions

diff --git a/spec/features/markdown/markdown_spec.rb b/spec/features/markdown/markdown_spec.rb
index 3b37ede8579..8815643ca96 100644
--- a/spec/features/markdown/markdown_spec.rb
+++ b/spec/features/markdown/markdown_spec.rb
@@ -13,7 +13,7 @@ require 'erb'
 #
 #   Raw Markdown
 #   -> `markdown` helper
-#     -> Redcarpet::Render::GitlabHTML converts Markdown to HTML
+#     -> CommonMark::Render::GitlabHTML converts Markdown to HTML
 #       -> Post-process HTML
 #         -> `gfm` helper
 #           -> HTML::Pipeline
@@ -324,31 +324,6 @@ describe 'GitLab Markdown', :aggregate_failures do
     end
   end
 
-  context 'Redcarpet documents' do
-    before do
-      allow_any_instance_of(Banzai::Filter::MarkdownFilter).to receive(:engine).and_return('Redcarpet')
-      @html = markdown(@feat.raw_markdown)
-    end
-
-    it 'processes certain elements differently' do
-      aggregate_failures 'parses superscript' do
-        expect(doc).to have_selector('sup', count: 3)
-      end
-
-      aggregate_failures 'permits style attribute in th elements' do
-        expect(doc.at_css('th:contains("Header")')['style']).to eq 'text-align: center'
-        expect(doc.at_css('th:contains("Row")')['style']).to eq 'text-align: right'
-        expect(doc.at_css('th:contains("Example")')['style']).to eq 'text-align: left'
-      end
-
-      aggregate_failures 'permits style attribute in td elements' do
-        expect(doc.at_css('td:contains("Foo")')['style']).to eq 'text-align: center'
-        expect(doc.at_css('td:contains("Bar")')['style']).to eq 'text-align: right'
-        expect(doc.at_css('td:contains("Baz")')['style']).to eq 'text-align: left'
-      end
-    end
-  end
-
   # Fake a `current_user` helper
   def current_user
     @feat.user
diff --git a/spec/features/projects/blobs/blob_show_spec.rb b/spec/features/projects/blobs/blob_show_spec.rb
index e2f9e7e9cc5..3edcc7ac2cd 100644
--- a/spec/features/projects/blobs/blob_show_spec.rb
+++ b/spec/features/projects/blobs/blob_show_spec.rb
@@ -5,8 +5,8 @@ describe 'File blob', :js do
 
   let(:project) { create(:project, :public, :repository) }
 
-  def visit_blob(path, anchor: nil, ref: 'master', legacy_render: nil)
-    visit project_blob_path(project, File.join(ref, path), anchor: anchor, legacy_render: legacy_render)
+  def visit_blob(path, anchor: nil, ref: 'master')
+    visit project_blob_path(project, File.join(ref, path), anchor: anchor)
 
     wait_for_requests
   end
@@ -171,21 +171,6 @@ describe 'File blob', :js do
         end
       end
     end
-
-    context 'when rendering legacy markdown' do
-      before do
-        visit_blob('files/commonmark/file.md', legacy_render: 1)
-
-        wait_for_requests
-      end
-
-      it 'renders using RedCarpet' do
-        aggregate_failures do
-          expect(page).to have_content("sublist")
-          expect(page).to have_xpath("//ol//li//ul")
-        end
-      end
-    end
   end
 
   context 'Markdown file (stored in LFS)' do
diff --git a/spec/features/projects/blobs/edit_spec.rb b/spec/features/projects/blobs/edit_spec.rb
index d5b20605860..6e6c299ee2e 100644
--- a/spec/features/projects/blobs/edit_spec.rb
+++ b/spec/features/projects/blobs/edit_spec.rb
@@ -81,17 +81,6 @@ describe 'Editing file blob', :js do
         expect(page).to have_content("sublist")
         expect(page).not_to have_xpath("//ol//li//ul")
       end
-
-      it 'renders content with RedCarpet when legacy_render is set' do
-        visit project_edit_blob_path(project, tree_join(branch, readme_file_path), legacy_render: 1)
-        fill_editor(content: "1. one\\n  - sublist\\n")
-        click_link 'Preview'
-        wait_for_requests
-
-        # the above generates a sublist list in RedCarpet
-        expect(page).to have_content("sublist")
-        expect(page).to have_xpath("//ol//li//ul")
-      end
     end
   end
 
diff --git a/spec/features/projects/wiki/markdown_preview_spec.rb b/spec/features/projects/wiki/markdown_preview_spec.rb
index f505023d1d0..3b469fee867 100644
--- a/spec/features/projects/wiki/markdown_preview_spec.rb
+++ b/spec/features/projects/wiki/markdown_preview_spec.rb
@@ -175,20 +175,6 @@ describe 'Projects > Wiki > User previews markdown changes', :js do
         expect(page).to have_content("sublist")
         expect(page).not_to have_xpath("//ol//li//ul")
       end
-
-      it 'renders content with RedCarpet when legacy_render is set' do
-        wiki_page = create(:wiki_page,
-                           wiki: project.wiki,
-                           attrs: { title: 'home', content: "Empty content" })
-        visit(project_wiki_edit_path(project, wiki_page, legacy_render: 1))
-
-        fill_in :wiki_content, with: "1. one\n  - sublist\n"
-        click_on "Preview"
-
-        # the above generates a sublist list in RedCarpet
-        expect(page).to have_content("sublist")
-        expect(page).to have_xpath("//ol//li//ul")
-      end
     end
   end
 
diff --git a/spec/features/snippets/show_spec.rb b/spec/features/snippets/show_spec.rb
index 367a479f62a..eb974c7c7fd 100644
--- a/spec/features/snippets/show_spec.rb
+++ b/spec/features/snippets/show_spec.rb
@@ -80,19 +80,6 @@ describe 'Snippet', :js do
           end
         end
 
-        context 'when rendering legacy markdown' do
-          before do
-            visit snippet_path(snippet, legacy_render: 1)
-
-            wait_for_requests
-          end
-
-          it 'renders using RedCarpet' do
-            expect(page).to have_content("sublist")
-            expect(page).to have_xpath("//ol//li//ul")
-          end
-        end
-
         context 'with cached CommonMark html' do
           let(:snippet) { create(:personal_snippet, :public, file_name: file_name, content: content, cached_markdown_version: CacheMarkdownField::CACHE_COMMONMARK_VERSION) }
 
@@ -100,14 +87,6 @@ describe 'Snippet', :js do
             expect(page).not_to have_xpath("//ol//li//ul")
           end
         end
-
-        context 'with cached Redcarpet html' do
-          let(:snippet) { create(:personal_snippet, :public, file_name: file_name, content: content, cached_markdown_version: CacheMarkdownField::CACHE_REDCARPET_VERSION) }
-
-          it 'renders correctly' do
-            expect(page).to have_xpath("//ol//li//ul")
-          end
-        end
       end
 
       context 'switching to the simple viewer' do
diff --git a/spec/features/task_lists_spec.rb b/spec/features/task_lists_spec.rb
index b549f2b5c62..6fe840dccf6 100644
--- a/spec/features/task_lists_spec.rb
+++ b/spec/features/task_lists_spec.rb
@@ -36,19 +36,6 @@ describe 'Task Lists' do
     MARKDOWN
   end
 
-  let(:nested_tasks_markdown_redcarpet) do
-    <<-EOT.strip_heredoc
-    - [ ] Task a
-      - [x] Task a.1
-      - [ ] Task a.2
-    - [ ] Task b
-
-    1. [ ] Task 1
-      1. [ ] Task 1.1
-      1. [x] Task 1.2
-    EOT
-  end
-
   let(:nested_tasks_markdown) do
     <<-EOT.strip_heredoc
     - [ ] Task a
@@ -153,61 +140,6 @@ describe 'Task Lists' do
         expect(page).to have_content("1 of 1 task completed")
       end
     end
-
-    shared_examples 'shared nested tasks' do
-      before do
-        allow(Banzai::Filter::MarkdownFilter).to receive(:engine).and_return('Redcarpet')
-        visit_issue(project, issue)
-      end
-      it 'renders' do
-        expect(page).to have_selector('ul.task-list',      count: 2)
-        expect(page).to have_selector('li.task-list-item', count: 7)
-        expect(page).to have_selector('ul input[checked]', count: 1)
-        expect(page).to have_selector('ol input[checked]', count: 1)
-      end
-
-      it 'solves tasks' do
-        expect(page).to have_content("2 of 7 tasks completed")
-
-        page.find('li.task-list-item', text: 'Task b').find('input').click
-        wait_for_requests
-        page.find('li.task-list-item ul li.task-list-item', text: 'Task a.2').find('input').click
-        wait_for_requests
-        page.find('li.task-list-item ol li.task-list-item', text: 'Task 1.1').find('input').click
-        wait_for_requests
-
-        expect(page).to have_content("5 of 7 tasks completed")
-
-        visit_issue(project, issue) # reload to see new system notes
-
-        expect(page).to have_content('marked the task Task b as complete')
-        expect(page).to have_content('marked the task Task a.2 as complete')
-        expect(page).to have_content('marked the task Task 1.1 as complete')
-      end
-    end
-
-    describe 'nested tasks', :js do
-      let(:cache_version) { CacheMarkdownField::CACHE_COMMONMARK_VERSION }
-      let!(:issue) do
-        create(:issue, description: nested_tasks_markdown, author: user, project: project,
-                       cached_markdown_version: cache_version)
-      end
-
-      before do
-        visit_issue(project, issue)
-      end
-
-      context 'with Redcarpet' do
-        let(:cache_version) { CacheMarkdownField::CACHE_REDCARPET_VERSION }
-        let(:nested_tasks_markdown) { nested_tasks_markdown_redcarpet }
-
-        it_behaves_like 'shared nested tasks'
-      end
-
-      context 'with CommonMark' do
-        it_behaves_like 'shared nested tasks'
-      end
-    end
   end
 
   describe 'for Notes' do