Merge pull request #404 from Safelite/refactor/initial-prefer-default

prefer default - initial attempt: application-config

src/constants/analytics.js

@@ -1,36 +1,36 @@
-const analyticsPageEvents = {
+const analyticsPageEvents = Object.freeze({
     ENTRY: 'ENTRY',
     EVENT: 'EVENT'
-};
+});
 
 // GA Constants
-const GaEvents = {
+const GaEvents = Object.freeze({
     GENERIC_EVENT: 'event',
     PAGE_VIEW_EVENT: 'logPageview'
-};
+});
 
-const GaCategories = {
+const GaCategories = Object.freeze({
     API_RESPONSE: 'Api_Response',
     EVOX: 'Evox'
-};
+});
 
-const GaActions = {
+const GaActions = Object.freeze({
     RESULT: 'Result',
     CLICKED: 'Clicked',
     VIF: 'vif',
     SUBMITTED: 'Submitted'
-};
+});
 
-const GaLabels = {
+const GaLabels = Object.freeze({
     SUCCESS: 'Success',
     ERROR: 'Error',
     LICENSE_PLATE_LOOKUP: 'License_Plate_Look_Up',
     VIN_LOOKUP: 'Vin_Look_Up',
     ADDRESS_LOOKUP: 'Address_Look_up'
-};
+});
 
-const ValueToLogTypes = {
+const ValueToLogTypes = Object.freeze({
     LAST_5: 'last_5'
-};
+});
 
 export { analyticsPageEvents, GaCategories, GaActions, GaLabels, GaEvents, ValueToLogTypes };

src/constants/application-config.js

@@ -14,4 +14,4 @@ const applicationConfig = {
     CASH_PARENT_ACCOUNT_NUMBER: 167132
 };
 
-export { applicationConfig };
+export default Object.freeze(applicationConfig);

src/constants/cookie-names.js

@@ -1,4 +1,4 @@
-import { applicationConfig } from '@/constants/application-config.js';
+import applicationConfig from '@/constants/application-config.js';
 
 const cookieNames = {
     ISS_SESSION_INFO: `ISSSessionInfo-${applicationConfig.CURRENT_ENVIRONMENT}`,

src/global-methods.js

@@ -2,7 +2,7 @@ import axios from 'axios';
 import analyticsMixIn from '@/mixins/analytics-mixin.js';
 import { useMainStore } from '@/store';
 
-import { applicationConfig } from '@/constants/application-config.js';
+import applicationConfig from '@/constants/application-config.js';
 import { GaCategories, GaActions, GaLabels } from '@/constants/analytics';
 import { headerKeys } from '@/constants/header-keys';
 

src/helpers/cookie-helper.js

@@ -1,5 +1,5 @@
 import { cookieNames } from '@/constants/cookie-names';
-import { applicationConfig } from '@/constants/application-config';
+import applicationConfig from '@/constants/application-config';
 import { useMainStore } from '@/store';
 
 /*

src/helpers/session-helper.js

@@ -1,4 +1,4 @@
-import { applicationConfig } from '@/constants/application-config';
+import applicationConfig from '@/constants/application-config';
 import { getISSCookie } from '@/helpers/cookie-helper.js';
 
 /*

src/helpers/session-helper.spec.js

@@ -4,7 +4,7 @@ import {
     isSavedSessionStillActive,
     getDateForSavedSessionTimeout
 } from '@/helpers/session-helper';
-import { applicationConfig } from '@/constants/application-config';
+import applicationConfig from '@/constants/application-config';
 
 describe('isAnalyticsSessionStillActive', () => {
     test('isAnalyticsSessionStillActive, should return true', () => {

src/iss-components/address-questions/address-questions.vue

@@ -92,7 +92,7 @@
 import textboxQuestion from '@/digital-components/textbox-question/textbox-question';
 import dropdownQuestion from '@/digital-components/dropdown-question/dropdown-question';
 import alert from '@/ux-components/alert/alert';
-import { applicationConfig } from '@/constants/application-config.js';
+import applicationConfig from '@/constants/application-config.js';
 import { defineRule } from 'vee-validate';
 import { required, regex } from '@/helpers/validation-rules';
 import { errorMessages } from '@/constants/error-messages';

src/layouts/vehicle-parts/vehicle-parts.spec.js

@@ -11,7 +11,7 @@ import { nextTick } from 'vue';
 import baseMixin from '@/mixins/base-mixin.js';
 import { useMainStore } from '@/store';
 import { navigationScenarios } from '@/router/router-constants/navigation-scenarios';
-import { applicationConfig } from '@/constants/application-config';
+import applicationConfig from '@/constants/application-config';
 
 // Mock our module for promises.
 jest.mock('@/helpers/layout-helper.js', () => ({

src/layouts/welcome-page/welcome-page.spec.js

@@ -7,7 +7,7 @@ import { settleAllPromises } from '@/helpers/layout-helper.js';
 import { fetchCmsContentForPage } from '@/helpers/cms-content-helper';
 import baseMixin from '@/mixins/base-mixin.js';
 import { getMountOptions } from '@/helpers/unit-test-helper.js';
-import { applicationConfig } from '@/constants/application-config';
+import applicationConfig from '@/constants/application-config';
 import { useMainStore } from '@/store';
 import { navigationScenarios } from '@/router/router-constants/navigation-scenarios';
 

src/router/index.js

@@ -12,7 +12,7 @@ import {
     updateSessionIdCookie
 } from '@/helpers/cookie-helper';
 import { experimentTriggers } from '@/constants/experiments';
-import { applicationConfig } from '@/constants/application-config';
+import applicationConfig from '@/constants/application-config';
 
 import analyticsMixin from '@/mixins/analytics-mixin';
 import { navigationScenarios } from './router-constants/navigation-scenarios';

src/store/index.js

@@ -6,7 +6,7 @@ import { getDateForSavedSessionTimeout } from '@/helpers/session-helper';
 // eslint-disable-next-line import/no-cycle
 import globalMethods from '@/global-methods';
 import { experimentTriggers } from '@/constants/experiments';
-import { applicationConfig } from '@/constants/application-config';
+import applicationConfig from '@/constants/application-config';
 import { issPageValues } from '@/router/router-constants/issPage-values';
 import { damageLocationsSelected } from '@/constants/damage-locations-selected';
 import { coverageStatuses } from '@/constants/coverage-statuses';

src/ux-components/alert/alert.vue

@@ -60,7 +60,7 @@ import {
     getRouterLinkDisplayTextFromCopy,
     splitCMSCopyOnParagraphTag
 } from '@/helpers/cms-content-helper';
-import { applicationConfig } from '@/constants/application-config';
+import applicationConfig from '@/constants/application-config';
 
 export default {
     name: 'alert',