Address technical debt in commit_block

This changes the specs and removes unused CSS
due to the use of utility classes

app/assets/javascripts/jobs/components/commit_block.vue

 <script>
-/* eslint-disable @gitlab/vue-require-i18n-strings */
 import { GlLink } from '@gitlab/ui';
 import ClipboardButton from '~/vue_shared/components/clipboard_button.vue';
 
@@ -23,9 +22,9 @@ export default {
 </script>
 <template>
   <div>
-    <span class="font-weight-bold">{{ __('Commit') }}</span>
+    <span class="gl-font-weight-bold">{{ __('Commit') }}</span>
 
-    <gl-link :href="commit.commit_path" class="js-commit-sha commit-sha link-commit">
+    <gl-link :href="commit.commit_path" class="gl-text-blue-600!" data-testid="commit-sha">
       {{ commit.short_id }}
     </gl-link>
 
@@ -37,8 +36,8 @@ export default {
     />
 
     <span v-if="mergeRequest">
-      in
-      <gl-link :href="mergeRequest.path" class="js-link-commit link-commit"
+      {{ __('in') }}
+      <gl-link :href="mergeRequest.path" class="gl-text-blue-600!" data-testid="link-commit"
         >!{{ mergeRequest.iid }}</gl-link
       >
     </span>

app/assets/stylesheets/page_bundles/build.scss

@@ -219,10 +219,6 @@
       }
     }
   }
-
-  .link-commit {
-    color: var(--blue-600, $blue-600);
-  }
 }
 
 .build-sidebar {

spec/frontend/jobs/components/commit_block_spec.js

-import Vue from 'vue';
-import mountComponent from 'helpers/vue_mount_component_helper';
-import component from '~/jobs/components/commit_block.vue';
+import { shallowMount } from '@vue/test-utils';
+import { extendedWrapper } from 'helpers/vue_test_utils_helper';
+import CommitBlock from '~/jobs/components/commit_block.vue';
+import ClipboardButton from '~/vue_shared/components/clipboard_button.vue';
 
 describe('Commit block', () => {
-  const Component = Vue.extend(component);
-  let vm;
+  let wrapper;
 
-  const props = {
+  const defaults = {
     commit: {
       short_id: '1f0fb84f',
       id: '1f0fb84fb6770d74d97eee58118fd3909cd4f48c',
@@ -20,70 +20,63 @@ describe('Commit block', () => {
     isLastBlock: true,
   };
 
+  const findCommitSha = () => wrapper.findByTestId('commit-sha');
+  const findLinkSha = () => wrapper.findByTestId('link-commit');
+
+  function mountComponent(props = {}) {
+    wrapper = extendedWrapper(
+      shallowMount(CommitBlock, {
+        propsData: {
+          ...props,
+        },
+      }),
+    );
+  }
+
   afterEach(() => {
-    vm.$destroy();
+    wrapper.destroy();
   });
 
   describe('pipeline short sha', () => {
     beforeEach(() => {
-      vm = mountComponent(Component, {
-        ...props,
-      });
+      mountComponent(defaults);
     });
 
     it('renders pipeline short sha link', () => {
-      expect(vm.$el.querySelector('.js-commit-sha').getAttribute('href')).toEqual(
-        props.commit.commit_path,
-      );
-
-      expect(vm.$el.querySelector('.js-commit-sha').textContent.trim()).toEqual(
-        props.commit.short_id,
-      );
+      expect(findCommitSha().attributes('href')).toBe(defaults.commit.commit_path);
+      expect(findCommitSha().text()).toBe(defaults.commit.short_id);
     });
 
     it('renders clipboard button', () => {
-      expect(vm.$el.querySelector('button').getAttribute('data-clipboard-text')).toEqual(
-        props.commit.id,
-      );
+      expect(wrapper.findComponent(ClipboardButton).attributes('text')).toBe(defaults.commit.id);
     });
   });
 
   describe('with merge request', () => {
     it('renders merge request link and reference', () => {
-      vm = mountComponent(Component, {
-        ...props,
-      });
-
-      expect(vm.$el.querySelector('.js-link-commit').getAttribute('href')).toEqual(
-        props.mergeRequest.path,
-      );
+      mountComponent(defaults);
 
-      expect(vm.$el.querySelector('.js-link-commit').textContent.trim()).toEqual(
-        `!${props.mergeRequest.iid}`,
-      );
+      expect(findLinkSha().attributes('href')).toBe(defaults.mergeRequest.path);
+      expect(findLinkSha().text()).toBe(`!${defaults.mergeRequest.iid}`);
     });
   });
 
   describe('without merge request', () => {
     it('does not render merge request', () => {
-      const copyProps = { ...props };
+      const copyProps = { ...defaults };
       delete copyProps.mergeRequest;
 
-      vm = mountComponent(Component, {
-        ...copyProps,
-      });
+      mountComponent(copyProps);
 
-      expect(vm.$el.querySelector('.js-link-commit')).toBeNull();
+      expect(findLinkSha().exists()).toBe(false);
     });
   });
 
   describe('git commit title', () => {
     it('renders git commit title', () => {
-      vm = mountComponent(Component, {
-        ...props,
-      });
+      mountComponent(defaults);
 
-      expect(vm.$el.textContent).toContain(props.commit.title);
+      expect(wrapper.text()).toContain(defaults.commit.title);
     });
   });
 });