diff options
author | Oswaldo Ferreira <oswaldo@gitlab.com> | 2019-02-19 16:29:10 -0300 |
---|---|---|
committer | Oswaldo Ferreira <oswaldo@gitlab.com> | 2019-06-05 08:38:09 -0300 |
commit | 5a89bcc4dae55c40bdd926797aba5e3f0ada05ae (patch) | |
tree | e64a364634faf196a3c59d40b50c437761d513d0 | |
parent | d9a761ed14ab5025626dee17faf1528c1264a1b0 (diff) | |
download | gitlab-ce-5a89bcc4dae55c40bdd926797aba5e3f0ada05ae.tar.gz |
Avoid 500's when commit is not reachable
-rw-r--r-- | app/models/diff_note.rb | 2 | ||||
-rw-r--r-- | changelogs/unreleased/osw-avoid-500-on-suggestions-check.yml | 5 | ||||
-rw-r--r-- | spec/models/diff_note_spec.rb | 8 |
3 files changed, 14 insertions, 1 deletions
diff --git a/app/models/diff_note.rb b/app/models/diff_note.rb index 1a87fc47c56..8221b7de2b6 100644 --- a/app/models/diff_note.rb +++ b/app/models/diff_note.rb @@ -77,7 +77,7 @@ class DiffNote < Note end def supports_suggestion? - return false unless noteable.supports_suggestion? && on_text? + return false unless noteable&.supports_suggestion? && on_text? # We don't want to trigger side-effects of `diff_file` call. return false unless file = latest_diff_file return false unless line = file.line_for_position(self.position) diff --git a/changelogs/unreleased/osw-avoid-500-on-suggestions-check.yml b/changelogs/unreleased/osw-avoid-500-on-suggestions-check.yml new file mode 100644 index 00000000000..d0a09385d4c --- /dev/null +++ b/changelogs/unreleased/osw-avoid-500-on-suggestions-check.yml @@ -0,0 +1,5 @@ +--- +title: Avoid 500 when rendering users ATOM data +merge_request: 25408 +author: +type: fixed diff --git a/spec/models/diff_note_spec.rb b/spec/models/diff_note_spec.rb index fa19cb47a0d..d9e1fe4b165 100644 --- a/spec/models/diff_note_spec.rb +++ b/spec/models/diff_note_spec.rb @@ -321,6 +321,14 @@ describe DiffNote do end describe '#supports_suggestion?' do + context 'when noteable does not exist' do + it 'returns false' do + allow(subject).to receive(:noteable) { nil } + + expect(subject.supports_suggestion?).to be(false) + end + end + context 'when noteable does not support suggestions' do it 'returns false' do allow(subject.noteable).to receive(:supports_suggestion?) { false } |