Merge branch 'master' into 'gl-ui-tooltip'gl-ui-tooltip

# Conflicts:
#   app/assets/javascripts/commons/gitlab_ui.js

89 files changed, 753 insertions, 597 deletions

diff --git a/spec/controllers/groups/milestones_controller_spec.rb b/spec/controllers/groups/milestones_controller_spec.rb
index 465f3499703..42723bb3820 100644
--- a/spec/controllers/groups/milestones_controller_spec.rb
+++ b/spec/controllers/groups/milestones_controller_spec.rb
@@ -63,7 +63,7 @@ describe Groups::MilestonesController do
     let(:group_milestone) { create(:milestone, group: group) }
 
     context 'when there is a title parameter' do
-      it 'searchs for a legacy group milestone' do
+      it 'searches for a legacy group milestone' do
         expect(GlobalMilestone).to receive(:build)
         expect(Milestone).not_to receive(:find_by_iid)
 
@@ -72,7 +72,7 @@ describe Groups::MilestonesController do
     end
 
     context 'when there is not a title parameter' do
-      it 'searchs for a group milestone' do
+      it 'searches for a group milestone' do
         expect(GlobalMilestone).not_to receive(:build)
         expect(Milestone).to receive(:find_by_iid)
 
diff --git a/spec/controllers/groups_controller_spec.rb b/spec/controllers/groups_controller_spec.rb
index a099cdafa58..4de61b65f71 100644
--- a/spec/controllers/groups_controller_spec.rb
+++ b/spec/controllers/groups_controller_spec.rb
@@ -529,7 +529,7 @@ describe GroupsController do
       sign_in(user)
     end
 
-    context 'when transfering to a subgroup goes right' do
+    context 'when transferring to a subgroup goes right' do
       let(:new_parent_group) { create(:group, :public) }
       let!(:group_member) { create(:group_member, :owner, group: group, user: user) }
       let!(:new_parent_group_member) { create(:group_member, :owner, group: new_parent_group, user: user) }
diff --git a/spec/controllers/profiles/keys_controller_spec.rb b/spec/controllers/profiles/keys_controller_spec.rb
index 363ed410bc0..ea26bc83353 100644
--- a/spec/controllers/profiles/keys_controller_spec.rb
+++ b/spec/controllers/profiles/keys_controller_spec.rb
@@ -4,7 +4,7 @@ describe Profiles::KeysController do
   let(:user) { create(:user) }
 
   describe "#get_keys" do
-    describe "non existant user" do
+    describe "non existent user" do
       it "does not generally work" do
         get :get_keys, username: 'not-existent'
 
diff --git a/spec/controllers/projects/environments_controller_spec.rb b/spec/controllers/projects/environments_controller_spec.rb
index b86029a4baf..bc17331f531 100644
--- a/spec/controllers/projects/environments_controller_spec.rb
+++ b/spec/controllers/projects/environments_controller_spec.rb
@@ -216,7 +216,7 @@ describe Projects::EnvironmentsController do
         expect(response).to have_gitlab_http_status(200)
       end
 
-      it 'loads the terminals for the enviroment' do
+      it 'loads the terminals for the environment' do
         expect_any_instance_of(Environment).to receive(:terminals)
 
         get :terminal, environment_params
diff --git a/spec/controllers/projects/merge_requests_controller_spec.rb b/spec/controllers/projects/merge_requests_controller_spec.rb
index 7b0459e0325..7463586621c 100644
--- a/spec/controllers/projects/merge_requests_controller_spec.rb
+++ b/spec/controllers/projects/merge_requests_controller_spec.rb
@@ -786,7 +786,7 @@ describe Projects::MergeRequestsController do
             merge_request.mark_as_merged!
           end
 
-          it 'returns the enviroment on the source project' do
+          it 'returns the environment on the source project' do
             get_ci_environments_status(environment_target: 'merge_commit')
 
             expect(response).to have_gitlab_http_status(:ok)
diff --git a/spec/features/dashboard/archived_projects_spec.rb b/spec/features/dashboard/archived_projects_spec.rb
index 6a0cd848345..d31df322d10 100644
--- a/spec/features/dashboard/archived_projects_spec.rb
+++ b/spec/features/dashboard/archived_projects_spec.rb
@@ -33,7 +33,7 @@ RSpec.describe 'Dashboard Archived Project' do
     expect(page).not_to have_content(project.name)
   end
 
-  it 'searchs archived projects', :js do
+  it 'searches archived projects', :js do
     click_button 'Last updated'
     click_link 'Show archived projects'
 
diff --git a/spec/features/import/manifest_import_spec.rb b/spec/features/import/manifest_import_spec.rb
index e381d073804..a90cdd8d920 100644
--- a/spec/features/import/manifest_import_spec.rb
+++ b/spec/features/import/manifest_import_spec.rb
@@ -22,7 +22,7 @@ describe 'Import multiple repositories by uploading a manifest file', :js, :post
     expect(page).to have_content('https://android-review.googlesource.com/platform/build/blueprint')
   end
 
-  it 'imports succesfully imports a project' do
+  it 'imports successfully imports a project' do
     visit new_import_manifest_path
 
     attach_file('manifest', Rails.root.join('spec/fixtures/aosp_manifest.xml'))
diff --git a/spec/features/issues/filtered_search/dropdown_emoji_spec.rb b/spec/features/issues/filtered_search/dropdown_emoji_spec.rb
index be229e8aa7d..c42fcd92a36 100644
--- a/spec/features/issues/filtered_search/dropdown_emoji_spec.rb
+++ b/spec/features/issues/filtered_search/dropdown_emoji_spec.rb
@@ -92,7 +92,7 @@ describe 'Dropdown emoji', :js do
       it 'shows the most populated emoji at top of dropdown' do
         send_keys_to_filtered_search('my-reaction:')
 
-        expect(first('#js-dropdown-my-reaction li')).to have_content(award_emoji_star.name)
+        expect(first('#js-dropdown-my-reaction .filter-dropdown li')).to have_content(award_emoji_star.name)
       end
     end
 
@@ -121,13 +121,29 @@ describe 'Dropdown emoji', :js do
         send_keys_to_filtered_search(':')
       end
 
+      it 'selects `None`' do
+        find('#js-dropdown-my-reaction .filter-dropdown-item', text: 'None').click
+
+        expect(page).to have_css(js_dropdown_emoji, visible: false)
+        expect_tokens([reaction_token('none', false)])
+        expect_filtered_search_input_empty
+      end
+
+      it 'selects `Any`' do
+        find('#js-dropdown-my-reaction .filter-dropdown-item', text: 'Any').click
+
+        expect(page).to have_css(js_dropdown_emoji, visible: false)
+        expect_tokens([reaction_token('any', false)])
+        expect_filtered_search_input_empty
+      end
+
       it 'fills in the my-reaction name' do
         click_emoji('thumbsup')
 
         wait_for_requests
 
         expect(page).to have_css(js_dropdown_emoji, visible: false)
-        expect_tokens([emoji_token('thumbsup')])
+        expect_tokens([reaction_token('thumbsup')])
         expect_filtered_search_input_empty
       end
     end
diff --git a/spec/features/issues/gfm_autocomplete_spec.rb b/spec/features/issues/gfm_autocomplete_spec.rb
index 593dc6b6690..605860b90cd 100644
--- a/spec/features/issues/gfm_autocomplete_spec.rb
+++ b/spec/features/issues/gfm_autocomplete_spec.rb
@@ -15,7 +15,7 @@ describe 'GFM autocomplete', :js do
     wait_for_requests
   end
 
-  it 'updates issue descripton with GFM reference' do
+  it 'updates issue description with GFM reference' do
     find('.js-issuable-edit').click
 
     simulate_input('#issue-description', "@#{user.name[0...3]}")
diff --git a/spec/features/projects/badges/pipeline_badge_spec.rb b/spec/features/projects/badges/pipeline_badge_spec.rb
index 8c4488b2ca6..dee81898928 100644
--- a/spec/features/projects/badges/pipeline_badge_spec.rb
+++ b/spec/features/projects/badges/pipeline_badge_spec.rb
@@ -19,7 +19,7 @@ describe 'Pipeline Badge' do
     let!(:pipeline) { create(:ci_empty_pipeline, project: project, ref: ref, sha: project.commit(ref).sha) }
     let!(:job) { create(:ci_build, pipeline: pipeline) }
 
-    context 'when the pipeline was successfull' do
+    context 'when the pipeline was successful' do
       it 'displays so on the badge' do
         job.success
 
diff --git a/spec/features/projects/clusters_spec.rb b/spec/features/projects/clusters_spec.rb
index 91eac9c8278..f13c35c00d3 100644
--- a/spec/features/projects/clusters_spec.rb
+++ b/spec/features/projects/clusters_spec.rb
@@ -40,7 +40,7 @@ describe 'Clusters', :js do
         expect(page).to have_selector('.js-project-feature-toggle')
       end
 
-      context 'with sucessfull request' do
+      context 'with successful request' do
         it 'user sees updated cluster' do
           expect do
             page.find('.js-project-feature-toggle').click
diff --git a/spec/features/projects/files/user_creates_files_spec.rb b/spec/features/projects/files/user_creates_files_spec.rb
index d9df4b50621..14b5bd58bd1 100644
--- a/spec/features/projects/files/user_creates_files_spec.rb
+++ b/spec/features/projects/files/user_creates_files_spec.rb
@@ -16,7 +16,7 @@ describe 'Projects > Files > User creates files' do
     sign_in(user)
   end
 
-  context 'without commiting a new file' do
+  context 'without committing a new file' do
     context 'when an user has write access' do
       before do
         visit(project_tree_path_root_ref)
@@ -49,7 +49,7 @@ describe 'Projects > Files > User creates files' do
     end
   end
 
-  context 'with commiting a new file' do
+  context 'with committing a new file' do
     context 'when an user has write access' do
       before do
         visit(project_tree_path_root_ref)
diff --git a/spec/finders/issues_finder_spec.rb b/spec/finders/issues_finder_spec.rb
index 2f164ffa8b0..c0488c83bd8 100644
--- a/spec/finders/issues_finder_spec.rb
+++ b/spec/finders/issues_finder_spec.rb
@@ -144,19 +144,31 @@ describe IssuesFinder do
       end
 
       context 'filtering by no milestone' do
-        let(:params) { { milestone_title: Milestone::None.title } }
+        let(:params) { { milestone_title: 'None' } }
 
         it 'returns issues with no milestone' do
           expect(issues).to contain_exactly(issue2, issue3, issue4)
         end
+
+        it 'returns issues with no milestone (deprecated)' do
+          params[:milestone_title] = Milestone::None.title
+
+          expect(issues).to contain_exactly(issue2, issue3, issue4)
+        end
       end
 
       context 'filtering by any milestone' do
-        let(:params) { { milestone_title: Milestone::Any.title } }
+        let(:params) { { milestone_title: 'Any' } }
 
         it 'returns issues with any assigned milestone' do
           expect(issues).to contain_exactly(issue1)
         end
+
+        it 'returns issues with any assigned milestone (deprecated)' do
+          params[:milestone_title] = Milestone::Any.title
+
+          expect(issues).to contain_exactly(issue1)
+        end
       end
 
       context 'filtering by upcoming milestone' do
@@ -360,6 +372,22 @@ describe IssuesFinder do
       end
 
       context 'filtering by reaction name' do
+        context 'user searches by no reaction' do
+          let(:params) { { my_reaction_emoji: 'None' } }
+
+          it 'returns issues that the user did not react to' do
+            expect(issues).to contain_exactly(issue2, issue4)
+          end
+        end
+
+        context 'user searches by any reaction' do
+          let(:params) { { my_reaction_emoji: 'Any' } }
+
+          it 'returns issues that the user reacted to' do
+            expect(issues).to contain_exactly(issue1, issue3)
+          end
+        end
+
         context 'user searches by "thumbsup" reaction' do
           let(:params) { { my_reaction_emoji: 'thumbsup' } }
 
diff --git a/spec/javascripts/awards_handler_spec.js b/spec/javascripts/awards_handler_spec.js
index b0689fc7cfe..ce5d2022441 100644
--- a/spec/javascripts/awards_handler_spec.js
+++ b/spec/javascripts/awards_handler_spec.js
@@ -219,7 +219,7 @@ describe('AwardsHandler', function() {
       expect($thumbsUpEmoji.data('originalTitle')).toBe('You, sam, jerry, max, and andy');
     });
 
