diff options
| -rw-r--r-- | lib/api/builds.rb | 2 | ||||
| -rw-r--r-- | spec/requests/api/builds_spec.rb | 5 |
2 files changed, 6 insertions, 1 deletions
diff --git a/lib/api/builds.rb b/lib/api/builds.rb index b265fa148ba..c058128ae54 100644 --- a/lib/api/builds.rb +++ b/lib/api/builds.rb @@ -138,7 +138,7 @@ module API return not_found!(build) unless build return forbidden!('Build is not erasable!') unless build.erasable? - build.erase + build.erase(erased_by: current_user) present build, with: Entities::Build, user_can_download_artifacts: can?(current_user, :download_build_artifacts, user_project) end diff --git a/spec/requests/api/builds_spec.rb b/spec/requests/api/builds_spec.rb index c634bb84b77..adbae5dcad8 100644 --- a/spec/requests/api/builds_spec.rb +++ b/spec/requests/api/builds_spec.rb @@ -184,6 +184,11 @@ describe API::API, api: true do expect(build.artifacts_file.exists?).to be_falsy expect(build.artifacts_metadata.exists?).to be_falsy end + + it 'should update build' do + expect(build.reload.erased_at).to be_truthy + expect(build.reload.erased_by).to eq user + end end context 'build is not erasable' do |