add getData method and

replace getCopyData method
with getData method

Author: dev-javascript

src/index.test.js

@@ -174,7 +174,7 @@ describe('calling some action inside the events options', () => {
       this.select('2');
     });
     renderApp();
-    expect(instance.getCopyData().selectedTabID === '2').toBe(true);
+    expect(instance.getData().selectedTabID === '2').toBe(true);
     expect(op.onLoad.mock.calls.length === 1).toBe(true);
     expect(op.onInit.mock.calls.length === 2).toBe(true);
     expect(op.onChange.mock.calls.length === 1).toBe(true);
@@ -205,7 +205,7 @@ describe('select method : ', () => {
     renderApp();
     return act(() => {
       return instance.select('2').then((result) => {
-        expect(result.currentData).toEqual(instance.getCopyData());
+        expect(result.currentData).toEqual(instance.getData());
         expect(result.instance).toBe(instance);
         expect(document.querySelector('[tab-id="2"]').className.includes('rc-dyn-tabs-selected')).toBe(true);
         expect(op.onInit.mock.calls.length === 2).toBe(true);
@@ -230,7 +230,7 @@ describe('select method : ', () => {
     return act(() => {
       return instance.select('3').then(() => {
         expect(document.querySelector('.rc-dyn-tabs-selected') == null).toBe(true);
-        expect(instance.getCopyData().selectedTabID === '3').toBe(true);
+        expect(instance.getData().selectedTabID === '3').toBe(true);
         expect(op.onChange.mock.calls.length === 1).toBe(true);
         expect(op.onInit.mock.calls.length === 2).toBe(true);
       });
@@ -258,7 +258,7 @@ describe('close method : ', () => {
     renderApp();
     return act(() => {
       return instance.close('2').then((result) => {
-        expect(result.currentData).toEqual(instance.getCopyData());
+        expect(result.currentData).toEqual(instance.getData());
         expect(result.instance).toBe(instance);
         expect(document.querySelector('[tab-id="1"]').className.includes('rc-dyn-tabs-selected')).toBe(true);
         expect(op.onInit.mock.calls.length === 2).toBe(true);
@@ -311,7 +311,7 @@ describe('open method : ', () => {
           title: 'tab3',
         })
         .then((result) => {
-          expect(result.currentData).toEqual(instance.getCopyData());
+          expect(result.currentData).toEqual(instance.getData());
           expect(result.instance).toBe(instance);
           expect(op.onInit.mock.calls.length === 2).toBe(true);
           expect(op.onChange.mock.calls.length === 1).toBe(true);
@@ -326,7 +326,7 @@ describe('open method : ', () => {
       return instance.open(op.tabs[0]).then(() => {
         expect(op.onChange.mock.calls.length === 0).toBe(true);
         expect(op.onInit.mock.calls.length === 2).toBe(true);
-        expect(instance.getCopyData().openTabIDs.length === 2).toBe(true);
+        expect(instance.getData().openTabIDs.length === 2).toBe(true);
       });
     });
   });
@@ -345,7 +345,7 @@ describe('refresh method : ', () => {
     renderApp();
     return act(() => {
       return instance.refresh().then((result) => {
-        expect(result.currentData).toEqual(instance.getCopyData());
+        expect(result.currentData).toEqual(instance.getData());
         expect(result.instance).toBe(instance);
         expect(op.onInit.mock.calls.length === 2).toBe(true);
         expect(op.onChange.mock.calls.length === 0).toBe(true);
@@ -410,7 +410,7 @@ describe('onChange callback : ', () => {
       openedTabIDs: [],
       closedTabIDs: [],
     });
-    expect(op.onChange.mock.calls[0][0].currentData).toEqual(instance.getCopyData());
+    expect(op.onChange.mock.calls[0][0].currentData).toEqual(instance.getData());
     expect(op.onChange.mock.calls[0][0].previousData).toEqual(instance.getPreviousData());
   });
 });

src/useDynamicTabs/useDynamicTabs.js

@@ -33,7 +33,7 @@ function useDynamicTabs(getDeps, options = {}) {
     api.onChange({newState: state, oldState, closedTabIDs, openedTabIDs, isSwitched});
   }, [state]);
   useLayoutEffect(() => {
-    api.trigger('_onFlushEffects', api.userProxy, {currentData: api.getCopyData(), instance: api.userProxy});
+    api.trigger('_onFlushEffects', api.userProxy, {currentData: api.getData(), instance: api.userProxy});
   }, [flushState]);
   if (!_ref.TabListComponent)
     _ref.TabListComponent = function TabListComponent(props = {}) {

src/utils/api/api.factory.js

@@ -83,9 +83,12 @@ const _apiProps = {
   getCopyPerviousData: function () {
     return this.getPreviousData();
   },
-  getCopyData: function () {
+  getData: function () {
     return this.helper.getCopyState(this.stateRef);
   },
+  getCopyData: function () {
+    return this.getData();
+  },
   isSelected: function (id = missingParamEr('isSelected')) {
     return this.stateRef.selectedTabID == id;
   },

src/utils/api/api.factory.test.js

@@ -420,12 +420,12 @@ describe('Api.prototype.eventHandlerFactory : ', () => {
     }
   });
 });
-describe('Api.prototype.getPreviousData', () => {
+describe('Api.prototype.getPreviousData and Api.prototype.getData : ', () => {
   test('returned data should be equal to optionsManager.initialState in onLoad event', () => {
     expect.assertions(2);
     obj.setOption('onLoad', function () {
       const previousData = this.getPreviousData();
-      const data = this.getCopyData();
+      const data = this.getData();
       expect(previousData).toEqual(obj.optionsManager.initialState);
       expect(previousData).not.toEqual(data);
     });
@@ -435,24 +435,39 @@ describe('Api.prototype.getPreviousData', () => {
   test('returned data should be equal to currentData in onLoad event', () => {
     expect.assertions(1);
     obj.setOption('onLoad', function () {
-      expect(this.getPreviousData()).toEqual(this.getCopyData());
+      expect(this.getPreviousData()).toEqual(this.getData());
     });
     obj.updateStateRef({selectedTabID: 'tab1', openTabIDs: ['tab1', 'tab2']}, () => {});
     obj.trigger('onLoad', obj.userProxy);
   });
-  test('In the onLoad event, return data is equal to getInitialState() and getCopyData()', () => {
+  test('In the onLoad event, return data is equal to getInitialState() and getData()', () => {
     expect.assertions(3);
     const _state = {selectedTabID: 'tab1', openTabIDs: ['tab1', 'tab2']};
     obj.setOption('onLoad', function () {
       const previousData = this.getPreviousData();
-      const data = this.getCopyData();
+      const data = this.getData();
       expect(previousData).toEqual(_state);
       expect(previousData).toEqual(data);
       expect(previousData !== null).toBe(true);
     });
     obj.updateStateRef(_state, () => {});
     obj.trigger('onLoad', obj.userProxy);
   });
+  test('getCopyData method should call getData internally and result of two method should be equal', () => {
+    obj.updateStateRef({selectedTabID: 'tab1', openTabIDs: ['tab1', 'tab2']}, () => {});
+    expect(obj.getData()).toEqual(obj.getCopyData());
+    obj.getData = jest.fn(() => {});
+    obj.getCopyData();
+    expect(obj.getData.mock.calls.length).toBe(1);
+  });
+  test('getCopyPerviousData method should call getPreviousData internally and result of two method should be equal', () => {
+    obj.updateStateRef({selectedTabID: 'tab1', openTabIDs: ['tab1', 'tab2']}, () => {});
+    expect(obj.getPreviousData()).toEqual(obj.getCopyPerviousData());
+    obj.getPreviousData = jest.fn(() => {});
+    obj.getCopyPerviousData();
+    expect(obj.getPreviousData.mock.calls.length).toBe(1);
+  });
+  //
 });
 describe('Api.prototype.setTab : ', () => {
   test('it should call optionsManager.validateObjectiveTabData and validatePanelComponent internally', () => {

src/utils/api/api.test.js

@@ -13,6 +13,7 @@ describe('user api : ', () => {
       'setOption',
       'getPreviousData',
       'getCopyPerviousData',
+      'getData',
       'getCopyData',
       'isSelected',
       'isOpen',