Merge branch 'master' into 'es-module-autosave'es-module-autosave

# Conflicts:
#   app/assets/javascripts/issuable_form.js
#   app/assets/javascripts/notes.js

1 files changed, 5 insertions, 6 deletions

diff --git a/spec/javascripts/shortcuts_spec.js b/spec/javascripts/shortcuts_spec.js
index 53e4c68beb3..a2a609edef9 100644
--- a/spec/javascripts/shortcuts_spec.js
+++ b/spec/javascripts/shortcuts_spec.js
@@ -1,4 +1,5 @@
-/* global Shortcuts */
+import Shortcuts from '~/shortcuts';
+
 describe('Shortcuts', () => {
   const fixtureName = 'merge_requests/diff_comment.html.raw';
   const createEvent = (type, target) => $.Event(type, {
@@ -8,19 +9,17 @@ describe('Shortcuts', () => {
   preloadFixtures(fixtureName);
 
   describe('toggleMarkdownPreview', () => {
-    let sc;
-
     beforeEach(() => {
       loadFixtures(fixtureName);
 
       spyOnEvent('.js-new-note-form .js-md-preview-button', 'focus');
       spyOnEvent('.edit-note .js-md-preview-button', 'focus');
 
-      sc = new Shortcuts();
+      new Shortcuts(); // eslint-disable-line no-new
     });
 
     it('focuses preview button in form', () => {
-      sc.toggleMarkdownPreview(
+      Shortcuts.toggleMarkdownPreview(
         createEvent('KeyboardEvent', document.querySelector('.js-new-note-form .js-note-text'),
       ));
 
@@ -31,7 +30,7 @@ describe('Shortcuts', () => {
       document.querySelector('.js-note-edit').click();
 
       setTimeout(() => {
-        sc.toggleMarkdownPreview(
+        Shortcuts.toggleMarkdownPreview(
           createEvent('KeyboardEvent', document.querySelector('.edit-note .js-note-text'),
         ));