Merge branch '241639-delete-manifest-api' into 'master'

Delete manifests in dependency proxy purge api

See merge request gitlab-org/gitlab!49056

app/workers/purge_dependency_proxy_cache_worker.rb

@@ -15,6 +15,7 @@ class PurgeDependencyProxyCacheWorker
     return unless valid?
 
     @group.dependency_proxy_blobs.destroy_all # rubocop:disable Cop/DestroyAll
+    @group.dependency_proxy_manifests.destroy_all # rubocop:disable Cop/DestroyAll
   end
 
   private

changelogs/unreleased/241639-delete-manifest-api.yml

+---
+title: Delete manifests when purging the dependency proxy using the API
+merge_request: 49056
+author:
+type: changed

doc/api/dependency_proxy.md

@@ -11,7 +11,7 @@ info: To determine the technical writer assigned to the Stage/Group associated w
 > - [Introduced](https://gitlab.com/gitlab-org/gitlab/-/issues/11631) in GitLab 12.10.
 > - [Moved](https://gitlab.com/gitlab-org/gitlab/-/issues/273655) to [GitLab Core](https://about.gitlab.com/pricing/) in GitLab 13.6.
 
-Deletes the cached blobs for a group. This endpoint requires group admin access.
+Deletes the cached manifests and blobs for a group. This endpoint requires group admin access.
 
 WARNING:
 [A bug exists](https://gitlab.com/gitlab-org/gitlab/-/issues/277161) for this API.

spec/workers/purge_dependency_proxy_cache_worker_spec.rb

@@ -6,6 +6,7 @@ RSpec.describe PurgeDependencyProxyCacheWorker do
   let_it_be(:user) { create(:admin) }
   let_it_be(:blob) { create(:dependency_proxy_blob )}
   let_it_be(:group, reload: true) { blob.group }
+  let_it_be(:manifest) { create(:dependency_proxy_manifest, group: group )}
   let_it_be(:group_id) { group.id }
 
   subject { described_class.new.perform(user.id, group_id) }
@@ -17,8 +18,9 @@ RSpec.describe PurgeDependencyProxyCacheWorker do
 
   describe '#perform' do
     shared_examples 'returns nil' do
-      it 'returns nil' do
+      it 'returns nil', :aggregate_failures do
         expect { subject }.not_to change { group.dependency_proxy_blobs.size }
+        expect { subject }.not_to change { group.dependency_proxy_manifests.size }
         expect(subject).to be_nil
       end
     end
@@ -27,12 +29,14 @@ RSpec.describe PurgeDependencyProxyCacheWorker do
       include_examples 'an idempotent worker' do
         let(:job_args) { [user.id, group_id] }
 
-        it 'deletes the blobs and returns ok' do
+        it 'deletes the blobs and returns ok', :aggregate_failures do
           expect(group.dependency_proxy_blobs.size).to eq(1)
+          expect(group.dependency_proxy_manifests.size).to eq(1)
 
           subject
 
           expect(group.dependency_proxy_blobs.size).to eq(0)
+          expect(group.dependency_proxy_manifests.size).to eq(0)
         end
       end
     end