1 files changed, 75 insertions, 0 deletions
diff --git a/spec/lib/gitlab/github_import/importer/single_endpoint_diff_notes_importer_spec.rb b/spec/lib/gitlab/github_import/importer/single_endpoint_diff_notes_importer_spec.rb
new file mode 100644
index 00000000000..8c71d7d0ed7
--- /dev/null
+++ b/spec/lib/gitlab/github_import/importer/single_endpoint_diff_notes_importer_spec.rb
@@ -0,0 +1,75 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe Gitlab::GithubImport::Importer::SingleEndpointDiffNotesImporter do
+  let(:client) { double }
+  let(:project) { create(:project, import_source: 'github/repo') }
+
+  subject { described_class.new(project, client) }
+
+  it { is_expected.to include_module(Gitlab::GithubImport::ParallelScheduling) }
+  it { is_expected.to include_module(Gitlab::GithubImport::SingleEndpointNotesImporting) }
+  it { expect(subject.representation_class).to eq(Gitlab::GithubImport::Representation::DiffNote) }
+  it { expect(subject.importer_class).to eq(Gitlab::GithubImport::Importer::DiffNoteImporter) }
+  it { expect(subject.collection_method).to eq(:pull_request_comments) }
+  it { expect(subject.object_type).to eq(:diff_note) }
+  it { expect(subject.id_for_already_imported_cache(double(id: 1))).to eq(1) }
+
+  describe '#each_object_to_import', :clean_gitlab_redis_cache do
+    let(:merge_request) do
+      create(
+        :merged_merge_request,
+        iid: 999,
+        source_project: project,
+        target_project: project
+      )
+    end
+
+    let(:note) { double(id: 1) }
+    let(:page) { double(objects: [note], number: 1) }
+
+    it 'fetches data' do
+      expect(client)
+        .to receive(:each_page)
+        .exactly(:once) # ensure to be cached on the second call
+        .with(:pull_request_comments, 'github/repo', merge_request.iid, page: 1)
+        .and_yield(page)
+
+      expect { |b| subject.each_object_to_import(&b) }.to yield_with_args(note)
+
+      subject.each_object_to_import {}
+
+      expect(
+        Gitlab::Cache::Import::Caching.set_includes?(
+          "github-importer/merge_request/diff_notes/already-imported/#{project.id}",
+          merge_request.iid
+        )
+      ).to eq(true)
+    end
+
+    it 'skips cached pages' do
+      Gitlab::GithubImport::PageCounter
+        .new(project, "merge_request/#{merge_request.id}/pull_request_comments")
+        .set(2)
+
+      expect(client)
+        .to receive(:each_page)
+        .exactly(:once) # ensure to be cached on the second call
+        .with(:pull_request_comments, 'github/repo', merge_request.iid, page: 2)
+
+      subject.each_object_to_import {}
+    end
+
+    it 'skips cached merge requests' do
+      Gitlab::Cache::Import::Caching.set_add(
+        "github-importer/merge_request/diff_notes/already-imported/#{project.id}",
+        merge_request.iid
+      )
+
+      expect(client).not_to receive(:each_page)
+
+      subject.each_object_to_import {}
+    end
+  end
+end