Merge pull request #14 from vueda/update-data-strategy

feat: UPDATE strategy for data update
angeliski · Aug 8, 2019 · 656f082 · 656f082
2 parents 87ed834 + cce0f15
commit 656f082
Show file tree

Hide file tree

Showing 3 changed files with 29 additions and 2 deletions.
diff --git a/src/components/TabulatorComponent.vue b/src/components/TabulatorComponent.vue
@@ -57,6 +57,9 @@ export default class TabulatorComponent extends Vue {
     if (this.tabulatorInstance) {
       if (this.integration && this.integration.updateStrategy === UpdateStrategy.REPLACE) {
         this.tabulatorInstance.replaceData(this.tableData);
+      } else if (this.integration && this.tableData
+      && this.integration.updateStrategy === UpdateStrategy.UPDATE) {
+        this.tabulatorInstance.updateData(this.tableData);
       } else {
         this.tabulatorInstance.setData(this.tableData);
       }

diff --git a/src/types/index.ts b/src/types/index.ts
@@ -1,7 +1,8 @@
 /* eslint-disable import/prefer-default-export */
 export enum UpdateStrategy {
     DATA = 'DATA',
-    REPLACE= 'REPLACE'
+    REPLACE = 'REPLACE',
+    UPDATE = 'UPDATE'
 }
 
 export interface IntegrationOptions {

diff --git a/tests/unit/TabulatorComponent-IntegrationModule.spec.ts b/tests/unit/TabulatorComponent-IntegrationModule.spec.ts
@@ -4,13 +4,15 @@ import { UpdateStrategy } from '@/types';
 
 const mockSetData = jest.fn();
 const mockReplaceData = jest.fn();
+const mockUpdateData = jest.fn();
+
 jest.mock('tabulator-tables', () => jest.fn().mockImplementation(() => ({
   setData: mockSetData,
   replaceData: mockReplaceData,
+  updateData: mockUpdateData,
 })));
 const Tabulator = require('tabulator-tables');
 
-
 const options : Tabulator.Options = {
   columns: [
     {
@@ -29,6 +31,7 @@ describe('TabulatorComponent.vue | Integration Module', () => {
     Tabulator.mockClear();
     mockSetData.mockClear();
     mockReplaceData.mockClear();
+    mockUpdateData.mockClear();
   });
 
   describe('Watchers', () => {
@@ -66,5 +69,25 @@ describe('TabulatorComponent.vue | Integration Module', () => {
       expect(mockSetData).not.toHaveBeenCalled();
       expect(mockReplaceData).toHaveBeenCalled();
     });
+
+    test('update de v-model should use UpdateStrategy.UPDATE', () => {
+      const wrapper = mount(TabulatorComponent, {
+        propsData: {
+          tableData: [{ id: 1, name: 'Someone to update' }],
+          options,
+          integration: {
+            updateStrategy: UpdateStrategy.UPDATE,
+          },
+        },
+      });
+
+      wrapper.setProps({
+        tableData: [{ id: 1, name: 'Someone updated' }],
+      });
+
+      expect(mockSetData).not.toHaveBeenCalled();
+      expect(mockReplaceData).not.toHaveBeenCalled();
+      expect(mockUpdateData).toHaveBeenCalled();
+    });
   });
 });