-    it('handles the special case where "You" is not cleanly comma seperated', function() {
+    it('handles the special case where "You" is not cleanly comma separated', function() {
       const awardUrl = awardsHandler.getAwardUrl();
       const $votesBlock = $('.js-awards-block').eq(0);
       const $thumbsUpEmoji = $votesBlock.find('[data-name=thumbsup]').parent();
@@ -244,7 +244,7 @@ describe('AwardsHandler', function() {
       expect($thumbsUpEmoji.data('originalTitle')).toBe('sam, jerry, max, and andy');
     });
 
-    it('handles the special case where "You" is not cleanly comma seperated', function() {
+    it('handles the special case where "You" is not cleanly comma separated', function() {
       const awardUrl = awardsHandler.getAwardUrl();
       const $votesBlock = $('.js-awards-block').eq(0);
       const $thumbsUpEmoji = $votesBlock.find('[data-name=thumbsup]').parent();
diff --git a/spec/javascripts/boards/board_list_spec.js b/spec/javascripts/boards/board_list_spec.js
index 037e06cf3b2..2642c8b1bdb 100644
--- a/spec/javascripts/boards/board_list_spec.js
+++ b/spec/javascripts/boards/board_list_spec.js
@@ -18,7 +18,7 @@ describe('Board list component', () => {
   let mock;
   let component;
 
-  beforeEach((done) => {
+  beforeEach(done => {
     const el = document.createElement('div');
 
     document.body.appendChild(el);
@@ -62,122 +62,102 @@ describe('Board list component', () => {
   });
 
   it('renders component', () => {
-    expect(
-      component.$el.classList.contains('board-list-component'),
-    ).toBe(true);
+    expect(component.$el.classList.contains('board-list-component')).toBe(true);
   });
 
-  it('renders loading icon', (done) => {
+  it('renders loading icon', done => {
     component.loading = true;
 
     Vue.nextTick(() => {
-      expect(
-        component.$el.querySelector('.board-list-loading'),
-      ).not.toBeNull();
+      expect(component.$el.querySelector('.board-list-loading')).not.toBeNull();
 
       done();
     });
   });
 
   it('renders issues', () => {
-    expect(
-      component.$el.querySelectorAll('.board-card').length,
-    ).toBe(1);
+    expect(component.$el.querySelectorAll('.board-card').length).toBe(1);
   });
 
   it('sets data attribute with issue id', () => {
-    expect(
-      component.$el.querySelector('.board-card').getAttribute('data-issue-id'),
-    ).toBe('1');
+    expect(component.$el.querySelector('.board-card').getAttribute('data-issue-id')).toBe('1');
   });
 
-  it('shows new issue form', (done) => {
+  it('shows new issue form', done => {
     component.toggleForm();
 
     Vue.nextTick(() => {
-      expect(
-        component.$el.querySelector('.board-new-issue-form'),
-      ).not.toBeNull();
+      expect(component.$el.querySelector('.board-new-issue-form')).not.toBeNull();
 
-      expect(
-        component.$el.querySelector('.is-smaller'),
-      ).not.toBeNull();
+      expect(component.$el.querySelector('.is-smaller')).not.toBeNull();
 
       done();
     });
   });
 
-  it('shows new issue form after eventhub event', (done) => {
+  it('shows new issue form after eventhub event', done => {
     eventHub.$emit(`hide-issue-form-${component.list.id}`);
 
     Vue.nextTick(() => {
-      expect(
-        component.$el.querySelector('.board-new-issue-form'),
-      ).not.toBeNull();
+      expect(component.$el.querySelector('.board-new-issue-form')).not.toBeNull();
 
-      expect(
-        component.$el.querySelector('.is-smaller'),
-      ).not.toBeNull();
+      expect(component.$el.querySelector('.is-smaller')).not.toBeNull();
 
       done();
     });
   });
 
-  it('does not show new issue form for closed list', (done) => {
+  it('does not show new issue form for closed list', done => {
     component.list.type = 'closed';
     component.toggleForm();
 
     Vue.nextTick(() => {
-      expect(
-        component.$el.querySelector('.board-new-issue-form'),
-      ).toBeNull();
+      expect(component.$el.querySelector('.board-new-issue-form')).toBeNull();
 
       done();
     });
   });
 
-  it('shows count list item', (done) => {
+  it('shows count list item', done => {
     component.showCount = true;
 
     Vue.nextTick(() => {
-      expect(
-        component.$el.querySelector('.board-list-count'),
-      ).not.toBeNull();
+      expect(component.$el.querySelector('.board-list-count')).not.toBeNull();
 
-      expect(
-        component.$el.querySelector('.board-list-count').textContent.trim(),
-      ).toBe('Showing all issues');
+      expect(component.$el.querySelector('.board-list-count').textContent.trim()).toBe(
+        'Showing all issues',
+      );
 
       done();
     });
   });
 
-  it('sets data attribute with invalid id', (done) => {
+  it('sets data attribute with invalid id', done => {
     component.showCount = true;
 
     Vue.nextTick(() => {
-      expect(
-        component.$el.querySelector('.board-list-count').getAttribute('data-issue-id'),
-      ).toBe('-1');
+      expect(component.$el.querySelector('.board-list-count').getAttribute('data-issue-id')).toBe(
+        '-1',
+      );
 
       done();
     });
   });
 
-  it('shows how many more issues to load', (done) => {
+  it('shows how many more issues to load', done => {
     component.showCount = true;
     component.list.issuesSize = 20;
 
     Vue.nextTick(() => {
-      expect(
-        component.$el.querySelector('.board-list-count').textContent.trim(),
-      ).toBe('Showing 1 of 20 issues');
+      expect(component.$el.querySelector('.board-list-count').textContent.trim()).toBe(
+        'Showing 1 of 20 issues',
+      );
 
       done();
     });
   });
 
-  it('loads more issues after scrolling', (done) => {
+  it('loads more issues after scrolling', done => {
     spyOn(component.list, 'nextPage');
     component.$refs.list.style.height = '100px';
     component.$refs.list.style.overflow = 'scroll';
@@ -200,7 +180,9 @@ describe('Board list component', () => {
   });
 
   it('does not load issues if already loading', () => {
-    component.list.nextPage = spyOn(component.list, 'nextPage').and.returnValue(new Promise(() => {}));
+    component.list.nextPage = spyOn(component.list, 'nextPage').and.returnValue(
+      new Promise(() => {}),
+    );
 
     component.onScroll();
     component.onScroll();
@@ -208,14 +190,12 @@ describe('Board list component', () => {
     expect(component.list.nextPage).toHaveBeenCalledTimes(1);
   });
 
-  it('shows loading more spinner', (done) => {
+  it('shows loading more spinner', done => {
     component.showCount = true;
     component.list.loadingMore = true;
 
     Vue.nextTick(() => {
-      expect(
-        component.$el.querySelector('.board-list-count .fa-spinner'),
-      ).not.toBeNull();
+      expect(component.$el.querySelector('.board-list-count .fa-spinner')).not.toBeNull();
 
       done();
     });
diff --git a/spec/javascripts/boards/components/board_spec.js b/spec/javascripts/boards/components/board_spec.js
index d4c53bd5a7d..dee7841c088 100644
--- a/spec/javascripts/boards/components/board_spec.js
+++ b/spec/javascripts/boards/components/board_spec.js
@@ -8,7 +8,7 @@ describe('Board component', () => {
   let vm;
   let el;
 
-  beforeEach((done) => {
+  beforeEach(done => {
     loadFixtures('boards/show.html.raw');
 
     el = document.createElement('div');
@@ -50,56 +50,46 @@ describe('Board component', () => {
   });
 
   it('board is expandable when list type is backlog', () => {
-    expect(
-      vm.$el.classList.contains('is-expandable'),
-    ).toBe(true);
+    expect(vm.$el.classList.contains('is-expandable')).toBe(true);
   });
 
-  it('board is expandable when list type is closed', (done) => {
+  it('board is expandable when list type is closed', done => {
     vm.list.type = 'closed';
 
     Vue.nextTick(() => {
-      expect(
-        vm.$el.classList.contains('is-expandable'),
-      ).toBe(true);
+      expect(vm.$el.classList.contains('is-expandable')).toBe(true);
 
       done();
     });
   });
 
-  it('board is not expandable when list type is label', (done) => {
+  it('board is not expandable when list type is label', done => {
     vm.list.type = 'label';
     vm.list.isExpandable = false;
 
     Vue.nextTick(() => {
-      expect(
-        vm.$el.classList.contains('is-expandable'),
-      ).toBe(false);
+      expect(vm.$el.classList.contains('is-expandable')).toBe(false);
 
       done();
     });
   });
 
-  it('collapses when clicking header', (done) => {
+  it('collapses when clicking header', done => {
     vm.$el.querySelector('.board-header').click();
 
     Vue.nextTick(() => {
-      expect(
-        vm.$el.classList.contains('is-collapsed'),
-      ).toBe(true);
+      expect(vm.$el.classList.contains('is-collapsed')).toBe(true);
 
       done();
     });
   });
 
-  it('created sets isExpanded to true from localStorage', (done) => {
+  it('created sets isExpanded to true from localStorage', done => {
     vm.$el.querySelector('.board-header').click();
 
     return Vue.nextTick()
       .then(() => {
-        expect(
-          vm.$el.classList.contains('is-collapsed'),
-        ).toBe(true);
+        expect(vm.$el.classList.contains('is-collapsed')).toBe(true);
 
         // call created manually
         vm.$options.created[0].call(vm);
@@ -107,11 +97,10 @@ describe('Board component', () => {
         return Vue.nextTick();
       })
       .then(() => {
-        expect(
-          vm.$el.classList.contains('is-collapsed'),
-        ).toBe(true);
+        expect(vm.$el.classList.contains('is-collapsed')).toBe(true);
 
         done();
-      }).catch(done.fail);
+      })
+      .catch(done.fail);
   });
 });
diff --git a/spec/javascripts/commit/commit_pipeline_status_component_spec.js b/spec/javascripts/commit/commit_pipeline_status_component_spec.js
index 4fc56fd9a27..f6b36e88a5f 100644
--- a/spec/javascripts/commit/commit_pipeline_status_component_spec.js
+++ b/spec/javascripts/commit/commit_pipeline_status_component_spec.js
@@ -22,7 +22,7 @@ describe('Commit pipeline status component', () => {
     Component = Vue.extend(commitPipelineStatus);
   });
 
-  describe('While polling pipeline data succesfully', () => {
+  describe('While polling pipeline data successfully', () => {
     beforeEach(() => {
       mock = new MockAdapter(axios);
       mock.onGet('/dummy/endpoint').reply(() => {
@@ -59,14 +59,14 @@ describe('Commit pipeline status component', () => {
       });
     });
 
-    it('contains a ciStatus when the polling is succesful ', done => {
+    it('contains a ciStatus when the polling is successful ', done => {
       setTimeout(() => {
         expect(vm.ciStatus).toEqual(mockCiStatus);
         done();
       });
     });
 
-    it('contains a ci-status icon when polling is succesful', done => {
+    it('contains a ci-status icon when polling is successful', done => {
       setTimeout(() => {
         expect(vm.$el.querySelector('.ci-status-icon')).not.toBe(null);
         expect(vm.$el.querySelector('.ci-status-icon').classList).toContain(
@@ -77,7 +77,7 @@ describe('Commit pipeline status component', () => {
     });
   });
 
-  describe('When polling data was not succesful', () => {
+  describe('When polling data was not successful', () => {
     beforeEach(() => {
       mock = new MockAdapter(axios);
       mock.onGet('/dummy/endpoint').reply(502, {});
diff --git a/spec/javascripts/environments/emtpy_state_spec.js b/spec/javascripts/environments/emtpy_state_spec.js
index d71dfe8197e..1f986d49bc7 100644
--- a/spec/javascripts/environments/emtpy_state_spec.js
+++ b/spec/javascripts/environments/emtpy_state_spec.js
@@ -1,4 +1,3 @@
-
 import Vue from 'vue';
 import emptyState from '~/environments/components/empty_state.vue';
 import mountComponent from 'spec/helpers/vue_mount_component_helper';
@@ -25,13 +24,13 @@ describe('environments empty state', () => {
     });
 
     it('renders empty state and new environment button', () => {
-      expect(
-        vm.$el.querySelector('.js-blank-state-title').textContent.trim(),
-      ).toEqual('You don\'t have any environments right now');
+      expect(vm.$el.querySelector('.js-blank-state-title').textContent.trim()).toEqual(
+        "You don't have any environments right now",
+      );
 
-      expect(
-        vm.$el.querySelector('.js-new-environment-button').getAttribute('href'),
-      ).toEqual('foo');
+      expect(vm.$el.querySelector('.js-new-environment-button').getAttribute('href')).toEqual(
+        'foo',
+      );
     });
   });
 
@@ -45,13 +44,11 @@ describe('environments empty state', () => {
     });
 
     it('renders empty state without new button', () => {
-      expect(
-        vm.$el.querySelector('.js-blank-state-title').textContent.trim(),
-      ).toEqual('You don\'t have any environments right now');
+      expect(vm.$el.querySelector('.js-blank-state-title').textContent.trim()).toEqual(
+        "You don't have any environments right now",
+      );
 
-      expect(
-        vm.$el.querySelector('.js-new-environment-button'),
-      ).toBeNull();
+      expect(vm.$el.querySelector('.js-new-environment-button')).toBeNull();
     });
   });
 });
diff --git a/spec/javascripts/environments/environment_item_spec.js b/spec/javascripts/environments/environment_item_spec.js
index 0b933dda431..7618c2f50ce 100644
--- a/spec/javascripts/environments/environment_item_spec.js
+++ b/spec/javascripts/environments/environment_item_spec.js
@@ -38,7 +38,9 @@ describe('Environment item', () => {
     });
 
     it('Should render the number of children in a badge', () => {
-      expect(component.$el.querySelector('.folder-name .badge').textContent).toContain(mockItem.size);
+      expect(component.$el.querySelector('.folder-name .badge').textContent).toContain(
+        mockItem.size,
+      );
     });
   });
 
@@ -68,7 +70,8 @@ describe('Environment item', () => {
             username: 'root',
             id: 1,
             state: 'active',
-            avatar_url: 'https://www.gravatar.com/avatar/e64c7d89f26bd1972efa854d13d7dd61?s=80\u0026d=identicon',
+            avatar_url:
+              'https://www.gravatar.com/avatar/e64c7d89f26bd1972efa854d13d7dd61?s=80\u0026d=identicon',
             web_url: 'http://localhost:3000/root',
           },
           commit: {
@@ -84,7 +87,8 @@ describe('Environment item', () => {
               username: 'root',
               id: 1,
               state: 'active',
-              avatar_url: 'https://www.gravatar.com/avatar/e64c7d89f26bd1972efa854d13d7dd61?s=80\u0026d=identicon',
+              avatar_url:
+                'https://www.gravatar.com/avatar/e64c7d89f26bd1972efa854d13d7dd61?s=80\u0026d=identicon',
               web_url: 'http://localhost:3000/root',
             },
             commit_path: '/root/ci-folders/tree/500aabcb17c97bdcf2d0c410b70cb8556f0362dd',
@@ -121,18 +125,18 @@ describe('Environment item', () => {
     });
 
     it('should render environment name', () => {
-      expect(component.$el.querySelector('.environment-name').textContent).toContain(environment.name);
+      expect(component.$el.querySelector('.environment-name').textContent).toContain(
+        environment.name,
+      );
     });
 
     describe('With deployment', () => {
       it('should render deployment internal id', () => {
-        expect(
-          component.$el.querySelector('.deployment-column span').textContent,
-        ).toContain(environment.last_deployment.iid);
+        expect(component.$el.querySelector('.deployment-column span').textContent).toContain(
+          environment.last_deployment.iid,
+        );
 
-        expect(
-          component.$el.querySelector('.deployment-column span').textContent,
-        ).toContain('#');
+        expect(component.$el.querySelector('.deployment-column span').textContent).toContain('#');
       });
 
       it('should render last deployment date', () => {
@@ -156,56 +160,46 @@ describe('Environment item', () => {
 
       describe('With build url', () => {
         it('Should link to build url provided', () => {
-          expect(
-            component.$el.querySelector('.build-link').getAttribute('href'),
-          ).toEqual(environment.last_deployment.deployable.build_path);
+          expect(component.$el.querySelector('.build-link').getAttribute('href')).toEqual(
+            environment.last_deployment.deployable.build_path,
+          );
         });
 
         it('Should render deployable name and id', () => {
-          expect(
-            component.$el.querySelector('.build-link').getAttribute('href'),
-          ).toEqual(environment.last_deployment.deployable.build_path);
+          expect(component.$el.querySelector('.build-link').getAttribute('href')).toEqual(
+            environment.last_deployment.deployable.build_path,
+          );
         });
       });
 
       describe('With commit information', () => {
         it('should render commit component', () => {
-          expect(
-            component.$el.querySelector('.js-commit-component'),
-          ).toBeDefined();
+          expect(component.$el.querySelector('.js-commit-component')).toBeDefined();
         });
       });
     });
 
     describe('With manual actions', () => {
       it('Should render actions component', () => {
-        expect(
-          component.$el.querySelector('.js-manual-actions-container'),
-        ).toBeDefined();
+        expect(component.$el.querySelector('.js-manual-actions-container')).toBeDefined();
       });
     });
 
     describe('With external URL', () => {
       it('should render external url component', () => {
-        expect(
-          component.$el.querySelector('.js-external-url-container'),
-        ).toBeDefined();
+        expect(component.$el.querySelector('.js-external-url-container')).toBeDefined();
       });
     });
 
     describe('With stop action', () => {
       it('Should render stop action component', () => {
-        expect(
-          component.$el.querySelector('.js-stop-component-container'),
-        ).toBeDefined();
+        expect(component.$el.querySelector('.js-stop-component-container')).toBeDefined();
       });
     });
 
     describe('With retry action', () => {
       it('Should render rollback component', () => {
-        expect(
-          component.$el.querySelector('.js-rollback-component-container'),
-        ).toBeDefined();
+        expect(component.$el.querySelector('.js-rollback-component-container')).toBeDefined();
       });
     });
   });
diff --git a/spec/javascripts/environments/environments_app_spec.js b/spec/javascripts/environments/environments_app_spec.js
index 7edc0ccac0b..e2d81eb454a 100644
--- a/spec/javascripts/environments/environments_app_spec.js
+++ b/spec/javascripts/environments/environments_app_spec.js
@@ -31,9 +31,9 @@ describe('Environment', () => {
     mock.restore();
   });
 
-  describe('successfull request', () => {
+  describe('successful request', () => {
     describe('without environments', () => {
-      beforeEach((done) => {
+      beforeEach(done => {
         mock.onGet(mockData.endpoint).reply(200, { environments: [] });
 
         component = mountComponent(EnvironmentsComponent, mockData);
@@ -44,30 +44,34 @@ describe('Environment', () => {
       });
 
       it('should render the empty state', () => {
-        expect(
-          component.$el.querySelector('.js-new-environment-button').textContent,
-        ).toContain('New environment');
+        expect(component.$el.querySelector('.js-new-environment-button').textContent).toContain(
+          'New environment',
+        );
 
-        expect(
-          component.$el.querySelector('.js-blank-state-title').textContent,
-        ).toContain('You don\'t have any environments right now');
+        expect(component.$el.querySelector('.js-blank-state-title').textContent).toContain(
+          "You don't have any environments right now",
+        );
       });
     });
 
     describe('with paginated environments', () => {
-      beforeEach((done) => {
-        mock.onGet(mockData.endpoint).reply(200, {
-          environments: [environment],
-          stopped_count: 1,
-          available_count: 0,
-        }, {
-          'X-nExt-pAge': '2',
-          'x-page': '1',
-          'X-Per-Page': '1',
-          'X-Prev-Page': '',
-          'X-TOTAL': '37',
-          'X-Total-Pages': '2',
-        });
+      beforeEach(done => {
+        mock.onGet(mockData.endpoint).reply(
+          200,
+          {
+            environments: [environment],
+            stopped_count: 1,
+            available_count: 0,
+          },
+          {
+            'X-nExt-pAge': '2',
+            'x-page': '1',
+            'X-Per-Page': '1',
+            'X-Prev-Page': '',
+            'X-TOTAL': '37',
+            'X-Total-Pages': '2',
+          },
+        );
 
         component = mountComponent(EnvironmentsComponent, mockData);
 
@@ -78,19 +82,17 @@ describe('Environment', () => {
 
       it('should render a table with environments', () => {
         expect(component.$el.querySelectorAll('table')).not.toBeNull();
-        expect(
-          component.$el.querySelector('.environment-name').textContent.trim(),
-        ).toEqual(environment.name);
+        expect(component.$el.querySelector('.environment-name').textContent.trim()).toEqual(
+          environment.name,
+        );
       });
 
       describe('pagination', () => {
         it('should render pagination', () => {
-          expect(
-            component.$el.querySelectorAll('.gl-pagination li').length,
-          ).toEqual(5);
+          expect(component.$el.querySelectorAll('.gl-pagination li').length).toEqual(5);
         });
 
-        it('should make an API request when page is clicked', (done) => {
+        it('should make an API request when page is clicked', done => {
           spyOn(component, 'updateContent');
           setTimeout(() => {
             component.$el.querySelector('.gl-pagination li:nth-child(5) a').click();
@@ -100,7 +102,7 @@ describe('Environment', () => {
           }, 0);
         });
 
-        it('should make an API request when using tabs', (done) => {
+        it('should make an API request when using tabs', done => {
           setTimeout(() => {
             spyOn(component, 'updateContent');
             component.$el.querySelector('.js-environments-tab-stopped').click();
@@ -114,7 +116,7 @@ describe('Environment', () => {
   });
 
   describe('unsuccessfull request', () => {
-    beforeEach((done) => {
+    beforeEach(done => {
       mock.onGet(mockData.endpoint).reply(500, {});
 
       component = mountComponent(EnvironmentsComponent, mockData);
@@ -125,15 +127,16 @@ describe('Environment', () => {
     });
 
     it('should render empty state', () => {
-      expect(
-        component.$el.querySelector('.js-blank-state-title').textContent,
-      ).toContain('You don\'t have any environments right now');
+      expect(component.$el.querySelector('.js-blank-state-title').textContent).toContain(
+        "You don't have any environments right now",
+      );
     });
   });
 
   describe('expandable folders', () => {
     beforeEach(() => {
-      mock.onGet(mockData.endpoint).reply(200,
+      mock.onGet(mockData.endpoint).reply(
+        200,
         {
           environments: [folder],
           stopped_count: 0,
@@ -154,7 +157,7 @@ describe('Environment', () => {
       component = mountComponent(EnvironmentsComponent, mockData);
     });
 
-    it('should open a closed folder', (done) => {
+    it('should open a closed folder', done => {
       setTimeout(() => {
         component.$el.querySelector('.folder-name').click();
 
@@ -165,7 +168,7 @@ describe('Environment', () => {
       }, 0);
     });
 
-    it('should close an opened folder', (done) => {
+    it('should close an opened folder', done => {
       setTimeout(() => {
         // open folder
         component.$el.querySelector('.folder-name').click();
@@ -182,7 +185,7 @@ describe('Environment', () => {
       }, 0);
     });
 
-    it('should show children environments and a button to show all environments', (done) => {
+    it('should show children environments and a button to show all environments', done => {
       setTimeout(() => {
         // open folder
         component.$el.querySelector('.folder-name').click();
@@ -191,7 +194,9 @@ describe('Environment', () => {
           // wait for next async request
           setTimeout(() => {
             expect(component.$el.querySelectorAll('.js-child-row').length).toEqual(1);
-            expect(component.$el.querySelector('.text-center > a.btn').textContent).toContain('Show all');
+            expect(component.$el.querySelector('.text-center > a.btn').textContent).toContain(
+              'Show all',
+            );
             done();
           });
         });
@@ -201,7 +206,8 @@ describe('Environment', () => {
 
   describe('methods', () => {
     beforeEach(() => {
-      mock.onGet(mockData.endpoint).reply(200,
+      mock.onGet(mockData.endpoint).reply(
+        200,
         {
           environments: [],
           stopped_count: 0,
@@ -215,8 +221,9 @@ describe('Environment', () => {
     });
 
     describe('updateContent', () => {
-      it('should set given parameters', (done) => {
-        component.updateContent({ scope: 'stopped', page: '3' })
+      it('should set given parameters', done => {
+        component
+          .updateContent({ scope: 'stopped', page: '3' })
           .then(() => {
             expect(component.page).toEqual('3');
             expect(component.scope).toEqual('stopped');
diff --git a/spec/javascripts/environments/folder/environments_folder_view_spec.js b/spec/javascripts/environments/folder/environments_folder_view_spec.js
index 51d4213c38f..7f0a9475d5f 100644
--- a/spec/javascripts/environments/folder/environments_folder_view_spec.js
+++ b/spec/javascripts/environments/folder/environments_folder_view_spec.js
@@ -30,39 +30,43 @@ describe('Environments Folder View', () => {
     component.$destroy();
   });
 
-  describe('successfull request', () => {
+  describe('successful request', () => {
     beforeEach(() => {
-      mock.onGet(mockData.endpoint).reply(200, {
-        environments: environmentsList,
-        stopped_count: 1,
-        available_count: 0,
-      }, {
-        'X-nExt-pAge': '2',
-        'x-page': '1',
-        'X-Per-Page': '2',
-        'X-Prev-Page': '',
-        'X-TOTAL': '20',
-        'X-Total-Pages': '10',
-      });
+      mock.onGet(mockData.endpoint).reply(
+        200,
+        {
+          environments: environmentsList,
+          stopped_count: 1,
+          available_count: 0,
+        },
+        {
+          'X-nExt-pAge': '2',
+          'x-page': '1',
+          'X-Per-Page': '2',
+          'X-Prev-Page': '',
+          'X-TOTAL': '20',
+          'X-Total-Pages': '10',
+        },
+      );
 
       component = mountComponent(Component, mockData);
     });
 
-    it('should render a table with environments', (done) => {
+    it('should render a table with environments', done => {
       setTimeout(() => {
         expect(component.$el.querySelectorAll('table')).not.toBeNull();
-        expect(
-          component.$el.querySelector('.environment-name').textContent.trim(),
-        ).toEqual(environmentsList[0].name);
+        expect(component.$el.querySelector('.environment-name').textContent.trim()).toEqual(
+          environmentsList[0].name,
+        );
         done();
       }, 0);
     });
 
-    it('should render available tab with count', (done) => {
+    it('should render available tab with count', done => {
       setTimeout(() => {
-        expect(
-          component.$el.querySelector('.js-environments-tab-available').textContent,
-        ).toContain('Available');
+        expect(component.$el.querySelector('.js-environments-tab-available').textContent).toContain(
+          'Available',
+        );
 
         expect(
           component.$el.querySelector('.js-environments-tab-available .badge').textContent,
@@ -71,11 +75,11 @@ describe('Environments Folder View', () => {
       }, 0);
     });
 
-    it('should render stopped tab with count', (done) => {
+    it('should render stopped tab with count', done => {
       setTimeout(() => {
-        expect(
-          component.$el.querySelector('.js-environments-tab-stopped').textContent,
-        ).toContain('Stopped');
+        expect(component.$el.querySelector('.js-environments-tab-stopped').textContent).toContain(
+          'Stopped',
+        );
 
         expect(
           component.$el.querySelector('.js-environments-tab-stopped .badge').textContent,
@@ -84,36 +88,37 @@ describe('Environments Folder View', () => {
       }, 0);
     });
 
-    it('should render parent folder name', (done) => {
+    it('should render parent folder name', done => {
       setTimeout(() => {
-        expect(
-          component.$el.querySelector('.js-folder-name').textContent.trim(),
-        ).toContain('Environments / review');
+        expect(component.$el.querySelector('.js-folder-name').textContent.trim()).toContain(
+          'Environments / review',
+        );
         done();
       }, 0);
     });
 
     describe('pagination', () => {
-      it('should render pagination', (done) => {
+      it('should render pagination', done => {
         setTimeout(() => {
-          expect(
-            component.$el.querySelectorAll('.gl-pagination'),
-          ).not.toBeNull();
+          expect(component.$el.querySelectorAll('.gl-pagination')).not.toBeNull();
           done();
         }, 0);
       });
 
-      it('should make an API request when changing page', (done) => {
+      it('should make an API request when changing page', done => {
         spyOn(component, 'updateContent');
         setTimeout(() => {
           component.$el.querySelector('.gl-pagination .js-last-button a').click();
 
-          expect(component.updateContent).toHaveBeenCalledWith({ scope: component.scope, page: '10' });
+          expect(component.updateContent).toHaveBeenCalledWith({
+            scope: component.scope,
+            page: '10',
+          });
           done();
         }, 0);
       });
 
-      it('should make an API request when using tabs', (done) => {
+      it('should make an API request when using tabs', done => {
         setTimeout(() => {
           spyOn(component, 'updateContent');
           component.$el.querySelector('.js-environments-tab-stopped').click();
@@ -134,20 +139,18 @@ describe('Environments Folder View', () => {
       component = mountComponent(Component, mockData);
     });
 
-    it('should not render a table', (done) => {
+    it('should not render a table', done => {
       setTimeout(() => {
-        expect(
-          component.$el.querySelector('table'),
-        ).toBe(null);
+        expect(component.$el.querySelector('table')).toBe(null);
         done();
       }, 0);
     });
 
-    it('should render available tab with count 0', (done) => {
+    it('should render available tab with count 0', done => {
       setTimeout(() => {
-        expect(
-          component.$el.querySelector('.js-environments-tab-available').textContent,
-        ).toContain('Available');
+        expect(component.$el.querySelector('.js-environments-tab-available').textContent).toContain(
+          'Available',
+        );
 
         expect(
           component.$el.querySelector('.js-environments-tab-available .badge').textContent,
@@ -156,11 +159,11 @@ describe('Environments Folder View', () => {
       }, 0);
     });
 
-    it('should render stopped tab with count 0', (done) => {
+    it('should render stopped tab with count 0', done => {
       setTimeout(() => {
-        expect(
-          component.$el.querySelector('.js-environments-tab-stopped').textContent,
-        ).toContain('Stopped');
+        expect(component.$el.querySelector('.js-environments-tab-stopped').textContent).toContain(
+          'Stopped',
+        );
 
         expect(
           component.$el.querySelector('.js-environments-tab-stopped .badge').textContent,
@@ -181,8 +184,9 @@ describe('Environments Folder View', () => {
     });
 
     describe('updateContent', () => {
-      it('should set given parameters', (done) => {
-        component.updateContent({ scope: 'stopped', page: '4' })
+      it('should set given parameters', done => {
+        component
+          .updateContent({ scope: 'stopped', page: '4' })
           .then(() => {
             expect(component.page).toEqual('4');
             expect(component.scope).toEqual('stopped');
diff --git a/spec/javascripts/issue_show/components/title_spec.js b/spec/javascripts/issue_show/components/title_spec.js
index 9e6f236b687..9754c8a6755 100644
--- a/spec/javascripts/issue_show/components/title_spec.js
+++ b/spec/javascripts/issue_show/components/title_spec.js
@@ -25,25 +25,21 @@ describe('Title component', () => {
   });
 
   it('renders title HTML', () => {
-    expect(
-      vm.$el.querySelector('.title').innerHTML.trim(),
-    ).toBe('Testing <img>');
+    expect(vm.$el.querySelector('.title').innerHTML.trim()).toBe('Testing <img>');
   });
 
-  it('updates page title when changing titleHtml', (done) => {
+  it('updates page title when changing titleHtml', done => {
     spyOn(vm, 'setPageTitle');
     vm.titleHtml = 'test';
 
     Vue.nextTick(() => {
-      expect(
-        vm.setPageTitle,
-      ).toHaveBeenCalled();
+      expect(vm.setPageTitle).toHaveBeenCalled();
 
       done();
     });
   });
 
-  it('animates title changes', (done) => {
+  it('animates title changes', done => {
     vm.titleHtml = 'test';
 
     Vue.nextTick(() => {
@@ -61,14 +57,12 @@ describe('Title component', () => {
     });
   });
 
-  it('updates page title after changing title', (done) => {
+  it('updates page title after changing title', done => {
     vm.titleHtml = 'changed';
     vm.titleText = 'changed';
 
     Vue.nextTick(() => {
-      expect(
-        document.querySelector('title').textContent.trim(),
-      ).toContain('changed');
+      expect(document.querySelector('title').textContent.trim()).toContain('changed');
 
       done();
     });
diff --git a/spec/javascripts/jobs/components/job_app_spec.js b/spec/javascripts/jobs/components/job_app_spec.js
index 288c06d6615..ba1889c4dcd 100644
--- a/spec/javascripts/jobs/components/job_app_spec.js
+++ b/spec/javascripts/jobs/components/job_app_spec.js
@@ -52,7 +52,7 @@ describe('Job App ', () => {
     });
   });
 
-  describe('with successfull request', () => {
+  describe('with successful request', () => {
     beforeEach(() => {
       mock.onGet(`${props.pagePath}/trace.json`).replyOnce(200, {});
     });
@@ -234,7 +234,7 @@ describe('Job App ', () => {
             );
             done();
           }, 0);
-        })
+        });
       });
 
       it('does not renders stuck block when there are no runners', done => {
diff --git a/spec/javascripts/jobs/store/actions_spec.js b/spec/javascripts/jobs/store/actions_spec.js
index 45130b983e7..77b44995b12 100644
--- a/spec/javascripts/jobs/store/actions_spec.js
+++ b/spec/javascripts/jobs/store/actions_spec.js
@@ -68,41 +68,20 @@ describe('Job State actions', () => {
 
   describe('hideSidebar', () => {
     it('should commit HIDE_SIDEBAR mutation', done => {
-      testAction(
-        hideSidebar,
-        null,
-        mockedState,
-        [{ type: types.HIDE_SIDEBAR }],
-        [],
-        done,
-      );
+      testAction(hideSidebar, null, mockedState, [{ type: types.HIDE_SIDEBAR }], [], done);
     });
   });
 
   describe('showSidebar', () => {
     it('should commit HIDE_SIDEBAR mutation', done => {
-      testAction(
-        showSidebar,
-        null,
-        mockedState,
-        [{ type: types.SHOW_SIDEBAR }],
-        [],
-        done,
-      );
+      testAction(showSidebar, null, mockedState, [{ type: types.SHOW_SIDEBAR }], [], done);
     });
   });
 
   describe('toggleSidebar', () => {
     describe('when isSidebarOpen is true', () => {
       it('should dispatch hideSidebar', done => {
-        testAction(
-          toggleSidebar,
-          null,
-          mockedState,
-          [],
-          [{ type: 'hideSidebar' }],
-          done,
-        );
+        testAction(toggleSidebar, null, mockedState, [], [{ type: 'hideSidebar' }], done);
       });
     });
 
@@ -110,14 +89,7 @@ describe('Job State actions', () => {
       it('should dispatch showSidebar', done => {
         mockedState.isSidebarOpen = false;
 
-        testAction(
-          toggleSidebar,
-          null,
-          mockedState,
-          [],
-          [{ type: 'showSidebar' }],
-          done,
-        );
+        testAction(toggleSidebar, null, mockedState, [], [{ type: 'showSidebar' }], done);
       });
     });
   });
diff --git a/spec/javascripts/jobs/store/getters_spec.js b/spec/javascripts/jobs/store/getters_spec.js
index 34e9707eadd..4195d9d3680 100644
--- a/spec/javascripts/jobs/store/getters_spec.js
+++ b/spec/javascripts/jobs/store/getters_spec.js
@@ -180,7 +180,7 @@ describe('Job Store Getters', () => {
       it('returns true', () => {
         localState.job.runners = {
           available: true,
-          online: false
+          online: false,
         };
 
         expect(getters.hasRunnersForProject(localState)).toEqual(true);
@@ -191,7 +191,7 @@ describe('Job Store Getters', () => {
       it('returns false', () => {
         localState.job.runners = {
           available: false,
-          online: false
+          online: false,
         };
 
         expect(getters.hasRunnersForProject(localState)).toEqual(false);
@@ -202,7 +202,7 @@ describe('Job Store Getters', () => {
       it('returns false', () => {
         localState.job.runners = {
           available: false,
-          online: true
+          online: true,
         };
 
         expect(getters.hasRunnersForProject(localState)).toEqual(false);
diff --git a/spec/javascripts/lib/utils/common_utils_spec.js b/spec/javascripts/lib/utils/common_utils_spec.js
index 514d6ddeae5..0fb90c3b78c 100644
--- a/spec/javascripts/lib/utils/common_utils_spec.js
+++ b/spec/javascripts/lib/utils/common_utils_spec.js
@@ -35,9 +35,7 @@ describe('common_utils', () => {
     });
 
     it('should decode params', () => {
-      expect(
-        commonUtils.urlParamsToArray('?label_name%5B%5D=test')[0],
-      ).toBe('label_name[]=test');
+      expect(commonUtils.urlParamsToArray('?label_name%5B%5D=test')[0]).toBe('label_name[]=test');
     });
 
     it('should remove the question mark from the search params', () => {
@@ -49,25 +47,19 @@ describe('common_utils', () => {
 
   describe('urlParamsToObject', () => {
     it('parses path for label with trailing +', () => {
-      expect(
-        commonUtils.urlParamsToObject('label_name[]=label%2B', {}),
-      ).toEqual({
+      expect(commonUtils.urlParamsToObject('label_name[]=label%2B', {})).toEqual({
         label_name: ['label+'],
       });
     });
 
     it('parses path for milestone with trailing +', () => {
-      expect(
-        commonUtils.urlParamsToObject('milestone_title=A%2B', {}),
-      ).toEqual({
+      expect(commonUtils.urlParamsToObject('milestone_title=A%2B', {})).toEqual({
         milestone_title: 'A+',
       });
     });
 
     it('parses path for search terms with spaces', () => {
-      expect(
-        commonUtils.urlParamsToObject('search=two+words', {}),
-      ).toEqual({
+      expect(commonUtils.urlParamsToObject('search=two+words', {})).toEqual({
         search: 'two words',
       });
     });
@@ -187,7 +179,11 @@ describe('common_utils', () => {
 
   describe('parseQueryStringIntoObject', () => {
     it('should return object with query parameters', () => {
-      expect(commonUtils.parseQueryStringIntoObject('scope=all&page=2')).toEqual({ scope: 'all', page: '2' });
+      expect(commonUtils.parseQueryStringIntoObject('scope=all&page=2')).toEqual({
+        scope: 'all',
+        page: '2',
+      });
+
       expect(commonUtils.parseQueryStringIntoObject('scope=all')).toEqual({ scope: 'all' });
       expect(commonUtils.parseQueryStringIntoObject()).toEqual({});
     });
@@ -211,7 +207,9 @@ describe('common_utils', () => {
   describe('buildUrlWithCurrentLocation', () => {
     it('should build an url with current location and given parameters', () => {
       expect(commonUtils.buildUrlWithCurrentLocation()).toEqual(window.location.pathname);
-      expect(commonUtils.buildUrlWithCurrentLocation('?page=2')).toEqual(`${window.location.pathname}?page=2`);
+      expect(commonUtils.buildUrlWithCurrentLocation('?page=2')).toEqual(
+        `${window.location.pathname}?page=2`,
+      );
     });
   });
 
@@ -266,21 +264,24 @@ describe('common_utils', () => {
   });
 
   describe('normalizeCRLFHeaders', () => {
-    beforeEach(function () {
-      this.CLRFHeaders = 'a-header: a-value\nAnother-Header: ANOTHER-VALUE\nLaSt-HeAdEr: last-VALUE';
+    beforeEach(function() {
+      this.CLRFHeaders =
+        'a-header: a-value\nAnother-Header: ANOTHER-VALUE\nLaSt-HeAdEr: last-VALUE';
       spyOn(String.prototype, 'split').and.callThrough();
       this.normalizeCRLFHeaders = commonUtils.normalizeCRLFHeaders(this.CLRFHeaders);
     });
 
-    it('should split by newline', function () {
+    it('should split by newline', function() {
       expect(String.prototype.split).toHaveBeenCalledWith('\n');
     });
 
-    it('should split by colon+space for each header', function () {
-      expect(String.prototype.split.calls.allArgs().filter(args => args[0] === ': ').length).toBe(3);
+    it('should split by colon+space for each header', function() {
+      expect(String.prototype.split.calls.allArgs().filter(args => args[0] === ': ').length).toBe(
+        3,
+      );
     });
 
-    it('should return a normalized headers object', function () {
+    it('should return a normalized headers object', function() {
       expect(this.normalizeCRLFHeaders).toEqual({
         'A-HEADER': 'a-value',
         'ANOTHER-HEADER': 'ANOTHER-VALUE',
@@ -359,67 +360,79 @@ describe('common_utils', () => {
       spyOn(window, 'setTimeout').and.callFake(cb => origSetTimeout(cb, 0));
     });
 
-    it('solves the promise from the callback', (done) => {
+    it('solves the promise from the callback', done => {
       const expectedResponseValue = 'Success!';
-      commonUtils.backOff((next, stop) => (
-        new Promise((resolve) => {
-          resolve(expectedResponseValue);
-        }).then((resp) => {
-          stop(resp);
+      commonUtils
+        .backOff((next, stop) =>
+          new Promise(resolve => {
+            resolve(expectedResponseValue);
+          })
+            .then(resp => {
+              stop(resp);
+            })
+            .catch(done.fail),
+        )
+        .then(respBackoff => {
+          expect(respBackoff).toBe(expectedResponseValue);
+          done();
         })
-      ).catch(done.fail)).then((respBackoff) => {
-        expect(respBackoff).toBe(expectedResponseValue);
-        done();
-      }).catch(done.fail);
+        .catch(done.fail);
     });
 
-    it('catches the rejected promise from the callback ', (done) => {
+    it('catches the rejected promise from the callback ', done => {
       const errorMessage = 'Mistakes were made!';
-      commonUtils.backOff((next, stop) => {
-        new Promise((resolve, reject) => {
-          reject(new Error(errorMessage));
-        }).then((resp) => {
-          stop(resp);
-        }).catch(err => stop(err));
-      }).catch((errBackoffResp) => {
-        expect(errBackoffResp instanceof Error).toBe(true);
-        expect(errBackoffResp.message).toBe(errorMessage);
-        done();
-      });
+      commonUtils
+        .backOff((next, stop) => {
+          new Promise((resolve, reject) => {
+            reject(new Error(errorMessage));
+          })
+            .then(resp => {
+              stop(resp);
+            })
+            .catch(err => stop(err));
+        })
+        .catch(errBackoffResp => {
+          expect(errBackoffResp instanceof Error).toBe(true);
+          expect(errBackoffResp.message).toBe(errorMessage);
+          done();
+        });
     });
 
-    it('solves the promise correctly after retrying a third time', (done) => {
+    it('solves the promise correctly after retrying a third time', done => {
       let numberOfCalls = 1;
       const expectedResponseValue = 'Success!';
-      commonUtils.backOff((next, stop) => (
-        Promise.resolve(expectedResponseValue)
-          .then((resp) => {
-            if (numberOfCalls < 3) {
-              numberOfCalls += 1;
-              next();
-            } else {
-              stop(resp);
-            }
-          })
-      ).catch(done.fail)).then((respBackoff) => {
-        const timeouts = window.setTimeout.calls.allArgs().map(([, timeout]) => timeout);
+      commonUtils
+        .backOff((next, stop) =>
+          Promise.resolve(expectedResponseValue)
+            .then(resp => {
+              if (numberOfCalls < 3) {
+                numberOfCalls += 1;
+                next();
+              } else {
+                stop(resp);
+              }
+            })
+            .catch(done.fail),
+        )
+        .then(respBackoff => {
+          const timeouts = window.setTimeout.calls.allArgs().map(([, timeout]) => timeout);
 
-        expect(timeouts).toEqual([2000, 4000]);
-        expect(respBackoff).toBe(expectedResponseValue);
-        done();
-      }).catch(done.fail);
+          expect(timeouts).toEqual([2000, 4000]);
+          expect(respBackoff).toBe(expectedResponseValue);
+          done();
+        })
+        .catch(done.fail);
     });
 
-    it('rejects the backOff promise after timing out', (done) => {
-      commonUtils.backOff(next => next(), 64000)
-        .catch((errBackoffResp) => {
-          const timeouts = window.setTimeout.calls.allArgs().map(([, timeout]) => timeout);
+    it('rejects the backOff promise after timing out', done => {
+      commonUtils.backOff(next => next(), 64000).catch(errBackoffResp => {
+        const timeouts = window.setTimeout.calls.allArgs().map(([, timeout]) => timeout);
 
-          expect(timeouts).toEqual([2000, 4000, 8000, 16000, 32000, 32000]);
-          expect(errBackoffResp instanceof Error).toBe(true);
-          expect(errBackoffResp.message).toBe('BACKOFF_TIMEOUT');
-          done();
-        });
+        expect(timeouts).toEqual([2000, 4000, 8000, 16000, 32000, 32000]);
+        expect(errBackoffResp instanceof Error).toBe(true);
+        expect(errBackoffResp.message).toBe('BACKOFF_TIMEOUT');
+        done();
+      });
     });
   });
 
@@ -466,11 +479,14 @@ describe('common_utils', () => {
   });
 
   describe('createOverlayIcon', () => {
-    it('should return the favicon with the overlay', (done) => {
-      commonUtils.createOverlayIcon(faviconDataUrl, overlayDataUrl).then((url) => {
-        expect(url).toEqual(faviconWithOverlayDataUrl);
-        done();
-      }).catch(done.fail);
+    it('should return the favicon with the overlay', done => {
+      commonUtils
+        .createOverlayIcon(faviconDataUrl, overlayDataUrl)
+        .then(url => {
+          expect(url).toEqual(faviconWithOverlayDataUrl);
+          done();
+        })
+        .catch(done.fail);
     });
   });
 
@@ -486,11 +502,16 @@ describe('common_utils', () => {
       document.body.removeChild(document.getElementById('favicon'));
     });
 
-    it('should set page favicon to provided favicon overlay', (done) => {
-      commonUtils.setFaviconOverlay(overlayDataUrl).then(() => {
-        expect(document.getElementById('favicon').getAttribute('href')).toEqual(faviconWithOverlayDataUrl);
-        done();
-      }).catch(done.fail);
+    it('should set page favicon to provided favicon overlay', done => {
+      commonUtils
+        .setFaviconOverlay(overlayDataUrl)
+        .then(() => {
+          expect(document.getElementById('favicon').getAttribute('href')).toEqual(
+            faviconWithOverlayDataUrl,
+          );
+          done();
+        })
+        .catch(done.fail);
     });
   });
 
@@ -512,24 +533,24 @@ describe('common_utils', () => {
       document.body.removeChild(document.getElementById('favicon'));
     });
 
-    it('should reset favicon in case of error', (done) => {
+    it('should reset favicon in case of error', done => {
       mock.onGet(BUILD_URL).replyOnce(500);
 
-      commonUtils.setCiStatusFavicon(BUILD_URL)
-        .catch(() => {
-          const favicon = document.getElementById('favicon');
+      commonUtils.setCiStatusFavicon(BUILD_URL).catch(() => {
+        const favicon = document.getElementById('favicon');
 
-          expect(favicon.getAttribute('href')).toEqual(faviconDataUrl);
-          done();
-        });
+        expect(favicon.getAttribute('href')).toEqual(faviconDataUrl);
+        done();
+      });
     });
 
-    it('should set page favicon to CI status favicon based on provided status', (done) => {
+    it('should set page favicon to CI status favicon based on provided status', done => {
       mock.onGet(BUILD_URL).reply(200, {
         favicon: overlayDataUrl,
       });
 
-      commonUtils.setCiStatusFavicon(BUILD_URL)
+      commonUtils
+        .setCiStatusFavicon(BUILD_URL)
         .then(() => {
           const favicon = document.getElementById('favicon');
 
@@ -554,11 +575,15 @@ describe('common_utils', () => {
     });
 
     it('should return the svg for a linked icon', () => {
-      expect(commonUtils.spriteIcon('test')).toEqual('<svg ><use xlink:href="icons.svg#test" /></svg>');
+      expect(commonUtils.spriteIcon('test')).toEqual(
+        '<svg ><use xlink:href="icons.svg#test" /></svg>',
+      );
     });
 
     it('should set svg className when passed', () => {
-      expect(commonUtils.spriteIcon('test', 'fa fa-test')).toEqual('<svg class="fa fa-test"><use xlink:href="icons.svg#test" /></svg>');
+      expect(commonUtils.spriteIcon('test', 'fa fa-test')).toEqual(
+        '<svg class="fa fa-test"><use xlink:href="icons.svg#test" /></svg>',
+      );
     });
   });
 
@@ -578,7 +603,7 @@ describe('common_utils', () => {
 
       const convertedObj = commonUtils.convertObjectPropsToCamelCase(mockObj);
 
-      Object.keys(convertedObj).forEach((prop) => {
+      Object.keys(convertedObj).forEach(prop => {
         expect(snakeRegEx.test(prop)).toBeFalsy();
         expect(convertedObj[prop]).toBe(mockObj[mappings[prop]]);
       });
@@ -597,9 +622,7 @@ describe('common_utils', () => {
         },
       };
 
-      expect(
-        commonUtils.convertObjectPropsToCamelCase(obj),
-      ).toEqual({
+      expect(commonUtils.convertObjectPropsToCamelCase(obj)).toEqual({
         snakeKey: {
           child_snake_key: 'value',
         },
@@ -614,9 +637,7 @@ describe('common_utils', () => {
           },
         };
 
-        expect(
-          commonUtils.convertObjectPropsToCamelCase(obj, { deep: true }),
-        ).toEqual({
+        expect(commonUtils.convertObjectPropsToCamelCase(obj, { deep: true })).toEqual({
           snakeKey: {
             childSnakeKey: 'value',
           },
@@ -630,9 +651,7 @@ describe('common_utils', () => {
           },
         ];
 
-        expect(
-          commonUtils.convertObjectPropsToCamelCase(arr, { deep: true }),
-        ).toEqual([
+        expect(commonUtils.convertObjectPropsToCamelCase(arr, { deep: true })).toEqual([
           {
             childSnakeKey: 'value',
           },
@@ -648,9 +667,7 @@ describe('common_utils', () => {
           ],
         ];
 
-        expect(
-          commonUtils.convertObjectPropsToCamelCase(arr, { deep: true }),
-        ).toEqual([
+        expect(commonUtils.convertObjectPropsToCamelCase(arr, { deep: true })).toEqual([
           [
             {
               childSnakeKey: 'value',
diff --git a/spec/javascripts/lib/utils/datetime_utility_spec.js b/spec/javascripts/lib/utils/datetime_utility_spec.js
index de6b96aab57..d699e66b8ca 100644
--- a/spec/javascripts/lib/utils/datetime_utility_spec.js
+++ b/spec/javascripts/lib/utils/datetime_utility_spec.js
@@ -199,11 +199,11 @@ describe('datefix', () => {
       expect(datetimeUtility.pad(2)).toEqual('02');
     });
 
-    it('should not add a zero when lenght matches the default', () => {
+    it('should not add a zero when length matches the default', () => {
       expect(datetimeUtility.pad(12)).toEqual('12');
     });
 
-    it('should add a 0 when lenght is smaller than the provided', () => {
+    it('should add a 0 when length is smaller than the provided', () => {
       expect(datetimeUtility.pad(12, 3)).toEqual('012');
     });
   });
diff --git a/spec/javascripts/lib/utils/number_utility_spec.js b/spec/javascripts/lib/utils/number_utility_spec.js
index a5099a2a3b8..94c6214c86a 100644
--- a/spec/javascripts/lib/utils/number_utility_spec.js
+++ b/spec/javascripts/lib/utils/number_utility_spec.js
@@ -1,4 +1,10 @@
-import { formatRelevantDigits, bytesToKiB, bytesToMiB, bytesToGiB, numberToHumanSize } from '~/lib/utils/number_utils';
+import {
+  formatRelevantDigits,
+  bytesToKiB,
+  bytesToMiB,
+  bytesToGiB,
+  numberToHumanSize,
+} from '~/lib/utils/number_utils';
 
 describe('Number Utils', () => {
   describe('formatRelevantDigits', () => {
diff --git a/spec/javascripts/lib/utils/text_utility_spec.js b/spec/javascripts/lib/utils/text_utility_spec.js
index ac3270baef5..92ebfc38722 100644
--- a/spec/javascripts/lib/utils/text_utility_spec.js
+++ b/spec/javascripts/lib/utils/text_utility_spec.js
@@ -120,7 +120,7 @@ describe('text_utility', () => {
   });
 
   describe('getFirstCharacterCapitalized', () => {
-    it('returns the first character captialized, if first character is alphabetic', () => {
+    it('returns the first character capitalized, if first character is alphabetic', () => {
       expect(textUtils.getFirstCharacterCapitalized('loremIpsumDolar')).toEqual('L');
       expect(textUtils.getFirstCharacterCapitalized('Sit amit !')).toEqual('S');
     });
diff --git a/spec/javascripts/monitoring/graph/flag_spec.js b/spec/javascripts/monitoring/graph/flag_spec.js
index a837b71db0b..038bfffd44f 100644
--- a/spec/javascripts/monitoring/graph/flag_spec.js
+++ b/spec/javascripts/monitoring/graph/flag_spec.js
@@ -2,7 +2,7 @@ import Vue from 'vue';
 import GraphFlag from '~/monitoring/components/graph/flag.vue';
 import { deploymentData } from '../mock_data';
 
-const createComponent = (propsData) => {
+const createComponent = propsData => {
   const Component = Vue.extend(GraphFlag);
 
   return new Component({
@@ -51,8 +51,7 @@ describe('GraphFlag', () => {
   it('has a line at the currentXCoordinate', () => {
     component = createComponent(defaultValuesComponent);
 
-    expect(component.$el.style.left)
-      .toEqual(`${70 + component.currentXCoordinate}px`);
+    expect(component.$el.style.left).toEqual(`${70 + component.currentXCoordinate}px`);
   });
 
   describe('Deployment flag', () => {
@@ -62,9 +61,7 @@ describe('GraphFlag', () => {
         deploymentFlagData,
       });
 
-      expect(
-        deploymentFlagComponent.$el.querySelector('.popover-title'),
-      ).toContainText('Deployed');
+      expect(deploymentFlagComponent.$el.querySelector('.popover-title')).toContainText('Deployed');
     });
 
     it('contains the ref when a tag is available', () => {
@@ -78,13 +75,13 @@ describe('GraphFlag', () => {
         },
       });
 
-      expect(
-        deploymentFlagComponent.$el.querySelector('.deploy-meta-content'),
-      ).toContainText('f5bcd1d9');
+      expect(deploymentFlagComponent.$el.querySelector('.deploy-meta-content')).toContainText(
+        'f5bcd1d9',
+      );
 
-      expect(
-        deploymentFlagComponent.$el.querySelector('.deploy-meta-content'),
-      ).toContainText('1.0');
+      expect(deploymentFlagComponent.$el.querySelector('.deploy-meta-content')).toContainText(
+        '1.0',
+      );
     });
 
     it('does not contain the ref when a tag is unavailable', () => {
@@ -98,13 +95,13 @@ describe('GraphFlag', () => {
         },
       });
 
-      expect(
-        deploymentFlagComponent.$el.querySelector('.deploy-meta-content'),
-      ).toContainText('f5bcd1d9');
+      expect(deploymentFlagComponent.$el.querySelector('.deploy-meta-content')).toContainText(
+        'f5bcd1d9',
+      );
 
-      expect(
-        deploymentFlagComponent.$el.querySelector('.deploy-meta-content'),
-      ).not.toContainText('1.0');
+      expect(deploymentFlagComponent.$el.querySelector('.deploy-meta-content')).not.toContainText(
+        '1.0',
+      );
     });
   });
 
diff --git a/spec/javascripts/notes/components/discussion_filter_spec.js b/spec/javascripts/notes/components/discussion_filter_spec.js
index 70dd5bb3be5..a81bdf618a3 100644
--- a/spec/javascripts/notes/components/discussion_filter_spec.js
+++ b/spec/javascripts/notes/components/discussion_filter_spec.js
@@ -11,11 +11,13 @@ describe('DiscussionFilter component', () => {
   beforeEach(() => {
     store = createStore();
 
-    const discussions = [{
-      ...discussionMock,
-      id: discussionMock.id,
-      notes: [{ ...discussionMock.notes[0], resolvable: true, resolved: true }],
-    }];
+    const discussions = [
+      {
+        ...discussionMock,
+        id: discussionMock.id,
+        notes: [{ ...discussionMock.notes[0], resolvable: true, resolved: true }],
+      },
+    ];
     const Component = Vue.extend(DiscussionFilter);
     const defaultValue = discussionFiltersMock[0].value;
 
@@ -35,11 +37,15 @@ describe('DiscussionFilter component', () => {
   });
 
   it('renders the all filters', () => {
-    expect(vm.$el.querySelectorAll('.dropdown-menu li').length).toEqual(discussionFiltersMock.length);
+    expect(vm.$el.querySelectorAll('.dropdown-menu li').length).toEqual(
+      discussionFiltersMock.length,
+    );
   });
 
   it('renders the default selected item', () => {
-    expect(vm.$el.querySelector('#discussion-filter-dropdown').textContent.trim()).toEqual(discussionFiltersMock[0].title);
+    expect(vm.$el.querySelector('#discussion-filter-dropdown').textContent.trim()).toEqual(
+      discussionFiltersMock[0].title,
+    );
   });
 
   it('updates to the selected item', () => {
diff --git a/spec/javascripts/notes/components/note_app_spec.js b/spec/javascripts/notes/components/note_app_spec.js
index 06b30375306..3e289a6b8e6 100644
--- a/spec/javascripts/notes/components/note_app_spec.js
+++ b/spec/javascripts/notes/components/note_app_spec.js
@@ -97,7 +97,8 @@ describe('note_app', () => {
     });
 
     it('should render list of notes', done => {
-      const note = mockData.INDIVIDUAL_NOTE_RESPONSE_MAP.GET[
+      const note =
+        mockData.INDIVIDUAL_NOTE_RESPONSE_MAP.GET[
           '/gitlab-org/gitlab-ce/issues/26/discussions.json'
         ][0].notes[0];
 
diff --git a/spec/javascripts/notes/stores/mutation_spec.js b/spec/javascripts/notes/stores/mutation_spec.js
index 9d652ba9f1e..380ab59099d 100644
--- a/spec/javascripts/notes/stores/mutation_spec.js
+++ b/spec/javascripts/notes/stores/mutation_spec.js
@@ -78,7 +78,7 @@ describe('Notes Store mutations', () => {
   });
 
   describe('COLLAPSE_DISCUSSION', () => {
-    it('should collpase an expanded discussion', () => {
+    it('should collapse an expanded discussion', () => {
       const discussion = Object.assign({}, discussionMock, { expanded: true });
 
       const state = {
diff --git a/spec/javascripts/pipelines/empty_state_spec.js b/spec/javascripts/pipelines/empty_state_spec.js
index e21dca45fa1..f12950b8fce 100644
--- a/spec/javascripts/pipelines/empty_state_spec.js
+++ b/spec/javascripts/pipelines/empty_state_spec.js
@@ -24,7 +24,7 @@ describe('Pipelines Empty State', () => {
     expect(component.$el.querySelector('.svg-content svg')).toBeDefined();
   });
 
-  it('should render emtpy state information', () => {
+  it('should render empty state information', () => {
     expect(component.$el.querySelector('h4').textContent).toContain('Build with confidence');
 
     expect(
diff --git a/spec/javascripts/pipelines/graph/action_component_spec.js b/spec/javascripts/pipelines/graph/action_component_spec.js
index 027066e1d4d..3d2232ff239 100644
--- a/spec/javascripts/pipelines/graph/action_component_spec.js
+++ b/spec/javascripts/pipelines/graph/action_component_spec.js
@@ -50,7 +50,7 @@ describe('pipeline graph action component', () => {
   });
 
   describe('on click', () => {
-    it('emits `pipelineActionRequestComplete` after a successfull request', done => {
+    it('emits `pipelineActionRequestComplete` after a successful request', done => {
       spyOn(component, '$emit');
 
       component.$el.click();
diff --git a/spec/javascripts/pipelines/graph/graph_component_spec.js b/spec/javascripts/pipelines/graph/graph_component_spec.js
index b6fa4272c8b..96a2d5f62fa 100644
--- a/spec/javascripts/pipelines/graph/graph_component_spec.js
+++ b/spec/javascripts/pipelines/graph/graph_component_spec.js
@@ -40,7 +40,9 @@ describe('graph component', () => {
       ).toEqual(true);
 
       expect(
-        component.$el.querySelector('.stage-column:nth-child(2) .build:nth-child(1)').classList.contains('left-connector'),
+        component.$el
+          .querySelector('.stage-column:nth-child(2) .build:nth-child(1)')
+          .classList.contains('left-connector'),
       ).toEqual(true);
 
       expect(component.$el.querySelector('loading-icon')).toBe(null);
@@ -56,7 +58,9 @@ describe('graph component', () => {
         pipeline: graphJSON,
       });
 
-      expect(component.$el.querySelector('.stage-column:nth-child(2) .stage-name').textContent.trim()).toEqual('Deploy &lt;img src=x onerror=alert(document.domain)&gt;');
+      expect(
+        component.$el.querySelector('.stage-column:nth-child(2) .stage-name').textContent.trim(),
+      ).toEqual('Deploy &lt;img src=x onerror=alert(document.domain)&gt;');
     });
   });
 });
diff --git a/spec/javascripts/pipelines/pipelines_spec.js b/spec/javascripts/pipelines/pipelines_spec.js
index 37908153e0e..97ded16db69 100644
--- a/spec/javascripts/pipelines/pipelines_spec.js
+++ b/spec/javascripts/pipelines/pipelines_spec.js
@@ -372,7 +372,7 @@ describe('Pipelines', () => {
     });
   });
 
-  describe('successfull request', () => {
+  describe('successful request', () => {
     describe('with pipelines', () => {
       beforeEach(() => {
         mock.onGet('twitter/flight/pipelines.json').reply(200, pipelines);
@@ -667,7 +667,7 @@ describe('Pipelines', () => {
         });
       });
 
-      it('returns false when state is emtpy state', done => {
+      it('returns false when state is empty state', done => {
         vm.isLoading = false;
         vm.hasMadeRequest = true;
         vm.hasGitlabCi = false;
diff --git a/spec/javascripts/pipelines/stage_spec.js b/spec/javascripts/pipelines/stage_spec.js
index a3caaeb44dc..3c8b8032de8 100644
--- a/spec/javascripts/pipelines/stage_spec.js
+++ b/spec/javascripts/pipelines/stage_spec.js
@@ -40,7 +40,7 @@ describe('Pipelines stage component', () => {
     expect(component.$el.querySelector('button').getAttribute('data-toggle')).toEqual('dropdown');
   });
 
-  describe('with successfull request', () => {
+  describe('with successful request', () => {
     beforeEach(() => {
       mock.onGet('path.json').reply(200, stageReply);
     });
diff --git a/spec/javascripts/sidebar/assignees_spec.js b/spec/javascripts/sidebar/assignees_spec.js
index e7f8f4f9936..eced4925489 100644
--- a/spec/javascripts/sidebar/assignees_spec.js
+++ b/spec/javascripts/sidebar/assignees_spec.js
@@ -78,9 +78,7 @@ describe('Assignee component', () => {
       component = new AssigneeComponent({
         propsData: {
           rootPath: 'http://localhost:3000',
-          users: [
-            UsersMock.user,
-          ],
+          users: [UsersMock.user],
           editable: false,
         },
       }).$mount();
@@ -90,7 +88,10 @@ describe('Assignee component', () => {
 
       expect(collapsed.childElementCount).toEqual(1);
       expect(assignee.querySelector('.avatar').getAttribute('src')).toEqual(UsersMock.user.avatar);
-      expect(assignee.querySelector('.avatar').getAttribute('alt')).toEqual(`${UsersMock.user.name}'s avatar`);
+      expect(assignee.querySelector('.avatar').getAttribute('alt')).toEqual(
+        `${UsersMock.user.name}'s avatar`,
+      );
+
       expect(assignee.querySelector('.author').innerText.trim()).toEqual(UsersMock.user.name);
     });
 
@@ -98,34 +99,38 @@ describe('Assignee component', () => {
       component = new AssigneeComponent({
         propsData: {
           rootPath: 'http://localhost:3000/',
-          users: [
-            UsersMock.user,
-          ],
+          users: [UsersMock.user],
           editable: true,
         },
       }).$mount();
 
       expect(component.$el.querySelector('.author-link')).not.toBeNull();
       // The image
-      expect(component.$el.querySelector('.author-link img').getAttribute('src')).toEqual(UsersMock.user.avatar);
+      expect(component.$el.querySelector('.author-link img').getAttribute('src')).toEqual(
+        UsersMock.user.avatar,
+      );
       // Author name
-      expect(component.$el.querySelector('.author-link .author').innerText.trim()).toEqual(UsersMock.user.name);
+      expect(component.$el.querySelector('.author-link .author').innerText.trim()).toEqual(
+        UsersMock.user.name,
+      );
       // Username
-      expect(component.$el.querySelector('.author-link .username').innerText.trim()).toEqual(`@${UsersMock.user.username}`);
+      expect(component.$el.querySelector('.author-link .username').innerText.trim()).toEqual(
+        `@${UsersMock.user.username}`,
+      );
     });
 
     it('has the root url present in the assigneeUrl method', () => {
       component = new AssigneeComponent({
         propsData: {
           rootPath: 'http://localhost:3000/',
-          users: [
-            UsersMock.user,
-          ],
+          users: [UsersMock.user],
           editable: true,
         },
       }).$mount();
 
-      expect(component.assigneeUrl(UsersMock.user).indexOf('http://localhost:3000/')).not.toEqual(-1);
+      expect(component.assigneeUrl(UsersMock.user).indexOf('http://localhost:3000/')).not.toEqual(
+        -1,
+      );
     });
   });
 
@@ -147,13 +152,19 @@ describe('Assignee component', () => {
       const first = collapsed.children[0];
 
       expect(first.querySelector('.avatar').getAttribute('src')).toEqual(users[0].avatar);
-      expect(first.querySelector('.avatar').getAttribute('alt')).toEqual(`${users[0].name}'s avatar`);
+      expect(first.querySelector('.avatar').getAttribute('alt')).toEqual(
+        `${users[0].name}'s avatar`,
+      );
+
       expect(first.querySelector('.author').innerText.trim()).toEqual(users[0].name);
 
       const second = collapsed.children[1];
 
       expect(second.querySelector('.avatar').getAttribute('src')).toEqual(users[1].avatar);
-      expect(second.querySelector('.avatar').getAttribute('alt')).toEqual(`${users[1].name}'s avatar`);
+      expect(second.querySelector('.avatar').getAttribute('alt')).toEqual(
+        `${users[1].name}'s avatar`,
+      );
+
       expect(second.querySelector('.author').innerText.trim()).toEqual(users[1].name);
     });
 
@@ -174,7 +185,10 @@ describe('Assignee component', () => {
       const first = collapsed.children[0];
 
       expect(first.querySelector('.avatar').getAttribute('src')).toEqual(users[0].avatar);
-      expect(first.querySelector('.avatar').getAttribute('alt')).toEqual(`${users[0].name}'s avatar`);
+      expect(first.querySelector('.avatar').getAttribute('alt')).toEqual(
+        `${users[0].name}'s avatar`,
+      );
+
       expect(first.querySelector('.author').innerText.trim()).toEqual(users[0].name);
 
       const second = collapsed.children[1];
@@ -196,7 +210,7 @@ describe('Assignee component', () => {
       expect(component.$el.querySelector('.user-list-more')).toBe(null);
     });
 
-    it('Shows the "show-less" assignees label', (done) => {
+    it('Shows the "show-less" assignees label', done => {
       const users = UsersMockHelper.createNumberRandomUsers(6);
       component = new AssigneeComponent({
         propsData: {
@@ -206,21 +220,26 @@ describe('Assignee component', () => {
         },
       }).$mount();
 
-      expect(component.$el.querySelectorAll('.user-item').length).toEqual(component.defaultRenderCount);
+      expect(component.$el.querySelectorAll('.user-item').length).toEqual(
+        component.defaultRenderCount,
+      );
+
       expect(component.$el.querySelector('.user-list-more')).not.toBe(null);
       const usersLabelExpectation = users.length - component.defaultRenderCount;
 
-      expect(component.$el.querySelector('.user-list-more .btn-link').innerText.trim())
-        .not.toBe(`+${usersLabelExpectation} more`);
+      expect(component.$el.querySelector('.user-list-more .btn-link').innerText.trim()).not.toBe(
+        `+${usersLabelExpectation} more`,
+      );
       component.toggleShowLess();
       Vue.nextTick(() => {
-        expect(component.$el.querySelector('.user-list-more .btn-link').innerText.trim())
-          .toBe('- show less');
+        expect(component.$el.querySelector('.user-list-more .btn-link').innerText.trim()).toBe(
+          '- show less',
+        );
         done();
       });
     });
 
-    it('Shows the "show-less" when "n+ more " label is clicked', (done) => {
+    it('Shows the "show-less" when "n+ more " label is clicked', done => {
       const users = UsersMockHelper.createNumberRandomUsers(6);
       component = new AssigneeComponent({
         propsData: {
@@ -232,8 +251,9 @@ describe('Assignee component', () => {
 
       component.$el.querySelector('.user-list-more .btn-link').click();
       Vue.nextTick(() => {
-        expect(component.$el.querySelector('.user-list-more .btn-link').innerText.trim())
-          .toBe('- show less');
+        expect(component.$el.querySelector('.user-list-more .btn-link').innerText.trim()).toBe(
+          '- show less',
+        );
         done();
       });
     });
@@ -264,16 +284,18 @@ describe('Assignee component', () => {
       });
 
       it('shows "+1 more" label', () => {
-        expect(component.$el.querySelector('.user-list-more .btn-link').innerText.trim())
-          .toBe('+ 1 more');
+        expect(component.$el.querySelector('.user-list-more .btn-link').innerText.trim()).toBe(
+          '+ 1 more',
+        );
       });
 
-      it('shows "show less" label', (done) => {
+      it('shows "show less" label', done => {
         component.toggleShowLess();
 
         Vue.nextTick(() => {
-          expect(component.$el.querySelector('.user-list-more .btn-link').innerText.trim())
-            .toBe('- show less');
+          expect(component.$el.querySelector('.user-list-more .btn-link').innerText.trim()).toBe(
+            '- show less',
+          );
           done();
         });
       });
diff --git a/spec/javascripts/vue_mr_widget/components/mr_widget_pipeline_spec.js b/spec/javascripts/vue_mr_widget/components/mr_widget_pipeline_spec.js
index 689580e6b91..6c7637eed13 100644
--- a/spec/javascripts/vue_mr_widget/components/mr_widget_pipeline_spec.js
+++ b/spec/javascripts/vue_mr_widget/components/mr_widget_pipeline_spec.js
@@ -69,12 +69,12 @@ describe('MRWidgetPipeline', () => {
         pipeline: mockData.pipeline,
         hasCi: true,
         ciStatus: null,
-          troubleshootingDocsPath: 'help',
+        troubleshootingDocsPath: 'help',
       });
 
-      expect(
-        vm.$el.querySelector('.media-body').textContent.trim(),
-      ).toContain('Could not retrieve the pipeline status. For troubleshooting steps, read the <a href="help">documentation.</a>');
+      expect(vm.$el.querySelector('.media-body').textContent.trim()).toContain(
+        'Could not retrieve the pipeline status. For troubleshooting steps, read the <a href="help">documentation.</a>',
+      );
     });
 
     describe('with a pipeline', () => {
@@ -88,34 +88,36 @@ describe('MRWidgetPipeline', () => {
       });
 
       it('should render pipeline ID', () => {
-        expect(
-          vm.$el.querySelector('.pipeline-id').textContent.trim(),
-        ).toEqual(`#${mockData.pipeline.id}`);
+        expect(vm.$el.querySelector('.pipeline-id').textContent.trim()).toEqual(
+          `#${mockData.pipeline.id}`,
+        );
       });
 
       it('should render pipeline status and commit id', () => {
-        expect(
-          vm.$el.querySelector('.media-body').textContent.trim(),
-        ).toContain(mockData.pipeline.details.status.label);
+        expect(vm.$el.querySelector('.media-body').textContent.trim()).toContain(
+          mockData.pipeline.details.status.label,
+        );
 
-        expect(
-          vm.$el.querySelector('.js-commit-link').textContent.trim(),
-        ).toEqual(mockData.pipeline.commit.short_id);
+        expect(vm.$el.querySelector('.js-commit-link').textContent.trim()).toEqual(
+          mockData.pipeline.commit.short_id,
+        );
 
-        expect(
-          vm.$el.querySelector('.js-commit-link').getAttribute('href'),
-        ).toEqual(mockData.pipeline.commit.commit_path);
+        expect(vm.$el.querySelector('.js-commit-link').getAttribute('href')).toEqual(
+          mockData.pipeline.commit.commit_path,
+        );
       });
 
       it('should render pipeline graph', () => {
         expect(vm.$el.querySelector('.mr-widget-pipeline-graph')).toBeDefined();
-        expect(vm.$el.querySelectorAll('.stage-container').length).toEqual(mockData.pipeline.details.stages.length);
+        expect(vm.$el.querySelectorAll('.stage-container').length).toEqual(
+          mockData.pipeline.details.stages.length,
+        );
       });
 
       it('should render coverage information', () => {
-        expect(
-          vm.$el.querySelector('.media-body').textContent,
-        ).toContain(`Coverage ${mockData.pipeline.coverage}`);
+        expect(vm.$el.querySelector('.media-body').textContent).toContain(
+          `Coverage ${mockData.pipeline.coverage}`,
+        );
       });
     });
 
@@ -133,30 +135,30 @@ describe('MRWidgetPipeline', () => {
       });
 
       it('should render pipeline ID', () => {
-        expect(
-          vm.$el.querySelector('.pipeline-id').textContent.trim(),
-        ).toEqual(`#${mockData.pipeline.id}`);
+        expect(vm.$el.querySelector('.pipeline-id').textContent.trim()).toEqual(
+          `#${mockData.pipeline.id}`,
+        );
       });
 
       it('should render pipeline status', () => {
-        expect(
-          vm.$el.querySelector('.media-body').textContent.trim(),
-        ).toContain(mockData.pipeline.details.status.label);
+        expect(vm.$el.querySelector('.media-body').textContent.trim()).toContain(
+          mockData.pipeline.details.status.label,
+        );
 
-        expect(
-          vm.$el.querySelector('.js-commit-link'),
-        ).toBeNull();
+        expect(vm.$el.querySelector('.js-commit-link')).toBeNull();
       });
 
       it('should render pipeline graph', () => {
         expect(vm.$el.querySelector('.mr-widget-pipeline-graph')).toBeDefined();
-        expect(vm.$el.querySelectorAll('.stage-container').length).toEqual(mockData.pipeline.details.stages.length);
+        expect(vm.$el.querySelectorAll('.stage-container').length).toEqual(
+          mockData.pipeline.details.stages.length,
+        );
       });
 
       it('should render coverage information', () => {
-        expect(
-          vm.$el.querySelector('.media-body').textContent,
-        ).toContain(`Coverage ${mockData.pipeline.coverage}`);
+        expect(vm.$el.querySelector('.media-body').textContent).toContain(
+          `Coverage ${mockData.pipeline.coverage}`,
+        );
       });
     });
 
@@ -172,9 +174,7 @@ describe('MRWidgetPipeline', () => {
           troubleshootingDocsPath: 'help',
         });
 
-        expect(
-          vm.$el.querySelector('.media-body').textContent,
-        ).not.toContain('Coverage');
+        expect(vm.$el.querySelector('.media-body').textContent).not.toContain('Coverage');
       });
     });
 
diff --git a/spec/javascripts/vue_mr_widget/components/states/mr_widget_merged_spec.js b/spec/javascripts/vue_mr_widget/components/states/mr_widget_merged_spec.js
index d68342635ef..da5cb752c6f 100644
--- a/spec/javascripts/vue_mr_widget/components/states/mr_widget_merged_spec.js
+++ b/spec/javascripts/vue_mr_widget/components/states/mr_widget_merged_spec.js
@@ -69,7 +69,7 @@ describe('MRWidgetMerged', () => {
         expect(vm.shouldShowRemoveSourceBranch).toEqual(true);
       });
 
-      it('returns false wehn sourceBranchRemoved is true', () => {
+      it('returns false when sourceBranchRemoved is true', () => {
         vm.mr.sourceBranchRemoved = true;
 
         expect(vm.shouldShowRemoveSourceBranch).toEqual(false);
diff --git a/spec/javascripts/vue_mr_widget/mock_data.js b/spec/javascripts/vue_mr_widget/mock_data.js
index c5e30a730cb..17554c4fe42 100644
--- a/spec/javascripts/vue_mr_widget/mock_data.js
+++ b/spec/javascripts/vue_mr_widget/mock_data.js
@@ -218,6 +218,7 @@ export default {
   diverged_commits_count: 0,
   only_allow_merge_if_pipeline_succeeds: false,
   commit_change_content_path: '/root/acets-app/merge_requests/22/commit_change_content',
-  merge_commit_path: 'http://localhost:3000/root/acets-app/commit/53027d060246c8f47e4a9310fb332aa52f221775',
-  troubleshooting_docs_path: 'help'
+  merge_commit_path:
+    'http://localhost:3000/root/acets-app/commit/53027d060246c8f47e4a9310fb332aa52f221775',
+  troubleshooting_docs_path: 'help',
 };
diff --git a/spec/javascripts/vue_mr_widget/mr_widget_options_spec.js b/spec/javascripts/vue_mr_widget/mr_widget_options_spec.js
index 27b6c91e154..09fbe87b27e 100644
--- a/spec/javascripts/vue_mr_widget/mr_widget_options_spec.js
+++ b/spec/javascripts/vue_mr_widget/mr_widget_options_spec.js
@@ -454,7 +454,7 @@ describe('mrWidgetOptions', () => {
       deployed_at: '2017-03-22T22:44:42.258Z',
       deployed_at_formatted: 'Mar 22, 2017 10:44pm',
       changes,
-      status: 'success'
+      status: 'success',
     };
 
     beforeEach(done => {
@@ -607,33 +607,36 @@ describe('mrWidgetOptions', () => {
 
       describe('with post merge deployments', () => {
         beforeEach(done => {
-          vm.mr.postMergeDeployments = [{
-            id: 15,
-            name: 'review/diplo',
-            url: '/root/acets-review-apps/environments/15',
-            stop_url: '/root/acets-review-apps/environments/15/stop',
-            metrics_url: '/root/acets-review-apps/environments/15/deployments/1/metrics',
-            metrics_monitoring_url: '/root/acets-review-apps/environments/15/metrics',
-            external_url: 'http://diplo.',
-            external_url_formatted: 'diplo.',
-            deployed_at: '2017-03-22T22:44:42.258Z',
-            deployed_at_formatted: 'Mar 22, 2017 10:44pm',
-            changes: [
-              {
-                path: 'index.html',
-                external_url: 'http://root-master-patch-91341.volatile-watch.surge.sh/index.html',
-              },
-              {
-                path: 'imgs/gallery.html',
-                external_url: 'http://root-master-patch-91341.volatile-watch.surge.sh/imgs/gallery.html',
-              },
-              {
-                path: 'about/',
-                external_url: 'http://root-master-patch-91341.volatile-watch.surge.sh/about/',
-              },
-            ],
-            status: 'success'
-          }];
+          vm.mr.postMergeDeployments = [
+            {
+              id: 15,
+              name: 'review/diplo',
+              url: '/root/acets-review-apps/environments/15',
+              stop_url: '/root/acets-review-apps/environments/15/stop',
+              metrics_url: '/root/acets-review-apps/environments/15/deployments/1/metrics',
+              metrics_monitoring_url: '/root/acets-review-apps/environments/15/metrics',
+              external_url: 'http://diplo.',
+              external_url_formatted: 'diplo.',
+              deployed_at: '2017-03-22T22:44:42.258Z',
+              deployed_at_formatted: 'Mar 22, 2017 10:44pm',
+              changes: [
+                {
+                  path: 'index.html',
+                  external_url: 'http://root-master-patch-91341.volatile-watch.surge.sh/index.html',
+                },
+                {
+                  path: 'imgs/gallery.html',
+                  external_url:
+                    'http://root-master-patch-91341.volatile-watch.surge.sh/imgs/gallery.html',
+                },
+                {
+                  path: 'about/',
+                  external_url: 'http://root-master-patch-91341.volatile-watch.surge.sh/about/',
+                },
+              ],
+              status: 'success',
+            },
+          ];
 
           vm.$nextTick(done);
         });
diff --git a/spec/javascripts/vue_shared/components/filtered_search_dropdown_spec.js b/spec/javascripts/vue_shared/components/filtered_search_dropdown_spec.js
index b71cb36ecf6..b84b5ae67a8 100644
--- a/spec/javascripts/vue_shared/components/filtered_search_dropdown_spec.js
+++ b/spec/javascripts/vue_shared/components/filtered_search_dropdown_spec.js
@@ -41,7 +41,7 @@ describe('Filtered search dropdown', () => {
     });
   });
 
-  describe('when visible number is bigger than the items lenght', () => {
+  describe('when visible number is bigger than the items length', () => {
     beforeEach(() => {
       vm = mountComponent(Component, {
         items: [{ title: 'One' }, { title: 'Two' }, { title: 'Three' }],
diff --git a/spec/javascripts/vue_shared/components/sidebar/collapsed_grouped_date_picker_spec.js b/spec/javascripts/vue_shared/components/sidebar/collapsed_grouped_date_picker_spec.js
index 3483b7d387d..c507a97d37e 100644
--- a/spec/javascripts/vue_shared/components/sidebar/collapsed_grouped_date_picker_spec.js
+++ b/spec/javascripts/vue_shared/components/sidebar/collapsed_grouped_date_picker_spec.js
@@ -12,7 +12,7 @@ describe('collapsedGroupedDatePicker', () => {
   });
 
   describe('toggleCollapse events', () => {
-    beforeEach((done) => {
+    beforeEach(done => {
       spyOn(vm, 'toggleSidebar');
       vm.minDate = new Date('07/17/2016');
       Vue.nextTick(done);
@@ -26,7 +26,7 @@ describe('collapsedGroupedDatePicker', () => {
   });
 
   describe('minDate and maxDate', () => {
-    beforeEach((done) => {
+    beforeEach(done => {
       vm.minDate = new Date('07/17/2016');
       vm.maxDate = new Date('07/17/2017');
       Vue.nextTick(done);
@@ -42,7 +42,7 @@ describe('collapsedGroupedDatePicker', () => {
   });
 
   describe('minDate', () => {
-    beforeEach((done) => {
+    beforeEach(done => {
       vm.minDate = new Date('07/17/2016');
       Vue.nextTick(done);
     });
@@ -56,7 +56,7 @@ describe('collapsedGroupedDatePicker', () => {
   });
 
   describe('maxDate', () => {
-    beforeEach((done) => {
+    beforeEach(done => {
       vm.maxDate = new Date('07/17/2017');
       Vue.nextTick(done);
     });
diff --git a/spec/javascripts/vue_shared/components/sidebar/date_picker_spec.js b/spec/javascripts/vue_shared/components/sidebar/date_picker_spec.js
index 1581f4e3eb1..805ba7b9947 100644
--- a/spec/javascripts/vue_shared/components/sidebar/date_picker_spec.js
+++ b/spec/javascripts/vue_shared/components/sidebar/date_picker_spec.js
@@ -41,7 +41,7 @@ describe('sidebarDatePicker', () => {
     expect(vm.$el.querySelector('.value-content span').innerText.trim()).toEqual('None');
   });
 
-  it('should render date-picker when editing', (done) => {
+  it('should render date-picker when editing', done => {
     vm.editing = true;
     Vue.nextTick(() => {
       expect(vm.$el.querySelector('.pika-label')).toBeDefined();
@@ -50,7 +50,7 @@ describe('sidebarDatePicker', () => {
   });
 
   describe('editable', () => {
-    beforeEach((done) => {
+    beforeEach(done => {
       vm.editable = true;
       Vue.nextTick(done);
     });
@@ -59,7 +59,7 @@ describe('sidebarDatePicker', () => {
       expect(vm.$el.querySelector('.title .btn-blank').innerText.trim()).toEqual('Edit');
     });
 
-    it('should enable editing when edit button is clicked', (done) => {
+    it('should enable editing when edit button is clicked', done => {
       vm.isLoading = false;
       Vue.nextTick(() => {
         vm.$el.querySelector('.title .btn-blank').click();
@@ -70,7 +70,7 @@ describe('sidebarDatePicker', () => {
     });
   });
 
-  it('should render date if selectedDate', (done) => {
+  it('should render date if selectedDate', done => {
     vm.selectedDate = new Date('07/07/2017');
     Vue.nextTick(() => {
       expect(vm.$el.querySelector('.value-content strong').innerText.trim()).toEqual('Jul 7, 2017');
@@ -79,7 +79,7 @@ describe('sidebarDatePicker', () => {
   });
 
   describe('selectedDate and editable', () => {
-    beforeEach((done) => {
+    beforeEach(done => {
       vm.selectedDate = new Date('07/07/2017');
       vm.editable = true;
       Vue.nextTick(done);
@@ -100,7 +100,7 @@ describe('sidebarDatePicker', () => {
   });
 
   describe('showToggleSidebar', () => {
-    beforeEach((done) => {
+    beforeEach(done => {
       vm.showToggleSidebar = true;
       Vue.nextTick(done);
     });
diff --git a/spec/javascripts/vue_shared/components/sidebar/labels_select/dropdown_value_collapsed_spec.js b/spec/javascripts/vue_shared/components/sidebar/labels_select/dropdown_value_collapsed_spec.js
index 9a691116cf8..804b33422bd 100644
--- a/spec/javascripts/vue_shared/components/sidebar/labels_select/dropdown_value_collapsed_spec.js
+++ b/spec/javascripts/vue_shared/components/sidebar/labels_select/dropdown_value_collapsed_spec.js
@@ -49,7 +49,9 @@ describe('DropdownValueCollapsedComponent', () => {
 
         const vmMoreLabels = createComponent(mockMoreLabels);
 
-        expect(vmMoreLabels.labelsList).toBe('Foo Label, Foo Label, Foo Label, Foo Label, Foo Label, and 2 more');
+        expect(vmMoreLabels.labelsList).toBe(
+          'Foo Label, Foo Label, Foo Label, Foo Label, Foo Label, and 2 more',
+        );
         vmMoreLabels.$destroy();
       });
 
diff --git a/spec/javascripts/vue_shared/components/user_avatar/user_avatar_link_spec.js b/spec/javascripts/vue_shared/components/user_avatar/user_avatar_link_spec.js
index 50b8d49d4bd..e022245d3ea 100644
--- a/spec/javascripts/vue_shared/components/user_avatar/user_avatar_link_spec.js
+++ b/spec/javascripts/vue_shared/components/user_avatar/user_avatar_link_spec.js
@@ -44,7 +44,7 @@ describe('User Avatar Link Component', function() {
     expect(this.userAvatarLink.$el.querySelector('img')).not.toBeNull();
   });
 
-  it('should return neccessary props as defined', function() {
+  it('should return necessary props as defined', function() {
     _.each(this.propsData, (val, key) => {
       expect(this.userAvatarLink[key]).toBeDefined();
     });
diff --git a/spec/lib/banzai/filter/autolink_filter_spec.rb b/spec/lib/banzai/filter/autolink_filter_spec.rb
index a50329473ad..7a457403b51 100644
--- a/spec/lib/banzai/filter/autolink_filter_spec.rb
+++ b/spec/lib/banzai/filter/autolink_filter_spec.rb
@@ -76,7 +76,7 @@ describe Banzai::Filter::AutolinkFilter do
       expect(doc.at_css('a')['href']).to eq link
     end
 
-    it 'autolinks multiple occurences of smb' do
+    it 'autolinks multiple occurrences of smb' do
       link1 = 'smb:///Volumes/shared/foo.pdf'
       link2 = 'smb:///Volumes/shared/bar.pdf'
 
diff --git a/spec/lib/banzai/filter/relative_link_filter_spec.rb b/spec/lib/banzai/filter/relative_link_filter_spec.rb
index ed1ebe9ebf6..415ded05e6e 100644
--- a/spec/lib/banzai/filter/relative_link_filter_spec.rb
+++ b/spec/lib/banzai/filter/relative_link_filter_spec.rb
@@ -226,7 +226,7 @@ describe Banzai::Filter::RelativeLinkFilter do
       let(:ref) {'mark#\'@],+;-._/#@!$&()+down'}
 
       it 'correctly escapes the ref' do
-        # Adressable won't escape the '#', so we do this manually
+        # Addressable won't escape the '#', so we do this manually
         ref_escaped = 'mark%23\'@%5D,+;-._/%23@!$&()+down'
 
         # Stub this method so the branch doesn't actually need to be in the repo
diff --git a/spec/lib/container_registry/blob_spec.rb b/spec/lib/container_registry/blob_spec.rb
index c73faa55513..d3fff5bad42 100644
--- a/spec/lib/container_registry/blob_spec.rb
+++ b/spec/lib/container_registry/blob_spec.rb
@@ -64,7 +64,7 @@ describe ContainerRegistry::Blob do
         .to_return(status: 200)
     end
 
-    it 'returns true when blob has been successfuly deleted' do
+    it 'returns true when blob has been successfully deleted' do
       expect(blob.delete).to be_truthy
     end
   end
diff --git a/spec/lib/gitlab/background_migration/create_fork_network_memberships_range_spec.rb b/spec/lib/gitlab/background_migration/create_fork_network_memberships_range_spec.rb
index e1c4f9cfea7..5076996474f 100644
--- a/spec/lib/gitlab/background_migration/create_fork_network_memberships_range_spec.rb
+++ b/spec/lib/gitlab/background_migration/create_fork_network_memberships_range_spec.rb
@@ -118,7 +118,7 @@ describe Gitlab::BackgroundMigration::CreateForkNetworkMembershipsRange, :migrat
       expect(fork_network_members.count).to eq(12)
     end
 
-    it 'knows when not all memberships withing a batch have been created' do
+    it 'knows when not all memberships within a batch have been created' do
       expect(migration.missing_members?(8, 10)).to be_truthy
     end
   end
diff --git a/spec/lib/gitlab/checks/lfs_integrity_spec.rb b/spec/lib/gitlab/checks/lfs_integrity_spec.rb
index 0488720cec8..887ea8fc1e0 100644
--- a/spec/lib/gitlab/checks/lfs_integrity_spec.rb
+++ b/spec/lib/gitlab/checks/lfs_integrity_spec.rb
@@ -68,7 +68,7 @@ describe Gitlab::Checks::LfsIntegrity do
         expect(subject.objects_missing?).to be_truthy
       end
 
-      it 'is false parent project already conatins LFS objects for the fork' do
+      it 'is false parent project already contains LFS objects for the fork' do
         lfs_object = create(:lfs_object, oid: blob_object.lfs_oid)
         create(:lfs_objects_project, project: parent_project, lfs_object: lfs_object)
 
diff --git a/spec/lib/gitlab/ci/ansi2html_spec.rb b/spec/lib/gitlab/ci/ansi2html_spec.rb
index 7549e9941b6..5a5c071c639 100644
--- a/spec/lib/gitlab/ci/ansi2html_spec.rb
+++ b/spec/lib/gitlab/ci/ansi2html_spec.rb
@@ -7,7 +7,7 @@ describe Gitlab::Ci::Ansi2html do
     expect(convert_html("Hello")).to eq('Hello')
   end
 
-  it "strips non-color-changing controll sequences" do
+  it "strips non-color-changing control sequences" do
     expect(convert_html("Hello \e[2Kworld")).to eq('Hello world')
   end
 
diff --git a/spec/lib/gitlab/ci/build/policy/variables_spec.rb b/spec/lib/gitlab/ci/build/policy/variables_spec.rb
index 854c4cb718c..c2c0742efc3 100644
--- a/spec/lib/gitlab/ci/build/policy/variables_spec.rb
+++ b/spec/lib/gitlab/ci/build/policy/variables_spec.rb
@@ -24,7 +24,7 @@ describe Gitlab::Ci::Build::Policy::Variables do
       expect(policy).to be_satisfied_by(pipeline, seed)
     end
 
-    it 'is not satisfied by an overriden empty variable' do
+    it 'is not satisfied by an overridden empty variable' do
       policy = described_class.new(['$CI_PROJECT_NAME'])
 
       expect(policy).not_to be_satisfied_by(pipeline, seed)
diff --git a/spec/lib/gitlab/ci/config/entry/global_spec.rb b/spec/lib/gitlab/ci/config/entry/global_spec.rb
index 1860ed79bfd..7c18514934e 100644
--- a/spec/lib/gitlab/ci/config/entry/global_spec.rb
+++ b/spec/lib/gitlab/ci/config/entry/global_spec.rb
@@ -219,7 +219,7 @@ describe Gitlab::Ci::Config::Entry::Global do
 
     ##
     # When nodes are specified but not defined, we assume that
-    # configuration is valid, and we asume that entry is simply undefined,
+    # configuration is valid, and we assume that entry is simply undefined,
     # despite the fact, that key is present. See issue #18775 for more
     # details.
     #
diff --git a/spec/lib/gitlab/cross_project_access/check_info_spec.rb b/spec/lib/gitlab/cross_project_access/check_info_spec.rb
index bc9dbf2bece..239fa364f5e 100644
--- a/spec/lib/gitlab/cross_project_access/check_info_spec.rb
+++ b/spec/lib/gitlab/cross_project_access/check_info_spec.rb
@@ -50,7 +50,7 @@ describe Gitlab::CrossProjectAccess::CheckInfo do
       expect(info.should_run?(dummy_controller)).to be_truthy
     end
 
-    it 'returns the the oposite of #should_skip? when the check is a skip' do
+    it 'returns the the opposite of #should_skip? when the check is a skip' do
       info = described_class.new({}, nil, nil, true)
 
       expect(info).to receive(:should_skip?).with(dummy_controller).and_return(false)
@@ -101,7 +101,7 @@ describe Gitlab::CrossProjectAccess::CheckInfo do
       expect(info.should_skip?(dummy_controller)).to be_truthy
     end
 
-    it 'returns the the oposite of #should_run? when the check is not a skip' do
+    it 'returns the the opposite of #should_run? when the check is not a skip' do
       info = described_class.new({}, nil, nil, false)
 
       expect(info).to receive(:should_run?).with(dummy_controller).and_return(false)
diff --git a/spec/lib/gitlab/database/rename_reserved_paths_migration/v1/rename_base_spec.rb b/spec/lib/gitlab/database/rename_reserved_paths_migration/v1/rename_base_spec.rb
index cc7cb3f23fd..248cca25a2c 100644
--- a/spec/lib/gitlab/database/rename_reserved_paths_migration/v1/rename_base_spec.rb
+++ b/spec/lib/gitlab/database/rename_reserved_paths_migration/v1/rename_base_spec.rb
@@ -20,7 +20,7 @@ describe Gitlab::Database::RenameReservedPathsMigration::V1::RenameBase, :delete
   end
 
   describe "#remove_last_ocurrence" do
-    it "removes only the last occurance of a string" do
+    it "removes only the last occurrence of a string" do
       input = "this/is/a-word-to-replace/namespace/with/a-word-to-replace"
 
       expect(subject.remove_last_occurrence(input, "a-word-to-replace"))
diff --git a/spec/lib/gitlab/diff/position_spec.rb b/spec/lib/gitlab/diff/position_spec.rb
index 2d94356f386..cc4faf6f10b 100644
--- a/spec/lib/gitlab/diff/position_spec.rb
+++ b/spec/lib/gitlab/diff/position_spec.rb
@@ -566,13 +566,13 @@ describe Gitlab::Diff::Position do
       end
     end
 
-    context "for text positon" do
+    context "for text position" do
       let(:args) { args_for_text }
 
       it_behaves_like "diff position json"
     end
 
-    context "for image positon" do
+    context "for image position" do
       let(:args) { args_for_img }
 
       it_behaves_like "diff position json"
@@ -592,13 +592,13 @@ describe Gitlab::Diff::Position do
       end
     end
 
-    context "for text positon" do
+    context "for text position" do
       let(:args) { args_for_text }
 
       it_behaves_like "diff position json"
     end
 
-    context "for image positon" do
+    context "for image position" do
       let(:args) { args_for_img }
 
       it_behaves_like "diff position json"
diff --git a/spec/lib/gitlab/diff/position_tracer_spec.rb b/spec/lib/gitlab/diff/position_tracer_spec.rb
index ddc4f6c5b5c..a2eed07ca55 100644
--- a/spec/lib/gitlab/diff/position_tracer_spec.rb
+++ b/spec/lib/gitlab/diff/position_tracer_spec.rb
@@ -43,7 +43,7 @@ describe Gitlab::Diff::PositionTracer do
   #
   # In any case, all of this means that the tests below will be extremely
   # (excessively, unjustifiably) thorough for scenarios where "the file was
-  # created in the old diff" and then drop off to comparitively lackluster
+  # created in the old diff" and then drop off to comparatively lackluster
   # testing of other scenarios.
   #
   # I did still try to cover most of the obvious and potentially tricky
diff --git a/spec/lib/gitlab/git/attributes_at_ref_parser_spec.rb b/spec/lib/gitlab/git/attributes_at_ref_parser_spec.rb
index ca067a29174..134bd5657e7 100644
--- a/spec/lib/gitlab/git/attributes_at_ref_parser_spec.rb
+++ b/spec/lib/gitlab/git/attributes_at_ref_parser_spec.rb
@@ -17,7 +17,7 @@ describe Gitlab::Git::AttributesAtRefParser, :seed_helper do
   end
 
   it 'handles missing blobs' do
-    expect { described_class.new(repository, 'non-existant-branch') }.not_to raise_error
+    expect { described_class.new(repository, 'non-existent-branch') }.not_to raise_error
   end
 
   describe '#attributes' do
diff --git a/spec/lib/gitlab/gpg_spec.rb b/spec/lib/gitlab/gpg_spec.rb
index 47f37cae98f..39d09c49989 100644
--- a/spec/lib/gitlab/gpg_spec.rb
+++ b/spec/lib/gitlab/gpg_spec.rb
@@ -96,7 +96,7 @@ describe Gitlab::Gpg do
       expect(described_class.current_home_dir).to eq default_home_dir
     end
 
-    it 'returns the explicitely set home dir' do
+    it 'returns the explicitly set home dir' do
       GPGME::Engine.home_dir = '/tmp/gpg'
 
       expect(described_class.current_home_dir).to eq '/tmp/gpg'
@@ -104,7 +104,7 @@ describe Gitlab::Gpg do
       GPGME::Engine.home_dir = GPGME::Engine.dirinfo('homedir')
     end
 
-    it 'returns the default value when explicitely setting the home dir to nil' do
+    it 'returns the default value when explicitly setting the home dir to nil' do
       GPGME::Engine.home_dir = nil
 
       expect(described_class.current_home_dir).to eq default_home_dir
diff --git a/spec/lib/gitlab/import_export/project_tree_restorer_spec.rb b/spec/lib/gitlab/import_export/project_tree_restorer_spec.rb
index b0570680d5a..365bfae0d88 100644
--- a/spec/lib/gitlab/import_export/project_tree_restorer_spec.rb
+++ b/spec/lib/gitlab/import_export/project_tree_restorer_spec.rb
@@ -321,7 +321,7 @@ describe Gitlab::ImportExport::ProjectTreeRestorer do
       end
     end
 
-    context 'when the project has overriden params in import data' do
+    context 'when the project has overridden params in import data' do
       it 'overwrites the params stored in the JSON' do
         project.create_import_data(data: { override_params: { description: "Overridden" } })
 
diff --git a/spec/lib/gitlab/view/presenter/base_spec.rb b/spec/lib/gitlab/view/presenter/base_spec.rb
index 4eca53032a2..02c2fd47197 100644
--- a/spec/lib/gitlab/view/presenter/base_spec.rb
+++ b/spec/lib/gitlab/view/presenter/base_spec.rb
@@ -40,7 +40,7 @@ describe Gitlab::View::Presenter::Base do
       end
     end
 
-    context 'subject is overriden' do
+    context 'subject is overridden' do
       it 'returns true' do
         presenter = presenter_class.new(build_stubbed(:project, :public))
 
diff --git a/spec/lib/microsoft_teams/notifier_spec.rb b/spec/lib/microsoft_teams/notifier_spec.rb
index c9756544bd6..2aaa7c24ad8 100644
--- a/spec/lib/microsoft_teams/notifier_spec.rb
+++ b/spec/lib/microsoft_teams/notifier_spec.rb
@@ -48,7 +48,7 @@ describe MicrosoftTeams::Notifier do
       stub_request(:post, webhook_url).with(body: JSON(body), headers: { 'Content-Type' => 'application/json' }).to_return(status: 200, body: "", headers: {})
     end
 
-    it 'expects to receive successfull answer' do
+    it 'expects to receive successful answer' do
       expect(subject.ping(options)).to be true
     end
   end
diff --git a/spec/migrations/migrate_old_artifacts_spec.rb b/spec/migrations/migrate_old_artifacts_spec.rb
index 4187ab149a5..af77d64fdbf 100644
--- a/spec/migrations/migrate_old_artifacts_spec.rb
+++ b/spec/migrations/migrate_old_artifacts_spec.rb
@@ -76,7 +76,7 @@ describe MigrateOldArtifacts do
         end
       end
 
-      context 'when there are aritfacts in old and new directory' do
+      context 'when there are artifacts in old and new directory' do
         before do
           store_artifacts_in_legacy_path(build2)
 
diff --git a/spec/models/concerns/awardable_spec.rb b/spec/models/concerns/awardable_spec.rb
index 69083bdc125..debc02fa51f 100644
--- a/spec/models/concerns/awardable_spec.rb
+++ b/spec/models/concerns/awardable_spec.rb
@@ -24,13 +24,29 @@ describe Awardable do
       end
     end
 
-    describe ".awarded" do
+    describe "#awarded" do
       it "filters by user and emoji name" do
         expect(Issue.awarded(award_emoji.user, "thumbsup")).to be_empty
         expect(Issue.awarded(award_emoji.user, "thumbsdown")).to eq [issue]
         expect(Issue.awarded(award_emoji2.user, "thumbsup")).to eq [issue2]
         expect(Issue.awarded(award_emoji2.user, "thumbsdown")).to be_empty
       end
+
+      it "filters by user and any emoji" do
+        issue3 = create(:issue)
+        create(:award_emoji, awardable: issue3, name: "star", user: award_emoji.user)
+        create(:award_emoji, awardable: issue3, name: "star", user: award_emoji2.user)
+
+        expect(Issue.awarded(award_emoji.user)).to eq [issue, issue3]
+        expect(Issue.awarded(award_emoji2.user)).to eq [issue2, issue3]
+      end
+    end
+
+    describe "#not_awarded" do
+      it "returns issues not awarded by user" do
+        expect(Issue.not_awarded(award_emoji.user)).to eq [issue2]
+        expect(Issue.not_awarded(award_emoji2.user)).to eq [issue]
+      end
     end
   end
 
diff --git a/spec/models/concerns/cacheable_attributes_spec.rb b/spec/models/concerns/cacheable_attributes_spec.rb
index f8c2e29fadd..827fbc9d7d5 100644
--- a/spec/models/concerns/cacheable_attributes_spec.rb
+++ b/spec/models/concerns/cacheable_attributes_spec.rb
@@ -41,7 +41,7 @@ describe CacheableAttributes do
       expect(minimal_test_class.current_without_cache).to eq(minimal_test_class.last)
     end
 
-    it 'can be overriden' do
+    it 'can be overridden' do
       minimal_test_class.define_singleton_method(:current_without_cache) do
         first
       end
@@ -64,7 +64,7 @@ describe CacheableAttributes do
     context 'with defaults defined' do
       include_context 'with defaults'
 
-      it 'can be overriden' do
+      it 'can be overridden' do
         expect(minimal_test_class.defaults).to eq({ foo: 'a', bar: 'b', baz: 'c' })
       end
     end
diff --git a/spec/models/concerns/issuable_spec.rb b/spec/models/concerns/issuable_spec.rb
index ec6374f3963..a4bf3e2350a 100644
--- a/spec/models/concerns/issuable_spec.rb
+++ b/spec/models/concerns/issuable_spec.rb
@@ -519,7 +519,7 @@ describe Issuable do
       end
     end
 
-    context 'substracting time' do
+    context 'subtracting time' do
       before do
         spend_time(1800)
       end
@@ -530,7 +530,7 @@ describe Issuable do
         expect(issue.total_time_spent).to eq(900)
       end
 
-      context 'when time to substract exceeds the total time spent' do
+      context 'when time to subtract exceeds the total time spent' do
         it 'raise a validation error' do
           Timecop.travel(1.minute.from_now) do
             expect do
diff --git a/spec/models/lfs_object_spec.rb b/spec/models/lfs_object_spec.rb
index 911f85d7b28..3f929710862 100644
--- a/spec/models/lfs_object_spec.rb
+++ b/spec/models/lfs_object_spec.rb
@@ -8,7 +8,7 @@ describe LfsObject do
       expect(subject.local_store?).to eq true
     end
 
-    it 'returns false whe file_store is equal to LfsObjectUploader::Store::REMOTE' do
+    it 'returns false when file_store is equal to LfsObjectUploader::Store::REMOTE' do
       subject.file_store = LfsObjectUploader::Store::REMOTE
 
       expect(subject.local_store?).to eq false
diff --git a/spec/models/merge_request_spec.rb b/spec/models/merge_request_spec.rb
index c8943f2d86f..85a4ebac66c 100644
--- a/spec/models/merge_request_spec.rb
+++ b/spec/models/merge_request_spec.rb
@@ -13,6 +13,20 @@ describe MergeRequest do
     it { is_expected.to belong_to(:merge_user).class_name("User") }
     it { is_expected.to belong_to(:assignee) }
     it { is_expected.to have_many(:merge_request_diffs) }
+
+    context 'for forks' do
+      let!(:project) { create(:project) }
+      let!(:fork) { fork_project(project) }
+      let!(:merge_request) { create(:merge_request, target_project: project, source_project: fork) }
+
+      it 'does not load another project due to inverse relationship' do
+        expect(project.merge_requests.first.target_project.object_id).to eq(project.object_id)
+      end
+
+      it 'finds the associated merge request' do
+        expect(project.merge_requests.find(merge_request.id)).to eq(merge_request)
+      end
+    end
   end
 
   describe '#squash_in_progress?' do
diff --git a/spec/models/project_spec.rb b/spec/models/project_spec.rb
index e66838edd1a..d4b9a4c8cd6 100644
--- a/spec/models/project_spec.rb
+++ b/spec/models/project_spec.rb
@@ -88,6 +88,10 @@ describe Project do
     it { is_expected.to have_many(:project_deploy_tokens) }
     it { is_expected.to have_many(:deploy_tokens).through(:project_deploy_tokens) }
 
+    it 'has an inverse relationship with merge requests' do
+      expect(described_class.reflect_on_association(:merge_requests).has_inverse?).to eq(:target_project)
+    end
+
     context 'after initialized' do
       it "has a project_feature" do
         expect(described_class.new.project_feature).to be_present
@@ -2746,7 +2750,7 @@ describe Project do
         .to raise_error(ActiveRecord::RecordNotSaved, error_message)
     end
 
-    it 'updates the project succesfully' do
+    it 'updates the project successfully' do
       merge_request = create(:merge_request, target_project: project, source_project: project)
 
       expect { project.append_or_update_attribute(:merge_requests, [merge_request]) }
@@ -3314,7 +3318,7 @@ describe Project do
       end
     end
 
-    context 'when explicitely enabled' do
+    context 'when explicitly enabled' do
       context 'when domain is empty' do
         before do
           create(:project_auto_devops, project: project, domain: nil)
diff --git a/spec/models/project_wiki_spec.rb b/spec/models/project_wiki_spec.rb
index f38fc191943..cc5e34782ec 100644
--- a/spec/models/project_wiki_spec.rb
+++ b/spec/models/project_wiki_spec.rb
@@ -145,7 +145,7 @@ describe ProjectWiki do
       end
 
       it "returns nil if the page does not exist" do
-        expect(subject.find_page("non-existant")).to eq(nil)
+        expect(subject.find_page("non-existent")).to eq(nil)
       end
 
       it "can find a page by slug" do
@@ -226,7 +226,7 @@ describe ProjectWiki do
       end
 
       it 'returns nil if the page does not exist' do
-        expect(subject.find_file('non-existant')).to eq(nil)
+        expect(subject.find_file('non-existent')).to eq(nil)
       end
 
       it 'returns a Gitlab::Git::WikiFile instance' do
diff --git a/spec/models/upload_spec.rb b/spec/models/upload_spec.rb
index 36b8e5d304f..3c89e99abf0 100644
--- a/spec/models/upload_spec.rb
+++ b/spec/models/upload_spec.rb
@@ -91,7 +91,7 @@ describe Upload do
         .to change { upload.checksum }.from(nil).to(expected)
     end
 
-    it 'sets `checksum` to nil for a non-existant file' do
+    it 'sets `checksum` to nil for a non-existent file' do
       expect(upload).to receive(:exist?).and_return(false)
 
       checksum = Digest::SHA256.file(__FILE__).hexdigest
diff --git a/spec/presenters/project_presenter_spec.rb b/spec/presenters/project_presenter_spec.rb
index 3eb2f149311..7b0192fa9c8 100644
--- a/spec/presenters/project_presenter_spec.rb
+++ b/spec/presenters/project_presenter_spec.rb
@@ -239,7 +239,7 @@ describe ProjectPresenter do
         expect(presenter.new_file_anchor_data).to have_attributes(enabled: false,
                                                                   label: "New file",
                                                                   link: presenter.project_new_blob_path(project, 'master'),
-                                                                  class_modifier: 'new')
+                                                                  class_modifier: 'success')
       end
 
       it 'returns nil if user cannot push' do
diff --git a/spec/requests/api/commits_spec.rb b/spec/requests/api/commits_spec.rb
index 98399471f9a..2963dea634a 100644
--- a/spec/requests/api/commits_spec.rb
+++ b/spec/requests/api/commits_spec.rb
@@ -565,7 +565,7 @@ describe API::Commits do
         }
       end
 
-      it 'are commited as one in project repo' do
+      it 'are committed as one in project repo' do
         post api(url, user), valid_mo_params
 
         expect(response).to have_gitlab_http_status(201)
diff --git a/spec/requests/api/issues_spec.rb b/spec/requests/api/issues_spec.rb
index 9cda39a569b..3d532dd83c7 100644
--- a/spec/requests/api/issues_spec.rb
+++ b/spec/requests/api/issues_spec.rb
@@ -55,8 +55,8 @@ describe API::Issues do
   end
   let!(:note) { create(:note_on_issue, author: user, project: project, noteable: issue) }
 
-  let(:no_milestone_title) { URI.escape(Milestone::None.title) }
-  let(:any_milestone_title) { URI.escape(Milestone::Any.title) }
+  let(:no_milestone_title) { "None" }
+  let(:any_milestone_title) { "Any" }
 
   before(:all) do
     project.add_reporter(user)
@@ -196,14 +196,24 @@ describe API::Issues do
         expect_paginated_array_response(size: 3)
       end
 
-      it 'returns issues reacted by the authenticated user by the given emoji' do
+      it 'returns issues reacted by the authenticated user' do
         issue2 = create(:issue, project: project, author: user, assignees: [user])
-        award_emoji = create(:award_emoji, awardable: issue2, user: user2, name: 'star')
+        create(:award_emoji, awardable: issue2, user: user2, name: 'star')
 
-        get api('/issues', user2), my_reaction_emoji: award_emoji.name, scope: 'all'
+        create(:award_emoji, awardable: issue, user: user2, name: 'thumbsup')
 
-        expect_paginated_array_response(size: 1)
-        expect(first_issue['id']).to eq(issue2.id)
+        get api('/issues', user2), my_reaction_emoji: 'Any', scope: 'all'
+
+        expect_paginated_array_response(size: 2)
+      end
+
+      it 'returns issues not reacted by the authenticated user' do
+        issue2 = create(:issue, project: project, author: user, assignees: [user])
+        create(:award_emoji, awardable: issue2, user: user2, name: 'star')
+
+        get api('/issues', user2), my_reaction_emoji: 'None', scope: 'all'
+
+        expect_paginated_array_response(size: 2)
       end
 
       it 'returns issues matching given search string for title' do
@@ -1791,6 +1801,74 @@ describe API::Issues do
     end
   end
 
+  describe 'GET :id/issues/:issue_iid/related_merge_requests' do
+    def get_related_merge_requests(project_id, issue_iid, user = nil)
+      get api("/projects/#{project_id}/issues/#{issue_iid}/related_merge_requests", user)
+    end
+
+    def create_referencing_mr(user, project, issue)
+      attributes = {
+        author: user,
+        source_project: project,
+        target_project: project,
+        source_branch: "master",
+        target_branch: "test",
+        description: "See #{issue.to_reference}"
+      }
+      create(:merge_request, attributes).tap do |merge_request|
+        create(:note, :system, project: project, noteable: issue, author: user, note: merge_request.to_reference(full: true))
+      end
+    end
+
+    let!(:related_mr) { create_referencing_mr(user, project, issue) }
+
+    context 'when unauthenticated' do
+      it 'return list of referenced merge requests from issue' do
+        get_related_merge_requests(project.id, issue.iid)
+
+        expect_paginated_array_response(size: 1)
+      end
+
+      it 'renders 404 if project is not visible' do
+        private_project = create(:project, :private)
+        private_issue = create(:issue, project: private_project)
+        create_referencing_mr(user, private_project, private_issue)
+
+        get_related_merge_requests(private_project.id, private_issue.iid)
+
+        expect(response).to have_gitlab_http_status(404)
+      end
+    end
+
+    it 'returns merge requests that mentioned a issue' do
+      create(:merge_request,
+            :simple,
+            author: user,
+            source_project: project,
+            target_project: project,
+            description: "Some description")
+
+      get_related_merge_requests(project.id, issue.iid, user)
+
+      expect_paginated_array_response(size: 1)
+      expect(json_response.first['id']).to eq(related_mr.id)
+    end
+
+    context 'no merge request mentioned a issue' do
+      it 'returns empty array' do
+        get_related_merge_requests(project.id, closed_issue.iid, user)
+
+        expect_paginated_array_response(size: 0)
+      end
+    end
+
+    it "returns 404 when issue doesn't exists" do
+      get_related_merge_requests(project.id, 999999, user)
+
+      expect(response).to have_gitlab_http_status(404)
+    end
+  end
+
   describe "GET /projects/:id/issues/:issue_iid/user_agent_detail" do
     let!(:user_agent_detail) { create(:user_agent_detail, subject: issue) }
 
diff --git a/spec/services/clusters/gcp/fetch_operation_service_spec.rb b/spec/services/clusters/gcp/fetch_operation_service_spec.rb
index e2fa93904c5..55f123ee786 100644
--- a/spec/services/clusters/gcp/fetch_operation_service_spec.rb
+++ b/spec/services/clusters/gcp/fetch_operation_service_spec.rb
@@ -24,7 +24,7 @@ describe Clusters::Gcp::FetchOperationService do
       end
     end
 
-    context 'when suceeded to fetch operation' do
+    context 'when succeeded to fetch operation' do
       before do
         stub_cloud_platform_get_zone_operation(gcp_project_id, zone, operation_id)
       end
diff --git a/spec/services/clusters/gcp/finalize_creation_service_spec.rb b/spec/services/clusters/gcp/finalize_creation_service_spec.rb
index 0f484222228..303d45495ef 100644
--- a/spec/services/clusters/gcp/finalize_creation_service_spec.rb
+++ b/spec/services/clusters/gcp/finalize_creation_service_spec.rb
@@ -30,7 +30,7 @@ describe Clusters::Gcp::FinalizeCreationService do
       end
     end
 
-    context 'when suceeded to fetch gke cluster info' do
+    context 'when succeeded to fetch gke cluster info' do
       let(:endpoint) { '111.111.111.111' }
       let(:api_url) { 'https://' + endpoint }
       let(:username) { 'sample-username' }
diff --git a/spec/services/clusters/gcp/provision_service_spec.rb b/spec/services/clusters/gcp/provision_service_spec.rb
index f48afdc83b2..c0bdac40938 100644
--- a/spec/services/clusters/gcp/provision_service_spec.rb
+++ b/spec/services/clusters/gcp/provision_service_spec.rb
@@ -26,7 +26,7 @@ describe Clusters::Gcp::ProvisionService do
       end
     end
 
-    context 'when suceeded to request provision' do
+    context 'when succeeded to request provision' do
       before do
         stub_cloud_platform_create_cluster(gcp_project_id, zone)
       end
diff --git a/spec/services/groups/transfer_service_spec.rb b/spec/services/groups/transfer_service_spec.rb
index 1289d3ce01f..dd8a1cee074 100644
--- a/spec/services/groups/transfer_service_spec.rb
+++ b/spec/services/groups/transfer_service_spec.rb
@@ -347,7 +347,7 @@ describe Groups::TransferService, :postgresql do
         end
       end
 
-      context 'when transfering a group with nested groups and projects' do
+      context 'when transferring a group with nested groups and projects' do
         let!(:group) { create(:group, :public) }
         let!(:project1) { create(:project, :repository, :private, namespace: group) }
         let!(:subgroup1) { create(:group, :private, parent: group) }
diff --git a/spec/services/projects/import_service_spec.rb b/spec/services/projects/import_service_spec.rb
index e6ffa2b957b..06f865dc848 100644
--- a/spec/services/projects/import_service_spec.rb
+++ b/spec/services/projects/import_service_spec.rb
@@ -125,7 +125,7 @@ describe Projects::ImportService do
           project.import_type = 'bitbucket'
         end
 
-        it 'succeeds if repository import is successfull' do
+        it 'succeeds if repository import is successful' do
           expect_any_instance_of(Gitlab::Shell).to receive(:import_repository).and_return(true)
           expect_any_instance_of(Gitlab::BitbucketImport::Importer).to receive(:execute).and_return(true)
           expect_any_instance_of(Projects::LfsPointers::LfsImportService).to receive(:execute).and_return({})
diff --git a/spec/services/projects/transfer_service_spec.rb b/spec/services/projects/transfer_service_spec.rb
index 1411723fb9e..2e07d4f8013 100644
--- a/spec/services/projects/transfer_service_spec.rb
+++ b/spec/services/projects/transfer_service_spec.rb
@@ -125,7 +125,7 @@ describe Projects::TransferService do
     it { expect(project.errors.messages[:new_namespace].first).to eq 'Please select a new namespace for your project.' }
   end
 
-  context 'disallow transfering of project with tags' do
+  context 'disallow transferring of project with tags' do
     let(:container_repository) { create(:container_repository) }
 
     before do
diff --git a/spec/services/quick_actions/interpret_service_spec.rb b/spec/services/quick_actions/interpret_service_spec.rb
index 41a170e4f25..e513ee7ae44 100644
--- a/spec/services/quick_actions/interpret_service_spec.rb
+++ b/spec/services/quick_actions/interpret_service_spec.rb
@@ -315,7 +315,7 @@ describe QuickActions::InterpretService do
     end
 
     shared_examples 'award command' do
-      it 'toggle award 100 emoji if content containts /award :100:' do
+      it 'toggle award 100 emoji if content contains /award :100:' do
         _, updates = service.execute(content, issuable)
 
         expect(updates).to eq(emoji_award: "100")
@@ -1395,7 +1395,7 @@ describe QuickActions::InterpretService do
       it 'includes the formatted duration and proper verb' do
         _, explanations = service.explain(content, issue)
 
-        expect(explanations).to eq(['Substracts 2h spent time.'])
+        expect(explanations).to eq(['Subtracts 2h spent time.'])
       end
     end
 
diff --git a/spec/support/capybara.rb b/spec/support/capybara.rb
index c0ceb0f6605..18a7a392c12 100644
--- a/spec/support/capybara.rb
+++ b/spec/support/capybara.rb
@@ -100,7 +100,7 @@ RSpec.configure do |config|
 
     # capybara/rspec already calls Capybara.reset_sessions! in an `after` hook,
     # but `block_and_wait_for_requests_complete` is called before it so by
-    # calling it explicitely here, we prevent any new requests from being fired
+    # calling it explicitly here, we prevent any new requests from being fired
     # See https://github.com/teamcapybara/capybara/blob/ffb41cfad620de1961bb49b1562a9fa9b28c0903/lib/capybara/rspec.rb#L20-L25
     # We don't reset the session when the example failed, because we need capybara-screenshot to have access to it.
     Capybara.reset_sessions! unless example.exception
diff --git a/spec/support/helpers/filtered_search_helpers.rb b/spec/support/helpers/filtered_search_helpers.rb
index 5f42ff77fb2..6569feec39b 100644
--- a/spec/support/helpers/filtered_search_helpers.rb
+++ b/spec/support/helpers/filtered_search_helpers.rb
@@ -120,8 +120,12 @@ module FilteredSearchHelpers
     create_token('Label', label_name, symbol)
   end
 
-  def emoji_token(emoji_name = nil)
-    { name: 'My-Reaction', emoji_name: emoji_name }
+  def reaction_token(reaction_name = nil, is_emoji = true)
+    if is_emoji
+      { name: 'My-Reaction', emoji_name: reaction_name }
+    else
+      create_token('My-Reaction', reaction_name)
+    end
   end
 
   def default_placeholder
diff --git a/spec/support/helpers/project_forks_helper.rb b/spec/support/helpers/project_forks_helper.rb
index 6a7132c3093..9a86560da2a 100644
--- a/spec/support/helpers/project_forks_helper.rb
+++ b/spec/support/helpers/project_forks_helper.rb
@@ -35,7 +35,7 @@ module ProjectForksHelper
     if create_repository
       # The call to project.repository.after_import in RepositoryForkWorker does
       # not reset the @exists variable of this forked_project.repository
-      # so we have to explicitely call this method to clear the @exists variable.
+      # so we have to explicitly call this method to clear the @exists variable.
       # of the instance we're returning here.
       forked_project.repository.after_import
     end
diff --git a/spec/support/shared_examples/services/boards/lists_move_service.rb b/spec/support/shared_examples/services/boards/lists_move_service.rb
index 07c98cb29b7..2cdb968a45d 100644
--- a/spec/support/shared_examples/services/boards/lists_move_service.rb
+++ b/spec/support/shared_examples/services/boards/lists_move_service.rb
@@ -14,7 +14,7 @@ shared_examples 'lists move service' do
       expect(current_list_positions).to eq [0, 1, 2, 3]
     end
 
-    it 'keeps position of lists when new positon is equal to old position' do
+    it 'keeps position of lists when new position is equal to old position' do
       service = described_class.new(parent, user, position: planning.position)
 
       service.execute(planning)
@@ -22,7 +22,7 @@ shared_examples 'lists move service' do
       expect(current_list_positions).to eq [0, 1, 2, 3]
     end
 
-    it 'keeps position of lists when new positon is negative' do
+    it 'keeps position of lists when new position is negative' do
       service = described_class.new(parent, user, position: -1)
 
       service.execute(planning)
@@ -30,7 +30,7 @@ shared_examples 'lists move service' do
       expect(current_list_positions).to eq [0, 1, 2, 3]
     end
 
-    it 'keeps position of lists when new positon is equal to number of labels lists' do
+    it 'keeps position of lists when new position is equal to number of labels lists' do
       service = described_class.new(parent, user, position: board.lists.label.size)
 
       service.execute(planning)
@@ -38,7 +38,7 @@ shared_examples 'lists move service' do
       expect(current_list_positions).to eq [0, 1, 2, 3]
     end
 
-    it 'keeps position of lists when new positon is greater than number of labels lists' do
+    it 'keeps position of lists when new position is greater than number of labels lists' do
       service = described_class.new(parent, user, position: board.lists.label.size + 1)
 
       service.execute(planning)
@@ -46,7 +46,7 @@ shared_examples 'lists move service' do
       expect(current_list_positions).to eq [0, 1, 2, 3]
     end
 
-    it 'increments position of intermediate lists when new positon is equal to first position' do
+    it 'increments position of intermediate lists when new position is equal to first position' do
       service = described_class.new(parent, user, position: 0)
 
       service.execute(staging)
@@ -54,7 +54,7 @@ shared_examples 'lists move service' do
       expect(current_list_positions).to eq [1, 2, 3, 0]
     end
 
-    it 'decrements position of intermediate lists when new positon is equal to last position' do
+    it 'decrements position of intermediate lists when new position is equal to last position' do
       service = described_class.new(parent, user, position: board.lists.label.last.position)
 
       service.execute(planning)
diff --git a/spec/tasks/gitlab/backup_rake_spec.rb b/spec/tasks/gitlab/backup_rake_spec.rb
index 3ba6caf1337..8c4360d4cf0 100644
--- a/spec/tasks/gitlab/backup_rake_spec.rb
+++ b/spec/tasks/gitlab/backup_rake_spec.rb
@@ -251,7 +251,7 @@ describe 'gitlab:app namespace rake task' do
 
         allow(Gitlab.config.repositories).to receive(:storages).and_return(storages)
 
-        # Avoid asking gitaly about the root ref (which will fail beacuse of the
+        # Avoid asking gitaly about the root ref (which will fail because of the
         # mocked storages)
         allow_any_instance_of(Repository).to receive(:empty?).and_return(false)
       end