groupcore-utils 3.0.1 → 3.0.3

package/database/crud.js

@@ -109,15 +109,17 @@ module.exports = class {
     let updateQuery = '';
 
     // set this to be used inside the map function to check when we get to the last index key so that we dont add the comma to the last key in the generated sql query
-    const count = Object.keys(data).length;
-    Object.keys(data).map((key, i) => {
-      if (count - i === 1) {
-        updateQuery += `${key} = '${escapeQuotes(data[key])}'`;
-      } else {
-        updateQuery += `${key} = '${escapeQuotes(data[key])}', `;
-      }
-      return true;
-    });
+    const count = Object.keys(data).length - 1; // adding -1 because we are taking out 'id' in the filter method
+    Object.keys(data)
+      .filter((key) => key !== 'id')
+      .forEach((key, i) => {
+        if (count - i === 1) {
+          updateQuery += `${key} = '${escapeQuotes(data[key])}'`;
+        } else {
+          updateQuery += `${key} = '${escapeQuotes(data[key])}', `;
+        }
+        return true;
+      });
 
     const query = `update ${this.dbTable} set ${updateQuery} where id = '${id}'`;
 

package/database/crud.spec.js

@@ -142,3 +142,20 @@ describe('read()', () => {
     expect(querySpy).toHaveBeenCalledWith("select * from test where id = 'test'");
   });
 });
+
+describe('update()', () => {
+  beforeEach(() => {
+    jest.clearAllMocks();
+  });
+  it('should generate the right query', async () => {
+    expect.hasAssertions();
+
+    const querySpy = jest.spyOn(Init.prototype, 'query').mockImplementation(() => {
+      console.log('query called');
+    });
+
+    await new Crud({ dbTable: 'test', db }).update({ data: { id: 1, tag: 'test', slug: 'test' }, id: 1 });
+    expect(querySpy).toHaveBeenCalledWith("update test set tag = 'test', slug = 'test' where id = '1'");
+    expect(querySpy).toHaveBeenCalledTimes(1);
+  });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "groupcore-utils",
-  "version": "3.0.1",
+  "version": "3.0.3",
   "description": "Utilities for working with some core features",
   "main": "Utils.js",
   "scripts": {