Merge branch 'acet-fix-double-note-render' into 'master'

Fix rendering double note issue

Closes #38139

See merge request gitlab-org/gitlab-ce!14429

app/assets/javascripts/notes/components/issue_comment_form.vue

@@ -97,6 +97,8 @@
     methods: {
       ...mapActions([
         'saveNote',
+        'stopPolling',
+        'restartPolling',
         'removePlaceholderNotes',
       ]),
       setIsSubmitButtonDisabled(note, isSubmitting) {
@@ -126,10 +128,13 @@
           this.isSubmitting = true;
           this.note = ''; // Empty textarea while being requested. Repopulate in catch
           this.resizeTextarea();
+          this.stopPolling();
 
           this.saveNote(noteData)
             .then((res) => {
               this.isSubmitting = false;
+              this.restartPolling();
+
               if (res.errors) {
                 if (res.errors.commands_only) {
                   this.discard();

app/assets/javascripts/notes/stores/actions.js

@@ -187,6 +187,14 @@ export const poll = ({ commit, state, getters }) => {
   });
 };
 
+export const stopPolling = () => {
+  eTagPoll.stop();
+};
+
+export const restartPolling = () => {
+  eTagPoll.restart();
+};
+
 export const fetchData = ({ commit, state, getters }) => {
   const requestData = { endpoint: state.notesData.notesPath, lastFetchedAt: state.lastFetchedAt };
 

app/assets/javascripts/notes/stores/mutations.js

@@ -5,6 +5,9 @@ import * as constants from '../constants';
 export default {
   [types.ADD_NEW_NOTE](state, note) {
     const { discussion_id, type } = note;
+    const [exists] = state.notes.filter(n => n.id === note.discussion_id);
+
+    if (!exists) {
       const noteData = {
         expanded: true,
         id: discussion_id,
@@ -14,6 +17,7 @@ export default {
       };
 
       state.notes.push(noteData);
+    }
   },
 
   [types.ADD_NEW_REPLY_TO_DISCUSSION](state, note) {

spec/javascripts/notes/stores/actions_spec.js

-
 import * as actions from '~/notes/stores/actions';
 import testAction from './helpers';
 import { discussionMock, notesDataMock, userDataMock, issueDataMock, individualNote } from '../mock_data';

spec/javascripts/notes/stores/mutation_spec.js

@@ -3,19 +3,31 @@ import { note, discussionMock, notesDataMock, userDataMock, issueDataMock, indiv
 
 describe('Mutation Notes Store', () => {
   describe('ADD_NEW_NOTE', () => {
-    it('should add a new note to an array of notes', () => {
-      const state = { notes: [] };
-      mutations.ADD_NEW_NOTE(state, note);
+    let state;
+    let noteData;
 
-      expect(state).toEqual({
-        notes: [{
+    beforeEach(() => {
+      state = { notes: [] };
+      noteData = {
         expanded: true,
         id: note.discussion_id,
         individual_note: true,
         notes: [note],
         reply_id: note.discussion_id,
-        }],
+      };
+      mutations.ADD_NEW_NOTE(state, note);
+    });
+
+    it('should add a new note to an array of notes', () => {
+      expect(state).toEqual({
+        notes: [noteData],
       });
+      expect(state.notes.length).toBe(1);
+    });
+
+    it('should not add the same note to the notes array', () => {
+      mutations.ADD_NEW_NOTE(state, note);
+      expect(state.notes.length).toBe(1);
     });
   });