diff options
| author | Rémy Coutable <remy@rymai.me> | 2016-06-17 15:08:59 +0000 |
|---|---|---|
| committer | Rémy Coutable <remy@rymai.me> | 2016-06-17 15:08:59 +0000 |
| commit | 439e154c1d635128c23ca1bbb46be2225bb0b18c (patch) | |
| tree | 987088c86e02d2de2abe897210676e575c8eff66 /spec/controllers/commit_controller_spec.rb | |
| parent | 2d622d44fd4e08536d6a79cf480da8e751dae1bc (diff) | |
| parent | d63673d6df703664aa95909c723df08449c573b5 (diff) | |
| download | gitlab-ce-439e154c1d635128c23ca1bbb46be2225bb0b18c.tar.gz | |
Merge branch 'make-sure-that-artifacts-file-is-saved' into 'master'
Make sure that artifacts_file is nullified after removing artifacts
## What does this MR do?
Fixes a problem that `ExpireBuildArtifactsWorker` is executed for all previously removed artifacts.
## Why was this MR needed?
Currently the `Ci::Build::erase_artifacts!` doesn't ensure that data are saved to database.
The bang at end of this method lets you believe that it should do so.
This adds a missing `save` to this method.
## What are the relevant issue numbers?
None, yet.
## CHANGELOG
Since this is regression in feature introduce in RC4 no CHANGELOG entry is needed.
cc @grzesiek
See merge request !4741
Diffstat (limited to 'spec/controllers/commit_controller_spec.rb')
0 files changed, 0 insertions, 0 deletions