added reviewed changes

src/components/common/Actions.vue

@@ -76,7 +76,6 @@ export default class Actions extends Mixins(DateMixin, FilingTemplateMixin, Name
   @Getter(useStore) getFilingId!: number
   // @Getter(useStore) getNameRequestNumber!: string
   @Getter(useStore) hasAlterationDataChanged!: boolean // for testing state-getters
-  // @Getter(useStore) hasCorrectionDataChanged!: boolean
   @Getter(useStore) havePeopleAndRolesChanged!: boolean // for testing state-getters
   @Getter(useStore) isBusySaving!: boolean
   @Getter(useStore) isCorrectionEditing!: boolean

src/store/store.ts

@@ -558,6 +558,7 @@ export const useStore = defineStore('store', {
     },
 
     /**
+     * Commenting out for now, might be used in future.
      * Whether any correction data has changed (for the purpose of showing the
      * fee summary), ie, does not include:
      * - completing party
@@ -1193,7 +1194,7 @@ export const useStore = defineStore('store', {
     },
 
     /**
-     * Whether to show the fee summary.
+     * Whether to show the fee summary [show fee summary in all cases].
      * This is a safety check to ensure that fee summary component is not loaded
      * until there is a valid filing type and entity code.
      */

tests/unit/CoopCorrection.spec.ts

@@ -455,7 +455,7 @@ describe('Coop Correction component', () => {
     expect(store.isCorrectionEditing).toBe(true)
   })
 
-  it('isCorrectionValid and hasCorrectionDataChanged', async () => {
+  it('isCorrectionValid', async () => {
     store.stateModel.correctionInformation.type = CorrectionErrorTypes.CLIENT
     await Vue.nextTick()
 
@@ -473,7 +473,6 @@ describe('Coop Correction component', () => {
     // These validation flags plus more make up isCorrectionValid
     // This property is used enable/disable the file and pay button.
     expect(store.isCorrectionValid).toBe(false)
-    // expect(store.hasCorrectionDataChanged).toBe(false)
 
     store.stateModel.nameRequestLegalName = 'SUPER SUPER COOP 2'
     expect(store.hasBusinessNameChanged).toBe(true)
@@ -485,8 +484,5 @@ describe('Coop Correction component', () => {
     expect(store.hasSpecialResolutionRulesChanged).toBe(true)
     store.stateModel.specialResolution.resolution = 'NEW RESOLUTION'
     expect(store.hasSpecialResolutionResolutionChanged).toBe(true)
-    // These change flags make up hasCorrectionDataChanged
-    // This property is used to enable/disable the file and pay button.
-    // expect(store.hasCorrectionDataChanged).toBe(true)
   })
 })

tests/unit/state-getters.spec.ts

@@ -331,9 +331,6 @@ describe('BEN correction getters', () => {
   })
 
   it('returns correct values for "Has Correction Changed" getter', async () => {
-    // initially, this getter should be false (default value)
-    // expect(vm.hasCorrectionDataChanged).toBe(false)
-
     // verify that business name changes are detected
     store.stateModel.nameRequestLegalName = 'MyLegalName2'
     expect(vm.hasBusinessNameChanged).toBe(true)
@@ -394,9 +391,6 @@ describe('BEN correction getters', () => {
     expect(vm.hasShareStructureChanged).toBe(true)
     store.stateModel.shareStructureStep.shareClasses = []
     expect(vm.hasShareStructureChanged).toBe(false)
-
-    // finally, this getter should be false
-    // expect(vm.hasCorrectionDataChanged).toBe(false)
   })
 })
 
@@ -459,9 +453,6 @@ describe('SP/GP correction getters - with easy legal name fix', () => {
   })
 
   it('returns correct values for "Has Correction Changed" getter', async () => {
-    // initially, this getter should be false (default value)
-    // expect(vm.hasCorrectionDataChanged).toBe(false)
-
     // verify that business name changes are detected
     store.stateModel.nameRequestLegalName = 'MyLegalName2'
     expect(vm.hasBusinessNameChanged).toBe(true)
@@ -537,9 +528,6 @@ describe('SP/GP correction getters - with easy legal name fix', () => {
     expect(vm.hasNaicsChanged).toBe(true)
     store.stateModel.businessInformation = naics as any
     expect(vm.hasNaicsChanged).toBe(false)
-
-    // finally, this getter should be false
-    // expect(vm.hasCorrectionDataChanged).toBe(false)
   })
 })