Merge branch 'master' into 38869-datetime

* master: (85 commits)
  Export old code into es6 modules
  Use relative URL for projects to avoid storing domains
  Fix spec by avoiding monkeypatching
  Return the noteable in Note#touch_noteable
  add CHANGELOG.md entry for !15889
  fix broken empty state assets for environment monitoring page
  removed tab indexes from tag form
  Refactor entrypoint override docs
  Present member collection at the controller level
  Adds i18n
  check the import_status field before doing SQL operations to check the import url
  Fix rubocop offence
  Fix entrypoint overriding documentation
  Remove unnecessary js-issuable-edit
  Use memoization for commits on diffs
  Move Repository#write_ref to Git::Repository#write_ref
  Treat empty markdown and html strings as valid cached text, not missing cache that needs to be updated
  Center cluster title
  Remove noisy notification from QA base page
  Remove unused page from hashed storage QA scenario
  ...

14 files changed, 363 insertions, 246 deletions

diff --git a/spec/javascripts/image_diff/helpers/badge_helper_spec.js b/spec/javascripts/image_diff/helpers/badge_helper_spec.js
index fb9c7e59031..ce3add1fd90 100644
--- a/spec/javascripts/image_diff/helpers/badge_helper_spec.js
+++ b/spec/javascripts/image_diff/helpers/badge_helper_spec.js
@@ -89,15 +89,8 @@ describe('badge helper', () => {
     });
 
     it('should create icon comment button', () => {
-      const iconEl = buttonEl.querySelector('i');
+      const iconEl = buttonEl.querySelector('svg');
       expect(iconEl).toBeDefined();
-      expect(iconEl.classList.contains('fa')).toEqual(true);
-      expect(iconEl.classList.contains('fa-comment-o')).toEqual(true);
-    });
-
-    it('should have .image-comment-badge.inverted in button class', () => {
-      expect(buttonEl.classList.contains('image-comment-badge')).toEqual(true);
-      expect(buttonEl.classList.contains('inverted')).toEqual(true);
     });
   });
 
diff --git a/spec/javascripts/notes/components/issue_comment_form_spec.js b/spec/javascripts/notes/components/comment_form_spec.js
index 04a7f8e32f1..20e352dd8bd 100644
--- a/spec/javascripts/notes/components/issue_comment_form_spec.js
+++ b/spec/javascripts/notes/components/comment_form_spec.js
@@ -1,7 +1,7 @@
 import Vue from 'vue';
 import Autosize from 'autosize';
 import store from '~/notes/stores';
-import issueCommentForm from '~/notes/components/issue_comment_form.vue';
+import issueCommentForm from '~/notes/components/comment_form.vue';
 import { loggedOutnoteableData, notesDataMock, userDataMock, noteableDataMock } from '../mock_data';
 import { keyboardDownEvent } from '../../issue_show/helpers';
 
diff --git a/spec/javascripts/notes/components/issue_note_app_spec.js b/spec/javascripts/notes/components/note_app_spec.js
index 8e43037f356..7c8d6685ee1 100644
--- a/spec/javascripts/notes/components/issue_note_app_spec.js
+++ b/spec/javascripts/notes/components/note_app_spec.js
@@ -1,26 +1,15 @@
 import Vue from 'vue';
-import issueNotesApp from '~/notes/components/issue_notes_app.vue';
+import notesApp from '~/notes/components/notes_app.vue';
 import service from '~/notes/services/notes_service';
 import * as mockData from '../mock_data';
+import getSetTimeoutPromise from '../../helpers/set_timeout_promise_helper';
 
