diff options
author | Stan Hu <stanhu@gmail.com> | 2019-05-06 14:42:21 -0700 |
---|---|---|
committer | Stan Hu <stanhu@gmail.com> | 2019-05-06 21:21:56 -0700 |
commit | b7b852c250a95648d5954e0ead6c37b5ff8b35d9 (patch) | |
tree | b022cb42e2a902d09b5874469412cab53893c041 /spec/models/issue_spec.rb | |
parent | e1aa7f5074aab8190d16a6f25121d33b99a07238 (diff) | |
download | gitlab-ce-b7b852c250a95648d5954e0ead6c37b5ff8b35d9.tar.gz |
Additional fix to handle NULL lock_versionsh-fix-activerecord-patch-mark2
If the UI sends a string value for lock_version (e.g. "0"), then the
previous monkey patch did not properly handle that properly. This
commit casts the value to an integer to determine whether to look for
NULL lock_versions.
For merge requests, GitLab sends a POST request to
`namespace/project/merge_requests/:iid` with the
`merge_request[lock_version]` parameter with a string `0`. The string
value comes from the form field, which explains why
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/28145 wasn't
sufficient.
Diffstat (limited to 'spec/models/issue_spec.rb')
-rw-r--r-- | spec/models/issue_spec.rb | 21 |
1 files changed, 16 insertions, 5 deletions
diff --git a/spec/models/issue_spec.rb b/spec/models/issue_spec.rb index 0ce4add5669..cc777cbf749 100644 --- a/spec/models/issue_spec.rb +++ b/spec/models/issue_spec.rb @@ -56,14 +56,25 @@ describe Issue do end describe 'locking' do - it 'works when an issue has a NULL lock_version' do - issue = create(:issue) + using RSpec::Parameterized::TableSyntax - described_class.where(id: issue.id).update_all('lock_version = NULL') + where(:lock_version) do + [ + [0], + ["0"] + ] + end - issue.update!(lock_version: 0, title: 'locking test') + with_them do + it 'works when an issue has a NULL lock_version' do + issue = create(:issue) - expect(issue.reload.title).to eq('locking test') + described_class.where(id: issue.id).update_all('lock_version = NULL') + + issue.update!(lock_version: lock_version, title: 'locking test') + + expect(issue.reload.title).to eq('locking test') + end end end |