From 2f130527c0959df1a79d8a4bec648b8f3ead73ca Mon Sep 17 00:00:00 2001 From: "Luke \"Jared\" Bennett" Date: Fri, 27 Oct 2017 20:42:57 +0100 Subject: FE review changes - remove internal details assertion and look for closest form if no discussion_line_code --- app/assets/javascripts/notes.js | 2 +- spec/javascripts/notes_spec.js | 4 ---- 2 files changed, 1 insertion(+), 5 deletions(-) diff --git a/app/assets/javascripts/notes.js b/app/assets/javascripts/notes.js index 7f163e7e0e8..b111c117556 100644 --- a/app/assets/javascripts/notes.js +++ b/app/assets/javascripts/notes.js @@ -415,7 +415,7 @@ export default class Notes { this.note_ids.push(noteEntity.id); form = $form || $(`.js-discussion-note-form[data-discussion-id="${noteEntity.discussion_id}"]`); - row = form.length ? form.closest('tr') : $(`#${noteEntity.discussion_line_code}`); + row = (form.length || !noteEntity.discussion_line_code) ? form.closest('tr') : $(`#${noteEntity.discussion_line_code}`); if (noteEntity.on_image) { row = form; diff --git a/spec/javascripts/notes_spec.js b/spec/javascripts/notes_spec.js index 8514c7ac5a1..d7aa7a3d9e6 100644 --- a/spec/javascripts/notes_spec.js +++ b/spec/javascripts/notes_spec.js @@ -380,13 +380,9 @@ import '~/notes'; document.body.appendChild(line); $form.closest.and.returnValues($form); - spyOn(document, 'getElementById').and.callThrough(); - spyOn($.fn, 'after').and.callThrough(); Notes.prototype.renderDiscussionNote.call(notes, note, $form); - expect(document.getElementById).toHaveBeenCalledWith(note.discussion_line_code); - expect($.fn.after).toHaveBeenCalled(); expect(line.nextSibling.outerHTML).toEqual(note.diff_discussion_html); }); }); -- cgit v1.2.1