1 files changed, 107 insertions, 0 deletions
diff --git a/spec/services/packages/mark_packages_for_destruction_service_spec.rb b/spec/services/packages/mark_packages_for_destruction_service_spec.rb
new file mode 100644
index 00000000000..5c043b89de8
--- /dev/null
+++ b/spec/services/packages/mark_packages_for_destruction_service_spec.rb
@@ -0,0 +1,107 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe Packages::MarkPackagesForDestructionService, :sidekiq_inline do
+  let_it_be(:project) { create(:project) }
+  let_it_be_with_reload(:packages) { create_list(:npm_package, 3, project: project) }
+
+  let(:user) { project.owner }
+
+  # The service only accepts ActiveRecord relationships and not arrays.
+  let(:service) { described_class.new(packages: ::Packages::Package.id_in(package_ids), current_user: user) }
+  let(:package_ids) { packages.map(&:id) }
+
+  describe '#execute' do
+    subject { service.execute }
+
+    context 'when the user is authorized' do
+      before do
+        project.add_maintainer(user)
+      end
+
+      context 'when it is successful' do
+        it 'marks the packages as pending destruction' do
+          expect(::Packages::Maven::Metadata::SyncService).not_to receive(:new)
+
+          expect { subject }.to change { ::Packages::Package.pending_destruction.count }.from(0).to(3)
+                                  .and change { Packages::PackageFile.pending_destruction.count }.from(0).to(3)
+          packages.each { |pkg| expect(pkg.reload).to be_pending_destruction }
+
+          expect(subject).to be_a(ServiceResponse)
+          expect(subject).to be_success
+          expect(subject.message).to eq('Packages were successfully marked as pending destruction')
+        end
+
+        context 'with maven packages' do
+          let_it_be_with_reload(:packages) { create_list(:maven_package, 3, project: project) }
+
+          it 'marks the packages as pending destruction' do
+            expect(::Packages::Maven::Metadata::SyncService).to receive(:new).once.and_call_original
+
+            expect { subject }.to change { ::Packages::Package.pending_destruction.count }.from(0).to(3)
+                                    .and change { Packages::PackageFile.pending_destruction.count }.from(0).to(9)
+            packages.each { |pkg| expect(pkg.reload).to be_pending_destruction }
+
+            expect(subject).to be_a(ServiceResponse)
+            expect(subject).to be_success
+            expect(subject.message).to eq('Packages were successfully marked as pending destruction')
+          end
+
+          context 'without version' do
+            before do
+              ::Packages::Package.id_in(package_ids).update_all(version: nil)
+            end
+
+            it 'marks the packages as pending destruction' do
+              expect(::Packages::Maven::Metadata::SyncService).not_to receive(:new)
+
+              expect { subject }.to change { ::Packages::Package.pending_destruction.count }.from(0).to(3)
+                                      .and change { Packages::PackageFile.pending_destruction.count }.from(0).to(9)
+              packages.each { |pkg| expect(pkg.reload).to be_pending_destruction }
+
+              expect(subject).to be_a(ServiceResponse)
+              expect(subject).to be_success
+              expect(subject.message).to eq('Packages were successfully marked as pending destruction')
+            end
+          end
+        end
+      end
+
+      context 'when it is not successful' do
+        before do
+          allow(service).to receive(:can_destroy_packages?).and_raise(StandardError, 'test')
+        end
+
+        it 'returns an error ServiceResponse' do
+          expect(::Packages::Maven::Metadata::SyncService).not_to receive(:new)
+
+          expect { subject }.to not_change { ::Packages::Package.pending_destruction.count }
+                                  .and not_change { ::Packages::PackageFile.pending_destruction.count }
+
+          expect(subject).to be_a(ServiceResponse)
+          expect(subject).to be_error
+          expect(subject.message).to eq("Failed to mark the packages as pending destruction")
+          expect(subject.status).to eq(:error)
+        end
+      end
+    end
+
+    context 'when the user is not authorized' do
+      let(:user) { nil }
+
+      it 'returns an error ServiceResponse' do
+        expect(::Packages::Maven::Metadata::SyncService).not_to receive(:new)
+
+        expect { subject }.to not_change { ::Packages::Package.pending_destruction.count }
+                                .and not_change { ::Packages::PackageFile.pending_destruction.count }
+
+        expect(subject).to be_a(ServiceResponse)
+        expect(subject).to be_error
+        expect(subject.message).to eq("You don't have the permission to perform this action")
+        expect(subject.status).to eq(:error)
+        expect(subject.reason).to eq(:unauthorized)
+      end
+    end
+  end
+end