-describe('issue_note_app', () => {
+describe('note_app', () => {
   let mountComponent;
   let vm;
 
-  const individualNoteInterceptor = (request, next) => {
-    next(request.respondWith(JSON.stringify(mockData.individualNoteServerResponse), {
-      status: 200,
-    }));
-  };
-
-  const discussionNoteInterceptor = (request, next) => {
-    next(request.respondWith(JSON.stringify(mockData.discussionNoteServerResponse), {
-      status: 200,
-    }));
-  };
-
   beforeEach(() => {
-    const IssueNotesApp = Vue.extend(issueNotesApp);
+    const IssueNotesApp = Vue.extend(notesApp);
 
     mountComponent = (data) => {
       const props = data || {
@@ -74,16 +63,16 @@ describe('issue_note_app', () => {
 
   describe('render', () => {
     beforeEach(() => {
-      Vue.http.interceptors.push(individualNoteInterceptor);
+      Vue.http.interceptors.push(mockData.individualNoteInterceptor);
       vm = mountComponent();
     });
 
     afterEach(() => {
-      Vue.http.interceptors = _.without(Vue.http.interceptors, individualNoteInterceptor);
+      Vue.http.interceptors = _.without(Vue.http.interceptors, mockData.individualNoteInterceptor);
     });
 
     it('should render list of notes', (done) => {
-      const note = mockData.individualNoteServerResponse[0].notes[0];
+      const note = mockData.INDIVIDUAL_NOTE_RESPONSE_MAP.GET['/gitlab-org/gitlab-ce/issues/26/discussions.json'][0].notes[0];
 
       setTimeout(() => {
         expect(
@@ -129,13 +118,16 @@ describe('issue_note_app', () => {
   describe('update note', () => {
     describe('individual note', () => {
       beforeEach(() => {
-        Vue.http.interceptors.push(individualNoteInterceptor);
-        spyOn(service, 'updateNote').and.callFake(() => Promise.resolve());
+        Vue.http.interceptors.push(mockData.individualNoteInterceptor);
+        spyOn(service, 'updateNote').and.callThrough();
         vm = mountComponent();
       });
 
       afterEach(() => {
-        Vue.http.interceptors = _.without(Vue.http.interceptors, individualNoteInterceptor);
+        Vue.http.interceptors = _.without(
+          Vue.http.interceptors,
+          mockData.individualNoteInterceptor,
+        );
       });
 
       it('renders edit form', (done) => {
@@ -149,28 +141,36 @@ describe('issue_note_app', () => {
       });
 
       it('calls the service to update the note', (done) => {
-        setTimeout(() => {
-          vm.$el.querySelector('.js-note-edit').click();
-          Vue.nextTick(() => {
+        getSetTimeoutPromise()
+          .then(() => {
+            vm.$el.querySelector('.js-note-edit').click();
+          })
+          .then(Vue.nextTick)
+          .then(() => {
             vm.$el.querySelector('.js-vue-issue-note-form').value = 'this is a note';
             vm.$el.querySelector('.js-vue-issue-save').click();
 
             expect(service.updateNote).toHaveBeenCalled();
-            done();
-          });
-        }, 0);
+          })
+          // Wait for the requests to finish before destroying
+          .then(Vue.nextTick)
+          .then(done)
+          .catch(done.fail);
       });
     });
 
     describe('dicussion note', () => {
       beforeEach(() => {
-        Vue.http.interceptors.push(discussionNoteInterceptor);
-        spyOn(service, 'updateNote').and.callFake(() => Promise.resolve());
+        Vue.http.interceptors.push(mockData.discussionNoteInterceptor);
+        spyOn(service, 'updateNote').and.callThrough();
         vm = mountComponent();
       });
 
       afterEach(() => {
-        Vue.http.interceptors = _.without(Vue.http.interceptors, discussionNoteInterceptor);
+        Vue.http.interceptors = _.without(
+          Vue.http.interceptors,
+          mockData.discussionNoteInterceptor,
+        );
       });
 
       it('renders edit form', (done) => {
@@ -184,16 +184,21 @@ describe('issue_note_app', () => {
       });
 
       it('updates the note and resets the edit form', (done) => {
-        setTimeout(() => {
-          vm.$el.querySelector('.js-note-edit').click();
-          Vue.nextTick(() => {
+        getSetTimeoutPromise()
+          .then(() => {
+            vm.$el.querySelector('.js-note-edit').click();
+          })
+          .then(Vue.nextTick)
+          .then(() => {
             vm.$el.querySelector('.js-vue-issue-note-form').value = 'this is a note';
             vm.$el.querySelector('.js-vue-issue-save').click();
 
             expect(service.updateNote).toHaveBeenCalled();
-            done();
-          });
-        }, 0);
+          })
+          // Wait for the requests to finish before destroying
+          .then(Vue.nextTick)
+          .then(done)
+          .catch(done.fail);
       });
     });
   });
@@ -216,12 +221,12 @@ describe('issue_note_app', () => {
 
   describe('edit form', () => {
     beforeEach(() => {
-      Vue.http.interceptors.push(individualNoteInterceptor);
+      Vue.http.interceptors.push(mockData.individualNoteInterceptor);
       vm = mountComponent();
     });
 
     afterEach(() => {
-      Vue.http.interceptors = _.without(Vue.http.interceptors, individualNoteInterceptor);
+      Vue.http.interceptors = _.without(Vue.http.interceptors, mockData.individualNoteInterceptor);
     });
 
     it('should render markdown docs url', (done) => {
diff --git a/spec/javascripts/notes/components/issue_note_body_spec.js b/spec/javascripts/notes/components/note_body_spec.js
index 37aad50737b..b42e7943b98 100644
--- a/spec/javascripts/notes/components/issue_note_body_spec.js
+++ b/spec/javascripts/notes/components/note_body_spec.js
@@ -1,7 +1,7 @@
 
 import Vue from 'vue';
 import store from '~/notes/stores';
-import noteBody from '~/notes/components/issue_note_body.vue';
+import noteBody from '~/notes/components/note_body.vue';
 import { noteableDataMock, notesDataMock, note } from '../mock_data';
 
 describe('issue_note_body component', () => {
diff --git a/spec/javascripts/notes/components/issue_note_form_spec.js b/spec/javascripts/notes/components/note_form_spec.js
index d42ef239711..86e9e2a32a9 100644
--- a/spec/javascripts/notes/components/issue_note_form_spec.js
+++ b/spec/javascripts/notes/components/note_form_spec.js
@@ -1,6 +1,6 @@
 import Vue from 'vue';
 import store from '~/notes/stores';
-import issueNoteForm from '~/notes/components/issue_note_form.vue';
+import issueNoteForm from '~/notes/components/note_form.vue';
 import { noteableDataMock, notesDataMock } from '../mock_data';
 import { keyboardDownEvent } from '../../issue_show/helpers';
 
diff --git a/spec/javascripts/notes/components/issue_discussion_spec.js b/spec/javascripts/notes/components/noteable_discussion_spec.js
index b6ae55d44f5..19504e4f7c8 100644
--- a/spec/javascripts/notes/components/issue_discussion_spec.js
+++ b/spec/javascripts/notes/components/noteable_discussion_spec.js
@@ -1,6 +1,6 @@
 import Vue from 'vue';
 import store from '~/notes/stores';
-import issueDiscussion from '~/notes/components/issue_discussion.vue';
+import issueDiscussion from '~/notes/components/noteable_discussion.vue';
 import { noteableDataMock, discussionMock, notesDataMock } from '../mock_data';
 
 describe('issue_discussion component', () => {
@@ -30,7 +30,7 @@ describe('issue_discussion component', () => {
 
   it('should render discussion header', () => {
     expect(vm.$el.querySelector('.discussion-header')).toBeDefined();
-    expect(vm.$el.querySelectorAll('.notes li').length).toEqual(discussionMock.notes.length);
+    expect(vm.$el.querySelector('.notes').children.length).toEqual(discussionMock.notes.length);
   });
 
   describe('actions', () => {
diff --git a/spec/javascripts/notes/components/issue_note_spec.js b/spec/javascripts/notes/components/noteable_note_spec.js
index bd888b2cbae..c8a6cb7e612 100644
--- a/spec/javascripts/notes/components/issue_note_spec.js
+++ b/spec/javascripts/notes/components/noteable_note_spec.js
@@ -1,7 +1,7 @@
 
 import Vue from 'vue';
 import store from '~/notes/stores';
-import issueNote from '~/notes/components/issue_note.vue';
+import issueNote from '~/notes/components/noteable_note.vue';
 import { noteableDataMock, notesDataMock, note } from '../mock_data';
 
 describe('issue_note', () => {
diff --git a/spec/javascripts/notes/mock_data.js b/spec/javascripts/notes/mock_data.js
index 42497de3c55..6b608adff15 100644
--- a/spec/javascripts/notes/mock_data.js
+++ b/spec/javascripts/notes/mock_data.js
@@ -312,138 +312,212 @@ export const loggedOutnoteableData = {
   "preview_note_path": "/gitlab-org/gitlab-ce/preview_markdown?quick_actions_target_id=98&quick_actions_target_type=Issue"
 }
 
-export const individualNoteServerResponse = [{
-  "id": "0fb4e0e3f9276e55ff32eb4195add694aece4edd",
-  "reply_id": "0fb4e0e3f9276e55ff32eb4195add694aece4edd",
-  "expanded": true,
-  "notes": [{
-    "id": 1390,
-    "attachment": {
-      "url": null,
-      "filename": null,
-      "image": false
-    },
-    "author": {
-      "id": 1,
-      "name": "Root",
-      "username": "root",
-      "state": "active",
-      "avatar_url": null,
-      "path": "/root"
-    },
-    "created_at": "2017-08-01T17:09:33.762Z",
-    "updated_at": "2017-08-01T17:09:33.762Z",
-    "system": false,
-    "noteable_id": 98,
-    "noteable_type": "Issue",
-    "type": null,
-    "human_access": "Owner",
-    "note": "sdfdsaf",
-    "note_html": "\u003cp dir=\"auto\"\u003esdfdsaf\u003c/p\u003e",
-    "current_user": {
-      "can_edit": true
+export const INDIVIDUAL_NOTE_RESPONSE_MAP = {
+  'GET': {
+    '/gitlab-org/gitlab-ce/issues/26/discussions.json': [{
+      "id": "0fb4e0e3f9276e55ff32eb4195add694aece4edd",
+      "reply_id": "0fb4e0e3f9276e55ff32eb4195add694aece4edd",
+      "expanded": true,
+      "notes": [{
+        "id": 1390,
+        "attachment": {
+          "url": null,
+          "filename": null,
+          "image": false
+        },
+        "author": {
+          "id": 1,
+          "name": "Root",
+          "username": "root",
+          "state": "active",
+          "avatar_url": null,
+          "path": "/root"
+        },
+        "created_at": "2017-08-01T17:09:33.762Z",
+        "updated_at": "2017-08-01T17:09:33.762Z",
+        "system": false,
+        "noteable_id": 98,
+        "noteable_type": "Issue",
+        "type": null,
+        "human_access": "Owner",
+        "note": "sdfdsaf",
+        "note_html": "\u003cp dir=\"auto\"\u003esdfdsaf\u003c/p\u003e",
+        "current_user": {
+          "can_edit": true
+        },
+        "discussion_id": "0fb4e0e3f9276e55ff32eb4195add694aece4edd",
+        "emoji_awardable": true,
+        "award_emoji": [{
+          "name": "baseball",
+          "user": {
+            "id": 1,
+            "name": "Root",
+            "username": "root"
+          }
+        }, {
+          "name": "art",
+          "user": {
+            "id": 1,
+            "name": "Root",
+            "username": "root"
+          }
+        }],
+        "toggle_award_path": "/gitlab-org/gitlab-ce/notes/1390/toggle_award_emoji",
+        "report_abuse_path": "/abuse_reports/new?ref_url=http%3A%2F%2Flocalhost%3A3000%2Fgitlab-org%2Fgitlab-ce%2Fissues%2F26%23note_1390\u0026user_id=1",
+        "path": "/gitlab-org/gitlab-ce/notes/1390"
+      }],
+      "individual_note": true
+      }, {
+      "id": "70d5c92a4039a36c70100c6691c18c27e4b0a790",
+      "reply_id": "70d5c92a4039a36c70100c6691c18c27e4b0a790",
+      "expanded": true,
+      "notes": [{
+        "id": 1391,
+        "attachment": {
+          "url": null,
+          "filename": null,
+          "image": false
+        },
+        "author": {
+          "id": 1,
+          "name": "Root",
+          "username": "root",
+          "state": "active",
+          "avatar_url": null,
+          "path": "/root"
+        },
+        "created_at": "2017-08-02T10:51:38.685Z",
+        "updated_at": "2017-08-02T10:51:38.685Z",
+        "system": false,
+        "noteable_id": 98,
+        "noteable_type": "Issue",
+        "type": null,
+        "human_access": "Owner",
+        "note": "New note!",
+        "note_html": "\u003cp dir=\"auto\"\u003eNew note!\u003c/p\u003e",
+        "current_user": {
+          "can_edit": true
+        },
+        "discussion_id": "70d5c92a4039a36c70100c6691c18c27e4b0a790",
+        "emoji_awardable": true,
+        "award_emoji": [],
+        "toggle_award_path": "/gitlab-org/gitlab-ce/notes/1391/toggle_award_emoji",
+        "report_abuse_path": "/abuse_reports/new?ref_url=http%3A%2F%2Flocalhost%3A3000%2Fgitlab-org%2Fgitlab-ce%2Fissues%2F26%23note_1391\u0026user_id=1",
+        "path": "/gitlab-org/gitlab-ce/notes/1391"
+      }],
+      "individual_note": true
+    }],
+    '/gitlab-org/gitlab-ce/noteable/issue/98/notes': {
+      last_fetched_at: 1512900838,
+      notes: [],
     },
-    "discussion_id": "0fb4e0e3f9276e55ff32eb4195add694aece4edd",
-    "emoji_awardable": true,
-    "award_emoji": [{
-      "name": "baseball",
-      "user": {
+  },
+  'PUT': {
+    '/gitlab-org/gitlab-ce/notes/1471': {
+      "commands_changes": null,
+      "valid": true,
+      "id": 1471,
+      "attachment": null,
+      "author": {
         "id": 1,
         "name": "Root",
-        "username": "root"
-      }
-    }, {
-      "name": "art",
-      "user": {
+        "username": "root",
+        "state": "active",
+        "avatar_url": null,
+        "path": "/root"
+      },
+      "created_at": "2017-08-08T16:53:00.666Z",
+      "updated_at": "2017-12-10T11:03:21.876Z",
+      "system": false,
+      "noteable_id": 124,
+      "noteable_type": "Issue",
+      "noteable_iid": 29,
+      "type": "DiscussionNote",
+      "human_access": "Owner",
+      "note": "Adding a comment",
+      "note_html": "\u003cp dir=\"auto\"\u003eAdding a comment\u003c/p\u003e",
+      "last_edited_at": "2017-12-10T11:03:21.876Z",
+      "last_edited_by": {
         "id": 1,
-        "name": "Root",
-        "username": "root"
-      }
-    }],
-    "toggle_award_path": "/gitlab-org/gitlab-ce/notes/1390/toggle_award_emoji",
-    "report_abuse_path": "/abuse_reports/new?ref_url=http%3A%2F%2Flocalhost%3A3000%2Fgitlab-org%2Fgitlab-ce%2Fissues%2F26%23note_1390\u0026user_id=1",
-    "path": "/gitlab-org/gitlab-ce/notes/1390"
-  }],
-  "individual_note": true
-  }, {
-  "id": "70d5c92a4039a36c70100c6691c18c27e4b0a790",
-  "reply_id": "70d5c92a4039a36c70100c6691c18c27e4b0a790",
-  "expanded": true,
-  "notes": [{
-    "id": 1391,
-    "attachment": {
-      "url": null,
-      "filename": null,
-      "image": false
-    },
-    "author": {
-      "id": 1,
-      "name": "Root",
-      "username": "root",
-      "state": "active",
-      "avatar_url": null,
-      "path": "/root"
-    },
-    "created_at": "2017-08-02T10:51:38.685Z",
-    "updated_at": "2017-08-02T10:51:38.685Z",
-    "system": false,
-    "noteable_id": 98,
-    "noteable_type": "Issue",
-    "type": null,
-    "human_access": "Owner",
-    "note": "New note!",
-    "note_html": "\u003cp dir=\"auto\"\u003eNew note!\u003c/p\u003e",
-    "current_user": {
-      "can_edit": true
+        "name": 'Root',
+        "username": 'root',
+        "state": 'active',
+        "avatar_url": null,
+        "path": '/root',
+      },
+      "current_user": {
+        "can_edit": true
+      },
+      "discussion_id": "a3ed36e29b1957efb3b68c53e2d7a2b24b1df052",
+      "emoji_awardable": true,
+      "award_emoji": [],
+      "toggle_award_path": "/gitlab-org/gitlab-ce/notes/1471/toggle_award_emoji",
+      "report_abuse_path": "/abuse_reports/new?ref_url=http%3A%2F%2Flocalhost%3A3000%2Fgitlab-org%2Fgitlab-ce%2Fissues%2F29%23note_1471\u0026user_id=1",
+      "path": "/gitlab-org/gitlab-ce/notes/1471"
     },
-    "discussion_id": "70d5c92a4039a36c70100c6691c18c27e4b0a790",
-    "emoji_awardable": true,
-    "award_emoji": [],
-    "toggle_award_path": "/gitlab-org/gitlab-ce/notes/1391/toggle_award_emoji",
-    "report_abuse_path": "/abuse_reports/new?ref_url=http%3A%2F%2Flocalhost%3A3000%2Fgitlab-org%2Fgitlab-ce%2Fissues%2F26%23note_1391\u0026user_id=1",
-    "path": "/gitlab-org/gitlab-ce/notes/1391"
-  }],
-  "individual_note": true
-}];
+  }
+};
 
-export const discussionNoteServerResponse = [{
-  "id": "a3ed36e29b1957efb3b68c53e2d7a2b24b1df052",
-  "reply_id": "a3ed36e29b1957efb3b68c53e2d7a2b24b1df052",
-  "expanded": true,
-  "notes": [{
-    "id": 1471,
-    "attachment": {
-      "url": null,
-      "filename": null,
-      "image": false
-    },
-    "author": {
-      "id": 1,
-      "name": "Root",
-      "username": "root",
-      "state": "active",
-      "avatar_url": null,
-      "path": "/root"
-    },
-    "created_at": "2017-08-08T16:53:00.666Z",
-    "updated_at": "2017-08-08T16:53:00.666Z",
-    "system": false,
-    "noteable_id": 124,
-    "noteable_type": "Issue",
-    "noteable_iid": 29,
-    "type": "DiscussionNote",
-    "human_access": "Owner",
-    "note": "Adding a comment",
-    "note_html": "\u003cp dir=\"auto\"\u003eAdding a comment\u003c/p\u003e",
-    "current_user": {
-      "can_edit": true
-    },
-    "discussion_id": "a3ed36e29b1957efb3b68c53e2d7a2b24b1df052",
-    "emoji_awardable": true,
-    "award_emoji": [],
-    "toggle_award_path": "/gitlab-org/gitlab-ce/notes/1471/toggle_award_emoji",
-    "report_abuse_path": "/abuse_reports/new?ref_url=http%3A%2F%2Flocalhost%3A3000%2Fgitlab-org%2Fgitlab-ce%2Fissues%2F29%23note_1471\u0026user_id=1",
-    "path": "/gitlab-org/gitlab-ce/notes/1471"
-  }],
-  "individual_note": false
-}];
+export const DISCUSSION_NOTE_RESPONSE_MAP = {
+  ...INDIVIDUAL_NOTE_RESPONSE_MAP,
+  'GET': {
+    ...INDIVIDUAL_NOTE_RESPONSE_MAP.GET,
+    '/gitlab-org/gitlab-ce/issues/26/discussions.json': [{
+      "id": "a3ed36e29b1957efb3b68c53e2d7a2b24b1df052",
+      "reply_id": "a3ed36e29b1957efb3b68c53e2d7a2b24b1df052",
+      "expanded": true,
+      "notes": [{
+        "id": 1471,
+        "attachment": {
+          "url": null,
+          "filename": null,
+          "image": false
+        },
+        "author": {
+          "id": 1,
+          "name": "Root",
+          "username": "root",
+          "state": "active",
+          "avatar_url": null,
+          "path": "/root"
+        },
+        "created_at": "2017-08-08T16:53:00.666Z",
+        "updated_at": "2017-08-08T16:53:00.666Z",
+        "system": false,
+        "noteable_id": 124,
+        "noteable_type": "Issue",
+        "noteable_iid": 29,
+        "type": "DiscussionNote",
+        "human_access": "Owner",
+        "note": "Adding a comment",
+        "note_html": "\u003cp dir=\"auto\"\u003eAdding a comment\u003c/p\u003e",
+        "current_user": {
+          "can_edit": true
+        },
+        "discussion_id": "a3ed36e29b1957efb3b68c53e2d7a2b24b1df052",
+        "emoji_awardable": true,
+        "award_emoji": [],
+        "toggle_award_path": "/gitlab-org/gitlab-ce/notes/1471/toggle_award_emoji",
+        "report_abuse_path": "/abuse_reports/new?ref_url=http%3A%2F%2Flocalhost%3A3000%2Fgitlab-org%2Fgitlab-ce%2Fissues%2F29%23note_1471\u0026user_id=1",
+        "path": "/gitlab-org/gitlab-ce/notes/1471"
+      }],
+      "individual_note": false
+    }],
+  },
+};
+
+export function individualNoteInterceptor(request, next) {
+  const body = INDIVIDUAL_NOTE_RESPONSE_MAP[request.method.toUpperCase()][request.url];
+
+  next(request.respondWith(JSON.stringify(body), {
+    status: 200,
+  }));
+}
+
+export function discussionNoteInterceptor(request, next) {
+  const body = DISCUSSION_NOTE_RESPONSE_MAP[request.method.toUpperCase()][request.url];
+
+  next(request.respondWith(JSON.stringify(body), {
+    status: 200,
+  }));
+}
diff --git a/spec/javascripts/pipelines/graph/job_component_spec.js b/spec/javascripts/pipelines/graph/job_component_spec.js
index 23c87610d83..35e36e9c353 100644
--- a/spec/javascripts/pipelines/graph/job_component_spec.js
+++ b/spec/javascripts/pipelines/graph/job_component_spec.js
@@ -113,4 +113,35 @@ describe('pipeline graph job component', () => {
       component.$el.querySelector('a').classList.contains('css-class-job-name'),
     ).toBe(true);
   });
+
+  describe('status label', () => {
+    it('should not render status label when it is not provided', () => {
+      component = mountComponent(JobComponent, {
+        job: {
+          id: 4256,
+          name: 'test',
+          status: {
+            icon: 'icon_status_success',
+          },
+        },
+      });
+
+      expect(component.$el.querySelector('.js-job-component-tooltip').getAttribute('data-original-title')).toEqual('test');
+    });
+
+    it('should not render status label when it is  provided', () => {
+      component = mountComponent(JobComponent, {
+        job: {
+          id: 4256,
+          name: 'test',
+          status: {
+            icon: 'icon_status_success',
+            label: 'success',
+          },
+        },
+      });
+
+      expect(component.$el.querySelector('.js-job-component-tooltip').getAttribute('data-original-title')).toEqual('test - success');
+    });
+  });
 });
diff --git a/spec/javascripts/search_autocomplete_spec.js b/spec/javascripts/search_autocomplete_spec.js
index 1e4c2c9faad..206f95abc1a 100644
--- a/spec/javascripts/search_autocomplete_spec.js
+++ b/spec/javascripts/search_autocomplete_spec.js
@@ -1,7 +1,7 @@
 /* eslint-disable space-before-function-paren, max-len, no-var, one-var, one-var-declaration-per-line, no-unused-expressions, consistent-return, no-param-reassign, default-case, no-return-assign, comma-dangle, object-shorthand, prefer-template, quotes, new-parens, vars-on-top, new-cap, max-len */
 
 import '~/gl_dropdown';
-import '~/search_autocomplete';
+import SearchAutocomplete from '~/search_autocomplete';
 import '~/lib/utils/common_utils';
 import * as urlUtils from '~/lib/utils/url_utility';
 
@@ -128,7 +128,7 @@ import * as urlUtils from '~/lib/utils/url_utility';
       window.gon.current_user_id = userId;
       window.gon.current_username = userName;
 
-      return widget = new gl.SearchAutocomplete;
+      return widget = new SearchAutocomplete();
     });
 
     afterEach(function() {
diff --git a/spec/javascripts/sidebar/sidebar_assignees_spec.js b/spec/javascripts/sidebar/sidebar_assignees_spec.js
index 929ba75e67d..b97e24d9dcf 100644
--- a/spec/javascripts/sidebar/sidebar_assignees_spec.js
+++ b/spec/javascripts/sidebar/sidebar_assignees_spec.js
@@ -4,20 +4,29 @@ import SidebarMediator from '~/sidebar/sidebar_mediator';
 import SidebarService from '~/sidebar/services/sidebar_service';
 import SidebarStore from '~/sidebar/stores/sidebar_store';
 import Mock from './mock_data';
+import mountComponent from '../helpers/vue_mount_component_helper';
 
 describe('sidebar assignees', () => {
-  let component;
-  let SidebarAssigneeComponent;
+  let vm;
+  let mediator;
+  let sidebarAssigneesEl;
   preloadFixtures('issues/open-issue.html.raw');
 
   beforeEach(() => {
     Vue.http.interceptors.push(Mock.sidebarMockInterceptor);
-    SidebarAssigneeComponent = Vue.extend(SidebarAssignees);
-    spyOn(SidebarMediator.prototype, 'saveAssignees').and.callThrough();
-    spyOn(SidebarMediator.prototype, 'assignYourself').and.callThrough();
-    this.mediator = new SidebarMediator(Mock.mediator);
+
     loadFixtures('issues/open-issue.html.raw');
-    this.sidebarAssigneesEl = document.querySelector('#js-vue-sidebar-assignees');
+
+    mediator = new SidebarMediator(Mock.mediator);
+    spyOn(mediator, 'saveAssignees').and.callThrough();
+    spyOn(mediator, 'assignYourself').and.callThrough();
+
+    const SidebarAssigneeComponent = Vue.extend(SidebarAssignees);
+    sidebarAssigneesEl = document.querySelector('#js-vue-sidebar-assignees');
+    vm = mountComponent(SidebarAssigneeComponent, {
+      mediator,
+      field: sidebarAssigneesEl.dataset.field,
+    }, sidebarAssigneesEl);
   });
 
   afterEach(() => {
@@ -28,30 +37,24 @@ describe('sidebar assignees', () => {
   });
 
   it('calls the mediator when saves the assignees', () => {
-    component = new SidebarAssigneeComponent()
-      .$mount(this.sidebarAssigneesEl);
-    component.saveAssignees();
-
-    expect(SidebarMediator.prototype.saveAssignees).toHaveBeenCalled();
+    vm.saveAssignees();
+    expect(mediator.saveAssignees).toHaveBeenCalled();
   });
 
   it('calls the mediator when "assignSelf" method is called', () => {
-    component = new SidebarAssigneeComponent()
-      .$mount(this.sidebarAssigneesEl);
-    component.assignSelf();
+    vm.assignSelf();
 
-    expect(SidebarMediator.prototype.assignYourself).toHaveBeenCalled();
-    expect(this.mediator.store.assignees.length).toEqual(1);
+    expect(mediator.assignYourself).toHaveBeenCalled();
+    expect(mediator.store.assignees.length).toEqual(1);
   });
 
   it('hides assignees until fetched', (done) => {
-    component = new SidebarAssigneeComponent().$mount(this.sidebarAssigneesEl);
-    const currentAssignee = this.sidebarAssigneesEl.querySelector('.value');
+    const currentAssignee = sidebarAssigneesEl.querySelector('.value');
     expect(currentAssignee).toBe(null);
 
-    component.store.isFetching.assignees = false;
+    vm.store.isFetching.assignees = false;
     Vue.nextTick(() => {
-      expect(component.$el.querySelector('.value')).toBeVisible();
+      expect(vm.$el.querySelector('.value')).toBeVisible();
       done();
     });
   });
diff --git a/spec/javascripts/sidebar/sidebar_subscriptions_spec.js b/spec/javascripts/sidebar/sidebar_subscriptions_spec.js
index 7adf22b0f1f..a6113cb0bae 100644
--- a/spec/javascripts/sidebar/sidebar_subscriptions_spec.js
+++ b/spec/javascripts/sidebar/sidebar_subscriptions_spec.js
@@ -26,11 +26,14 @@ describe('Sidebar Subscriptions', function () {
   });
 
   it('calls the mediator toggleSubscription on event', () => {
-    spyOn(SidebarMediator.prototype, 'toggleSubscription').and.returnValue(Promise.resolve());
-    vm = mountComponent(SidebarSubscriptions, {});
+    const mediator = new SidebarMediator();
+    spyOn(mediator, 'toggleSubscription').and.returnValue(Promise.resolve());
+    vm = mountComponent(SidebarSubscriptions, {
+      mediator,
+    });
 
     eventHub.$emit('toggleSubscription');
 
-    expect(SidebarMediator.prototype.toggleSubscription).toHaveBeenCalled();
+    expect(mediator.toggleSubscription).toHaveBeenCalled();
   });
 });
diff --git a/spec/javascripts/syntax_highlight_spec.js b/spec/javascripts/syntax_highlight_spec.js
index 946f98379ce..763a15e710b 100644
--- a/spec/javascripts/syntax_highlight_spec.js
+++ b/spec/javascripts/syntax_highlight_spec.js
@@ -1,44 +1,42 @@
 /* eslint-disable space-before-function-paren, no-var, no-return-assign, quotes */
 
-import '~/syntax_highlight';
+import syntaxHighlight from '~/syntax_highlight';
 
-(function() {
-  describe('Syntax Highlighter', function() {
-    var stubUserColorScheme;
-    stubUserColorScheme = function(value) {
-      if (window.gon == null) {
-        window.gon = {};
-      }
-      return window.gon.user_color_scheme = value;
-    };
-    describe('on a js-syntax-highlight element', function() {
-      beforeEach(function() {
-        return setFixtures('<div class="js-syntax-highlight"></div>');
-      });
-      return it('applies syntax highlighting', function() {
-        stubUserColorScheme('monokai');
-        $('.js-syntax-highlight').syntaxHighlight();
-        return expect($('.js-syntax-highlight')).toHaveClass('monokai');
-      });
+describe('Syntax Highlighter', function() {
+  var stubUserColorScheme;
+  stubUserColorScheme = function(value) {
+    if (window.gon == null) {
+      window.gon = {};
+    }
+    return window.gon.user_color_scheme = value;
+  };
+  describe('on a js-syntax-highlight element', function() {
+    beforeEach(function() {
+      return setFixtures('<div class="js-syntax-highlight"></div>');
     });
-    return describe('on a parent element', function() {
-      beforeEach(function() {
-        return setFixtures("<div class=\"parent\">\n  <div class=\"js-syntax-highlight\"></div>\n  <div class=\"foo\"></div>\n  <div class=\"js-syntax-highlight\"></div>\n</div>");
-      });
-      it('applies highlighting to all applicable children', function() {
-        stubUserColorScheme('monokai');
-        $('.parent').syntaxHighlight();
-        expect($('.parent, .foo')).not.toHaveClass('monokai');
-        return expect($('.monokai').length).toBe(2);
-      });
-      return it('prevents an infinite loop when no matches exist', function() {
-        var highlight;
-        setFixtures('<div></div>');
-        highlight = function() {
-          return $('div').syntaxHighlight();
-        };
-        return expect(highlight).not.toThrow();
-      });
+    return it('applies syntax highlighting', function() {
+      stubUserColorScheme('monokai');
+      syntaxHighlight($('.js-syntax-highlight'));
+      return expect($('.js-syntax-highlight')).toHaveClass('monokai');
     });
   });
-}).call(window);
+  return describe('on a parent element', function() {
+    beforeEach(function() {
+      return setFixtures("<div class=\"parent\">\n  <div class=\"js-syntax-highlight\"></div>\n  <div class=\"foo\"></div>\n  <div class=\"js-syntax-highlight\"></div>\n</div>");
+    });
+    it('applies highlighting to all applicable children', function() {
+      stubUserColorScheme('monokai');
+      syntaxHighlight($('.parent'));
+      expect($('.parent, .foo')).not.toHaveClass('monokai');
+      return expect($('.monokai').length).toBe(2);
+    });
+    return it('prevents an infinite loop when no matches exist', function() {
+      var highlight;
+      setFixtures('<div></div>');
+      highlight = function() {
+        return syntaxHighlight($('div'));
+      };
+      return expect(highlight).not.toThrow();
+    });
+  });
+});
diff --git a/spec/javascripts/test_bundle.js b/spec/javascripts/test_bundle.js
index fd7aa332d17..6897c991066 100644
--- a/spec/javascripts/test_bundle.js
+++ b/spec/javascripts/test_bundle.js
@@ -17,6 +17,12 @@ Vue.config.warnHandler = (msg, vm, trace) => {
   fail(`${msg}${trace}`);
 };
 
+let hasVueErrors = false;
+Vue.config.errorHandler = function (err) {
+  hasVueErrors = true;
+  fail(err);
+};
+
 Vue.use(VueResource);
 
 // enable test fixtures
@@ -72,7 +78,7 @@ testsContext.keys().forEach(function (path) {
 
 describe('test errors', () => {
   beforeAll((done) => {
-    if (hasUnhandledPromiseRejections || hasVueWarnings) {
+    if (hasUnhandledPromiseRejections || hasVueWarnings || hasVueErrors) {
       setTimeout(done, 1000);
     } else {
       done();
@@ -86,6 +92,10 @@ describe('test errors', () => {
   it('has no Vue warnings', () => {
     expect(hasVueWarnings).toBe(false);
   });
+
+  it('has no Vue error', () => {
+    expect(hasVueErrors).toBe(false);
+  });
 });
 
 // if we're generating coverage reports, make sure to include all files so