Refactored tests to improve coverage

Author: JackieBinya

tests/unit/specs/components/CopyrightWaiverStep.spec.js

@@ -58,14 +58,20 @@ describe('CopyrightWaiver Step: Check conditional rendering of markup', () => {
     })
 })
 
-describe('Test functionality of computed properties', () => {
+describe('Test the functionality of Computed properties', () => {
     let wrapper
 
     beforeEach(() => {
         wrapper = mount(CopyrightWaiverStep, {
             localVue,
+            data(){
+                return{
+                    agreed: false,
+                    confirmed: false  
+                }
+            },
             propsData: {
-                selected: undefined,
+                selected: true,
                 status: 'current',
                 stepId: 6,
                 stepName: 'CW'
@@ -80,11 +86,9 @@ describe('Test functionality of computed properties', () => {
         wrapper.destroy()
     })
 
-    it('User checks agreed with confirmed checked', () => {
-        wrapper.setData({
-            agreed: false,
-            confirmed: true
-        })
+    it('User checks confirmed then checks agreed', () => {
+        const checkbox1 = wrapper.findAll('input[type="checkbox"]').at(1)
+        checkbox1.setChecked()
 
         const checkbox = wrapper.findAll('input[type="checkbox"]').at(0)
         checkbox.setChecked()
@@ -93,63 +97,32 @@ describe('Test functionality of computed properties', () => {
         expect(wrapper.vm.copyrightWaiverAgreed).toBe(true)
     })
 
-    it('User checks confirmed with agreed checked', () => {
-        wrapper.setData({
-            agreed: true,
-            confirmed: false
-        })
-
-        const checkbox = wrapper.findAll('input[type="checkbox"]').at(1)
+    it('User checks agreed and then checks confirmed', () => {
+        const checkbox = wrapper.findAll('input[type="checkbox"]').at(0)
         checkbox.setChecked()
+       
+
+        const checkbox1 = wrapper.findAll('input[type="checkbox"]').at(1)
+        checkbox1.setChecked()
 
         expect(wrapper.emitted().change[0]).toStrictEqual(['CW', 6, true])
         expect(wrapper.vm.copyrightWaiverConfirmed).toBe(true)
     })
-})
-
-describe('Computed properties: user unchecks agreed and/ confirmed', () => {
-    let wrapper
-
-    beforeEach(() => {
-        wrapper = mount(CopyrightWaiverStep, {
-            localVue,
-            propsData: {
-                selected: true,
-                status: 'current',
-                stepId: 6,
-                stepName: 'CW'
-            },
-            mocks: {
-                $t: key => key
-            }
-        })
-    })
-
-    afterEach(() => {
-        wrapper.destroy()
-    })
 
     it('User unchecks agreed', () => {
-        wrapper.setData({
-            agreed: true,
-            confirmed: true
-        })
-
         const checkbox = wrapper.findAll('input[type="checkbox"]').at(0)
-        checkbox.trigger('change')
+        checkbox.setChecked()
+        checkbox.setChecked(false)
 
         expect(wrapper.emitted().change[0]).toStrictEqual(['CW', 6, undefined])
         expect(wrapper.vm.copyrightWaiverAgreed).toBe(false)
     })
 
     it('User unchecks confirmed', () => {
-        wrapper.setData({
-            agreed: true,
-            confirmed: true
-        })
-
         const checkbox = wrapper.findAll('input[type="checkbox"]').at(1)
-        checkbox.trigger('change')
+
+        checkbox.setChecked()
+        checkbox.setChecked(false)
 
         expect(wrapper.emitted().change[0]).toStrictEqual(['CW', 6, undefined])
         expect(wrapper.vm.copyrightWaiverConfirmed).toBe(false)