summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRémy Coutable <remy@rymai.me>2018-06-18 08:58:23 +0000
committerRémy Coutable <remy@rymai.me>2018-06-18 08:58:23 +0000
commit99fdbe4c230205510e1d7061e68bb1a003913d4b (patch)
tree05ccc7609dd5a3e48a938a0cd41770784d6c47ae
parentb7700e0963675ce4dde85d4f3fe13e0e047a1950 (diff)
parentf1955962d91eb25a847553b2900068cd9487d0be (diff)
downloadgitlab-ce-99fdbe4c230205510e1d7061e68bb1a003913d4b.tar.gz
Merge branch 'rails5-fix-47960' into 'master'
Rails5 fix update_attribute usage not causing a save Closes #47960 See merge request gitlab-org/gitlab-ce!19881
-rw-r--r--changelogs/unreleased/rails5-fix-47960.yml5
-rw-r--r--spec/controllers/projects/imports_controller_spec.rb8
2 files changed, 9 insertions, 4 deletions
diff --git a/changelogs/unreleased/rails5-fix-47960.yml b/changelogs/unreleased/rails5-fix-47960.yml
new file mode 100644
index 00000000000..2229511ccd6
--- /dev/null
+++ b/changelogs/unreleased/rails5-fix-47960.yml
@@ -0,0 +1,5 @@
+---
+title: Rails5 fix update_attribute usage not causing a save
+merge_request: 19881
+author: Jasper Maes
+type: fixed
diff --git a/spec/controllers/projects/imports_controller_spec.rb b/spec/controllers/projects/imports_controller_spec.rb
index 011843baffc..812833cc86b 100644
--- a/spec/controllers/projects/imports_controller_spec.rb
+++ b/spec/controllers/projects/imports_controller_spec.rb
@@ -29,7 +29,7 @@ describe Projects::ImportsController do
context 'when import is in progress' do
before do
- project.update_attribute(:import_status, :started)
+ project.update_attributes(import_status: :started)
end
it 'renders template' do
@@ -47,7 +47,7 @@ describe Projects::ImportsController do
context 'when import failed' do
before do
- project.update_attribute(:import_status, :failed)
+ project.update_attributes(import_status: :failed)
end
it 'redirects to new_namespace_project_import_path' do
@@ -59,7 +59,7 @@ describe Projects::ImportsController do
context 'when import finished' do
before do
- project.update_attribute(:import_status, :finished)
+ project.update_attributes(import_status: :finished)
end
context 'when project is a fork' do
@@ -108,7 +108,7 @@ describe Projects::ImportsController do
context 'when import never happened' do
before do
- project.update_attribute(:import_status, :none)
+ project.update_attributes(import_status: :none)
end
it 'redirects to namespace_project_path' do