Applying maintainer feedback

Removed unused directive
Changed dynamic css class
Modified test to use named arguments
Added case for null in utils_spec

ee/app/assets/javascripts/packages/list/components/packages_list.vue

 <script>
 import { mapState, mapGetters } from 'vuex';
-import { GlPagination, GlModal, GlTooltipDirective } from '@gitlab/ui';
+import { GlPagination, GlModal } from '@gitlab/ui';
 import Tracking from '~/tracking';
 import { s__, sprintf } from '~/locale';
 import { TrackingActions } from '../../shared/constants';
@@ -15,7 +15,6 @@ export default {
     PackagesListLoader,
     PackagesListRow,
   },
-  directives: { GlTooltip: GlTooltipDirective },
   mixins: [Tracking.mixin()],
   data() {
     return {

ee/app/assets/javascripts/packages/list/components/packages_list_row.vue

@@ -91,8 +91,8 @@ export default {
     </div>
 
     <div
-      class="table-section section-40 d-flex flex-md-column justify-content-between align-items-md-end"
-      :class="{ 'section-50': isGroupPage }"
+      class="table-section d-flex flex-md-column justify-content-between align-items-md-end"
+      :class="isGroupPage ? 'section-50' : 'section-40'"
     >
       <publish-method :package-entity="packageEntity" />
 

ee/spec/frontend/packages/list/components/__snapshots__/packages_list_row_spec.js.snap

@@ -66,7 +66,7 @@ exports[`packages_list_row renders 1`] = `
   </div>
    
   <div
-    class="table-section section-40 d-flex flex-md-column justify-content-between align-items-md-end"
+    class="table-section d-flex flex-md-column justify-content-between align-items-md-end section-40"
   >
     <publish-method-stub
       packageentity="[object Object]"

ee/spec/frontend/packages/list/stores/getters_spec.js

@@ -4,7 +4,7 @@ import { packageList } from '../../mock_data';
 describe('Getters registry list store', () => {
   let state;
 
-  const setState = (isGroupPage = false) => {
+  const setState = ({ isGroupPage = false } = {}) => {
     state = {
       packages: packageList,
       config: {
@@ -42,7 +42,7 @@ describe('Getters registry list store', () => {
     });
 
     describe('when isGroupPage is true', () => {
-      beforeEach(() => setState(true));
+      beforeEach(() => setState({ isGroupPage: true }));
 
       it('returns an absolute link matching project path', () => {
         const mavenPackage = packageList[0];

ee/spec/frontend/packages/shared/utils_spec.js

@@ -31,6 +31,7 @@ describe('Packages shared utils', () => {
       ${'maven'}  | ${'Maven'}
       ${'npm'}    | ${'NPM'}
       ${'nuget'}  | ${'NuGet'}
+      ${'foo'}    | ${null}
     `(`package type`, ({ packageType, expectedResult }) => {
       it(`${packageType} should show as ${expectedResult}`, () => {
         expect(getPackageType(packageType)).toBe(expectedResult);