Merge pull request #32 from sioked/bugfix/fix-update-issue

Fix for calling update multiple times

Author: fzaninotto

src/jsonGraphqlExpress.spec.js

@@ -96,4 +96,18 @@ describe('integration tests', () => {
                 },
             },
         }));
+    it('allows multiple mutations', () =>
+        gqlAgent(
+            'mutation{ updatePost(id:"2", title:"Foo bar", views: 200, user_id:"123") { id } }'
+        ).then(() =>
+            gqlAgent(
+                'mutation{ updatePost(id:"2", title:"Foo bar", views: 200, user_id:"123") { id } }'
+            ).expect({
+                data: {
+                    updatePost: {
+                        id: 2,
+                    },
+                },
+            })
+        ));
 });

src/resolver/Mutation/update.js

@@ -1,6 +1,8 @@
 export default (entityData = []) => (_, params) => {
     const parsedId = parseInt(params.id, 10); // FIXME fails for non-integer ids
-    const indexOfEntity = entityData.findIndex(e => e.id === parsedId);
+    const indexOfEntity = entityData.findIndex(
+        e => parseInt(e.id, 10) === parsedId
+    );
     if (indexOfEntity !== -1) {
         entityData[indexOfEntity] = Object.assign(
             {},